Dne Po 5. dubna 2010 15:41:07 Mark Brown napsal(a):
> On Mon, Apr 05, 2010 at 06:13:37AM +0200, Marek Vasut wrote:
> > This patch registers the WM8750 codec on a proper place on the SPITZ
> > machine after the WM8750 driver was converted to new API. Moreover, the
> > incorrect code
>
> We also need this for the Jive machine. FWIW I'm checking for affected
> machines by doing 'grep -l wm8750 sound/soc/*/*.c'.
Will send a separate patch for that
>
> > is hereby removed from the aSoC driver.
>
> ASoC.
>
> > diff --git a/sound/soc/pxa/spitz.c b/sound/soc/pxa/spitz.c
> > index c4cd2ac..d256f5f 100644
> > --- a/sound/soc/pxa/spitz.c
> > +++ b/sound/soc/pxa/spitz.c
> > @@ -322,17 +322,10 @@ static struct snd_soc_card snd_soc_spitz = {
> > .num_links = 1,
> > };
> >
> > -/* spitz audio private data */
> > -static struct wm8750_setup_data spitz_wm8750_setup = {
> > - .i2c_bus = 0,
> > - .i2c_address = 0x1b,
> > -};
> > -
> > /* spitz audio subsystem */
> > static struct snd_soc_device spitz_snd_devdata = {
> > .card = &snd_soc_spitz,
> > .codec_dev = &soc_codec_dev_wm8750,
> > - .codec_data = &spitz_wm8750_setup,
> > };
>
> So, this is going to break spitz if not merged in with the patch to
> update wm8750 since prior to the conversion of the driver the setup data
> is required while after the conversion the setup data structure no
> longer exists and will therefore cause compile failures.
>
> As I've mentioned several times before I urge you to look at how this
> transition was done for wm8731 for an example of how this sort of change
> can be done without too much disruption.
>
Can you possibly elaborate ? I was unable to find it.
Thanks
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
