[alsa-devel] [PATCH v8 1/5] video: rmk's HDMI notification prototype

[Philipp Zabel]

Am Donnerstag, den 11.08.2016, 11:18 +0100 schrieb Russell King - ARM
Linux:
> On Thu, Aug 11, 2016 at 11:20:23AM +0200, Philipp Zabel wrote:
> > This is Russell's HDMI notification prototype [1], currently waiting
> > for the HDMI CEC situation to resolve.
> > 
> > The use case for the notifications on MediaTek MT8173 is to let the
> > (dis)connection notifications control an ALSA jack object.
> > 
> > No Signed-off-by since this is not my code, and still up for discussion.
> 
> Well, I have two drivers (both CEC drivers) which use this, and I still
> don't see any alternative solution to the problem that this patch is
> solving.
> 
> I don't think it's really a CEC problem - there's three bits to HDMI
> that need to track each others state - the video, audio and CEC paths.
> 
> While the video and audio paths may be one block, the CEC path may
> actually be a separate block.  For example, the TDA998x devices
> integrate the HDMI video/audio block along with a TDA9950 on the
> same device - the TDA9950 being a CEC engine.  The TDA9950 is also
> available as a separate device, and even when integrated with HDMI,
> it appears on the I2C bus as a seperate device.
> 
> So, splitting the functionality is definitely the right model.  We
> just need some way to keep each blocks state in sync.  What's provided
> in this patch is the simple solution which seems to work for the use
> cases we have.

Yes, it works fine for the MT8173 use case.

> I think, in light of no comments against this approach, and no other
> approach being available, that this is good enough justification to
> merge this, especially as it is blocking other work.
> 
> So... if people want to give me reviewed-by/acked-bys, I'll add them
> to my patch, and I'll post that and the dw-hdmi and tda9950 CEC drivers.

Acked-by: Philipp Zabel <p.zabel at pengutronix.de>

regards
Philipp