>-----Original Message-----
>From: Phillips Kim-R1AAHA 
>Sent: Wednesday, October 28, 2009 2:24 AM
>To: Kumar Gopalpet-B05799
>Cc: u-boot@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.ku...@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 
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to