Re: [PATCHv2 2/3] drm/imx: convey the pixelclk-active and de-active flags from DT to the ipu-di driver

From	Philipp Zabel <p.zabel@pengutronix.de>
Newsgroups	linux.kernel
Subject	Re: [PATCHv2 2/3] drm/imx: convey the pixelclk-active and de-active flags from DT to the ipu-di driver
Date	2016-05-25 12:30 +0200
Message-ID	<rCEeD-8gC-35@gated-at.bofh.it> (permalink)
References	<rCeau-8d9-7@gated-at.bofh.it> <rCeau-8d9-9@gated-at.bofh.it> <rCllE-4jI-7@gated-at.bofh.it> <rCAkG-5XU-23@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Am Mittwoch, den 25.05.2016, 08:11 +0200 schrieb Lothar Waßmann:
[...]
> > > @@ -81,10 +87,27 @@ static int imx_pd_connector_get_modes(struct drm_connector *connector)
> > >  
> > >  	if (np) {
> > >  		struct drm_display_mode *mode = drm_mode_create(connector->dev);
> > > +		struct videomode vm;
> > > +		int ret;
> > >  
> > >  		if (!mode)
> > >  			return -EINVAL;
> > > -		of_get_drm_display_mode(np, &imxpd->mode, OF_USE_NATIVE_MODE);
> > > +
> > > +		ret = of_get_videomode(np, &vm, OF_USE_NATIVE_MODE);
> > > +		if (ret)
> > > +			return ret;
> > > +		drm_display_mode_from_videomode(&vm, &imxpd->mode);
> > > +		drm_mode_debug_printmodeline(&imxpd->mode);
> > > +
> > > +		if (vm.flags & DISPLAY_FLAGS_PIXDATA_POSEDGE)
> > > +			imxpd->bus_flags |= DRM_BUS_FLAG_PIXDATA_POSEDGE;
> > > +		if (vm.flags & DISPLAY_FLAGS_PIXDATA_NEGEDGE)
> > > +			imxpd->bus_flags |= DRM_BUS_FLAG_PIXDATA_NEGEDGE;
> > > +
> > > +		if (vm.flags & DISPLAY_FLAGS_DE_LOW)
> > > +			imxpd->bus_flags |= DRM_BUS_FLAG_DE_LOW;
> > > +		if (vm.flags & DISPLAY_FLAGS_DE_HIGH)
> > > +			imxpd->bus_flags |= DRM_BUS_FLAG_DE_HIGH;
> > 
> > This could be shared between ldb and parallel drivers if you extended
> > of_get_drm_display_mode to also return bus_flags, for example. The flags
> > translation could be put into a drm_bus_flags_from_videomode helper.
> > 
> I had that in mind also, but hesitated to change the generic
> of_get_drm_display_mode() to implement some i.MX specific functionality.

Since the videomode obtained from DT includes both the timing and pin
polarity information, and the drm way to express these are
drm_display_mode and bus_flags, respectively, there's nothing i.MX
specific about this.

regards
Philipp

Thread

[PATCH 0/3] drm/imx: convey the pixelclk-active and de-active flags to the ipu-di driver Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-20 16:00 +0200
  [PATCH 2/3] drm/imx: convey the pixelclk-active and de-active flags from DT to the ipu-di driver Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-20 16:00 +0200
    Re: [PATCH 2/3] drm/imx: convey the pixelclk-active and de-active  flags from DT to the ipu-di driver Philipp Zabel <p.zabel@pengutronix.de> - 2016-05-20 16:40 +0200
  [PATCHv2 0/3] drm/imx: convey the pixelclk-active and de-active flags from DT to the ipu-di driver Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-24 08:40 +0200
    [PATCHv2 2/3] drm/imx: convey the pixelclk-active and de-active flags from DT to the ipu-di driver Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-24 08:40 +0200
      Re: [PATCHv2 2/3] drm/imx: convey the pixelclk-active and de-active  flags from DT to the ipu-di driver Philipp Zabel <p.zabel@pengutronix.de> - 2016-05-24 16:20 +0200
        Re: [PATCHv2 2/3] drm/imx: convey the pixelclk-active and de-active  flags from DT to the ipu-di driver Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-25 08:20 +0200
          Re: [PATCHv2 2/3] drm/imx: convey the pixelclk-active and de-active  flags from DT to the ipu-di driver Philipp Zabel <p.zabel@pengutronix.de> - 2016-05-25 12:30 +0200
    [PATCHv2 3/3] drm/imx: remove dead code Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-24 08:40 +0200
    [PATCHv2 1/3] drm/imx: imx-ldb: honor 'native-mode' property when selecting video mode from DT Lothar Waßmann <LW@KARO-electronics.de> - 2016-05-24 08:40 +0200

csiph-web