[U-Boot] [Bug] IXP425 and e1000 network driver

[Wolfgang Denk]

Dear Ben Warren,

In message <49D68311.4090807 at gmail.com> you wrote:
> Wolfgang Denk wrote:
> > Dear Ben,
> >
> > In message <gijdmd$vhk$1 at ger.gmane.org> Stefan Althoefer wrote:
> >   
> >> Hi,
> >>
> >> I found that IXP425 (big endian ARM) did not work with e1000 network
> >> driver. The reason is broken access to controller registers.
> >>
> >> I get it working with this patch:
> >>
> >> --------
> >> --- a/drivers/net/e1000.c
> >> +++ b/drivers/net/e1000.c
> >> @@ -105,12 +105,15 @@ static void e1000_phy_hw_reset(struct e1000_hw *hw);
> >>  static int e1000_phy_reset(struct e1000_hw *hw);
> >>  static int e1000_detect_gig_phy(struct e1000_hw *hw);
> >>
> >> -#define E1000_WRITE_REG(a, reg, value) (writel((value), ((a)->hw_addr + E1000_##reg)))
> >> -#define E1000_READ_REG(a, reg) (readl((a)->hw_addr + E1000_##reg))
> >> -#define E1000_WRITE_REG_ARRAY(a, reg, offset, value) (\
> >> -			writel((value), ((a)->hw_addr + E1000_##reg + ((offset) << 2))))
> >> -#define E1000_READ_REG_ARRAY(a, reg, offset) ( \
> >> -	readl((a)->hw_addr + E1000_##reg + ((offset) << 2)))
> >> +#define E1000_WRITE_REG(a, reg, value) \
> >> +	(writel(cpu_to_le32(value), ((a)->hw_addr + E1000_##reg)))
> >> +#define E1000_READ_REG(a, reg) \
> >> +	(le32_to_cpu(readl((a)->hw_addr + E1000_##reg)))
> >> +#define E1000_WRITE_REG_ARRAY(a, reg, offset, value) \
> >> +	(writel(cpu_to_le32(value),\
> >> +		((a)->hw_addr + E1000_##reg + ((offset) << 2))))
> >> +#define E1000_READ_REG_ARRAY(a, reg, offset) \
> >> +	(le32_to_cpu(readl((a)->hw_addr + E1000_##reg + ((offset) << 2))))
> >>  #define E1000_WRITE_FLUSH(a) {uint32_t x; x = E1000_READ_REG(a, STATUS);}
> >>
> >>  #ifndef CONFIG_AP1000 /* remove for warnings */
> >> ---------
> >>
> >> However, I'm not sure it this is the correct fix.
> >>
> >> Is readl supposed to read raw data?
> >>
> >> Is le32_to_cpu/cpu_to_le32 a function or a macro? In the later case the
> >> code is not save or slow due to multiple argument expansion.
> >>
> >> -- Stefan
> >>     
> >
> > I have never seen any comments on this. Could you please have a look
> > at it?
> >
> > Best regards,
> >
> > Wolfgang Denk
> >
> >   
> Sure thing.

That was 4 months ago, but I did not see anything happen. Can you
please re-check?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Niklaus Wirth has lamented that, whereas Europeans pronounce his name
correctly  (Ni-klows  Virt),  Americans  invariably  mangle  it  into
(Nick-les  Worth).  Which  is to say that Europeans call him by name,
but Americans call him by value.