* [PATCH v2] Staging: cx25821: fix coding style issues
@ 2011-08-02 9:03 Leonid V. Fedorenchik
2011-08-23 21:44 ` Greg KH
0 siblings, 1 reply; 3+ messages in thread
From: Leonid V. Fedorenchik @ 2011-08-02 9:03 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Mauro Carvalho Chehab, Palash Bandyopadhyay, Ruslan Pisarev,
Ilya Gorskin, Joe Perches, Arnd Bergmann, linux-kernel, devel,
linux-media, Leonid V. Fedorenchik
Fix too long lines in cx25821-audio.h
Fix wrong brace placement in 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 since it is not used.
Get rid of typedef in cx25821.h.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
---
drivers/staging/cx25821/cx25821-audio.h | 6 ++++--
drivers/staging/cx25821/cx25821-cards.c | 2 +-
drivers/staging/cx25821/cx25821-core.c | 7 +++----
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, 17 insertions(+), 15 deletions(-)
delete mode 100644 drivers/staging/cx25821/cx25821-gpio.h
diff --git a/drivers/staging/cx25821/cx25821-audio.h b/drivers/staging/cx25821/cx25821-audio.h
index 2771725..a2098fb 100644
--- a/drivers/staging/cx25821/cx25821-audio.h
+++ b/drivers/staging/cx25821/cx25821-audio.h
@@ -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. */
diff --git a/drivers/staging/cx25821/cx25821-cards.c b/drivers/staging/cx25821/cx25821-cards.c
index 94e8d68..9f7febd 100644
--- a/drivers/staging/cx25821/cx25821-cards.c
+++ b/drivers/staging/cx25821/cx25821-cards.c
@@ -57,7 +57,7 @@ struct cx25821_subid cx25821_subids[] = {
.subvendor = 0x14f1,
.subdevice = 0x0920,
.card = CX25821_BOARD,
- },
+ },
};
void cx25821_card_setup(struct cx25821_dev *dev)
diff --git a/drivers/staging/cx25821/cx25821-core.c b/drivers/staging/cx25821/cx25821-core.c
index 523ac5e..f06c671 100644
--- a/drivers/staging/cx25821/cx25821-core.c
+++ b/drivers/staging/cx25821/cx25821-core.c
@@ -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) = {
{
/* 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);
diff --git a/drivers/staging/cx25821/cx25821-gpio.c b/drivers/staging/cx25821/cx25821-gpio.c
index 2f154b3..29e43b0 100644
--- a/drivers/staging/cx25821/cx25821-gpio.c
+++ b/drivers/staging/cx25821/cx25821-gpio.c
@@ -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)
diff --git a/drivers/staging/cx25821/cx25821-gpio.h b/drivers/staging/cx25821/cx25821-gpio.h
deleted file mode 100644
index ca07644..0000000
--- a/drivers/staging/cx25821/cx25821-gpio.h
+++ /dev/null
@@ -1,2 +0,0 @@
-
-void cx25821_gpio_init(struct athena_dev *dev);
diff --git a/drivers/staging/cx25821/cx25821-i2c.c b/drivers/staging/cx25821/cx25821-i2c.c
index 130dfeb..ea5a878 100644
--- a/drivers/staging/cx25821/cx25821-i2c.c
+++ b/drivers/staging/cx25821/cx25821-i2c.c
@@ -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;
diff --git a/drivers/staging/cx25821/cx25821.h b/drivers/staging/cx25821/cx25821.h
index 6230243..a282592 100644
--- a/drivers/staging/cx25821/cx25821.h
+++ b/drivers/staging/cx25821/cx25821.h
@@ -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;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] Staging: cx25821: fix coding style issues
2011-08-02 9:03 [PATCH v2] Staging: cx25821: fix coding style issues Leonid V. Fedorenchik
@ 2011-08-23 21:44 ` Greg KH
2011-08-24 3:14 ` Leonid V. Fedorenchik
0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2011-08-23 21:44 UTC (permalink / raw)
To: Leonid V. Fedorenchik
Cc: Greg Kroah-Hartman, Mauro Carvalho Chehab, Palash Bandyopadhyay,
Ruslan Pisarev, Ilya Gorskin, Joe Perches, Arnd Bergmann,
linux-kernel, devel, linux-media
On Tue, Aug 02, 2011 at 05:03:18PM +0800, Leonid V. Fedorenchik wrote:
> Fix too long lines in cx25821-audio.h
> Fix wrong brace placement in 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 since it is not used.
> Get rid of typedef in cx25821.h.
I prefer to get patches in a "one patch per thing you did" format.
So, care to split this up into a number of smaller patches, all doing
the individual thing you list above?
Yes, it's going to be a bunch of small patches, but then they will be
"obviously" correct and trivial to review and apply.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] Staging: cx25821: fix coding style issues
2011-08-23 21:44 ` Greg KH
@ 2011-08-24 3:14 ` Leonid V. Fedorenchik
0 siblings, 0 replies; 3+ messages in thread
From: Leonid V. Fedorenchik @ 2011-08-24 3:14 UTC (permalink / raw)
To: Greg KH
Cc: Greg Kroah-Hartman, Mauro Carvalho Chehab, Palash Bandyopadhyay,
Ruslan Pisarev, Ilya Gorskin, Joe Perches, Arnd Bergmann,
linux-kernel, devel, linux-media
On Tue, 23 Aug 2011 14:44:21 -0700
Greg KH <greg@kroah.com> wrote:
> On Tue, Aug 02, 2011 at 05:03:18PM +0800, Leonid V. Fedorenchik wrote:
> > Fix too long lines in cx25821-audio.h
> > Fix wrong brace placement in 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 since it is not used.
> > Get rid of typedef in cx25821.h.
>
> I prefer to get patches in a "one patch per thing you did" format.
>
> So, care to split this up into a number of smaller patches, all doing
> the individual thing you list above?
>
> Yes, it's going to be a bunch of small patches, but then they will be
> "obviously" correct and trivial to review and apply.
>
> thanks,
>
> greg k-h
OK, no problem. I'll do this.
Leonid V. Fedorenchik
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-08-24 3:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-02 9:03 [PATCH v2] Staging: cx25821: fix coding style issues Leonid V. Fedorenchik
2011-08-23 21:44 ` Greg KH
2011-08-24 3:14 ` Leonid V. Fedorenchik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox