[U-Boot] [PATCH] tsec: Configure the buffer descriptor bases to always include all of the descriptors

Joe Hershberger joe.hershberger at gmail.com
Wed Aug 10 21:20:11 CEST 2011


On Wed, Aug 10, 2011 at 9:10 AM, Andy Fleming <afleming at freescale.com> wrote:
>
> On Aug 10, 2011, at 2:12 AM, Joe Hershberger wrote:
>
>> Previously only the last N were included based on the current one in use.
>>
>> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
>> Cc: Joe Hershberger <joe.hershberger at gmail.com>
>> Cc: Mingkai Hu <Mingkai.hu at freescale.com>
>> Cc: Andy Fleming <afleming at freescale.com>
>> Cc: Kumar Gala <galak at kernel.crashing.org>
>> Cc: Detlev Zundel <dzu at denx.de>
>
>
> I'm curious if you were seeing a problem that this fixes?

I was searching for a performance problem on the MPC8313, and
discovered this, which seemed wrong.  It was not, however, the source
of my problem.

>> ---
>> drivers/net/tsec.c |    4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/net/tsec.c b/drivers/net/tsec.c
>> index 78ffc95..1805ca0 100644
>> --- a/drivers/net/tsec.c
>> +++ b/drivers/net/tsec.c
>> @@ -250,8 +250,8 @@ static void startup_tsec(struct eth_device *dev)
>>       txIdx = 0;
>>
>>       /* Point to the buffer descriptors */
>> -     out_be32(&regs->tbase, (unsigned int)(&rtx.txbd[txIdx]));
>> -     out_be32(&regs->rbase, (unsigned int)(&rtx.rxbd[rxIdx]));
>> +     out_be32(&regs->tbase, (unsigned int)(&rtx.txbd[0]));
>> +     out_be32(&regs->rbase, (unsigned int)(&rtx.rxbd[0]));
>
> However, while I don't believe this fixes a technical problem, I believe this makes the code more straightforward.

I agree.  It is more straightforward to use 0 explicitly.

> So if this is a fix to a problem, we need more information to understand what you're really fixing. If this is just fixing something that looked wrong...:
>
> Acked-by: Andy Fleming <afleming at freescale.com>

It fixes something that was wrong before you committed
063c12633d5ad74d52152d9c358e715475e17629, but at this point, it's just
cosmetic.

Best regards,
-Joe


More information about the U-Boot mailing list