From 1bf338da0ae61a46aa7aaf9a3921edfe5a02c305 Mon Sep 17 00:00:00 2001
From: Steven Toth <stoth@hauppauge.com>
Date: Mon, 19 Mar 2007 19:22:41 -0300
Subject: [PATCH] --- yaml --- r: 65847 b: refs/heads/master c:
 e133be0f587996f112d7984c03606af418a7ca05 h: refs/heads/master i:   65845:
 748b6a0b36db74c92d976eb58a908a61d57df388   65843:
 557fe784024bf4b46062ebbaeb924acfac25632f   65839:
 996f4f552246b03dea40399bdda84e66c3653485 v: v3

---
 [refs]                                        |  2 +-
 .../media/video/cx23885/cx23885-cards.c       |  3 ---
 .../media/video/cx23885/cx23885-core.c        | 24 +++++++++----------
 trunk/drivers/media/video/cx23885/cx23885.h   | 11 +++++----
 4 files changed, 18 insertions(+), 22 deletions(-)

diff --git a/[refs] b/[refs]
index c4b8eb2ef474..e346c508ea8a 100644
--- a/[refs]
+++ b/[refs]
@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 0fc0739ba940c07e97599a7ee04c24faae9808ed
+refs/heads/master: e133be0f587996f112d7984c03606af418a7ca05
diff --git a/trunk/drivers/media/video/cx23885/cx23885-cards.c b/trunk/drivers/media/video/cx23885/cx23885-cards.c
index 9344fb5d95f6..decf602ab164 100644
--- a/trunk/drivers/media/video/cx23885/cx23885-cards.c
+++ b/trunk/drivers/media/video/cx23885/cx23885-cards.c
@@ -32,7 +32,6 @@
 struct cx23885_board cx23885_boards[] = {
 	[CX23885_BOARD_UNKNOWN] = {
 		.name		= "UNKNOWN/GENERIC",
-		.bridge		= CX23885_BRIDGE_UNDEFINED,
 		.input          = {{
 			.type   = CX23885_VMUX_COMPOSITE1,
 			.vmux   = 0,
@@ -49,7 +48,6 @@ struct cx23885_board cx23885_boards[] = {
 	},
 	[CX23885_BOARD_HAUPPAUGE_HVR1800lp] = {
 		.name		= "Hauppauge WinTV-HVR1800lp",
-		.bridge		= CX23885_BRIDGE_885,
 		.portc		= CX23885_MPEG_DVB,
 		.input          = {{
 			.type   = CX23885_VMUX_TELEVISION,
@@ -71,7 +69,6 @@ struct cx23885_board cx23885_boards[] = {
 	},
 	[CX23885_BOARD_HAUPPAUGE_HVR1800] = {
 		.name		= "Hauppauge WinTV-HVR1800",
-		.bridge		= CX23885_BRIDGE_887,
 		.portc		= CX23885_MPEG_DVB,
 		.input          = {{
 			.type   = CX23885_VMUX_TELEVISION,
diff --git a/trunk/drivers/media/video/cx23885/cx23885-core.c b/trunk/drivers/media/video/cx23885/cx23885-core.c
index b1f75350e83d..d8e376dee68b 100644
--- a/trunk/drivers/media/video/cx23885/cx23885-core.c
+++ b/trunk/drivers/media/video/cx23885/cx23885-core.c
@@ -452,8 +452,8 @@ int cx23885_sram_channel_setup(struct cx23885_dev *dev,
 	cx_write(ch->cnt2_reg, (lines*16) >> 3);
 	cx_write(ch->cnt1_reg, (bpl >> 3) -1);
 
-	dprintk(2,"[bridged %d] sram setup %s: bpl=%d lines=%d\n",
-		cx23885_boards[dev->board].bridge,
+	dprintk(2,"[bridge %d] sram setup %s: bpl=%d lines=%d\n",
+		dev->bridge,
 		ch->name,
 		bpl,
 		lines);
@@ -770,18 +770,16 @@ static int cx23885_dev_setup(struct cx23885_dev *dev)
 		dev->board, card[dev->nr] == dev->board ?
 		"insmod option" : "autodetected");
 
-	/* Configure the hardware internal memory for fifos */
-	switch(cx23885_boards[dev->board].bridge) {
-	case CX23885_BRIDGE_UNDEFINED:
-	case CX23885_BRIDGE_885:
-		dev->sram_channels = cx23885_sram_channels;
-		break;
-	case CX23885_BRIDGE_887:
+	/* Configure the internal memory */
+	if(dev->pci->device == 0x8880) {
+		dev->bridge = CX23885_BRIDGE_887;
 		dev->sram_channels = cx23887_sram_channels;
-		break;
-	default:
-		printk(KERN_ERR "%s() error, default case", __FUNCTION__ );
+	} else
+	if(dev->pci->device == 0x8852) {
+		dev->bridge = CX23885_BRIDGE_885;
+		dev->sram_channels = cx23885_sram_channels;
 	}
+	dprintk(1, "%s() Memory configured for PCIe bridge type %d\n", __FUNCTION__, dev->bridge);
 
 	/* init hardware */
 	cx23885_reset(dev);
@@ -1037,7 +1035,7 @@ static int cx23885_start_dma(struct cx23885_tsport *port,
 	 * starting or stopping interrupts or dma. Avoid the bug for the time being,
 	 * enabling the developer to work on the demod/tuner locking work.
 	 */
-	switch(cx23885_boards[dev->board].bridge) {
+	switch(dev->bridge) {
 	case CX23885_BRIDGE_885:
 	case CX23885_BRIDGE_887:
 		/* enable irqs */
diff --git a/trunk/drivers/media/video/cx23885/cx23885.h b/trunk/drivers/media/video/cx23885/cx23885.h
index 04105bcf53f3..4e1fc68d4206 100644
--- a/trunk/drivers/media/video/cx23885/cx23885.h
+++ b/trunk/drivers/media/video/cx23885/cx23885.h
@@ -101,11 +101,6 @@ struct cx23885_board {
 		CX23885_MPEG_UNDEFINED = 0,
 		CX23885_MPEG_DVB
 	} portc;
-	enum {
-		CX23885_BRIDGE_UNDEFINED = 0,
-		CX23885_BRIDGE_885 = 885,
-		CX23885_BRIDGE_887 = 887,
-	} bridge;
 	struct cx23885_input    input[MAX_CX23885_INPUT];
 };
 
@@ -210,6 +205,12 @@ struct cx23885_dev {
 
 	/* sram configuration */
 	struct sram_channel        *sram_channels;
+
+	enum {
+		CX23885_BRIDGE_UNDEFINED = 0,
+		CX23885_BRIDGE_885 = 885,
+		CX23885_BRIDGE_887 = 887,
+	} bridge;
 };
 
 #define SRAM_CH01  0 /* Video A */