public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] pinctrl: samsung: Remove a redundant settting for ctrldesc->npins
@ 2012-11-06 15:28 Axel Lin
  2012-11-07 13:35 ` Kyungmin Park
  2012-11-11 18:16 ` Linus Walleij
  0 siblings, 2 replies; 3+ messages in thread
From: Axel Lin @ 2012-11-06 15:28 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Kyungmin Park, Thomas Abraham, Tomasz Figa, linux-kernel

Set it once is enough.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
 drivers/pinctrl/pinctrl-samsung.c |    1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl/pinctrl-samsung.c
index fc34cac..1307283 100644
--- a/drivers/pinctrl/pinctrl-samsung.c
+++ b/drivers/pinctrl/pinctrl-samsung.c
@@ -714,7 +714,6 @@ static int __devinit samsung_pinctrl_register(struct platform_device *pdev,
 	}
 	ctrldesc->pins = pindesc;
 	ctrldesc->npins = drvdata->ctrl->nr_pins;
-	ctrldesc->npins = drvdata->ctrl->nr_pins;
 
 	/* dynamically populate the pin number and pin name for pindesc */
 	for (pin = 0, pdesc = pindesc; pin < ctrldesc->npins; pin++, pdesc++)
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: samsung: Remove a redundant settting for ctrldesc->npins
  2012-11-06 15:28 [PATCH] pinctrl: samsung: Remove a redundant settting for ctrldesc->npins Axel Lin
@ 2012-11-07 13:35 ` Kyungmin Park
  2012-11-11 18:16 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Kyungmin Park @ 2012-11-07 13:35 UTC (permalink / raw)
  To: Axel Lin; +Cc: Linus Walleij, Thomas Abraham, Tomasz Figa, linux-kernel

Acked-by: Kyungmin Park <kyungmin.park@samsung.com>

On Wed, Nov 7, 2012 at 12:28 AM, Axel Lin <axel.lin@ingics.com> wrote:
> Set it once is enough.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
>  drivers/pinctrl/pinctrl-samsung.c |    1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl/pinctrl-samsung.c
> index fc34cac..1307283 100644
> --- a/drivers/pinctrl/pinctrl-samsung.c
> +++ b/drivers/pinctrl/pinctrl-samsung.c
> @@ -714,7 +714,6 @@ static int __devinit samsung_pinctrl_register(struct platform_device *pdev,
>         }
>         ctrldesc->pins = pindesc;
>         ctrldesc->npins = drvdata->ctrl->nr_pins;
> -       ctrldesc->npins = drvdata->ctrl->nr_pins;
>
>         /* dynamically populate the pin number and pin name for pindesc */
>         for (pin = 0, pdesc = pindesc; pin < ctrldesc->npins; pin++, pdesc++)
> --
> 1.7.9.5
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: samsung: Remove a redundant settting for ctrldesc->npins
  2012-11-06 15:28 [PATCH] pinctrl: samsung: Remove a redundant settting for ctrldesc->npins Axel Lin
  2012-11-07 13:35 ` Kyungmin Park
@ 2012-11-11 18:16 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2012-11-11 18:16 UTC (permalink / raw)
  To: Axel Lin; +Cc: Kyungmin Park, Thomas Abraham, Tomasz Figa, linux-kernel

On Tue, Nov 6, 2012 at 4:28 PM, Axel Lin <axel.lin@ingics.com> wrote:

> Set it once is enough.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Applied with Kyungmin's ACK, thanks!
Linus Walleij

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-11-11 18:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-06 15:28 [PATCH] pinctrl: samsung: Remove a redundant settting for ctrldesc->npins Axel Lin
2012-11-07 13:35 ` Kyungmin Park
2012-11-11 18:16 ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox