On 26/02/15 16:55, Jyri Sarha wrote: > Adds a DRM_TILCDC_INIT module for "ti,tilcdc,slave" node > conversion. The implementation is in tilcdc_boot_init.c and it uses > tilcdc_slave_convert.dts as a basis for creating a DTS overlay. The > DTS overlay adds an external tda998x encoder to tilcdc that > corresponds to the old tda998x based slave encoder. > > Signed-off-by: Jyri Sarha <jsarha at ti.com> > --- > drivers/gpu/drm/tilcdc/Kconfig | 6 + > drivers/gpu/drm/tilcdc/Makefile | 2 + > drivers/gpu/drm/tilcdc/tilcdc_boot_init.c | 270 > ++++++++++++++++++++++++ > drivers/gpu/drm/tilcdc/tilcdc_slave_convert.dts | 70 ++++++ > 4 files changed, 348 insertions(+) > create mode 100644 drivers/gpu/drm/tilcdc/tilcdc_boot_init.c > create mode 100644 drivers/gpu/drm/tilcdc/tilcdc_slave_convert.dts
All this is needed to support the old bindings, right? So I would suggest scratching "DRM_TILCDC_INIT", and create "DRM_TILCDC_COMPAT_SLAVE" or whatever, which would be a user visible option, enabled by default. This way one can just disable all this compatibility stuff if it's not needed. Tomi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150302/06532c20/attachment-0001.sig>