Re: [PATCH] pinctrl: samsung: remove duplicated line of samsung_pinctrl_register()
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()
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()
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