Re: [PATCH] drm/panel: ld9040: Update calls to gpiod_get*()

[Andrzej Hajda]

On 10/23/2014 10:16 AM, Alexandre Courbot wrote:
> Add the new flags argument to calls of (devm_)gpiod_get*() and
> remove any direction setting code afterwards.
> 
> Currently both forms (with or without the flags argument)
> are valid thanks to transitional macros in
> <linux/gpio/consumer.h>. These macros will be removed once
> all consumers are updated and the flags argument will become
> compulsary.
> 
> Signed-off-by: Alexandre Courbot <acourbot@xxxxxxxxxx>

It needs patch "gpio: Fix gpio direction flags not getting set" to work
correctly. It is not yet present in drm-next. Beside this:

Acked-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>

--
Regards
Andrzej

> ---
>  drivers/gpu/drm/panel/panel-ld9040.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/panel/panel-ld9040.c b/drivers/gpu/drm/panel/panel-ld9040.c
> index 42ac67b21e9f..dd1d3f2e624e 100644
> --- a/drivers/gpu/drm/panel/panel-ld9040.c
> +++ b/drivers/gpu/drm/panel/panel-ld9040.c
> @@ -336,17 +336,12 @@ static int ld9040_probe(struct spi_device *spi)
>  	if (ret < 0)
>  		return ret;
>  
> -	ctx->reset_gpio = devm_gpiod_get(dev, "reset");
> +	ctx->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH);
>  	if (IS_ERR(ctx->reset_gpio)) {
>  		dev_err(dev, "cannot get reset-gpios %ld\n",
>  			PTR_ERR(ctx->reset_gpio));
>  		return PTR_ERR(ctx->reset_gpio);
>  	}
> -	ret = gpiod_direction_output(ctx->reset_gpio, 1);
> -	if (ret < 0) {
> -		dev_err(dev, "cannot configure reset-gpios %d\n", ret);
> -		return ret;
> -	}
>  
>  	spi->bits_per_word = 9;
>  	ret = spi_setup(spi);
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/