Staging: cx25821: fix coding style issues
PATCH 1/1 Staging: cx25821: fix coding style issues
Fix too long lines in cx25821-audio.h and cx25821-core.c Fix wrong brace placement it cx25821-cards.c, cx25821-core.c, and cx25821-i2c.c
Use DEFINE_PCI_DEVICE_TABLE for cx25821_pci_tbl.
Move EXPORT_SYMBOL(cx25821_set_gpiopin_direction) to the right place. Delete file cx25821-gpio.h as it is not used.
Get rid of typedef in cx25821.h.
--- drivers/staging/cx25821/cx25821-audio.h | 6 ++++-- drivers/staging/cx25821/cx25821-cards.c | 2 +- drivers/staging/cx25821/cx25821-core.c | 15 +++++++-------- drivers/staging/cx25821/cx25821-gpio.c | 1 + drivers/staging/cx25821/cx25821-gpio.h | 2 -- drivers/staging/cx25821/cx25821-i2c.c | 6 +++--- drivers/staging/cx25821/cx25821.h | 8 +++++--- 7 files changed, 21 insertions(+), 19 deletions(-) delete mode 100644 drivers/staging/cx25821/cx25821-gpio.h
drivers/staging/cx25821/cx25821-audio.h +4 -2
@@ -36,13 +36,15 @@*/ #ifndef USE_RISC_NOOP #define MAX_BUFFER_PROGRAM_SIZE \ - (2*LINES_PER_BUFFER*RISC_WRITE_INSTRUCTION_SIZE + RISC_WRITECR_INSTRUCTION_SIZE*4) + (2*LINES_PER_BUFFER*RISC_WRITE_INSTRUCTION_SIZE + \ + RISC_WRITECR_INSTRUCTION_SIZE*4)#endif /* MAE 12 July 2005 Try to use NOOP RISC instruction instead */ #ifdef USE_RISC_NOOP #define MAX_BUFFER_PROGRAM_SIZE \ - (2*LINES_PER_BUFFER*RISC_WRITE_INSTRUCTION_SIZE + RISC_NOOP_INSTRUCTION_SIZE*4) + (2*LINES_PER_BUFFER*RISC_WRITE_INSTRUCTION_SIZE + \ + RISC_NOOP_INSTRUCTION_SIZE*4)#endif /* Sizes of various instructions in bytes. Used when adding instructions. */
drivers/staging/cx25821/cx25821-cards.c +1 -1
@@ -57,7 +57,7 @@ struct cx25821_subid cx25821_subids[] = { .subvendor = 0x14f1, .subdevice = 0x0920, .card = CX25821_BOARD, - }, + }, };void cx25821_card_setup (struct cx25821_dev *dev)
drivers/staging/cx25821/cx25821-core.c +7 -8
@@ -972,8 +972,8 @@ static int cx25821_dev_setup(struct cx25821_dev *dev) dev->lmmio =ioremap (dev->base_io_addr,pci_resource_len (dev->pci,0 ));if (!dev->lmmio) { -CX25821_ERR - ("ioremap failed, maybe increasing __VMALLOC_RESERVE in page.h\n" ); +CX25821_ERR ("ioremap failed, maybe increasing __VMALLOC_RESERVE" +" in page.h\n" );cx25821_iounmap (dev);return -ENOMEM ; }
@@ -1011,8 +1011,8 @@ static int cx25821_dev_setup(struct cx25821_dev *dev)if (video_register_device (dev->ioctl_dev,VFL_TYPE_GRABBER ,VIDEO_IOCTL_CH ) <0 ) {cx25821_videoioctl_unregister (dev); -pr_err ("%s(): Failed to register video adapter for IOCTL, so unregistering videoioctl device\n" , - __func__); +pr_err ("%s(): Failed to register video adapter for IOCTL, so " +"unregistering videoioctl device\n" , __func__); }cx25821_dev_checkrevision (dev);
@@ -1473,17 +1473,17 @@ static void __devexit cx25821_finidev(struct pci_dev *pci_dev)kfree (dev); } -static struct pci_device_id cx25821_pci_tbl[] = { +static DEFINE_PCI_DEVICE_TABLE (cx25821_pci_tbl)__used = { {/* CX25821 Athena */ .vendor =0x14f1 , .device =0x8210 , .subvendor =0x14f1 , .subdevice =0x0920 , - }, + }, {/* --- end of list --- */ - } + } };MODULE_DEVICE_TABLE (pci, cx25821_pci_tbl);
@@ -1512,7 +1512,6 @@ static void __exit cx25821_fini(void)pci_unregister_driver (&cx25821_pci_driver); } -EXPORT_SYMBOL (cx25821_set_gpiopin_direction);module_init (cx25821_init);module_exit (cx25821_fini);
drivers/staging/cx25821/cx25821-gpio.c +1 -0
@@ -50,6 +50,7 @@ void cx25821_set_gpiopin_direction(struct cx25821_dev *dev,cx_write (gpio_oe_reg, value); } +EXPORT_SYMBOL (cx25821_set_gpiopin_direction);static void cx25821_set_gpiopin_logicvalue (struct cx25821_dev *dev,int pin_number,int pin_logic_value)
drivers/staging/cx25821/cx25821-gpio.h deleted +0 -2
@@ -1,2 +0,0 @@ - -void cx25821_gpio_init (struct athena_dev *dev);
drivers/staging/cx25821/cx25821-i2c.c +3 -3
@@ -374,12 +374,12 @@ int cx25821_i2c_read(struct cx25821_i2c *bus, u16 reg_addr, int *value) .flags = 0, .len = 2, .buf = addr, - }, { + }, { .addr = client->addr, .flags = I2C_M_RD, .len = 4, .buf = buf, - } + } }; addr[0 ] = (reg_addr >>8 );
@@ -407,7 +407,7 @@ int cx25821_i2c_write(struct cx25821_i2c *bus, u16 reg_addr, int value) .flags = 0, .len = 6, .buf = buf, - } + } }; buf[0 ] = reg_addr >>8 ;
drivers/staging/cx25821/cx25821.h +5 -3
@@ -179,15 +179,17 @@ struct cx25821_input {u32 gpio0, gpio1, gpio2, gpio3; }; -typedef enum { +enum port {CX25821_UNDEFINED =0 ,CX25821_RAW ,CX25821_264 -}port_t ; +};struct cx25821_board {char *name; -port_t porta, portb, portc; +enum port porta; +enum port portb; +enum port portc;unsigned int tuner_type;unsigned int radio_type;unsigned char tuner_addr;
Signature · 1 line
1.7.0.4
Quoted 1 line · Leonid V. Fedorenchik
Fix too long lines in
cx25821-audio.handcx25821-core.c
[]
drivers/staging/cx25821/cx25821-core.c
Quoted 1 line
drivers/staging/cx25821/cx25821-core.c +0
-0
[]
drivers/staging/cx25821/cx25821-core.c static int cx25821_dev_setup(struct cx25821_dev *dev) · line 976
Quoted 8 lines
drivers/staging/cx25821/cx25821-core.c +2
-2@@ -972,8 +972,8 @@ static int cx25821_dev_setup(struct cx25821_dev *dev) dev->lmmio =ioremap (dev->base_io_addr,pci_resource_len (dev->pci,0 ));if (!dev->lmmio) { -CX25821_ERR - ("ioremap failed, maybe increasing __VMALLOC_RESERVE in page.h\n" ); +CX25821_ERR ("ioremap failed, maybe increasing __VMALLOC_RESERVE" +" in page.h\n" );
Please do not split formats.
drivers/staging/cx25821/cx25821-core.c static int cx25821_dev_setup(struct cx25821_dev *dev) · line 1015
Quoted 11 lines
drivers/staging/cx25821/cx25821-core.c +2
-2cx25821_iounmap (dev);return -ENOMEM ; }@@ -1011,8 +1011,8 @@ static int cx25821_dev_setup(struct cx25821_dev *dev)if (video_register_device (dev->ioctl_dev,VFL_TYPE_GRABBER ,VIDEO_IOCTL_CH ) <0 ) {cx25821_videoioctl_unregister (dev); -pr_err ("%s(): Failed to register video adapter for IOCTL, so unregistering videoioctl device\n" , - __func__); +pr_err ("%s(): Failed to register video adapter for IOCTL, so " +"unregistering videoioctl device\n" , __func__);
This was better before the change.
drivers/staging/cx25821/cx25821-core.c static void __devexit cx25821_finidev(struct pci_dev *pci_dev) · line 1476
Quoted 2 lines
drivers/staging/cx25821/cx25821-core.c +1
-1-static struct pci_device_id cx25821_pci_tbl[] = { +static DEFINE_PCI_DEVICE_TABLE (cx25821_pci_tbl)__used = {
__used? Hard to believe that's necessary here.
Thanks for the review. I sent v2. I have a question, however...
Quoted 8 lines · On Mon, 01 Aug 2011 09:14:22 -0700 Joe Perches <[email protected]> wrote:
Quoted 1 line · Leonid V. Fedorenchik
Fix too long lines in
cx25821-audio.handcx25821-core.c[]
drivers/staging/cx25821/cx25821-core.c
Quoted 1 line
drivers/staging/cx25821/cx25821-core.c +0
-0[]
Quoted 3 lines
drivers/staging/cx25821/cx25821-core.c +0
-0@@ -972,8 +972,8 @@ static int cx25821_dev_setup(struct cx25821_dev *dev) dev->lmmio =ioremap (dev->base_io_addr,pci_resource_len (dev->pci,0 ));
Quoted 7 lines
drivers/staging/cx25821/cx25821-core.c static int cx25821_dev_setup(struct cx25821_dev *dev) · line 976
Quoted 5 lines
drivers/staging/cx25821/cx25821-core.c +2
-2if (!dev->lmmio) { -CX25821_ERR - ("ioremap failed, maybe increasing __VMALLOC_RESERVE in page.h\n" ); +CX25821_ERR ("ioremap failed, maybe increasing __VMALLOC_RESERVE" +" in page.h\n" );Please do not split formats.
What is the reason to not split formats?
I can see that in Documentation/CodingStyle in Chapter 2 it is OK to split formats if it is not making readability worse.
So, is it for the sake of readability or is it new coding style rule for the kernel that is not in documentation yet or something else or am I misunderstood something?
Leonid V. Fedorenchik
Em 02-08-2011 06:18, Leonid V. Fedorenchik escreveu:
Quoted 21 lines
Thanks for the review. I sent v2. I have a question, however...
Quoted 8 lines · On Mon, 01 Aug 2011 09:14:22 -0700 Joe Perches <[email protected]> wrote:
Quoted 1 line · Leonid V. Fedorenchik
Fix too long lines in
cx25821-audio.handcx25821-core.c[]
drivers/staging/cx25821/cx25821-core.c
Quoted 1 line
drivers/staging/cx25821/cx25821-core.c +0
-0[]
Quoted 3 lines
drivers/staging/cx25821/cx25821-core.c +0
-0@@ -972,8 +972,8 @@ static int cx25821_dev_setup(struct cx25821_dev *dev) dev->lmmio =ioremap (dev->base_io_addr,pci_resource_len (dev->pci,0 ));Quoted 7 lines
drivers/staging/cx25821/cx25821-core.c static int cx25821_dev_setup(struct cx25821_dev *dev) · line 976
Quoted 5 lines
drivers/staging/cx25821/cx25821-core.c +2
-2if (!dev->lmmio) { -CX25821_ERR - ("ioremap failed, maybe increasing __VMALLOC_RESERVE in page.h\n" ); +CX25821_ERR ("ioremap failed, maybe increasing __VMALLOC_RESERVE" +" in page.h\n" );Please do not split formats.
Quoted 6 lines
What is the reason to not split formats?
I can see that inDocumentation/CodingStylein Chapter 2 it is OK to split formats if it is not making readability worse.
So, is it for the sake of readability or is it new coding style rule for the kernel that is not in documentation yet or something else or am I misunderstood something?
Breaking long printk lines break things like:
git grep "__VMALLOC_RESERVE in page.h"
Cheers,
Mauro