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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/14/2015 08:57 AM, Jean-Francois Moine wrote:
On Tue, 13 Jan 2015 19:12:25 +0200
Jyri Sarha <jsarha@xxxxxx> 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


Interesting. Would you still have the original branch somewhere? Manual applying the patches from the web-page is time consuming.

Best regards,
Jyri
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux