Patchwork pcnet APROMWE bit location (retry)

login
register
mail settings
Submitter Chris Kilgour
Date Feb. 24, 2010, 6:32 a.m.
Message ID <4B84C7EE.4050303@whiterocker.com>
Download mbox | patch
Permalink /patch/46113/
State New
Headers show

Comments

Chris Kilgour - Feb. 24, 2010, 6:32 a.m.
According to AMD document 21485D pp.141, APROMWE is bit 8 of BCR2.

Signed-off-by: Christopher Kilgour <techie@whiterocker.com>

---
Anthony Liguori - March 9, 2010, 2:24 p.m.
On 02/24/2010 12:32 AM, Chris Kilgour wrote:
> According to AMD document 21485D pp.141, APROMWE is bit 8 of BCR2.
>
> Signed-off-by: Christopher Kilgour<techie@whiterocker.com>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
> diff --git a/hw/pcnet.c b/hw/pcnet.c
> index 44b5b31..f889898 100644
> --- a/hw/pcnet.c
> +++ b/hw/pcnet.c
> @@ -1601,7 +1601,7 @@ static void pcnet_aprom_writeb(void *opaque, uint32_t addr, uint32_t val)
>       printf("pcnet_aprom_writeb addr=0x%08x val=0x%02x\n", addr, val);
>   #endif
>       /* Check APROMWE bit to enable write access */
> -    if (pcnet_bcr_readw(s,2)&  0x80)
> +    if (pcnet_bcr_readw(s,2)&  0x100)
>           s->prom[addr&  15] = val;
>   }
>
>
>
>
>

Patch

diff --git a/hw/pcnet.c b/hw/pcnet.c
index 44b5b31..f889898 100644
--- a/hw/pcnet.c
+++ b/hw/pcnet.c
@@ -1601,7 +1601,7 @@  static void pcnet_aprom_writeb(void *opaque, uint32_t addr, uint32_t val)
     printf("pcnet_aprom_writeb addr=0x%08x val=0x%02x\n", addr, val);
 #endif
     /* Check APROMWE bit to enable write access */
-    if (pcnet_bcr_readw(s,2) & 0x80)
+    if (pcnet_bcr_readw(s,2) & 0x100)
         s->prom[addr & 15] = val;
 }