[U-Boot] [PATCH] tsec: Configure the buffer descriptor bases to always include all of the descriptors
Andy Fleming
afleming at freescale.com
Wed Aug 10 16:10:48 CEST 2011
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?
> ---
> 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(®s->tbase, (unsigned int)(&rtx.txbd[txIdx]));
> - out_be32(®s->rbase, (unsigned int)(&rtx.rxbd[rxIdx]));
> + out_be32(®s->tbase, (unsigned int)(&rtx.txbd[0]));
> + out_be32(®s->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.
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>
More information about the U-Boot
mailing list