On Monday 15 April 2013 09:52 PM, Michael Cashwell wrote:
> Hi Sricharan,
>
> I very much like how you've structured this. A vast improvement!
>
> I haven't yet tried to apply the whole series but have one quick comment. In
> the new function:
>
> static void save_omap_boot_params(void)
> {
> ...
> if (!(omap_hw_init_context() ==
> OMAP_INIT_CONTEXT_UBOOT_AFTER_SPL)) {
> ...
> } else {
> ...
> }
>
> wouldn't it be clearer to drop the boolean negation "!" and exchange the
> if/else bodies?
>
hmm, will do and add a comment as well.
Regards,
Sricharan
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
