Re: [PATCH] pinctrl: qcom: msm8660: rename some SDC1->SDC4

From: Bjorn Andersson
Date: Mon Jan 02 2017 - 13:20:34 EST


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.andersson@xxxxxxxxxx>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>

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
>