2008-02-08 11:22:25

by Adrian McMenamin

Subject: [PATCH] 8139too fix for Dreamcast

Updates the 8139too driver to work with recently added
(a724605cb7a66d423a494a395f9a8ba871b8a1eb) declared coherent memory
patch for the Dreamcast.

Jeff - I am assuming you are still the maintainer and I guess Paul
should also ack this as it is SH related.

Signed-off-by: Adrian McMenamin <[email protected]>

-----


diff --git a/drivers/net/8139too.c b/drivers/net/8139too.c
index d373545..80f1cce 100644
--- a/drivers/net/8139too.c
+++ b/drivers/net/8139too.c
@@ -168,7 +168,7 @@ static int debug = -1;
* Warning: 64K ring has hardware issues and may lock up.
*/
#if defined(CONFIG_SH_DREAMCAST)
-#define RX_BUF_IDX 1 /* 16K ring */
+#define RX_BUF_IDX 0 /* 8K ring */
#else
#define RX_BUF_IDX 2 /* 32K ring */
#endif

2008-02-11 15:31:31

by Jeff Garzik

Subject: Re: [PATCH] 8139too fix for Dreamcast

Adrian McMenamin wrote:
> Updates the 8139too driver to work with recently added
> (a724605cb7a66d423a494a395f9a8ba871b8a1eb) declared coherent memory
> patch for the Dreamcast.
>
> Jeff - I am assuming you are still the maintainer and I guess Paul
> should also ack this as it is SH related.
>
> Signed-off-by: Adrian McMenamin <[email protected]>
>
> -----
>
>
> diff --git a/drivers/net/8139too.c b/drivers/net/8139too.c
> index d373545..80f1cce 100644
> --- a/drivers/net/8139too.c
> +++ b/drivers/net/8139too.c
> @@ -168,7 +168,7 @@ static int debug = -1;
> * Warning: 64K ring has hardware issues and may lock up.
> */
> #if defined(CONFIG_SH_DREAMCAST)
> -#define RX_BUF_IDX 1 /* 16K ring */
> +#define RX_BUF_IDX 0 /* 8K ring */
> #else
> #define RX_BUF_IDX 2 /* 32K ring */
> #endif

applied... yell if that's a problem Paul