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