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

Kumar Gopalpet-B05799 B05799 at freescale.com
Wed Oct 28 08:18:08 CET 2009


 

>-----Original Message-----
>From: Phillips Kim-R1AAHA 
>Sent: Wednesday, October 28, 2009 2:24 AM
>To: Kumar Gopalpet-B05799
>Cc: u-boot at lists.denx.de
>Subject: Re: [U-Boot] [PATCH] NET: Move MDIO regs out of TSEC Space
>
>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, removed the warning and inconsistancy.

Resending the patches

-Thanks
Sandeep 


More information about the U-Boot mailing list