[PATCH] pcmcia: Add support TI PCI4510 CardBus bridge

[OGAWA Hirofumi]

Marco Rogantini <marco.rogantini@xxxxxxxx> writes:

>> +#define PCI_DEVICE_ID_TI_4510		0xac44
>> +	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_4510, TI12XX),
>
> :-) It solved the problem! However I still must use the 'disable_clkrun'
> parameter to get the bridge working correctly.

Great! Thanks. I checked the datasheet, and added zoom_video
support. New patch is attached.

Andrew, could you apply this patch to your tree for testing?
-- 
OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>



This does setup PCI4510 properly, and supports the "disable_clkrun" option.

1) Add PCI_DEVICE_ID_TI_4510 to pci_id.h.
2) Add PCI4510 to yenta_table (that uses TI12XX handlers).
3) Add zoom_video handler support.

TI12XX handlers can disable CLKRUN feature with "disable_clkrun" option.
Some devices or bridge itself seems to be needing this option as workaround.

Signed-off-by: OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>
---

 drivers/pcmcia/ti113x.h       |    1 +
 drivers/pcmcia/yenta_socket.c |    1 +
 include/linux/pci_ids.h       |    1 +
 3 files changed, 3 insertions(+)

diff -puN drivers/pcmcia/ti113x.h~yenta-pci4510-support drivers/pcmcia/ti113x.h
--- linux-2.6.11-rc3/drivers/pcmcia/ti113x.h~yenta-pci4510-support	2005-02-07 01:46:12.000000000 +0900
+++ linux-2.6.11-rc3-hirofumi/drivers/pcmcia/ti113x.h	2005-02-07 01:46:12.000000000 +0900
@@ -262,6 +262,7 @@ static void ti_set_zv(struct yenta_socke
 			case PCI_DEVICE_ID_TI_1220:
 			case PCI_DEVICE_ID_TI_1221:
 			case PCI_DEVICE_ID_TI_1225:
+			case PCI_DEVICE_ID_TI_4510:
 				socket->socket.zoom_video = ti_zoom_video;
 				break;	
 			case PCI_DEVICE_ID_TI_1250:
diff -puN drivers/pcmcia/yenta_socket.c~yenta-pci4510-support drivers/pcmcia/yenta_socket.c
--- linux-2.6.11-rc3/drivers/pcmcia/yenta_socket.c~yenta-pci4510-support	2005-02-07 01:46:12.000000000 +0900
+++ linux-2.6.11-rc3-hirofumi/drivers/pcmcia/yenta_socket.c	2005-02-07 01:46:12.000000000 +0900
@@ -1105,6 +1105,7 @@ static struct pci_device_id yenta_table 
 	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_4410, TI12XX),
 	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_4450, TI12XX),
 	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_4451, TI12XX),
+	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_4510, TI12XX),
 	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_4520, TI12XX),
 
 	CB_ID(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_1250, TI1250),
diff -puN include/linux/pci_ids.h~yenta-pci4510-support include/linux/pci_ids.h
--- linux-2.6.11-rc3/include/linux/pci_ids.h~yenta-pci4510-support	2005-02-07 01:46:12.000000000 +0900
+++ linux-2.6.11-rc3-hirofumi/include/linux/pci_ids.h	2005-02-07 01:46:12.000000000 +0900
@@ -753,6 +753,7 @@
 #define PCI_DEVICE_ID_TI_1251B		0xac1f
 #define PCI_DEVICE_ID_TI_4410		0xac41
 #define PCI_DEVICE_ID_TI_4451		0xac42
+#define PCI_DEVICE_ID_TI_4510		0xac44
 #define PCI_DEVICE_ID_TI_4520		0xac46
 #define PCI_DEVICE_ID_TI_1410		0xac50
 #define PCI_DEVICE_ID_TI_1420		0xac51
_
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/