[U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood

Prafulla Wadaskar prafulla at marvell.com
Mon Jul 12 07:45:28 CEST 2010


 

> -----Original Message-----
> From: u-boot-bounces at lists.denx.de 
> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert Aribaud
> Sent: Sunday, July 11, 2010 1:32 PM
> To: u-boot at lists.denx.de
> Subject: [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other 
> than kirkwood
> 
> Rename all references to kirkwood in mv_egiga symbols
> throughout the whole codebase.
> 
> Signed-off-by: Albert Aribaud <albert.aribaud at free.fr>
> ---
>  arch/arm/cpu/arm926ejs/kirkwood/cpu.c         |    4 +-
>  arch/arm/include/asm/arch-kirkwood/kirkwood.h |    5 +
>  drivers/net/Makefile                          |    2 +-
>  drivers/net/mv_egiga.c                        |  322 
> +++++++++---------
>  drivers/net/mv_egiga.h                        |  466 
> ++++++++++++------------
>  include/configs/guruplug.h                    |    4 +-
>  include/configs/km_arm.h                      |    4 +-
>  include/configs/mv88f6281gtw_ge.h             |    4 +-
>  include/configs/openrd_base.h                 |    4 +-
>  include/configs/rd6281a.h                     |    4 +-
>  include/configs/sheevaplug.h                  |    4 +-
>  include/netdev.h                              |    2 +-
>  12 files changed, 418 insertions(+), 407 deletions(-)
> 
> diff --git a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c 
> b/arch/arm/cpu/arm926ejs/kirkwood/cpu.c
> index 6fc3902..786ffc6 100644
> --- a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c
> +++ b/arch/arm/cpu/arm926ejs/kirkwood/cpu.c
> @@ -378,10 +378,10 @@ int arch_misc_init(void)
>  }
>  #endif /* CONFIG_ARCH_MISC_INIT */
>  
> -#ifdef CONFIG_KIRKWOOD_EGIGA
> +#ifdef CONFIG_MV_EGIGA
>  int cpu_eth_init(bd_t *bis)
>  {
> -	kirkwood_egiga_initialize(bis);
> +	mv_egiga_initialize(bis);
>  	return 0;
>  }
>  #endif
> diff --git a/arch/arm/include/asm/arch-kirkwood/kirkwood.h 
> b/arch/arm/include/asm/arch-kirkwood/kirkwood.h
> index 2470efb..9200605 100644
> --- a/arch/arm/include/asm/arch-kirkwood/kirkwood.h
> +++ b/arch/arm/include/asm/arch-kirkwood/kirkwood.h
> @@ -60,6 +60,11 @@
>  #define KW_EGIGA0_BASE			(KW_REGISTER(0x72000))
>  #define KW_EGIGA1_BASE			(KW_REGISTER(0x76000))
>  
> +#if defined (CONFIG_MV_EGIGA)

I think you don't need ifdef here

> +#define MV_EGIGA0_BASE			KW_EGIGA0_BASE
> +#define MV_EGIGA1_BASE			KW_EGIGA1_BASE
> +#endif
> +
>  #if defined (CONFIG_KW88F6281)
>  #include <asm/arch/kw88f6281.h>
>  #elif defined (CONFIG_KW88F6192)
> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
> index 1599c26..04f8de0 100644
> --- a/drivers/net/Makefile
> +++ b/drivers/net/Makefile
> @@ -52,7 +52,7 @@ COBJS-$(CONFIG_MACB) += macb.o
>  COBJS-$(CONFIG_MCFFEC) += mcffec.o mcfmii.o
>  COBJS-$(CONFIG_MPC5xxx_FEC) += mpc5xxx_fec.o
>  COBJS-$(CONFIG_MPC512x_FEC) += mpc512x_fec.o
> -COBJS-$(CONFIG_KIRKWOOD_EGIGA) += mv_egiga.o
> +COBJS-$(CONFIG_MV_EGIGA) += mv_egiga.o
>  COBJS-$(CONFIG_NATSEMI) += natsemi.o
>  COBJS-$(CONFIG_DRIVER_NE2000) += ne2000.o ne2000_base.o
>  COBJS-$(CONFIG_DRIVER_AX88796L) += ax88796.o ne2000_base.o
> diff --git a/drivers/net/mv_egiga.c b/drivers/net/mv_egiga.c
> index b7ecc9d..4fecf1e 100644
> --- a/drivers/net/mv_egiga.c
> +++ b/drivers/net/mv_egiga.c
> @@ -35,13 +35,17 @@
>  #include <asm/errno.h>
>  #include <asm/types.h>
>  #include <asm/byteorder.h>
> +
> +#if defined (CONFIG_KIRKWOOD)
>  #include <asm/arch/kirkwood.h>
> +#endif
> +
>  #include "mv_egiga.h"
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> -#define KIRKWOOD_PHY_ADR_REQUEST 0xee
> -#define KWGBE_SMI_REG (((struct kwgbe_registers 
> *)KW_EGIGA0_BASE)->smi)
> +#define MV_PHY_ADR_REQUEST 0xee
> +#define MV_EGIGA_SMI_REG (((struct mv_egiga_registers 
> *)MV_EGIGA0_BASE)->smi)
>  
>  /*
>   * smi_reg_read - miiphy_read callback function.
> @@ -51,16 +55,16 @@ DECLARE_GLOBAL_DATA_PTR;
>  static int smi_reg_read(char *devname, u8 phy_adr, u8 
> reg_ofs, u16 * data)
>  {
>  	struct eth_device *dev = eth_get_dev_by_name(devname);
> -	struct kwgbe_device *dkwgbe = to_dkwgbe(dev);
> -	struct kwgbe_registers *regs = dkwgbe->regs;
> +	struct mv_egiga_device *dmvegiga = to_mv_egiga(dev);
> +	struct mv_egiga_registers *regs = dmvegiga->regs;

I suggest to keep name as mvgbe here instead of mv_egiga, 3 additional chars, increases overall code size

>  	u32 smi_reg;
>  	u32 timeout;
>  
>  	/* Phyadr read request */
> -	if (phy_adr == KIRKWOOD_PHY_ADR_REQUEST &&
> -			reg_ofs == KIRKWOOD_PHY_ADR_REQUEST) {
> +	if (phy_adr == MV_PHY_ADR_REQUEST &&
> +			reg_ofs == MV_PHY_ADR_REQUEST) {
>  		/* */
> -		*data = (u16) (KWGBEREG_RD(regs->phyadr) & PHYADR_MASK);
> +		*data = (u16) (MV_EGIGA_REG_RD(regs->phyadr) & 

Same here, pls use MVGBEREG instead of MV_EGIGA_REG, 4 extra chars

Pls just replace kw/KW by mv/MV to avoide indentation issues

I think these are the only changes in this patch (i.e. find and replace), are there any other?

Regards..
Prafulla . .


More information about the U-Boot mailing list