On Wed, Nov 16, 2016 at 02:42:41PM -0200, Mauro Carvalho Chehab wrote:
> This driver is old, and have lots of checkpatch violations.
> As we're touching a lot on this driver due to the printk
> conversions, let's run checkpatch --fix on it, in order to
> solve some of those issues. Also, let's remove the FSF
> address and use the usual coding style for the initial comments.
Good idea to give checkpatch a run.
Good job by checkpatch, really powerful tool.
Have proofread, no weirdness except for few places where vertical
"table-alike" alignment across lines got broken.
>
> Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>
Reviewed-by: Andrey Utkin <andrey_ut...@fastmail.com>
> --- a/drivers/media/pci/cx88/cx88-cards.c
> +++ b/drivers/media/pci/cx88/cx88-cards.c
> @@ -2911,33 +2906,33 @@ static const struct {
> int fm;
> const char *name;
> } gdi_tuner[] = {
> - [ 0x01 ] = { .id = UNSET,
> + [0x01] = { .id = UNSET,
> .name = "NTSC_M" },
Alignment got broken
> --- a/drivers/media/pci/cx88/cx88-vbi.c
> +++ b/drivers/media/pci/cx88/cx88-vbi.c
> @@ -57,9 +57,9 @@ static int cx8800_start_vbi_dma(struct cx8800_dev *dev,
> cx88_sram_channel_setup(dev->core, &cx88_sram_channels[SRAM_CH24],
> VBI_LINE_LENGTH, buf->risc.dma);
>
> - cx_write(MO_VBOS_CONTROL, ( (1 << 18) | // comb filter delay fixup
> + cx_write(MO_VBOS_CONTROL, ((1 << 18) | // comb filter delay fixup
Alignment got broken.
> --- a/drivers/media/pci/cx88/cx88.h
> +++ b/drivers/media/pci/cx88/cx88.h
> @@ -385,8 +381,8 @@ struct cx88_core {
> /* state info */
> struct task_struct *kthread;
> v4l2_std_id tvnorm;
> - unsigned width, height;
> - unsigned field;
> + unsigned int width, height;
> + unsigned int field;
Alignment got broken
> @@ -591,23 +587,23 @@ struct cx8802_dev {
> /* ----------------------------------------------------------- */
>
> #define cx_read(reg) readl(core->lmmio + ((reg)>>2))
> -#define cx_write(reg,value) writel((value), core->lmmio + ((reg)>>2))
> -#define cx_writeb(reg,value) writeb((value), core->bmmio + (reg))
> +#define cx_write(reg, value) writel((value), core->lmmio + ((reg)>>2))
> +#define cx_writeb(reg, value) writeb((value), core->bmmio + (reg))
Alignment got broken
>
> -#define cx_andor(reg,mask,value) \
> +#define cx_andor(reg, mask, value) \
> writel((readl(core->lmmio+((reg)>>2)) & ~(mask)) |\
> ((value) & (mask)), core->lmmio+((reg)>>2))
> -#define cx_set(reg,bit) cx_andor((reg),(bit),(bit))
> -#define cx_clear(reg,bit) cx_andor((reg),(bit),0)
> +#define cx_set(reg, bit) cx_andor((reg), (bit), (bit))
> +#define cx_clear(reg, bit) cx_andor((reg), (bit), 0)
Alignment got broken
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
