[U-Boot] [PATCH] ARM: davici_emac: Fix condition for number of phy detects

Tom Rini tom.rini at gmail.com
Tue Nov 29 21:38:10 CET 2011


On Mon, Nov 28, 2011 at 9:30 PM, Prabhakar Lad
<prabhakar.csengg at gmail.com> wrote:
> Sandeep,
>
> Can you take this patch ?
>
> Regards,
> --Prabhakar Lad
>
> On Thu, Nov 17, 2011 at 6:34 PM, Heiko Schocher <hs at denx.de> wrote:
>
>> Hello prabhakar.csengg at gmail.com
>>
>> prabhakar.csengg at gmail.com wrote:
>> > From: Prabhakar Lad <prabhakar.csengg at gmail.com>
>> >
>> > Fix the condition for number of phys in
>> > davinci_eth_phy_detect() function.
>> > CONFIG_SYS_DAVINCI_EMAC_PHY_COUNT indicates number of
>> > phys. From this commit id dc02badab480563b0bf9d3908046ea9d6b22ae63
>> > davinci emac initilazed one less than the number of phy count.
>> >
>> > Signed-off-by: Prabhakar Lad <prabhakar.csengg at gmail.com>
>> > ---
>> >  drivers/net/davinci_emac.c |    2 +-
>> >  1 files changed, 1 insertions(+), 1 deletions(-)
>> >
>> > diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
>> > index 36c33af..34b6f94 100644
>> > --- a/drivers/net/davinci_emac.c
>> > +++ b/drivers/net/davinci_emac.c
>> > @@ -176,7 +176,7 @@ static int davinci_eth_phy_detect(void)
>> >       for (i = 0, j = 0; i < 32; i++)
>> >               if (phy_act_state & (1 << i)) {
>> >                       count++;
>> > -                     if (count < CONFIG_SYS_DAVINCI_EMAC_PHY_COUNT) {
>> > +                     if (count <= CONFIG_SYS_DAVINCI_EMAC_PHY_COUNT) {
>> >                               active_phy_addr[j++] = i;
>> >                       } else {
>> >                               printf("%s: to many PHYs detected.\n",
>>
>> good catch!
>>
>> Acked-by: Heiko Schocher <hs at denx.de>

(Will be) queued to u-boot-ti, thanks!

-- 
Tom


More information about the U-Boot mailing list