2013-03-20 12:10:29

by Laurent Navet

[permalink] [raw]
Subject: GPIO - Fix checkpatch errors

This series fix almost all checkpatch errors in drivers/gpio/

drivers/gpio/gpio-mvebu.c | 26 +++++++++++++-------------
drivers/gpio/gpio-omap.c | 2 +-
drivers/gpio/gpio-pca953x.c | 3 +--
drivers/gpio/gpio-pxa.c | 4 ++--
drivers/gpio/gpio-sch.c | 74 +++++++++++++++++++++++++++++++++++---------------------------------------
drivers/gpio/gpio-stp-xway.c | 2 +-
drivers/gpio/gpio-tc3589x.c | 8 ++++----
drivers/gpio/gpio-timberdale.c | 3 +--
drivers/gpio/gpio-tps65910.c | 2 +-
drivers/gpio/gpiolib-of.c | 2 +-
10 files changed, 60 insertions(+), 66 deletions(-)

Regards,
Laurent Navet.


2013-03-20 12:10:33

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 01/10] gpio: gpiolib-of.c: fix checkpatch error

Fix :
gpio/gpiolib-of.c:64: ERROR: code indent should use tabs where possible

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpiolib-of.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpiolib-of.c b/drivers/gpio/gpiolib-of.c
index a71a54a..8940793 100644
--- a/drivers/gpio/gpiolib-of.c
+++ b/drivers/gpio/gpiolib-of.c
@@ -61,7 +61,7 @@ static int of_gpiochip_find_and_xlate(struct gpio_chip *gc, void *data)
* in flags for the GPIO.
*/
int of_get_named_gpio_flags(struct device_node *np, const char *propname,
- int index, enum of_gpio_flags *flags)
+ int index, enum of_gpio_flags *flags)
{
/* Return -EPROBE_DEFER to support probe() functions to be called
* later when the GPIO actually becomes available
--
1.7.10.4

2013-03-20 12:10:37

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 02/10] gpio: gpio-mvebu.c: fix checkpatch errors

Fix :
gpio/gpio-mvebu.c:120: ERROR: space required before the open parenthesis '('
gpio/gpio-mvebu.c:136: ERROR: space required before the open parenthesis '('
gpio/gpio-mvebu.c:154: ERROR: space required before the open parenthesis '('
gpio/gpio-mvebu.c:404: ERROR: space required before the open parenthesis '('
gpio/gpio-mvebu.c:476: ERROR: "(foo*)" should be "(foo *)"
gpio/gpio-mvebu.c:480: ERROR: "(foo*)" should be "(foo *)"
gpio/gpio-mvebu.c:484: ERROR: "(foo*)" should be "(foo *)"
gpio/gpio-mvebu.c:512: ERROR: space prohibited after that '!' (ctx:BxW)
gpio/gpio-mvebu.c:518: ERROR: space prohibited after that '!' (ctx:BxW)
gpio/gpio-mvebu.c:518: ERROR: space required before the open brace '{'
gpio/gpio-mvebu.c:563: ERROR: space prohibited after that '!' (ctx:BxW)
gpio/gpio-mvebu.c:570: ERROR: trailing whitespace
gpio/gpio-mvebu.c:577: ERROR: space required before the open parenthesis '('
gpio/gpio-mvebu.c:635: ERROR: space prohibited after that '!' (ctx:BxW)

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-mvebu.c | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/drivers/gpio/gpio-mvebu.c b/drivers/gpio/gpio-mvebu.c
index 61a6fde..ca6d4ac 100644
--- a/drivers/gpio/gpio-mvebu.c
+++ b/drivers/gpio/gpio-mvebu.c
@@ -117,7 +117,7 @@ static inline void __iomem *mvebu_gpioreg_edge_cause(struct mvebu_gpio_chip *mvc
{
int cpu;

- switch(mvchip->soc_variant) {
+ switch (mvchip->soc_variant) {
case MVEBU_GPIO_SOC_VARIANT_ORION:
case MVEBU_GPIO_SOC_VARIANT_MV78200:
return mvchip->membase + GPIO_EDGE_CAUSE_OFF;
@@ -133,7 +133,7 @@ static inline void __iomem *mvebu_gpioreg_edge_mask(struct mvebu_gpio_chip *mvch
{
int cpu;

- switch(mvchip->soc_variant) {
+ switch (mvchip->soc_variant) {
case MVEBU_GPIO_SOC_VARIANT_ORION:
return mvchip->membase + GPIO_EDGE_MASK_OFF;
case MVEBU_GPIO_SOC_VARIANT_MV78200:
@@ -151,7 +151,7 @@ static void __iomem *mvebu_gpioreg_level_mask(struct mvebu_gpio_chip *mvchip)
{
int cpu;

- switch(mvchip->soc_variant) {
+ switch (mvchip->soc_variant) {
case MVEBU_GPIO_SOC_VARIANT_ORION:
return mvchip->membase + GPIO_LEVEL_MASK_OFF;
case MVEBU_GPIO_SOC_VARIANT_MV78200:
@@ -401,7 +401,7 @@ static int mvebu_gpio_irq_set_type(struct irq_data *d, unsigned int type)
/*
* Configure interrupt polarity.
*/
- switch(type) {
+ switch (type) {
case IRQ_TYPE_EDGE_RISING:
case IRQ_TYPE_LEVEL_HIGH:
u = readl_relaxed(mvebu_gpioreg_in_pol(mvchip));
@@ -473,15 +473,15 @@ static void mvebu_gpio_irq_handler(unsigned int irq, struct irq_desc *desc)
static struct of_device_id mvebu_gpio_of_match[] = {
{
.compatible = "marvell,orion-gpio",
- .data = (void*) MVEBU_GPIO_SOC_VARIANT_ORION,
+ .data = (void *) MVEBU_GPIO_SOC_VARIANT_ORION,
},
{
.compatible = "marvell,mv78200-gpio",
- .data = (void*) MVEBU_GPIO_SOC_VARIANT_MV78200,
+ .data = (void *) MVEBU_GPIO_SOC_VARIANT_MV78200,
},
{
.compatible = "marvell,armadaxp-gpio",
- .data = (void*) MVEBU_GPIO_SOC_VARIANT_ARMADAXP,
+ .data = (void *) MVEBU_GPIO_SOC_VARIANT_ARMADAXP,
},
{
/* sentinel */
@@ -509,13 +509,13 @@ static int mvebu_gpio_probe(struct platform_device *pdev)
soc_variant = MVEBU_GPIO_SOC_VARIANT_ORION;

res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- if (! res) {
+ if (!res) {
dev_err(&pdev->dev, "Cannot get memory resource\n");
return -ENODEV;
}

mvchip = devm_kzalloc(&pdev->dev, sizeof(struct mvebu_gpio_chip), GFP_KERNEL);
- if (! mvchip){
+ if (!mvchip) {
dev_err(&pdev->dev, "Cannot allocate memory\n");
return -ENOMEM;
}
@@ -560,21 +560,21 @@ static int mvebu_gpio_probe(struct platform_device *pdev)
* per-CPU registers */
if (soc_variant == MVEBU_GPIO_SOC_VARIANT_ARMADAXP) {
res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
- if (! res) {
+ if (!res) {
dev_err(&pdev->dev, "Cannot get memory resource\n");
return -ENODEV;
}

mvchip->percpu_membase = devm_ioremap_resource(&pdev->dev,
res);
- if (IS_ERR(mvchip->percpu_membase))
+ if (IS_ERR(mvchip->percpu_membase))
return PTR_ERR(mvchip->percpu_membase);
}

/*
* Mask and clear GPIO interrupts.
*/
- switch(soc_variant) {
+ switch (soc_variant) {
case MVEBU_GPIO_SOC_VARIANT_ORION:
writel_relaxed(0, mvchip->membase + GPIO_EDGE_CAUSE_OFF);
writel_relaxed(0, mvchip->membase + GPIO_EDGE_MASK_OFF);
@@ -632,7 +632,7 @@ static int mvebu_gpio_probe(struct platform_device *pdev)

gc = irq_alloc_generic_chip("mvebu_gpio_irq", 2, mvchip->irqbase,
mvchip->membase, handle_level_irq);
- if (! gc) {
+ if (!gc) {
dev_err(&pdev->dev, "Cannot allocate generic irq_chip\n");
return -ENOMEM;
}
--
1.7.10.4

2013-03-20 12:10:43

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 04/10] gpio: gpio-pca953x.c: fix checkpatch error

Fix :
gpio/gpio-pca953x.c:150: ERROR: else should follow close brace '}'

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-pca953x.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c
index 2405946..15dbc36 100644
--- a/drivers/gpio/gpio-pca953x.c
+++ b/drivers/gpio/gpio-pca953x.c
@@ -146,8 +146,7 @@ static int pca953x_write_regs(struct pca953x_chip *chip, int reg, u8 *val)
ret = i2c_smbus_write_i2c_block_data(chip->client,
(reg << bank_shift) | REG_ADDR_AI,
NBANK(chip), val);
- }
- else {
+ } else {
switch (chip->chip_type) {
case PCA953X_TYPE:
ret = i2c_smbus_write_word_data(chip->client,
--
1.7.10.4

2013-03-20 12:10:52

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 08/10] gpio: gpio-tc3589x.c: fix checkpatch errors

Fix :
gpio/gpio-tc3589x.c:285: ERROR: code indent should use tabs where possible
gpio/gpio-tc3589x.c:286: ERROR: code indent should use tabs where possible
gpio/gpio-tc3589x.c:287: ERROR: code indent should use tabs where possible
gpio/gpio-tc3589x.c:347: ERROR: code indent should use tabs where possible

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-tc3589x.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpio/gpio-tc3589x.c b/drivers/gpio/gpio-tc3589x.c
index c0595bb..d34d80d 100644
--- a/drivers/gpio/gpio-tc3589x.c
+++ b/drivers/gpio/gpio-tc3589x.c
@@ -282,9 +282,9 @@ static void tc3589x_gpio_irq_unmap(struct irq_domain *d, unsigned int virq)
}

static struct irq_domain_ops tc3589x_irq_ops = {
- .map = tc3589x_gpio_irq_map,
- .unmap = tc3589x_gpio_irq_unmap,
- .xlate = irq_domain_xlate_twocell,
+ .map = tc3589x_gpio_irq_map,
+ .unmap = tc3589x_gpio_irq_unmap,
+ .xlate = irq_domain_xlate_twocell,
};

static int tc3589x_gpio_irq_init(struct tc3589x_gpio *tc3589x_gpio,
@@ -344,7 +344,7 @@ static int tc3589x_gpio_probe(struct platform_device *pdev)
tc3589x_gpio->chip.base = (pdata) ? pdata->gpio_base : -1;

#ifdef CONFIG_OF_GPIO
- tc3589x_gpio->chip.of_node = np;
+ tc3589x_gpio->chip.of_node = np;
#endif

tc3589x_gpio->irq_base = tc3589x->irq_base ?
--
1.7.10.4

2013-03-20 12:10:48

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 06/10] gpio: gpio-sch.c: fix checkpatch error

Fix :
gpio/gpio-sch.c:206: ERROR: switch and case should be at the same indent

Also remove blank lines

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-sch.c | 74 ++++++++++++++++++++++-------------------------
1 file changed, 35 insertions(+), 39 deletions(-)

diff --git a/drivers/gpio/gpio-sch.c b/drivers/gpio/gpio-sch.c
index edae963..cb60081 100644
--- a/drivers/gpio/gpio-sch.c
+++ b/drivers/gpio/gpio-sch.c
@@ -204,45 +204,41 @@ static int sch_gpio_probe(struct platform_device *pdev)
gpio_ba = res->start;

switch (id) {
- case PCI_DEVICE_ID_INTEL_SCH_LPC:
- sch_gpio_core.base = 0;
- sch_gpio_core.ngpio = 10;
-
- sch_gpio_resume.base = 10;
- sch_gpio_resume.ngpio = 4;
-
- /*
- * GPIO[6:0] enabled by default
- * GPIO7 is configured by the CMC as SLPIOVR
- * Enable GPIO[9:8] core powered gpios explicitly
- */
- outb(0x3, gpio_ba + CGEN + 1);
- /*
- * SUS_GPIO[2:0] enabled by default
- * Enable SUS_GPIO3 resume powered gpio explicitly
- */
- outb(0x8, gpio_ba + RGEN);
- break;
-
- case PCI_DEVICE_ID_INTEL_ITC_LPC:
- sch_gpio_core.base = 0;
- sch_gpio_core.ngpio = 5;
-
- sch_gpio_resume.base = 5;
- sch_gpio_resume.ngpio = 9;
- break;
-
- case PCI_DEVICE_ID_INTEL_CENTERTON_ILB:
- sch_gpio_core.base = 0;
- sch_gpio_core.ngpio = 21;
-
- sch_gpio_resume.base = 21;
- sch_gpio_resume.ngpio = 9;
- break;
-
- default:
- err = -ENODEV;
- goto err_sch_gpio_core;
+ case PCI_DEVICE_ID_INTEL_SCH_LPC:
+ sch_gpio_core.base = 0;
+ sch_gpio_core.ngpio = 10;
+ sch_gpio_resume.base = 10;
+ sch_gpio_resume.ngpio = 4;
+ /*
+ * GPIO[6:0] enabled by default
+ * GPIO7 is configured by the CMC as SLPIOVR
+ * Enable GPIO[9:8] core powered gpios explicitly
+ */
+ outb(0x3, gpio_ba + CGEN + 1);
+ /*
+ * SUS_GPIO[2:0] enabled by default
+ * Enable SUS_GPIO3 resume powered gpio explicitly
+ */
+ outb(0x8, gpio_ba + RGEN);
+ break;
+
+ case PCI_DEVICE_ID_INTEL_ITC_LPC:
+ sch_gpio_core.base = 0;
+ sch_gpio_core.ngpio = 5;
+ sch_gpio_resume.base = 5;
+ sch_gpio_resume.ngpio = 9;
+ break;
+
+ case PCI_DEVICE_ID_INTEL_CENTERTON_ILB:
+ sch_gpio_core.base = 0;
+ sch_gpio_core.ngpio = 21;
+ sch_gpio_resume.base = 21;
+ sch_gpio_resume.ngpio = 9;
+ break;
+
+ default:
+ err = -ENODEV;
+ goto err_sch_gpio_core;
}

sch_gpio_core.dev = &pdev->dev;
--
1.7.10.4

2013-03-20 12:11:00

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 09/10] gpio: gpio-timberdale.c: fix checkpatch error

Fix :
gpio/gpio-timberdale.c:171: ERROR: else should follow close brace '}'

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-timberdale.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpio/gpio-timberdale.c b/drivers/gpio/gpio-timberdale.c
index 702cca9..4377405 100644
--- a/drivers/gpio/gpio-timberdale.c
+++ b/drivers/gpio/gpio-timberdale.c
@@ -167,8 +167,7 @@ static int timbgpio_irq_type(struct irq_data *d, unsigned trigger)
if (ver < 3) {
ret = -EINVAL;
goto out;
- }
- else {
+ } else {
flr |= 1 << offset;
bflr |= 1 << offset;
}
--
1.7.10.4

2013-03-20 12:10:58

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 10/10] gpio: gpio-tps65910.c: fix checkpatch error

Fix :
gpio/gpio-tps65910.c:136: ERROR: space required before the open parenthesis '('

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-tps65910.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-tps65910.c b/drivers/gpio/gpio-tps65910.c
index 5083825..0614621 100644
--- a/drivers/gpio/gpio-tps65910.c
+++ b/drivers/gpio/gpio-tps65910.c
@@ -133,7 +133,7 @@ static int tps65910_gpio_probe(struct platform_device *pdev)
tps65910_gpio->gpio_chip.owner = THIS_MODULE;
tps65910_gpio->gpio_chip.label = tps65910->i2c_client->name;

- switch(tps65910_chip_id(tps65910)) {
+ switch (tps65910_chip_id(tps65910)) {
case TPS65910:
tps65910_gpio->gpio_chip.ngpio = TPS65910_NUM_GPIO;
break;
--
1.7.10.4

2013-03-20 12:12:19

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 07/10] gpio: gpio-stp-xway.c: fix checkpatch error

Fix :
gpio/gpio-stp-xway.c:220: ERROR: trailing whitespace

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-stp-xway.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-stp-xway.c b/drivers/gpio/gpio-stp-xway.c
index c20e051..04882a9 100644
--- a/drivers/gpio/gpio-stp-xway.c
+++ b/drivers/gpio/gpio-stp-xway.c
@@ -217,7 +217,7 @@ static int xway_stp_probe(struct platform_device *pdev)
chip->virt = devm_ioremap_resource(&pdev->dev, res);
if (IS_ERR(chip->virt))
return PTR_ERR(chip->virt);
-
+
chip->gc.dev = &pdev->dev;
chip->gc.label = "stp-xway";
chip->gc.direction_output = xway_stp_dir_out;
--
1.7.10.4

2013-03-20 12:12:43

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 05/10] gpio: gpio-pxa.c: fix checkpatch errors

Fix :
gpio/gpio-pxa.c:605: ERROR: space required after that ',' (ctx:VxV)
gpio/gpio-pxa.c:672: ERROR: space prohibited after that open parenthesis '('

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-pxa.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/gpio-pxa.c b/drivers/gpio/gpio-pxa.c
index 9cc108d..6d01914 100644
--- a/drivers/gpio/gpio-pxa.c
+++ b/drivers/gpio/gpio-pxa.c
@@ -602,7 +602,7 @@ static int pxa_gpio_probe(struct platform_device *pdev)
for_each_gpio_chip(gpio, c) {
writel_relaxed(0, c->regbase + GFER_OFFSET);
writel_relaxed(0, c->regbase + GRER_OFFSET);
- writel_relaxed(~0,c->regbase + GEDR_OFFSET);
+ writel_relaxed(~0, c->regbase + GEDR_OFFSET);
/* unmask GPIO edge detect for AP side */
if (gpio_is_mmp_type(gpio_type))
writel_relaxed(~0, c->regbase + ED_MASK_OFFSET);
@@ -669,7 +669,7 @@ static void pxa_gpio_resume(void)

for_each_gpio_chip(gpio, c) {
/* restore level with set/clear */
- writel_relaxed( c->saved_gplr, c->regbase + GPSR_OFFSET);
+ writel_relaxed(c->saved_gplr, c->regbase + GPSR_OFFSET);
writel_relaxed(~c->saved_gplr, c->regbase + GPCR_OFFSET);

writel_relaxed(c->saved_grer, c->regbase + GRER_OFFSET);
--
1.7.10.4

2013-03-20 12:13:07

by Laurent Navet

[permalink] [raw]
Subject: [PATCH 03/10] gpio: gpio-omap.c: fix checkpatch error

Fix :
gpio/gpio-omap.c:697: ERROR: space required before the open parenthesis '('

Signed-off-by: Laurent Navet <[email protected]>
---
drivers/gpio/gpio-omap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c
index 159f5c5..7e8f791 100644
--- a/drivers/gpio/gpio-omap.c
+++ b/drivers/gpio/gpio-omap.c
@@ -694,7 +694,7 @@ static void gpio_irq_handler(unsigned int irq, struct irq_desc *desc)
if (WARN_ON(!isr_reg))
goto exit;

- while(1) {
+ while (1) {
u32 isr_saved, level_mask = 0;
u32 enabled;

--
1.7.10.4

2013-03-20 12:22:46

by Santosh Shilimkar

[permalink] [raw]
Subject: Re: [PATCH 03/10] gpio: gpio-omap.c: fix checkpatch error

On Wednesday 20 March 2013 05:45 PM, Laurent Navet wrote:
> Fix :
> gpio/gpio-omap.c:697: ERROR: space required before the open parenthesis '('
>
> Signed-off-by: Laurent Navet <[email protected]>
> ---
Acked-by: Santosh Shilimkar <[email protected]>

2013-03-20 12:31:21

by Santosh Shilimkar

[permalink] [raw]
Subject: Re: [PATCH 03/10] gpio: gpio-omap.c: fix checkpatch error

On Wednesday 20 March 2013 05:54 PM, Santosh Shilimkar wrote:
> On Wednesday 20 March 2013 05:45 PM, Laurent Navet wrote:
>> Fix :
>> gpio/gpio-omap.c:697: ERROR: space required before the open parenthesis '('
>>
>> Signed-off-by: Laurent Navet <[email protected]>
>> ---
> Acked-by: Santosh Shilimkar <[email protected]>
>
Minor suggestion.
You might want to use $subject like "coding style fixes".
No strong opinion though.

2013-03-27 12:36:35

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 01/10] gpio: gpiolib-of.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:15 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpiolib-of.c:64: ERROR: code indent should use tabs where possible
>
> Signed-off-by: Laurent Navet <[email protected]>

Applied.

Thanks!
Linus Walleij

2013-03-27 12:39:51

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 02/10] gpio: gpio-mvebu.c: fix checkpatch errors

On Wed, Mar 20, 2013 at 1:15 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-mvebu.c:120: ERROR: space required before the open parenthesis '('
> gpio/gpio-mvebu.c:136: ERROR: space required before the open parenthesis '('
> gpio/gpio-mvebu.c:154: ERROR: space required before the open parenthesis '('
> gpio/gpio-mvebu.c:404: ERROR: space required before the open parenthesis '('
> gpio/gpio-mvebu.c:476: ERROR: "(foo*)" should be "(foo *)"
> gpio/gpio-mvebu.c:480: ERROR: "(foo*)" should be "(foo *)"
> gpio/gpio-mvebu.c:484: ERROR: "(foo*)" should be "(foo *)"
> gpio/gpio-mvebu.c:512: ERROR: space prohibited after that '!' (ctx:BxW)
> gpio/gpio-mvebu.c:518: ERROR: space prohibited after that '!' (ctx:BxW)
> gpio/gpio-mvebu.c:518: ERROR: space required before the open brace '{'
> gpio/gpio-mvebu.c:563: ERROR: space prohibited after that '!' (ctx:BxW)
> gpio/gpio-mvebu.c:570: ERROR: trailing whitespace
> gpio/gpio-mvebu.c:577: ERROR: space required before the open parenthesis '('
> gpio/gpio-mvebu.c:635: ERROR: space prohibited after that '!' (ctx:BxW)
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:41:38

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 03/10] gpio: gpio-omap.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:15 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-omap.c:697: ERROR: space required before the open parenthesis '('
>
> Signed-off-by: Laurent Navet <[email protected]>

Applied with Santosh's ACK.

Yours,
Linus Walleij

2013-03-27 12:42:31

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 04/10] gpio: gpio-pca953x.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:15 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-pca953x.c:150: ERROR: else should follow close brace '}'
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:43:21

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 05/10] gpio: gpio-pxa.c: fix checkpatch errors

On Wed, Mar 20, 2013 at 1:15 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-pxa.c:605: ERROR: space required after that ',' (ctx:VxV)
> gpio/gpio-pxa.c:672: ERROR: space prohibited after that open parenthesis '('
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:44:19

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 06/10] gpio: gpio-sch.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:16 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-sch.c:206: ERROR: switch and case should be at the same indent
>
> Also remove blank lines
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:45:04

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 07/10] gpio: gpio-stp-xway.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:16 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-stp-xway.c:220: ERROR: trailing whitespace
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:45:48

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 08/10] gpio: gpio-tc3589x.c: fix checkpatch errors

On Wed, Mar 20, 2013 at 1:16 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-tc3589x.c:285: ERROR: code indent should use tabs where possible
> gpio/gpio-tc3589x.c:286: ERROR: code indent should use tabs where possible
> gpio/gpio-tc3589x.c:287: ERROR: code indent should use tabs where possible
> gpio/gpio-tc3589x.c:347: ERROR: code indent should use tabs where possible
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:46:34

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 09/10] gpio: gpio-timberdale.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:16 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-timberdale.c:171: ERROR: else should follow close brace '}'
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij

2013-03-27 12:47:30

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 10/10] gpio: gpio-tps65910.c: fix checkpatch error

On Wed, Mar 20, 2013 at 1:16 PM, Laurent Navet <[email protected]> wrote:

> Fix :
> gpio/gpio-tps65910.c:136: ERROR: space required before the open parenthesis '('
>
> Signed-off-by: Laurent Navet <[email protected]>

Patch applied.

Thanks!
Linus Walleij