Dear Heiko Schocher,
In message <4ed3a6a8.5010...@denx.de> you wrote:
>
> >> + ret = gpio_request(gpio, enbw_gpio_config[i].name);
> >> + if (ret)
> >> + printf("%s: Could not get %s gpio\n", __func__,
> >> + enbw_gpio_config[i].name);
> >> + else
> >
> > instead of having that else and adding another level of indentation below
> > you can just add continue;
>
> Done.
Is this really a good idea??
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
One essential to success is that you desire be an all-obsessing one,
your thoughts and aims be co-ordinated, and your energy be concentra-
ted and applied without letup.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
