Markus Klotzbücher wrote:
> Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]> writes:
> 
>>> +
>>> +   reg16 = in_le16(hose_cfg_base + PEX_LTSSM_STAT);
>>> +   if (reg16 < 0x16) {
>>> +           printf("No link\n");
>>> +           hose->indirect_type = INDIRECT_TYPE_NO_PCIE_LINK;
>>> +   } else {
>>> +           printf("link\n");
>>> +   }
>> please remove the bracket on the else
> 
> Come on, this is being ridiculously pedantic.

Absolutely agree.

In my opinion, extra braces should just acceptable
and even encouraged.

jdl



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to