Commit 9d802222 authored by Maciej S. Szmigiero's avatar Maciej S. Szmigiero Committed by Mauro Carvalho Chehab
Browse files

media: saa7134: fix terminology around saa7134_i2c_eeprom_md7134_gate() 



saa7134_i2c_eeprom_md7134_gate() function and the associated comment uses
an inverted i2c gate open / closed terminology.
Let's fix this.

Signed-off-by: default avatarMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
[hverkuil-cisco@xs4all.nl: fix alignment checkpatch warning]
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent c1849372

drivers/media/pci/saa7134/saa7134-i2c.c

+8 −4
Original line number Diff line number Diff line
@@ -342,7 +342,11 @@ static const struct i2c_client saa7134_client_template = { 


/* ----------------------------------------------------------- */



/* On Medion 7134 reading EEPROM needs DVB-T demod i2c gate open */

/*

 * On Medion 7134 reading the SAA7134 chip config EEPROM needs DVB-T

 * demod i2c gate closed due to an address clash between this EEPROM

 * and the demod one.

 */

static void saa7134_i2c_eeprom_md7134_gate(struct saa7134_dev *dev)

{

	u8 subaddr = 0x7, dmdregval;
@@ -359,13 +363,13 @@ static void saa7134_i2c_eeprom_md7134_gate(struct saa7134_dev *dev) 


	ret = i2c_transfer(&dev->i2c_adap, i2cgatemsg_r, 2);

	if ((ret == 2) && (dmdregval & 0x2)) {

		pr_debug("%s: DVB-T demod i2c gate was left closed\n",

		pr_debug("%s: DVB-T demod i2c gate was left open\n",

			 dev->name);



		data[0] = subaddr;

		data[1] = (dmdregval & ~0x2);

		if (i2c_transfer(&dev->i2c_adap, i2cgatemsg_w, 1) != 1)

			pr_err("%s: EEPROM i2c gate open failure\n",

			pr_err("%s: EEPROM i2c gate close failure\n",

			       dev->name);

	}

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE