[riot-notifications] [RIOT] drivers/gpio: changed periph driver interface (#3095)
notifications at github.com
Fri Jun 5 10:20:40 CEST 2015
> +int main(void)
> + shell_t shell;
> + puts("GPIO peripheral driver test\n");
> + puts("In this test, pins are specified by integer port and pin numbers.\n"
> + "So if your platform has a pin PA01, it will be port=0 and pin=1,\n"
> + "PC14 would be port=2 and pin=14 etc.\n\n"
> + "NOTE: make sure the values you use exist on your platform! The\n"
> + " behavior for not existing ports/pins is not defined!");
> + /* start the shell */
> + (void) posix_open(uart0_handler_pid, 0);
> + shell_init(&shell, shell_commands, SHELL_BUFSIZE, uart0_readc, uart0_putc);
> + shell_run(&shell);
> return 0;
I agree with Kaspar. But this test is of course not the end of it, there is room for improvement. I could think of something like defining a set of board specific GPIO pins that should be tested and providing some shell commands, that work on this set. But Lets take it step-by-step and merge this first and improve afterwards, otherwise this PR gets too heavy and it will never be merged...
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications