From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 09 Nov 2011 07:56:55 +0100 Subject: [U-Boot] [PATCH] arm, davinci_emac: fixup In-Reply-To: <20111109065119.BB539189301F@gemini.denx.de> References: <1320819979-3861-1-git-send-email-hs@denx.de> <20111109065119.BB539189301F@gemini.denx.de> Message-ID: <4EBA2437.30807@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Wolfgang, Wolfgang Denk wrote: > Dear Heiko Schocher, > > In message <1320819979-3861-1-git-send-email-hs@denx.de> you wrote: > ... >> + if (count < MAX_PHY) >> + active_phy_addr[j++] = i; >> + else { >> + printf("%s: to much PHYs detected.\n", >> + __func__); >> + break; >> + } > > Coding Style: if the "else" branch uses braces, the "if" branch shall > use braces, too. Hups, you are right, just checked only with checkpatch ... sorry. Changed! > Also, please s/too much/too many/ in the printf() above. Done. Thanks! bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany