On Mon 02 Jan 00:42 PST 2017, Linus Walleij wrote:

> These four pins are for SDC4, not SDC1. They are grouped for
> SDC4 later in the file so this must be a typo.
> 
> Cc: Björn Andersson <bjorn.anders...@linaro.org>
> Signed-off-by: Linus Walleij <linus.wall...@linaro.org>

Reviewed-by: Bjorn Andersson <bjorn.anders...@linaro.org>

Regards,
Bjorn

> ---
>  drivers/pinctrl/qcom/pinctrl-msm8660.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/pinctrl/qcom/pinctrl-msm8660.c 
> b/drivers/pinctrl/qcom/pinctrl-msm8660.c
> index 5591d093bf78..bb71dd1e6279 100644
> --- a/drivers/pinctrl/qcom/pinctrl-msm8660.c
> +++ b/drivers/pinctrl/qcom/pinctrl-msm8660.c
> @@ -193,9 +193,9 @@ static const struct pinctrl_pin_desc msm8660_pins[] = {
>       PINCTRL_PIN(171, "GPIO_171"),
>       PINCTRL_PIN(172, "GPIO_172"),
>  
> -     PINCTRL_PIN(173, "SDC1_CLK"),
> -     PINCTRL_PIN(174, "SDC1_CMD"),
> -     PINCTRL_PIN(175, "SDC1_DATA"),
> +     PINCTRL_PIN(173, "SDC4_CLK"),
> +     PINCTRL_PIN(174, "SDC4_CMD"),
> +     PINCTRL_PIN(175, "SDC4_DATA"),
>       PINCTRL_PIN(176, "SDC3_CLK"),
>       PINCTRL_PIN(177, "SDC3_CMD"),
>       PINCTRL_PIN(178, "SDC3_DATA"),
> -- 
> 2.9.3
> 

Reply via email to