[U-Boot] [PATCH 1/1] at91: Add command to control up to 3 GPIO LEDs from the console
Stefan Roese
sr at denx.de
Fri May 8 08:21:05 CEST 2009
Hi Daniel,
On Friday 08 May 2009, Daniel Gorsulowski wrote:
> > I suggest to use something like this here:
> >
> > led_nr = simple_strtoul(argv[1], NULL, 10);
> > if (led_nr > CONFIG_LED_MAX) {
> > printf ("Usage:\n%s\n", cmdtp->usage);
> > return 1;
> > }
> >
> > if (strcmp(argv[2], "off") == 0) {
> > on = 1;
> > } else if (strcmp(argv[2], "on") == 0) {
> > on = 0;
> > } else {
> > printf ("Usage:\n%s\n", cmdtp->usage);
> > return 1;
> > }
> >
> > user_led(led_nr, on);
> >
> > No ugly #ifdef's in this case. What do you think?
> >
> > Best regards,
> > Stefan
>
> I agree with you.
Good. :)
> Please give me some days, to implement your basic approaches.
> I've many other things to do and it's not that easy (for me)
> to create a tidy patch.
Sure. Take your time.
Thanks.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
More information about the U-Boot
mailing list