rtl8169 support - Final

[Jon Shadow]

Fixed a few things. Still need someone to test it.

Signed-off-by: Sergey Lichack

[Michael Karcher]

Am Mittwoch, den 05.01.2011, 12:59 +0300 schrieb Jon Shadow:
> Fixed a few things. Still need someone to test it.
>
> Signed-off-by: Sergey Lichack

Code looks fine to me, as soon as it is tested, it is:
Acked-by: Michael Karcher

Regards,
  Michael Karcher

[Mathias Krause]

On 05.01.2011 10:59, Jon Shadow wrote:
> Fixed a few things. Still need someone to test it.
>
> Signed-off-by: Sergey Lichack

Haven't the hardware to do some testing but like to comment the patch.
> Index: nicrealtek.c
>
> --- nicrealtek.c	(revision 1250)
> +++ nicrealtek.c	(working copy)
> @@ -27,11 +27,11 @@
>  #define PCI_VENDOR_ID_REALTEK	0x10ec
>  #define PCI_VENDOR_ID_SMC1211	0x1113
>
> -#define BIOS_ROM_ADDR		0xD4
> -#define BIOS_ROM_DATA		0xD7
> +static int bios_rom_addr, bios_rom_data;
>
> const struct pcidev_status nics_realtek[] = {
>    {0x10ec, 0x8139, OK, "Realtek", "RTL8139/8139C/8139C+"},
> {0x10ec, 0x8169, OK, "Realtek", "RTL8169"},
> {},
>
> };
>
> @@ -42,6 +42,8 @@
>
> int nicrealtek_init(void)
>  {
> static uint16_t id;

Why static? It's a new variable, only visible in this function and not
checked before being (re)assigned.
> get_io_perms();
>
> io_base_addr = pcidev_init(PCI_VENDOR_ID_REALTEK, PCI_BASE_ADDRESS_0,
>
> @@ -49,6 +51,16 @@
>
> buses_supported = CHIP_BUSTYPE_PARALLEL;
> id = pcidev_dev->device_id;
> if(id == 0x8139) {
> bios_rom_addr = 0xD4;
> bios_rom_data = 0xD7;
> }
> else {
> bios_rom_addr = 0x30;
> bios_rom_data = 0x33;
> }
> return 0;
>
> }
>
> @@ -61,6 +73,9 @@
>
> buses_supported = CHIP_BUSTYPE_PARALLEL;
> bios_rom_addr = 0xD4;
> bios_rom_data = 0xD7;
> return 0;
>
> }
>
> @@ -78,12 +93,12 @@
>     * enable software access.
>     */
>    OUTL(((uint32_t)addr & 0x01FFFF) | 0x0A0000 | (val << 24),
>     io_base_addr + BIOS_ROM_ADDR);
>     io_base_addr + bios_rom_addr);
> /* Output addr and data, set WE to 1, set OE to 1, set CS to 1,
> enable software access.
> */
>  OUTL(((uint32_t)addr & 0x01FFFF) | 0x1E0000 | (val << 24),
>     io_base_addr + BIOS_ROM_ADDR);
>     io_base_addr + bios_rom_addr);
>
> }
>
> uint8_t nicrealtek_chip_readb(const chipaddr addr)
> @@ -92,20 +107,20 @@
>
> /* FIXME: Can we skip reading the old data and simply use 0? */
>    /* Read old data. */
> val = INB(io_base_addr + BIOS_ROM_DATA);
> val = INB(io_base_addr + bios_rom_data);
> /* Output new addr and old data, set WE to 1, set OE to 0, set CS to 0,
> enable software access.
> */
> OUTL(((uint32_t)addr & 0x01FFFF) | 0x060000 | (val << 24),
>     io_base_addr + BIOS_ROM_ADDR);
>     io_base_addr + bios_rom_addr);
>
> /* Read new data. */
> val = INB(io_base_addr + BIOS_ROM_DATA);
> val = INB(io_base_addr + bios_rom_data);
> /* Output addr and new data, set WE to 1, set OE to 1, set CS to 1,
> enable software access.
> */
> OUTL(((uint32_t)addr & 0x01FFFF) | 0x1E0000 | (val << 24),
>     io_base_addr + BIOS_ROM_ADDR);
>     io_base_addr + bios_rom_addr);
>
> return val;
>
> }

Otherwise Acked-by: Mathias Krause

[PATCH] Add support for Realtek RTL8169.

[Stefan Tauner]

And some cleanups including the removal of any left traces of the
nicsmc1211 programmer. The RTL8169 still needs someone to test it,
but it's good enough to be in the tree now.

Signed-off-by: Sergey Lichack 
Acked-by: Michael Karcher 
Acked-by: Mathias Krause 
Signed-off-by: Stefan Tauner 
Acked-by: Stefan Tauner 
---
 flashrom.8   |   10 ++++------
 flashrom.c   |    1 -
 nicrealtek.c |   44 +++++++++++++++++++++++++++-----------------
 3 files changed, 31 insertions(+), 24 deletions(-)

diff --git a/flashrom.8 b/flashrom.8
index 33e6e23..66b7129 100644
--- a/flashrom.8
+++ b/flashrom.8
@@ -172,9 +172,7 @@ involving any chip access (probe/read/write/...). Currently supported are:
 .sp
 .BR "* nic3com" " (for flash ROMs on 3COM network cards)"
 .sp
-.BR "* nicrealtek" " (for flash ROMs on Realtek network cards)"
-.sp
-.BR "* nicsmc1211" " (for flash ROMs on RTL8139-compatible SMC2 network cards)"
+.BR "* nicrealtek" " (for flash ROMs on Realtek and RTL8139-compatible SMC network cards)"
 .sp
 .BR "* nicnatsemi" " (for flash ROMs on National Semiconductor DP838* network \
 cards)"
@@ -532,7 +530,7 @@ syntax where
 .B content
 is an 8-bit hexadecimal value.
 .SS
-.BR "nic3com" , " nicrealtek" , " nicsmc1211" , " nicnatsemi" , " nicintel\
+.BR "nic3com" , " nicrealtek" , " nicnatsemi" , " nicintel\
 " , " nicintel_spi" , " gfxnvidia" , " ogp_spi" , " drkaiser" , " satasii\
 " , " satamv" ", and " atahpt " programmers
 These programmers have an option to specify the PCI address of the card
@@ -761,7 +759,7 @@ flashrom needs different access permissions for different programmers.
 needs raw memory access, PCI configuration space access, raw I/O port
 access (x86) and MSR access (x86).
 .sp
-.BR nic3com ", " nicrealtek ", " nicsmc1211 " and " nicnatsemi "
+.BR nic3com ", " nicrealtek " and " nicnatsemi "
 need PCI configuration space read access and raw I/O port access.
 .sp
 .B atahpt
@@ -792,7 +790,7 @@ need access to the USB device via libusb.
 .B dummy
 needs no access permissions at all.
 .sp
-.BR internal ", " nic3com ", " nicrealtek ", " nicsmc1211 ", " nicnatsemi ", "
+.BR internal ", " nic3com ", " nicrealtek ", " nicnatsemi ", "
 .BR gfxnvidia ", " drkaiser ", " satasii ", " satamv " and " atahpt
 have to be run as superuser/root, and need additional raw access permission.
 .sp
diff --git a/flashrom.c b/flashrom.c
index 44a3eba..98ab18a 100644
--- a/flashrom.c
+++ b/flashrom.c
@@ -95,7 +95,6 @@ const struct programmer_entry programmer_table[] = {
    {
    	/* This programmer works for Realtek RTL8139 and SMC 1211. */
    	.name			= "nicrealtek",
-		//.name			= "nicsmc1211",
    	.init			= nicrealtek_init,
    	.map_flash_region	= fallback_map,
    	.unmap_flash_region	= fallback_unmap,
diff --git a/nicrealtek.c b/nicrealtek.c
index afc3d0f..71bccfc 100644
--- a/nicrealtek.c
+++ b/nicrealtek.c
@@ -28,19 +28,17 @@
 #define PCI_VENDOR_ID_REALTEK	0x10ec
 #define PCI_VENDOR_ID_SMC1211	0x1113

-#define BIOS_ROM_ADDR		0xD4
-#define BIOS_ROM_DATA		0xD7
+static int bios_rom_addr, bios_rom_data;

const struct pcidev_status nics_realtek[] = {
    {0x10ec, 0x8139, OK, "Realtek", "RTL8139/8139C/8139C+"},
-	{0x1113, 0x1211, OK, "SMC2", "1211TX"}, /* RTL8139 clone */
+	{0x10ec, 0x8169, NT, "Realtek", "RTL8169"},
+	{0x1113, 0x1211, OK, "SMC", "1211TX"}, /* RTL8139 clone */
    {},
 };

-static void nicrealtek_chip_writeb(const struct flashctx *flash, uint8_t val,
-				   chipaddr addr);
-static uint8_t nicrealtek_chip_readb(const struct flashctx *flash,
-				     const chipaddr addr);
+static void nicrealtek_chip_writeb(const struct flashctx *flash, uint8_t val, chipaddr addr);
+static uint8_t nicrealtek_chip_readb(const struct flashctx *flash, const chipaddr addr);
 static const struct par_programmer par_programmer_nicrealtek = {
    	.chip_readb		= nicrealtek_chip_readb,
    	.chip_readw		= fallback_chip_readw,
@@ -61,6 +59,20 @@ static int nicrealtek_shutdown(void *data)

int nicrealtek_init(void)
 {
+	/* Beware, this ignores the vendor ID! */
+	switch (pcidev_dev->device_id) {
+	case 0x8139: /* RTL8139 */
+	case 0x1211: /* SMC 1211TX */
+	default:
+		bios_rom_addr = 0xD4;
+		bios_rom_data = 0xD7;
+		break;
+	case 0x8169: /* RTL8169 */
+		bios_rom_addr = 0x30;
+		bios_rom_data = 0x33;
+		break;
+	}
+
    if (rget_io_perms())
    	return 1;

@@ -74,42 +86,40 @@ int nicrealtek_init(void)
    return 0;
 }

-static void nicrealtek_chip_writeb(const struct flashctx *flash, uint8_t val,
-				   chipaddr addr)
+static void nicrealtek_chip_writeb(const struct flashctx *flash, uint8_t val, chipaddr addr)
 {
    /* Output addr and data, set WE to 0, set OE to 1, set CS to 0,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x0A0000 | (val << 24),
-	     io_base_addr + BIOS_ROM_ADDR);
+	     io_base_addr + bios_rom_addr);
    /* Output addr and data, set WE to 1, set OE to 1, set CS to 1,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x1E0000 | (val << 24),
-	     io_base_addr + BIOS_ROM_ADDR);
+	     io_base_addr + bios_rom_addr);
 }

-static uint8_t nicrealtek_chip_readb(const struct flashctx *flash,
-				     const chipaddr addr)
+static uint8_t nicrealtek_chip_readb(const struct flashctx *flash, const chipaddr addr)
 {
    uint8_t val;

/* FIXME: Can we skip reading the old data and simply use 0? */
    /* Read old data. */
-	val = INB(io_base_addr + BIOS_ROM_DATA);
+	val = INB(io_base_addr + bios_rom_data);
    /* Output new addr and old data, set WE to 1, set OE to 0, set CS to 0,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x060000 | (val << 24),
-	     io_base_addr + BIOS_ROM_ADDR);
+	     io_base_addr + bios_rom_addr);

/* Read new data. */
-	val = INB(io_base_addr + BIOS_ROM_DATA);
+	val = INB(io_base_addr + bios_rom_data);
    /* Output addr and new data, set WE to 1, set OE to 1, set CS to 1,
     * enable software access.
     */
    OUTL(((uint32_t)addr & 0x01FFFF) | 0x1E0000 | (val << 24),
-	     io_base_addr + BIOS_ROM_ADDR);
+	     io_base_addr + bios_rom_addr);

return val;
 }