[U-Boot] [PATCH v4 1/1] imximage: Add MX53 boot image support

Jason Liu liu.h.jason at gmail.com
Wed Jan 19 17:50:08 CET 2011


Hi, Wolfgang,

2011/1/20 Wolfgang Denk <wd at denx.de>:
> Dear Jason Liu,
>
> In message <AANLkTinx920ets6A-F5R-+RiJ06OQLx=5zUjbj9LK7wX at mail.gmail.com> you wrote:
>>
>> Do you mean the I need add the braces as the following?
>>
>>     if ((fhdr_v1->app_code_barker == APP_CODE_BARKER) &&
>>              (hdr_v1->dcd_table.preamble.barker == DCD_BARKER)) {
>>             return IMXIMAGE_V1;
>>     }
>
> Right. And especially with the blank lines you had before.

Yes,  but can we just change to the following since it just one statement,
if ((fhdr_v1->app_code_barker == APP_CODE_BARKER) &&
              (hdr_v1->dcd_table.preamble.barker == DCD_BARKER))
             return IMXIMAGE_V1;

Some checkpatch will give a warning, say, no need to add braces for only one
statement, something like that.


>
> 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: wd at denx.de
> Winners never talk about glorious victories. That's  because  they're
> the  ones  who  see  what the battlefield looks like afterwards. It's
> only the losers who have glorious victories.
>                                      - Terry Pratchett, _Small Gods_
>


More information about the U-Boot mailing list