Re: [PATCH 04/10] include: platform_data: Platform data header for OMAP4 ASoC audio

[Mark Brown <broonie@opensource.wolfsonmicro.com>]

On Mon, Dec 19, 2011 at 11:20:37AM -0800, Tony Lindgren wrote:
> * Peter Ujfalusi <peter.ujfalusi@ti.com> [111219 05:33]:

> > struct omap-abe-twl6040-connection sdp4430_asoc_route[] = {
> > 	{"MAINMIC", "Main Mic Bias"},
> > 	{"SUBMIC", "Main Mic Bias"},
> > 	{"Main Mic Bias", "Ext Mic"},

> Hmm does it make sense to describe all those in DT? If you can
> group things in some sane way, then maybe the routings could be
> stored in the driver itself in the .data associated with the DT
> compatible flag? That is of course assuming there are some sane
> ways to group the routings..

No, and it's not completely trivial to do so until we have a sensible
binding for the objects you find on boards, especially jacks which don't
map in any sort of straightforward fashion onto the DAPM routes which we
need internally as they'll often group a bunch of different signals into
a single connector that don't have any direct relationship at the driver
level.

The old style MICBIAS widgets that the CODEC driver is using would also
be something I'd consider a blocker to direct use in device tree - the
way they're hooked up is really Linux specific and not terribly clear
either.