[U-Boot] [PATCH] NET: Move MDIO regs out of TSEC Space

Kim Phillips kim.phillips at freescale.com
Tue Oct 27 21:54:27 CET 2009


On Sat, 24 Oct 2009 23:39:36 +0530
Sandeep Gopalpet <sandeep.kumar at freescale.com> wrote:

> @@ -306,12 +307,11 @@ static int init_phy(struct eth_device *dev)
>  {
>  	struct tsec_private *priv = (struct tsec_private *)dev->priv;
>  	struct phy_info *curphy;
> -	volatile tsec_t *phyregs = priv->phyregs;
> +	volatile tsec_mdio_t *phyregs = priv->phyregs;
>  	volatile tsec_t *regs = priv->regs;
>  
>  	/* Assign a Physical address to the TBI */
>  	regs->tbipa = CONFIG_SYS_TBIPA_VALUE;
> -	phyregs->tbipa = CONFIG_SYS_TBIPA_VALUE;
>  	asm("sync");

this generates a new warning:

tsec.c: In function 'init_phy':
tsec.c:307: warning: unused variable 'phyregs'

>  	.regs = (tsec_t *)(TSEC_BASE_ADDR + ((num - 1) * TSEC_SIZE)), \
> -	.miiregs = (tsec_t *)TSEC_BASE_ADDR, \
> +	.miiregs = (tsec_mdio_t *)(MDIO_BASE_ADDR), \
> +	.miiregs_sgmii = (tsec_mdio_t *)(MDIO_BASE_ADDR + (num -1)*MDIO_OFFSET ), \

and please use consistent codingstyle; here and..

>  	.devname = CONFIG_TSEC##num##_NAME, \
>  	.phyaddr = TSEC##num##_PHY_ADDR, \
>  	.flags = TSEC##num##_FLAGS \
> @@ -44,7 +33,8 @@
>  #define SET_STD_TSEC_INFO(x, num) \
>  {			\
>  	x.regs = (tsec_t *)(TSEC_BASE_ADDR + ((num - 1) * TSEC_SIZE)); \
> -	x.miiregs = (tsec_t *)TSEC_BASE_ADDR; \
> +	x.miiregs = (tsec_mdio_t *)(MDIO_BASE_ADDR); \
> +	x.miiregs_sgmii = (tsec_mdio_t *)(MDIO_BASE_ADDR + (num -1)*MDIO_OFFSET); \

..here.

Thanks,

Kim


More information about the U-Boot mailing list