[PATCH] pinctrl: samsung: remove duplicated line of samsung_pinctrl_register()

[PATCH] pinctrl: samsung: remove duplicated line of samsung_pinctrl_register()

[Jingoo Han]

This patch removes duplicated line of samsung_pinctrl_register(),
because the number of pins is redundantly assigned twice.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/pinctrl/pinctrl-samsung.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl/pinctrl-samsung.c
index 294bcdd..3d5cf63 100644
--- a/drivers/pinctrl/pinctrl-samsung.c
+++ b/drivers/pinctrl/pinctrl-samsung.c
@@ -716,7 +716,6 @@ static int 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.2.5

Re: [PATCH] pinctrl: samsung: remove duplicated line of samsung_pinctrl_register()

[Tomasz Figa]

On Friday 08 of February 2013 14:39:30 Jingoo Han wrote:
> This patch removes duplicated line of samsung_pinctrl_register(),
> because the number of pins is redundantly assigned twice.
> 
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> ---
>  drivers/pinctrl/pinctrl-samsung.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/pinctrl/pinctrl-samsung.c
> b/drivers/pinctrl/pinctrl-samsung.c index 294bcdd..3d5cf63 100644
> --- a/drivers/pinctrl/pinctrl-samsung.c
> +++ b/drivers/pinctrl/pinctrl-samsung.c
> @@ -716,7 +716,6 @@ static int 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++)

Reviewed-by: Tomasz Figa <t.figa@samsung.com>

Best regards,
-- 
Tomasz Figa
Samsung Poland R&D Center
SW Solution Development, Linux Platform

RE: [PATCH] pinctrl: samsung: remove duplicated line of samsung_pinctrl_register()

[Kukjin Kim]

Jingoo Han wrote:
> 
> This patch removes duplicated line of samsung_pinctrl_register(),
> because the number of pins is redundantly assigned twice.
> 
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Acked-by: Kukjin Kim <kgene.kim@samsung.com>

Thanks.

- Kukjin

> ---
>  drivers/pinctrl/pinctrl-samsung.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl/pinctrl-
> samsung.c
> index 294bcdd..3d5cf63 100644
> --- a/drivers/pinctrl/pinctrl-samsung.c
> +++ b/drivers/pinctrl/pinctrl-samsung.c
> @@ -716,7 +716,6 @@ static int 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.2.5

Re: [PATCH] pinctrl: samsung: remove duplicated line of samsung_pinctrl_register()

[Linus Walleij]

On Fri, Feb 8, 2013 at 6:39 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> This patch removes duplicated line of samsung_pinctrl_register(),
> because the number of pins is redundantly assigned twice.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied with the Reviewed-by/Acked-by tags from
the maintainers.

Thanks!
Linus Walleij