Dear Ben,
In message <gijdmd$vh...@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
--
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: w...@denx.de
Unix is like a toll road on which you have to stop every 50 feet to
pay another nickel. But hey! You only feel 5 cents poorer each time.
- Larry Wall in <1992aug13.192357.15...@netlabs.com>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
