[PATCH] gpiolib: remove unused variable 'i'

[PATCH] gpiolib: remove unused variable 'i'

[Colin King]

From: Colin Ian King <colin.king@canonical.com>

Fixes smatch warning: unused variable 'i' [-Wunused-variable]

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/gpio/gpiolib.c                   | 2 --
 sound/soc/generic/audio-graph-scu-card.c | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index be8097097326..62ffb4e293d2 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -1471,8 +1471,6 @@ static struct gpio_chip *find_chip_by_name(const char *name)
 
 static int gpiochip_irqchip_init_valid_mask(struct gpio_chip *gpiochip)
 {
-	int i;
-
 	if (!gpiochip->irq_need_valid_mask)
 		return 0;
 
diff --git a/sound/soc/generic/audio-graph-scu-card.c b/sound/soc/generic/audio-graph-scu-card.c
index c5c52d1f500c..e15f63c74f59 100644
--- a/sound/soc/generic/audio-graph-scu-card.c
+++ b/sound/soc/generic/audio-graph-scu-card.c
@@ -246,7 +246,7 @@ static int asoc_graph_card_parse_of(struct graph_card_data *priv)
 			continue;
 
 		if (rcpu_ep != cpu_ep) {
-			dev_err(dev, "remote-endpoint missmatch (%s/%s/%s)\n",
+			dev_err(dev, "remote-endpoint mismatch (%s/%s/%s)\n",
 				cpu_ep->name, codec_ep->name, rcpu_ep->name);
 			ret = -EINVAL;
 			goto parse_of_err;
-- 
2.11.0

Re: [PATCH] gpiolib: remove unused variable 'i'

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 528 bytes --]

On Tue, May 30, 2017 at 10:44:20AM +0100, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> Fixes smatch warning: unused variable 'i' [-Wunused-variable]
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  drivers/gpio/gpiolib.c                   | 2 --
>  sound/soc/generic/audio-graph-scu-card.c | 2 +-
>  2 files changed, 1 insertion(+), 3 deletions(-)

You've got two unrelated patches in one here (as a quick review of your
CC list would probably have warned you...).

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] gpiolib: remove unused variable 'i'

[Colin Ian King]

[-- Attachment #1.1.1: Type: text/plain, Size: 613 bytes --]

On 30/05/17 11:58, Mark Brown wrote:
> On Tue, May 30, 2017 at 10:44:20AM +0100, Colin King wrote:
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> Fixes smatch warning: unused variable 'i' [-Wunused-variable]
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>> ---
>>  drivers/gpio/gpiolib.c                   | 2 --
>>  sound/soc/generic/audio-graph-scu-card.c | 2 +-
>>  2 files changed, 1 insertion(+), 3 deletions(-)
> 
> You've got two unrelated patches in one here (as a quick review of your
> CC list would probably have warned you...).
> 
Oops, my bad. V2 resent.


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 837 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]