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 devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html