Re: Linux 3.2.3

[Tim Gardner]

On 02/03/2012 11:54 AM, Greg KH wrote:
> On Fri, Feb 03, 2012 at 10:53:19AM -0800, Greg KH wrote:
> > On Fri, Feb 03, 2012 at 10:51:57AM -0800, Greg KH wrote:
> > > On Fri, Feb 03, 2012 at 11:41:04AM -0700, Tim Gardner wrote:
> > > > On 02/03/2012 10:47 AM, Greg KH wrote:
> > > > > I'm announcing the release of the 3.2.3 kernel.
> > > > >
> > > > > All users of the 3.2 kernel series must upgrade.
> > > > >
> > > > > The updated 3.2.y git tree can be found at:
> > > > >          git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-3.2.y
> > > > > and can be browsed at the normal kernel.org git web browser:
> > > > >          http://git.kernel.org/?p=linux/kernel/git/stable/linux-stable.git;a=summary
> > > > >
> > > > > thanks,
> > > > >
> > > > > greg k-h
> > > >
> > > > Build failure in vanilla Linux 3.2.3:
> > > >
> > > >    CC [M]  sound/soc/codecs/wm5100.o
> > > > sound/soc/codecs/wm5100.c: In function âwm5100_set_bias_levelâ:
> > > > sound/soc/codecs/wm5100.c:1382:33: error: âstruct wm5100_privâ has
> > > > no member named âregmapâ
> > > > sound/soc/codecs/wm5100.c:1398:33: error: âstruct wm5100_privâ has
> > > > no member named âregmapâ
> > > > sound/soc/codecs/wm5100.c:1409:29: error: âstruct wm5100_privâ has
> > > > no member named âregmapâ
> > > > make[5]: *** [sound/soc/codecs/wm5100.o] Error 1
> > > >
> > > > Looks like it depends on bd132ec585c498ee27d7eedf8569703606743928
> > > > "ASoC: Convert wm5100 to direct regmap API usage", but I haven't
> > > > re-built with it yet.
> > > >
> > > > See attached config.
> > >
> > > Ugh, I didn't have CONFIG_SND_SOC=m in my test config files, I'll go fix
> > > that now and see what needs to be done here, thanks for letting me know.
> > >
> > > Mark any hints?
> >
> > Yes, the above commit fixes the build for me, thanks, I'll go release
> > 3.2.4 with this build fix in it, thanks Tim for letting me know.
>
> Hm, no, I don't think I'll really trust that patch for a -stable
> release, it looks like:
>   sound/soc/codecs/wm5100-tables.c | 1488 +++++++++++++++++++--------------------
>   sound/soc/codecs/wm5100.c        |   51 +
>   sound/soc/codecs/wm5100.h        |    7
>   3 files changed, 787 insertions(+), 759 deletions(-)
>
> Which is a bit big.
>
> Mark, any objection to me just reverting the SOC patch that causes this build
> failure instead?
>
> thanks,
>
> greg k-h

I think you'll have to revert both WM5100 patches that came in with v3.2.3:

78fd75340768980ce6ca646106762a6928289e0c ASoC: Don't go through cache 
when applying WM5100 rev A updates
11a17e56ac9c607c6eaecf64b618cd17c828ade0 ASoC: Mark WM5100 register map 
cache only when going into BIAS_OFF

Adding the dependent patch bd132ec585c498ee27d7eedf8569703606743928 does 
allow the compile to succeed. But, as you pointed out, its a huge patch 
and doesn't really qualify for stable.

rtg
--
Tim Gardner tim.gardner@xxxxxxxxxxxxx
--
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/