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

From: Jean-Francois Moine
Date: Thu Oct 02 2014 - 13:59:50 EST


On Wed, 1 Oct 2014 15:23:41 +0100
Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> 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/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/