[PATCH RFC 0/3] Make "ti,tilcdc,slave" DT binding more sensible

Jean-Francois Moine moinejf at free.fr
Tue Jan 13 22:57:29 PST 2015


On Tue, 13 Jan 2015 19:12:25 +0200
Jyri Sarha <jsarha at ti.com> wrote:

> These patches are needed for Beaglebone-back HDMI audio. There is no
> direct dependency between these patches and the other (dts and ASoC)
> changes needed for the HDMI audio so these changes can be merged
> independently. I also feel that these changes make sense even without
> the HDMI audio.
> 
> Best regards,
> Jyri
> 
> Jyri Sarha (3):
>   drm: encoder_slave: Add drm_i2c_encoder_attach()
>   drm/tilcdc: slave: Add support for "i2c-slave" DT-parameter
>   ARM: dts: am335x-boneblack: Use new binding in ti,tilcdc,slave node
> 
>  .../devicetree/bindings/drm/tilcdc/slave.txt       |    4 +-
>  arch/arm/boot/dts/am335x-boneblack.dts             |    9 +++-
>  drivers/gpu/drm/drm_encoder_slave.c                |   51 ++++++++++++++++++++
>  drivers/gpu/drm/tilcdc/tilcdc_slave.c              |   50 +++++++++++--------
>  include/drm/drm_encoder_slave.h                    |    3 ++
>  5 files changed, 95 insertions(+), 22 deletions(-)

Instead of adding code to have the slave encoder working, it would be
simpler to change the way tilcdc uses the tda998x.
I already proposed such a patch:

http://lists.freedesktop.org/archives/dri-devel/2014-March/056065.html

and the changes in the tda998x driver have been done by Russell:

commit: a8f4d4d63739e4bca459ff40636f1d9e4b7ef5e6
	drm/i2c: tda998x: allow re-use of tda998x support code
and
commit: c707c3619ca81f499a5ce032021405e989a96ff0
	drm/i2c: tda998x: add component support

-- 
Ken ar c'hentaƱ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/


More information about the dri-devel mailing list