From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerry Van Baren Date: Thu, 24 Sep 2009 09:35:02 -0400 Subject: [U-Boot] [PATCH 1/3 v3] Bit-banged MII driver with multi-bus support. In-Reply-To: <200909241452.05747.luigi.mantellini@idf-hit.com> References: <1253711456-6309-1-git-send-email-luigi.mantellini.ml@gmail.com> <1253711456-6309-2-git-send-email-luigi.mantellini.ml@gmail.com> <200909241452.05747.luigi.mantellini@idf-hit.com> Message-ID: <4ABB7586.9040001@ge.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Luigi, Luigi 'Comio' Mantellini wrote: > (autoreview) > > Hi Ben, > > I make some changes to the patch, but I will post after your review. > See my annotation, that will be present into the next patch release. > > best regards. > > Il mercoled? 23 settembre 2009 15:10:54 Luigi 'Comio' Mantellini ha scritto: >> From: Luigi 'Comio' Mantellini >> >> Signed-off-by: Luigi 'Comio' Mantellini >> --- >> drivers/net/phy/miiphybb.c | 324 [snip] >> +static inline struct bbmiibus *bb_miiphy_getbus(char *devname) >> +{ >> +#ifdef CONFIG_BITBANGMII_MULTI >> + /* Search the correct bus */ >> + for (j = 0; j < sizeof(bbmiibusses)/sizeof(bbmmis[0]); j++) { > > [COMIO] Changed j to i and add i declaration. You may have inherited the "j" loop counter from me. I stopped using "i" in loop counters and started with "j" because most code (variables and keywords) do not use words with the letter "j" or "k" in them. On the other hand, A LOT of variables and keywords have "i" in their names. The result is that it is a PAIN to find loop variable references for the loop variable "i" but it is usually trivial to find them for "j" and "k". Case in point: how many "i"s does your loop have? Just in the for() loop control line, I count five. Trivia: * I find I almost never need to nest loops more than 2 deep, so "j" and "k" cover almost all loop variable needs. * I believe Fortran coding conventions started the "ijk" counter mania. [snip] > Any comment is welcome > > best regards, > > luigi Best regards, gvb