Em 10-06-2010 01:25, Palash Bandyopadhyay escreveu:
> From 53df9742b92902b5fa9d28b2dcc949cb495725a5 Mon Sep 17 00:00:00 2001
> Message-Id: <53df9742b92902b5fa9d28b2dcc949cb495725a5.1276143429.git.palash.bandyopadhyay@xxxxxxxxxxxx>
> From: palash <palash.bandyopadhyay@xxxxxxxxxxxx>
> Date: Wed, 9 Jun 2010 21:13:17 -0700
> Subject: [cx231xx 1/2] Added support for s5h1432 demod
> To: linux-media@xxxxxxxxxxxxxxx
>
> Signed-off-by: palash <palash.bandyopadhyay@xxxxxxxxxxxx>
There are lots of CodingStyle issues that need fixes. Even fixing the bad whitespacing
with a script, there are still lots of other issues, as pointed by scripts/checkpatch.pl:
WARNING: please write a paragraph that describes the config symbol fully
#60: FILE: drivers/media/dvb/frontends/Kconfig:264:
+ help
WARNING: suspect code indent for conditional statements (7, 15)
#130: FILE: drivers/media/dvb/frontends/s5h1432.c:52:
+ if (debug) \
+ printk(arg); \
WARNING: suspect code indent for conditional statements (7, 15)
#145: FILE: drivers/media/dvb/frontends/s5h1432.c:67:
+ if (ret != 1)
+ printk(KERN_ERR "%s: writereg error 0x%02x 0x%02x 0x%04x, "
WARNING: suspect code indent for conditional statements (7, 15)
#164: FILE: drivers/media/dvb/frontends/s5h1432.c:86:
+ if (ret != 2)
+ printk(KERN_ERR "%s: readreg error (ret == %i)\n",
WARNING: unnecessary whitespace before a quoted newline
#256: FILE: drivers/media/dvb/frontends/s5h1432.c:178:
+ printk(KERN_INFO "Default IFFreq %d :reg value = 0x%x \n",
WARNING: suspect code indent for conditional statements (7, 15)
#279: FILE: drivers/media/dvb/frontends/s5h1432.c:201:
+ if (p->frequency == state->current_frequency) {
+ /*current_frequency = p->frequency;*/
WARNING: suspect code indent for conditional statements (7, 15)
#396: FILE: drivers/media/dvb/frontends/s5h1432.c:318:
+ for (i = 0; i < 0xFF; i++) {
+ reg = s5h1432_readreg(state, S5H1432_I2C_TOP_ADDR, i);
WARNING: suspect code indent for conditional statements (7, 15)
#463: FILE: drivers/media/dvb/frontends/s5h1432.c:385:
+ if (state == NULL)
+ goto error;
total: 0 errors, 8 warnings, 562 lines checked
patches/lmml_108408_cx231xx_1_2_added_support_for_s5h1432_demod.patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Please fix.
Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
