[PATCH] pinctrl: mvebu: fix checking for SoC specific controls

Sebastian Hesselbarth sebastian.hesselbarth at gmail.com
Tue Mar 19 15:41:41 EDT 2013


On 03/19/2013 08:07 PM, Simon Guinot wrote:
> This patch fixes a minor bug (probably due to a typo) while checking
> the SoC specific controls in mvebu_pinctrl_probe().
>
> Signed-off-by: Simon Guinot<simon.guinot at sequanux.org>

Simon,

good catch! The patch looks sane to me, so

Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth at gmail.com>

> ---
>   drivers/pinctrl/mvebu/pinctrl-mvebu.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/mvebu/pinctrl-mvebu.c b/drivers/pinctrl/mvebu/pinctrl-mvebu.c
> index c689c04..2d2f0a4 100644
> --- a/drivers/pinctrl/mvebu/pinctrl-mvebu.c
> +++ b/drivers/pinctrl/mvebu/pinctrl-mvebu.c
> @@ -620,7 +620,7 @@ int mvebu_pinctrl_probe(struct platform_device *pdev)
>
>   		/* special soc specific control */
>   		if (ctrl->mpp_get || ctrl->mpp_set) {
> -			if (!ctrl->name || !ctrl->mpp_set || !ctrl->mpp_set) {
> +			if (!ctrl->name || !ctrl->mpp_get || !ctrl->mpp_set) {
>   				dev_err(&pdev->dev, "wrong soc control info\n");
>   				return -EINVAL;
>   			}




More information about the linux-arm-kernel mailing list