On Tuesday, December 03, 2013 at 12:44:52 PM, Shawn Guo wrote: > On Mon, Dec 02, 2013 at 04:39:26PM +0100, Marek Vasut wrote: > > Add DRM flags for the LCD display clock polarity so the pixelclk-active > > DT property can be properly handled by drivers using the DRM API. > > > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > > Cc: Dave Airlie <airlied@xxxxxxxxx> > > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > Cc: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> > > Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > > Cc: Shawn Guo <shawn.guo@xxxxxxxxxx> > > --- > > > > drivers/gpu/drm/drm_modes.c | 5 +++++ > > include/uapi/drm/drm_mode.h | 3 +++ > > 2 files changed, 8 insertions(+) > > > > diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c > > index 85071a1..d1f3bfc 100644 > > --- a/drivers/gpu/drm/drm_modes.c > > +++ b/drivers/gpu/drm/drm_modes.c > > @@ -537,6 +537,11 @@ int drm_display_mode_from_videomode(const struct > > videomode *vm, > > > > dmode->flags |= DRM_MODE_FLAG_DBLSCAN; > > > > if (vm->flags & DISPLAY_FLAGS_DOUBLECLK) > > > > dmode->flags |= DRM_MODE_FLAG_DBLCLK; > > > > + if (vm->flags & DISPLAY_FLAGS_PIXDATA_POSEDGE) > > + dmode->flags |= DRM_MODE_FLAG_PIXELCLK_PPOL; > > + else if (vm->flags & DISPLAY_FLAGS_PIXDATA_NEGEDGE) > > + dmode->flags |= DRM_MODE_FLAG_PIXELCLK_NPOL; > > + > > > > drm_mode_set_name(dmode); > > > > return 0; > > > > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > > index f104c26..a6169ca 100644 > > --- a/include/uapi/drm/drm_mode.h > > +++ b/include/uapi/drm/drm_mode.h > > @@ -73,6 +73,9 @@ > > > > #define DRM_MODE_FLAG_3D_TOP_AND_BOTTOM (7<<14) > > #define DRM_MODE_FLAG_3D_SIDE_BY_SIDE_HALF (8<<14) > > > > +/* CRTC LCD clock polarity flags. */ > > +#define DRM_MODE_FLAG_PIXELCLK_PPOL (1<<19) > > +#define DRM_MODE_FLAG_PIXELCLK_NPOL (1<<20) > > Marek, > > It looks that Denis (copied) is working on the same problem, so you may > want to be aware of his effort [1][2]. He also just sent a new patchset, so I would focus on his work instead. It makes more sense as he is not breaking the userland stuff. Thank you Shawn, Russell! Best regards, Marek Vasut _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
