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 
> ---
>  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 

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

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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 

Acked-by: Kukjin Kim 

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

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Linus Walleij]

On Fri, Feb 8, 2013 at 6:39 AM, 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 

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

Thanks!
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html