Patchwork hw/pcnet.c: Fix EPROM contents to suit AMD netware drivers

login
register
mail settings
Submitter Gerhard Wiesinger
Date March 5, 2011, 12:44 p.m.
Message ID <alpine.LFD.2.02.1103051342010.22602@bbs.intern>
Download mbox | patch
Permalink /patch/85519/
State New
Headers show

Comments

Gerhard Wiesinger - March 5, 2011, 12:44 p.m.
bugfix under DOS for AMD netware driver:
AMD PCNTNW Ethernet MLID v3.10 (960115), network card not found

bugfix works well under DOS with:
1.) AMD NDIS driver v2.0.1
2.) AMD PCNTNW Ethernet MLID v3.10 (960115)
3.) Knoppix 6.2

Signed-off-by: Gerhard Wiesinger <lists@wiesinger.com>
---
  hw/pcnet.c |   16 ++++++++++++++++
  1 files changed, 16 insertions(+), 0 deletions(-)
Blue Swirl - March 5, 2011, 1:25 p.m.
Thanks, applied.

There were still some white space problem with the patch, the
unchanged lines had shifted by one space.

On Sat, Mar 5, 2011 at 2:44 PM, Gerhard Wiesinger <lists@wiesinger.com> wrote:
> bugfix under DOS for AMD netware driver:
> AMD PCNTNW Ethernet MLID v3.10 (960115), network card not found
>
> bugfix works well under DOS with:
> 1.) AMD NDIS driver v2.0.1
> 2.) AMD PCNTNW Ethernet MLID v3.10 (960115)
> 3.) Knoppix 6.2
>
> Signed-off-by: Gerhard Wiesinger <lists@wiesinger.com>
> ---
>  hw/pcnet.c |   16 ++++++++++++++++
>  1 files changed, 16 insertions(+), 0 deletions(-)
>
> diff --git a/hw/pcnet.c b/hw/pcnet.c
> index db52dc5..6dfdcc4 100644
> --- a/hw/pcnet.c
> +++ b/hw/pcnet.c
> @@ -1562,8 +1562,24 @@ void pcnet_h_reset(void *opaque)
>
>     /* Initialize the PROM */
>
> +    /*
> +      Datasheet: http://pdfdata.datasheetsite.com/web/24528/AM79C970A.pdf
> +      page 95
> +    */
>     memcpy(s->prom, s->conf.macaddr.a, 6);
> +    /* Reserved Location: must be 00h */
> +    s->prom[6] = s->prom[7] = 0x00;
> +    /* Reserved Location: must be 00h */
> +    s->prom[8] = 0x00;
> +    /* Hardware ID: must be 11h if compatibility to AMD drivers is desired
> */
> +    s->prom[9] = 0x11;
> +    /* User programmable space, init with 0 */
> +    s->prom[10] = s->prom[11] = 0x00;
> +    /* LSByte of two-byte checksum, which is the sum of bytes 00h-0Bh
> +       and bytes 0Eh and 0Fh, must therefore be initialized with 0! */
>     s->prom[12] = s->prom[13] = 0x00;
> +    /* Must be ASCII W (57h) if compatibility to AMD
> +       driver software is desired */
>     s->prom[14] = s->prom[15] = 0x57;
>
>     for (i = 0,checksum = 0; i < 16; i++)
> --
> 1.7.3.4
>
>
>
Gerhard Wiesinger - March 5, 2011, 1:41 p.m.
On Sat, 5 Mar 2011, Blue Swirl wrote:

> Thanks, applied.
>
> There were still some white space problem with the patch, the
> unchanged lines had shifted by one space.

I guess that comes from reading patch file into alpine and wrapping ...

Anyway, thnx.

Ciao,
Gehrard

--
http://www.wiesinger.com/

Patch

diff --git a/hw/pcnet.c b/hw/pcnet.c
index db52dc5..6dfdcc4 100644
--- a/hw/pcnet.c
+++ b/hw/pcnet.c
@@ -1562,8 +1562,24 @@  void pcnet_h_reset(void *opaque)

      /* Initialize the PROM */

+    /*
+      Datasheet: http://pdfdata.datasheetsite.com/web/24528/AM79C970A.pdf
+      page 95
+    */
      memcpy(s->prom, s->conf.macaddr.a, 6);
+    /* Reserved Location: must be 00h */
+    s->prom[6] = s->prom[7] = 0x00;
+    /* Reserved Location: must be 00h */
+    s->prom[8] = 0x00;
+    /* Hardware ID: must be 11h if compatibility to AMD drivers is desired */
+    s->prom[9] = 0x11;
+    /* User programmable space, init with 0 */
+    s->prom[10] = s->prom[11] = 0x00;
+    /* LSByte of two-byte checksum, which is the sum of bytes 00h-0Bh
+       and bytes 0Eh and 0Fh, must therefore be initialized with 0! */
      s->prom[12] = s->prom[13] = 0x00;
+    /* Must be ASCII W (57h) if compatibility to AMD
+       driver software is desired */
      s->prom[14] = s->prom[15] = 0x57;

      for (i = 0,checksum = 0; i < 16; i++)