Skip to content

L
linux
Commit 941830c9 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab
Browse files
Options

[media] xc4000: make checkpatch.pl happy 

Solve the ./scripts/checkpatch.pl compliants for the patches
that added xc4000 support, including a few changes at dib0700.

While here, remove a few printk noise by converting some msgs
into debug ones.

Cc: Istvan Varga <istvan_v@mailbox.hu>
Cc: Patrick Boettcher <pboettcher@kernellabs.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 8edeb6eb
Show whitespace changes
Inline Side-by-side
Showing with 71 additions and 73 deletions
drivers/media/common/tuners/xc4000.c
View file @ 941830c9
...@@ -244,8 +244,8 @@ static struct XC_TV_STANDARD xc4000_standard[MAX_TV_STANDARD] = { ...@@ -244,8 +244,8 @@ static struct XC_TV_STANDARD xc4000_standard[MAX_TV_STANDARD] = {
{"DTV8", 0x00C0, 0x800B, 0}, {"DTV8", 0x00C0, 0x800B, 0},
{"DTV7/8", 0x00C0, 0x801B, 0}, {"DTV7/8", 0x00C0, 0x801B, 0},
{"DTV7", 0x00C0, 0x8007, 0}, {"DTV7", 0x00C0, 0x8007, 0},
{"FM Radio-INPUT2", 0x0008, 0x9800,10700}, {"FM Radio-INPUT2", 0x0008, 0x9800, 10700},
{"FM Radio-INPUT1", 0x0008, 0x9000,10700} {"FM Radio-INPUT1", 0x0008, 0x9000, 10700}
}; };
static int xc4000_readreg(struct xc4000_priv *priv, u16 reg, u16 *val); static int xc4000_readreg(struct xc4000_priv *priv, u16 reg, u16 *val);
...@@ -261,7 +261,7 @@ static int xc_send_i2c_data(struct xc4000_priv *priv, u8 *buf, int len) ...@@ -261,7 +261,7 @@ static int xc_send_i2c_data(struct xc4000_priv *priv, u8 *buf, int len)
printk(KERN_ERR "xc4000: I2C write failed (len=%i)\n", printk(KERN_ERR "xc4000: I2C write failed (len=%i)\n",
len); len);
if (len == 4) { if (len == 4) {
printk("bytes %02x %02x %02x %02x\n", buf[0], printk(KERN_ERR "bytes %02x %02x %02x %02x\n", buf[0],
buf[1], buf[2], buf[3]); buf[1], buf[2], buf[3]);
} }
return -EREMOTEIO; return -EREMOTEIO;
...@@ -546,7 +546,7 @@ static int xc4000_readreg(struct xc4000_priv *priv, u16 reg, u16 *val) ...@@ -546,7 +546,7 @@ static int xc4000_readreg(struct xc4000_priv *priv, u16 reg, u16 *val)
}; };
if (i2c_transfer(priv->i2c_props.adap, msg, 2) != 2) { if (i2c_transfer(priv->i2c_props.adap, msg, 2) != 2) {
printk(KERN_WARNING "xc4000: I2C read failed\n"); printk(KERN_ERR "xc4000: I2C read failed\n");
return -EREMOTEIO; return -EREMOTEIO;
} }
...@@ -558,67 +558,67 @@ static int xc4000_readreg(struct xc4000_priv *priv, u16 reg, u16 *val) ...@@ -558,67 +558,67 @@ static int xc4000_readreg(struct xc4000_priv *priv, u16 reg, u16 *val)
static void dump_firm_type_and_int_freq(unsigned int type, u16 int_freq) static void dump_firm_type_and_int_freq(unsigned int type, u16 int_freq)
{ {
if (type & BASE) if (type & BASE)
printk("BASE "); printk(KERN_CONT "BASE ");
if (type & INIT1) if (type & INIT1)
printk("INIT1 "); printk(KERN_CONT "INIT1 ");
if (type & F8MHZ) if (type & F8MHZ)
printk("F8MHZ "); printk(KERN_CONT "F8MHZ ");
if (type & MTS) if (type & MTS)
printk("MTS "); printk(KERN_CONT "MTS ");
if (type & D2620) if (type & D2620)
printk("D2620 "); printk(KERN_CONT "D2620 ");
if (type & D2633) if (type & D2633)
printk("D2633 "); printk(KERN_CONT "D2633 ");
if (type & DTV6) if (type & DTV6)
printk("DTV6 "); printk(KERN_CONT "DTV6 ");
if (type & QAM) if (type & QAM)
printk("QAM "); printk(KERN_CONT "QAM ");
if (type & DTV7) if (type & DTV7)
printk("DTV7 "); printk(KERN_CONT "DTV7 ");
if (type & DTV78) if (type & DTV78)
printk("DTV78 "); printk(KERN_CONT "DTV78 ");
if (type & DTV8) if (type & DTV8)
printk("DTV8 "); printk(KERN_CONT "DTV8 ");
if (type & FM) if (type & FM)
printk("FM "); printk(KERN_CONT "FM ");
if (type & INPUT1) if (type & INPUT1)
printk("INPUT1 "); printk(KERN_CONT "INPUT1 ");
if (type & LCD) if (type & LCD)
printk("LCD "); printk(KERN_CONT "LCD ");
if (type & NOGD) if (type & NOGD)
printk("NOGD "); printk(KERN_CONT "NOGD ");
if (type & MONO) if (type & MONO)
printk("MONO "); printk(KERN_CONT "MONO ");
if (type & ATSC) if (type & ATSC)
printk("ATSC "); printk(KERN_CONT "ATSC ");
if (type & IF) if (type & IF)
printk("IF "); printk(KERN_CONT "IF ");
if (type & LG60) if (type & LG60)
printk("LG60 "); printk(KERN_CONT "LG60 ");
if (type & ATI638) if (type & ATI638)
printk("ATI638 "); printk(KERN_CONT "ATI638 ");
if (type & OREN538) if (type & OREN538)
printk("OREN538 "); printk(KERN_CONT "OREN538 ");
if (type & OREN36) if (type & OREN36)
printk("OREN36 "); printk(KERN_CONT "OREN36 ");
if (type & TOYOTA388) if (type & TOYOTA388)
printk("TOYOTA388 "); printk(KERN_CONT "TOYOTA388 ");
if (type & TOYOTA794) if (type & TOYOTA794)
printk("TOYOTA794 "); printk(KERN_CONT "TOYOTA794 ");
if (type & DIBCOM52) if (type & DIBCOM52)
printk("DIBCOM52 "); printk(KERN_CONT "DIBCOM52 ");
if (type & ZARLINK456) if (type & ZARLINK456)
printk("ZARLINK456 "); printk(KERN_CONT "ZARLINK456 ");
if (type & CHINA) if (type & CHINA)
printk("CHINA "); printk(KERN_CONT "CHINA ");
if (type & F6MHZ) if (type & F6MHZ)
printk("F6MHZ "); printk(KERN_CONT "F6MHZ ");
if (type & INPUT2) if (type & INPUT2)
printk("INPUT2 "); printk(KERN_CONT "INPUT2 ");
if (type & SCODE) if (type & SCODE)
printk("SCODE "); printk(KERN_CONT "SCODE ");
if (type & HAS_IF) if (type & HAS_IF)
printk("HAS_IF_%d ", int_freq); printk(KERN_CONT "HAS_IF_%d ", int_freq);
} }
static int seek_firmware(struct dvb_frontend *fe, unsigned int type, static int seek_firmware(struct dvb_frontend *fe, unsigned int type,
...@@ -666,7 +666,8 @@ static int seek_firmware(struct dvb_frontend *fe, unsigned int type, ...@@ -666,7 +666,8 @@ static int seek_firmware(struct dvb_frontend *fe, unsigned int type,
} }
if (best_nr_diffs > 0U) { if (best_nr_diffs > 0U) {
printk("Selecting best matching firmware (%u bits differ) for " printk(KERN_WARNING
"Selecting best matching firmware (%u bits differ) for "
"type=(%x), id %016llx:\n", "type=(%x), id %016llx:\n",
best_nr_diffs, type, (unsigned long long)*id); best_nr_diffs, type, (unsigned long long)*id);
i = best_i; i = best_i;
...@@ -677,10 +678,10 @@ static int seek_firmware(struct dvb_frontend *fe, unsigned int type, ...@@ -677,10 +678,10 @@ static int seek_firmware(struct dvb_frontend *fe, unsigned int type,
ret: ret:
if (debug) { if (debug) {
printk("%s firmware for type=", printk(KERN_DEBUG "%s firmware for type=",
(i < 0) ? "Can't find" : "Found"); (i < 0) ? "Can't find" : "Found");
dump_firm_type(type); dump_firm_type(type);
printk("(%x), id %016llx.\n", type, (unsigned long long)*id); printk(KERN_DEBUG "(%x), id %016llx.\n", type, (unsigned long long)*id);
} }
return i; return i;
} }
...@@ -723,13 +724,13 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -723,13 +724,13 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
else else
fname = XC4000_DEFAULT_FIRMWARE; fname = XC4000_DEFAULT_FIRMWARE;
printk("Reading firmware %s\n", fname); dprintk(1, "Reading firmware %s\n", fname);
rc = request_firmware(&fw, fname, priv->i2c_props.adap->dev.parent); rc = request_firmware(&fw, fname, priv->i2c_props.adap->dev.parent);
if (rc < 0) { if (rc < 0) {
if (rc == -ENOENT) if (rc == -ENOENT)
printk("Error: firmware %s not found.\n", fname); printk(KERN_ERR "Error: firmware %s not found.\n", fname);
else else
printk("Error %d while requesting firmware %s \n", printk(KERN_ERR "Error %d while requesting firmware %s\n",
rc, fname); rc, fname);
return rc; return rc;
...@@ -738,7 +739,8 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -738,7 +739,8 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
endp = p + fw->size; endp = p + fw->size;
if (fw->size < sizeof(name) - 1 + 2 + 2) { if (fw->size < sizeof(name) - 1 + 2 + 2) {
printk("Error: firmware file %s has invalid size!\n", fname); printk(KERN_ERR "Error: firmware file %s has invalid size!\n",
fname);
goto corrupt; goto corrupt;
} }
...@@ -758,9 +760,9 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -758,9 +760,9 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
priv->firm = kzalloc(sizeof(*priv->firm) * n_array, GFP_KERNEL); priv->firm = kzalloc(sizeof(*priv->firm) * n_array, GFP_KERNEL);
if (priv->firm == NULL) { if (priv->firm == NULL) {
printk("Not enough memory to load firmware file.\n"); printk(KERN_ERR "Not enough memory to load firmware file.\n");
rc = -ENOMEM; rc = -ENOMEM;
goto err; goto done;
} }
priv->firm_size = n_array; priv->firm_size = n_array;
...@@ -772,7 +774,7 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -772,7 +774,7 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
n++; n++;
if (n >= n_array) { if (n >= n_array) {
printk("More firmware images in file than " printk(KERN_ERR "More firmware images in file than "
"were expected!\n"); "were expected!\n");
goto corrupt; goto corrupt;
} }
...@@ -798,8 +800,7 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -798,8 +800,7 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
p += sizeof(size); p += sizeof(size);
if (!size || size > endp - p) { if (!size || size > endp - p) {
printk("Firmware type (%x), id %llx is corrupted " printk(KERN_ERR "Firmware type (%x), id %llx is corrupted (size=%d, expected %d)\n",
"(size=%d, expected %d)\n",
type, (unsigned long long)id, type, (unsigned long long)id,
(unsigned)(endp - p), size); (unsigned)(endp - p), size);
goto corrupt; goto corrupt;
...@@ -807,15 +808,15 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -807,15 +808,15 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
priv->firm[n].ptr = kzalloc(size, GFP_KERNEL); priv->firm[n].ptr = kzalloc(size, GFP_KERNEL);
if (priv->firm[n].ptr == NULL) { if (priv->firm[n].ptr == NULL) {
printk("Not enough memory to load firmware file.\n"); printk(KERN_ERR "Not enough memory to load firmware file.\n");
rc = -ENOMEM; rc = -ENOMEM;
goto err; goto done;
} }
if (debug) { if (debug) {
printk("Reading firmware type "); printk(KERN_DEBUG "Reading firmware type ");
dump_firm_type_and_int_freq(type, int_freq); dump_firm_type_and_int_freq(type, int_freq);
printk("(%x), id %llx, size=%d.\n", printk(KERN_DEBUG "(%x), id %llx, size=%d.\n",
type, (unsigned long long)id, size); type, (unsigned long long)id, size);
} }
...@@ -829,20 +830,17 @@ static int xc4000_fwupload(struct dvb_frontend *fe) ...@@ -829,20 +830,17 @@ static int xc4000_fwupload(struct dvb_frontend *fe)
} }
if (n + 1 != priv->firm_size) { if (n + 1 != priv->firm_size) {
printk("Firmware file is incomplete!\n"); printk(KERN_ERR "Firmware file is incomplete!\n");
goto corrupt; goto corrupt;
} }
goto done; goto done;
header: header:
printk("Firmware header is incomplete!\n"); printk(KERN_ERR "Firmware header is incomplete!\n");
corrupt: corrupt:
rc = -EINVAL; rc = -EINVAL;
printk("Error: firmware file is corrupted!\n"); printk(KERN_ERR "Error: firmware file is corrupted!\n");
err:
printk("Releasing partially loaded firmware file.\n");
done: done:
release_firmware(fw); release_firmware(fw);
...@@ -883,11 +881,13 @@ static int load_scode(struct dvb_frontend *fe, unsigned int type, ...@@ -883,11 +881,13 @@ static int load_scode(struct dvb_frontend *fe, unsigned int type,
return -EINVAL; return -EINVAL;
p += 12 * scode; p += 12 * scode;
if (debug) {
tuner_info("Loading SCODE for type="); tuner_info("Loading SCODE for type=");
dump_firm_type_and_int_freq(priv->firm[pos].type, dump_firm_type_and_int_freq(priv->firm[pos].type,
priv->firm[pos].int_freq); priv->firm[pos].int_freq);
printk("(%x), id %016llx.\n", priv->firm[pos].type, printk(KERN_CONT "(%x), id %016llx.\n", priv->firm[pos].type,
(unsigned long long)*id); (unsigned long long)*id);
}
scode_buf[0] = 0x00; scode_buf[0] = 0x00;
memcpy(&scode_buf[1], p, 12); memcpy(&scode_buf[1], p, 12);
...@@ -895,7 +895,7 @@ static int load_scode(struct dvb_frontend *fe, unsigned int type, ...@@ -895,7 +895,7 @@ static int load_scode(struct dvb_frontend *fe, unsigned int type,
/* Enter direct-mode */ /* Enter direct-mode */
rc = xc_write_reg(priv, XREG_DIRECTSITTING_MODE, 0); rc = xc_write_reg(priv, XREG_DIRECTSITTING_MODE, 0);
if (rc < 0) { if (rc < 0) {
printk("failed to put device into direct mode!\n"); printk(KERN_ERR "failed to put device into direct mode!\n");
return -EIO; return -EIO;
} }
...@@ -903,7 +903,7 @@ static int load_scode(struct dvb_frontend *fe, unsigned int type, ...@@ -903,7 +903,7 @@ static int load_scode(struct dvb_frontend *fe, unsigned int type,
if (rc != 0) { if (rc != 0) {
/* Even if the send failed, make sure we set back to indirect /* Even if the send failed, make sure we set back to indirect
mode */ mode */
printk("Failed to set scode %d\n", rc); printk(KERN_ERR "Failed to set scode %d\n", rc);
} }
/* Switch back to indirect-mode */ /* Switch back to indirect-mode */
...@@ -944,7 +944,7 @@ static int check_firmware(struct dvb_frontend *fe, unsigned int type, ...@@ -944,7 +944,7 @@ static int check_firmware(struct dvb_frontend *fe, unsigned int type,
dprintk(1, "checking firmware, user requested type="); dprintk(1, "checking firmware, user requested type=");
if (debug) { if (debug) {
dump_firm_type(new_fw.type); dump_firm_type(new_fw.type);
printk("(%x), id %016llx, ", new_fw.type, printk(KERN_CONT "(%x), id %016llx, ", new_fw.type,
(unsigned long long)new_fw.std_req); (unsigned long long)new_fw.std_req);
if (!int_freq) if (!int_freq)
printk(KERN_CONT "scode_tbl "); printk(KERN_CONT "scode_tbl ");
...@@ -971,7 +971,7 @@ static int check_firmware(struct dvb_frontend *fe, unsigned int type, ...@@ -971,7 +971,7 @@ static int check_firmware(struct dvb_frontend *fe, unsigned int type,
std0 = 0; std0 = 0;
rc = load_firmware(fe, BASE, &std0); rc = load_firmware(fe, BASE, &std0);
if (rc < 0) { if (rc < 0) {
printk("Error %d while loading base firmware\n", rc); printk(KERN_ERR "Error %d while loading base firmware\n", rc);
goto fail; goto fail;
} }
...@@ -1025,7 +1025,7 @@ static int check_firmware(struct dvb_frontend *fe, unsigned int type, ...@@ -1025,7 +1025,7 @@ static int check_firmware(struct dvb_frontend *fe, unsigned int type,
if (xc_get_version(priv, &hw_major, &hw_minor, &fw_major, if (xc_get_version(priv, &hw_major, &hw_minor, &fw_major,
&fw_minor) != 0) { &fw_minor) != 0) {
printk("Unable to read tuner registers.\n"); printk(KERN_ERR "Unable to read tuner registers.\n");
goto fail; goto fail;
} }
...@@ -1340,11 +1340,10 @@ static int xc4000_set_analog_params(struct dvb_frontend *fe, ...@@ -1340,11 +1340,10 @@ static int xc4000_set_analog_params(struct dvb_frontend *fe,
if (params->std & V4L2_STD_PAL_I) { if (params->std & V4L2_STD_PAL_I) {
/* default to NICAM audio standard */ /* default to NICAM audio standard */
params->std = V4L2_STD_PAL_I | V4L2_STD_NICAM; params->std = V4L2_STD_PAL_I | V4L2_STD_NICAM;
if (audio_std & XC4000_AUDIO_STD_MONO) { if (audio_std & XC4000_AUDIO_STD_MONO)
priv->video_standard = XC4000_I_PAL_NICAM_MONO; priv->video_standard = XC4000_I_PAL_NICAM_MONO;
} else { else
priv->video_standard = XC4000_I_PAL_NICAM; priv->video_standard = XC4000_I_PAL_NICAM;
}
goto tune_channel; goto tune_channel;
} }
......
drivers/media/common/tuners/xc4000.h
View file @ 941830c9
...@@ -50,8 +50,7 @@ struct xc4000_config { ...@@ -50,8 +50,7 @@ struct xc4000_config {
* it's passed back to a bridge during tuner_callback(). * it's passed back to a bridge during tuner_callback().
*/ */
#if defined(CONFIG_MEDIA_TUNER_XC4000) || \ #if defined(CONFIG_MEDIA_TUNER_XC4000) || (defined(CONFIG_MEDIA_TUNER_XC4000_MODULE) && defined(MODULE))
(defined(CONFIG_MEDIA_TUNER_XC4000_MODULE) && defined(MODULE))
extern struct dvb_frontend *xc4000_attach(struct dvb_frontend *fe, extern struct dvb_frontend *xc4000_attach(struct dvb_frontend *fe,
struct i2c_adapter *i2c, struct i2c_adapter *i2c,
struct xc4000_config *cfg); struct xc4000_config *cfg);
......
drivers/media/dvb/dvb-usb/dib0700_devices.c
View file @ 941830c9
...@@ -2794,7 +2794,7 @@ static int xc4000_tuner_attach(struct dvb_usb_adapter *adap) ...@@ -2794,7 +2794,7 @@ static int xc4000_tuner_attach(struct dvb_usb_adapter *adap)
tun_i2c = dib7000p_get_i2c_master(adap->fe, tun_i2c = dib7000p_get_i2c_master(adap->fe,
DIBX000_I2C_INTERFACE_TUNER, 1); DIBX000_I2C_INTERFACE_TUNER, 1);
if (tun_i2c == NULL) { if (tun_i2c == NULL) {
printk("Could not reach tuner i2c bus\n"); printk(KERN_ERR "Could not reach tuner i2c bus\n");
return 0; return 0;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7