[U-Boot] Change to miiphy_register() creates warnings

Simon Glass sjg at chromium.org
Wed Dec 7 17:19:37 CET 2011


Hi Wolfgang,

On Tue, Dec 6, 2011 at 11:39 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Simon Glass,
>
> In message <CAPnjgZ0tGPnZOQ7eBRwt5noHHVw_z3KY=QcyH_GSxkFxS2kgEw at mail.gmail.com> you wrote:
>>
>> In this commit:
>>
>> 5c45a22 mii: miiphy register address width change
>>
>> the required prototypes for the read and write functions passed to
>> miiphy_register() were changed. However, from what I can tell the
>> callers to miiphy_register() did not all get their functions changes.
>> So there are now a lot of warnings. I doubt they will affect actual
>> operation.
>>
>> Chandan, do you have time to do a patch for this? Files that use this
>> are listed below, but not all will require changes - just changing the
>> other use in miiphy.h would help a lot.
>>
>> ./arch/arm/cpu/ixp/npe/npe.c
>> ./arch/mips/cpu/mips32/au1x00/au1x00_eth.c
>> ./arch/powerpc/cpu/mpc85xx/ether_fcc.c
>> ./arch/powerpc/cpu/mpc8220/fec.c
>> ./arch/powerpc/cpu/mpc8xx/fec.c
>> ./arch/powerpc/cpu/mpc8260/ether_fcc.c
>> ./board/evb64260/eth.c
>> ./board/prodrive/p3mx/mv_eth.c
>> ./board/gdsys/405ex/io64.c
>> ./board/gdsys/405ep/io.c
>> ./common/miiphyutil.c
>> ./drivers/qe/uec.c
>> ./drivers/net/fsl_mcdmafec.c
>> ./drivers/net/macb.c
>> ./drivers/net/mpc5xxx_fec.c
>> ./drivers/net/davinci_emac.c
>> ./drivers/net/mcffec.c
>> ./drivers/net/enc28j60.c
>> ./drivers/net/smc911x.c
>> ./drivers/net/altera_tse.c
>> ./drivers/net/ep93xx_eth.c
>> ./drivers/net/at91_emac.c
>> ./drivers/net/sh_eth.c
>> ./drivers/net/bfin_mac.c
>> ./drivers/net/mpc512x_fec.c
>> ./drivers/net/xilinx_axi_emac.c
>> ./drivers/net/mvgbe.c
>> ./drivers/net/armada100_fec.c
>> ./drivers/net/designware.c
>> ./drivers/net/4xx_enet.c
>> ./drivers/net/fec_mxc.c
>> ./drivers/net/eepro100.c
>
> It makes no sense "fixing" all these drivers just to keep the old API
> longer alive.  I reverted the commit.

Well that will fix the problem, thank you.

Regards,
Simon

>
> 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
> Wait!  You have not been prepared!
>        -- Mr. Atoz, "Tomorrow is Yesterday", stardate 3113.2


More information about the U-Boot mailing list