Messages in this thread | | | Date | Thu, 2 Oct 2014 19:59:51 +0200 | From | Jean-Francois Moine <> | Subject | Re: [PATCH v6 2/2] drm/i2c:tda998x: Use the HDMI audio CODEC |
| |
On Wed, 1 Oct 2014 15:23:41 +0100 Russell King - ARM Linux <linux@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/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |