* [PATCH] fixes removal of normal_i2c_range on V4L
@ 2005-06-13 2:46 Mauro Carvalho Chehab
2005-06-13 2:57 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2005-06-13 2:46 UTC (permalink / raw)
To: Andrew Morton; +Cc: Linux and Kernel Video, linux-kernel
This patch is necessary to correct I2C detect after normal_i2c_range
removal applied by gregkh-i2c-i2c-address_range_removal.patch at -mm
series.
Signed-off-by: Mauro Carvalho Chehab <mchehab@brturbo.com.br>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fixes removal of normal_i2c_range on V4L
2005-06-13 2:46 Mauro Carvalho Chehab
@ 2005-06-13 2:57 ` Mauro Carvalho Chehab
2005-06-13 16:19 ` Jean Delvare
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2005-06-13 2:57 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Andrew Morton, Linux and Kernel Video, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 279 bytes --]
Mauro Carvalho Chehab wrote:
>This patch is necessary to correct I2C detect after normal_i2c_range
>removal applied by gregkh-i2c-i2c-address_range_removal.patch at -mm
>series.
>
>Signed-off-by: Mauro Carvalho Chehab <mchehab@brturbo.com.br>
>
>
>
>
Now with attachment :-)
[-- Attachment #2: patch07-remove_i2c_range.diff --]
[-- Type: text/x-patch, Size: 5761 bytes --]
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/bt832.c linux-2.6.12-rc6-mm1/drivers/media/video/bt832.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/bt832.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/bt832.c 2005-06-12 23:15:26.000000000 -0300
@@ -39,8 +39,8 @@
MODULE_LICENSE("GPL");
/* Addresses to scan */
-static unsigned short normal_i2c[] = {I2C_CLIENT_END};
-static unsigned short normal_i2c_range[] = {I2C_BT832_ALT1>>1,I2C_BT832_ALT2>>1,I2C_CLIENT_END};
+static unsigned short normal_i2c[] = { I2C_BT832_ALT1>>1, I2C_BT832_ALT2>>1,
+ I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
/* ---------------------------------------------------------------------- */
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/msp3400.c linux-2.6.12-rc6-mm1/drivers/media/video/msp3400.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/msp3400.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/msp3400.c 2005-06-12 23:15:26.000000000 -0300
@@ -147,7 +147,6 @@
I2C_MSP3400C_ALT >> 1,
I2C_CLIENT_END
};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END,I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
/* ----------------------------------------------------------------------- */
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/saa7134/saa6752hs.c linux-2.6.12-rc6-mm1/drivers/media/video/saa7134/saa6752hs.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/saa7134/saa6752hs.c 2005-06-12 12:14:13.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/saa7134/saa6752hs.c 2005-06-12 23:15:26.000000000 -0300
@@ -22,7 +22,6 @@
/* Addresses to scan */
static unsigned short normal_i2c[] = {0x20, I2C_CLIENT_END};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
MODULE_DESCRIPTION("device driver for saa6752hs MPEG2 encoder");
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tda7432.c linux-2.6.12-rc6-mm1/drivers/media/video/tda7432.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tda7432.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tda7432.c 2005-06-12 23:15:26.000000000 -0300
@@ -74,7 +74,6 @@
I2C_TDA7432 >> 1,
I2C_CLIENT_END,
};
-static unsigned short normal_i2c_range[] = { I2C_CLIENT_END, I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
/* Structure of address and subaddresses for the tda7432 */
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tda9875.c linux-2.6.12-rc6-mm1/drivers/media/video/tda9875.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tda9875.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tda9875.c 2005-06-12 23:15:26.000000000 -0300
@@ -44,7 +44,6 @@
I2C_TDA9875 >> 1,
I2C_CLIENT_END
};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
/* This is a superset of the TDA9875 */
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tda9887.c linux-2.6.12-rc6-mm1/drivers/media/video/tda9887.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tda9887.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tda9887.c 2005-06-12 23:15:26.000000000 -0300
@@ -33,7 +33,6 @@
0x96 >>1,
I2C_CLIENT_END,
};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END,I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
/* insmod options */
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tuner-core.c linux-2.6.12-rc6-mm1/drivers/media/video/tuner-core.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tuner-core.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tuner-core.c 2005-06-12 23:26:42.000000000 -0300
@@ -26,17 +26,14 @@
/*
* comment line bellow to return to old behavor, where only one I2C device is supported
*/
-#define CONFIG_TUNER_MULTI_I2C /**/
#define UNSET (-1U)
/* standard i2c insmod options */
static unsigned short normal_i2c[] = {
0x4b, /* tda8290 */
- I2C_CLIENT_END
-};
-static unsigned short normal_i2c_range[] = {
- 0x60, 0x6f,
+ 0x60, 0x61, 0x62, 0x63, 0x64, 0x65, 0x66, 0x67,
+ 0x68, 0x69, 0x6a, 0x6b, 0x6c, 0x6d, 0x6e, 0x6f,
I2C_CLIENT_END
};
I2C_CLIENT_INSMOD;
@@ -325,9 +322,8 @@
static int tuner_probe(struct i2c_adapter *adap)
{
if (0 != addr) {
- normal_i2c[0] = addr;
- normal_i2c_range[0] = addr;
- normal_i2c_range[1] = addr;
+ normal_i2c[0] = addr;
+ normal_i2c[1] = I2C_CLIENT_END;
}
this_adap = 0;
Somente em linux-2.6.12-rc6-mm1/drivers/media/video: tuner-core.c~
Somente em linux-2.6.12-rc6-mm1/drivers/media/video: tuner-core.c.orig
Somente em linux-2.6.12-rc6-mm1/drivers/media/video: tuner-core.c.rej
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tvaudio.c linux-2.6.12-rc6-mm1/drivers/media/video/tvaudio.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tvaudio.c 2005-06-12 12:14:10.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tvaudio.c 2005-06-12 23:15:26.000000000 -0300
@@ -148,7 +148,6 @@
I2C_TDA9874 >> 1,
I2C_PIC16C54 >> 1,
I2C_CLIENT_END };
-static unsigned short normal_i2c_range[2] = { I2C_CLIENT_END, I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
static struct i2c_driver driver;
diff -ur linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tveeprom.c linux-2.6.12-rc6-mm1/drivers/media/video/tveeprom.c
--- linux-2.6.12-rc6-mm1-v4l_patch/drivers/media/video/tveeprom.c 2005-06-12 12:14:13.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tveeprom.c 2005-06-12 23:15:26.000000000 -0300
@@ -482,7 +482,6 @@
0xa0 >> 1,
I2C_CLIENT_END,
};
-static unsigned short normal_i2c_range[] = { I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
struct i2c_driver i2c_driver_tveeprom;
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fixes removal of normal_i2c_range on V4L
2005-06-13 2:57 ` Mauro Carvalho Chehab
@ 2005-06-13 16:19 ` Jean Delvare
0 siblings, 0 replies; 4+ messages in thread
From: Jean Delvare @ 2005-06-13 16:19 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Andrew Morton, video4linux-list, LKML
Hi Mauro,
> This patch is necessary to correct I2C detect after normal_i2c_range
> removal applied by gregkh-i2c-i2c-address_range_removal.patch at -mm
> series.
No. Check 2.6.12-rc6-mm1, it already has all these changes. You must
have messed up somewhere.
--
Jean Delvare
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] fixes removal of normal_i2c_range on V4L
@ 2005-06-13 16:42 Mauro Carvalho Chehab
0 siblings, 0 replies; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2005-06-13 16:42 UTC (permalink / raw)
To: Andrew Morton, Linux and Kernel Video, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 525 bytes --]
Andrew, After sending patch07, I found it was generated with wrong tuner-core.c. I'm sending it again. It should apply correctly this time.
As a reference, all patches 01 to 07 against 2.6.12-rc6-mm1 are at
http://linuxtv.org/download/video4linux/patches/2.6.12-rc6-mm
Mauro Carvalho Chehab wrote:
>This patch is necessary to correct I2C detect after normal_i2c_range
>removal applied by gregkh-i2c-i2c-address_range_removal.patch at -mm
>series.
>
>Signed-off-by: Mauro Carvalho Chehab <mchehab@brturbo.com.br>
>
>
>
[-- Attachment #2: patch07-remove_i2c_range.diff --]
[-- Type: text/x-patch, Size: 5444 bytes --]
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/bt832.c linux-2.6.12-rc6-mm1/drivers/media/video/bt832.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/bt832.c 2005-06-13 11:51:08.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/bt832.c 2005-06-13 11:54:15.000000000 -0300
@@ -39,8 +39,8 @@
MODULE_LICENSE("GPL");
/* Addresses to scan */
-static unsigned short normal_i2c[] = {I2C_CLIENT_END};
-static unsigned short normal_i2c_range[] = {I2C_BT832_ALT1>>1,I2C_BT832_ALT2>>1,I2C_CLIENT_END};
+static unsigned short normal_i2c[] = { I2C_BT832_ALT1>>1, I2C_BT832_ALT2>>1,
+ I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
/* ---------------------------------------------------------------------- */
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/msp3400.c linux-2.6.12-rc6-mm1/drivers/media/video/msp3400.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/msp3400.c 2005-06-13 11:51:08.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/msp3400.c 2005-06-13 11:54:15.000000000 -0300
@@ -147,7 +147,6 @@
I2C_MSP3400C_ALT >> 1,
I2C_CLIENT_END
};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END,I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
/* ----------------------------------------------------------------------- */
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/saa7134/saa6752hs.c linux-2.6.12-rc6-mm1/drivers/media/video/saa7134/saa6752hs.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/saa7134/saa6752hs.c 2005-06-13 11:51:11.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/saa7134/saa6752hs.c 2005-06-13 11:54:15.000000000 -0300
@@ -22,7 +22,6 @@
/* Addresses to scan */
static unsigned short normal_i2c[] = {0x20, I2C_CLIENT_END};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
MODULE_DESCRIPTION("device driver for saa6752hs MPEG2 encoder");
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tda7432.c linux-2.6.12-rc6-mm1/drivers/media/video/tda7432.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tda7432.c 2005-06-13 11:51:08.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tda7432.c 2005-06-13 11:54:15.000000000 -0300
@@ -74,7 +74,6 @@
I2C_TDA7432 >> 1,
I2C_CLIENT_END,
};
-static unsigned short normal_i2c_range[] = { I2C_CLIENT_END, I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
/* Structure of address and subaddresses for the tda7432 */
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tda9875.c linux-2.6.12-rc6-mm1/drivers/media/video/tda9875.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tda9875.c 2005-06-13 11:51:08.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tda9875.c 2005-06-13 11:54:15.000000000 -0300
@@ -44,7 +44,6 @@
I2C_TDA9875 >> 1,
I2C_CLIENT_END
};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
/* This is a superset of the TDA9875 */
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tda9887.c linux-2.6.12-rc6-mm1/drivers/media/video/tda9887.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tda9887.c 2005-06-13 11:51:08.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tda9887.c 2005-06-13 11:54:15.000000000 -0300
@@ -33,7 +33,6 @@
0x96 >>1,
I2C_CLIENT_END,
};
-static unsigned short normal_i2c_range[] = {I2C_CLIENT_END,I2C_CLIENT_END};
I2C_CLIENT_INSMOD;
/* insmod options */
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tuner-core.c linux-2.6.12-rc6-mm1/drivers/media/video/tuner-core.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tuner-core.c 2005-06-13 11:51:24.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tuner-core.c 2005-06-13 11:55:51.000000000 -0300
@@ -32,13 +32,10 @@
/* standard i2c insmod options */
static unsigned short normal_i2c[] = {
0x4b, /* tda8290 */
+ 0x60, 0x61, 0x62, 0x63, 0x64, 0x65, 0x66, 0x67,
+ 0x68, 0x69, 0x6a, 0x6b, 0x6c, 0x6d, 0x6e, 0x6f,
I2C_CLIENT_END
};
-static unsigned short normal_i2c_range[] = {
- 0x60, 0x6f,
- I2C_CLIENT_END
-};
-I2C_CLIENT_INSMOD;
/* insmod options used at init time => read/only */
static unsigned int addr = 0;
@@ -324,9 +321,8 @@
static int tuner_probe(struct i2c_adapter *adap)
{
if (0 != addr) {
- normal_i2c[0] = addr;
- normal_i2c_range[0] = addr;
- normal_i2c_range[1] = addr;
+ normal_i2c[0] = addr;
+ normal_i2c[1] = I2C_CLIENT_END;
}
this_adap = 0;
Somente em linux-2.6.12-rc6-mm1/drivers/media/video: tuner-core.c.orig
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tvaudio.c linux-2.6.12-rc6-mm1/drivers/media/video/tvaudio.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tvaudio.c 2005-06-13 11:51:08.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tvaudio.c 2005-06-13 11:54:15.000000000 -0300
@@ -148,7 +148,6 @@
I2C_TDA9874 >> 1,
I2C_PIC16C54 >> 1,
I2C_CLIENT_END };
-static unsigned short normal_i2c_range[2] = { I2C_CLIENT_END, I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
static struct i2c_driver driver;
diff -ur linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tveeprom.c linux-2.6.12-rc6-mm1/drivers/media/video/tveeprom.c
--- linux-2.6.12-rc6-mm1-v4l/drivers/media/video/tveeprom.c 2005-06-13 11:51:11.000000000 -0300
+++ linux-2.6.12-rc6-mm1/drivers/media/video/tveeprom.c 2005-06-13 11:54:15.000000000 -0300
@@ -482,7 +482,6 @@
0xa0 >> 1,
I2C_CLIENT_END,
};
-static unsigned short normal_i2c_range[] = { I2C_CLIENT_END };
I2C_CLIENT_INSMOD;
struct i2c_driver i2c_driver_tveeprom;
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2005-06-13 16:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-06-13 16:42 [PATCH] fixes removal of normal_i2c_range on V4L Mauro Carvalho Chehab
-- strict thread matches above, loose matches on Subject: below --
2005-06-13 2:46 Mauro Carvalho Chehab
2005-06-13 2:57 ` Mauro Carvalho Chehab
2005-06-13 16:19 ` Jean Delvare
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox