[PATCH v6 2/2] drm/i2c:tda998x: Use the HDMI audio CODEC

Jean-Francois Moine moinejf at free.fr
Thu Oct 2 10:59:51 PDT 2014


On Wed, 1 Oct 2014 15:23:41 +0100
Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:

> I would prefer this structure to stay here, as code above this point should
> have no business knowing how these are packaged together.  I would suggest
> either:
> 
> - moving the audio codec code below this point, or
> - storing struct tda998x_priv in the device private pointer, and
>   converting it to struct tda998x_priv2 via container_of() where
>   necessary below this point.

The second option seems easier for use as a slave encoder.

Thanks for all your remarks. I will send an other version.

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


More information about the dri-devel mailing list