[PATCH] drm/bridge/ptn3460: add include for gpiod*

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Wed Jun 10 00:15:25 PDT 2015


Hello,

Cc += dri-devel at lists.freedesktop.org + kernel at pengutronix.de, forgot
that for the original submission. Also added (hopefully the right)
Geert.

On Mon, Jun 08, 2015 at 11:45:17AM +0200, Uwe Kleine-König wrote:
> Hello,
> 
> On Fri, Jun 05, 2015 at 04:46:25PM +0200, Thierry Reding wrote:
> > On Mon, Jun 01, 2015 at 04:43:32PM +0200, Uwe Kleine-König wrote:
> > > This fixes:
> > > 
> > > 	  CC      drivers/gpu/drm/bridge/ptn3460.o
> > > 	drivers/gpu/drm/bridge/ptn3460.c: In function ‘ptn3460_pre_enable’:
> > > 	drivers/gpu/drm/bridge/ptn3460.c:135:2: error: implicit declaration of function ‘gpiod_set_value’ [-Werror=implicit-function-declaration]
> > > 	  gpiod_set_value(ptn_bridge->gpio_pd_n, 1);
> > > 	  ^
> > > 	drivers/gpu/drm/bridge/ptn3460.c: In function ‘ptn3460_probe’:
> > > 	drivers/gpu/drm/bridge/ptn3460.c:333:2: error: implicit declaration of function ‘devm_gpiod_get’ [-Werror=implicit-function-declaration]
> > > 	  ptn_bridge->gpio_pd_n = devm_gpiod_get(&client->dev, "powerdown");
> > > 	  ^
> > > 	drivers/gpu/drm/bridge/ptn3460.c:333:24: warning: assignment makes pointer from integer without a cast
> > > 	  ptn_bridge->gpio_pd_n = devm_gpiod_get(&client->dev, "powerdown");
> > > 				^
> > > 	drivers/gpu/drm/bridge/ptn3460.c:340:2: error: implicit declaration of function ‘gpiod_direction_output’ [-Werror=implicit-function-declaration]
> > > 	  ret = gpiod_direction_output(ptn_bridge->gpio_pd_n, 1);
> > > 	  ^
> > > 	drivers/gpu/drm/bridge/ptn3460.c:346:25: warning: assignment makes pointer from integer without a cast
> > > 	  ptn_bridge->gpio_rst_n = devm_gpiod_get(&client->dev, "reset");
> > > 				 ^
> > > 
> > > Fixes: af478d8823a7 ("drm/bridge: ptn3460: use gpiod interface")
> > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> > > ---
> > >  drivers/gpu/drm/bridge/ptn3460.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > 
> > Geert had sent similar patches a while ago, so I applied his instead.
> I didn't find where you applied it. Did you send this patch to Linus for
> 4.1?
If you want to get this into 4.1 it's about time to send it to Linus. Or
don't you consider it that urgent?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |


More information about the dri-devel mailing list