[alsa-devel] Merged upstream to topic/asoc branch
tiwai at suse.de
Mon Dec 1 20:11:19 CET 2008
At Mon, 01 Dec 2008 18:32:27 +0100,
> At Mon, 1 Dec 2008 17:24:55 +0000,
> Mark Brown wrote:
> > On Mon, Dec 01, 2008 at 06:14:12PM +0100, Takashi Iwai wrote:
> > > Mark Brown wrote:
> > > > Hrm. Which compiler are you using?
> > > gcc-4.1 for ARM, if I remember correctly...
> > I'm at 4.2.4 for ARM.
> > > > The WM8903 warning patch you
> > > > introduced is not needed on any of my systems (primarily ARM and
> > > > x86-64). It smells like a problem with the BUG() definition on your
> > > > platform...
> > > Possibly. I'm not pretty sure on which systems, since I run test-builds
> > > regularly for ARM, PPC, x86 and x86-64 with cross compilers.
> > Could you investigate, please? I'd rather not add annotations like that
> > since they mask genuine compiler warnings and are liable to crop up
> > elsewhere. That particular function isn't likely to change but I'm
> > worried in case this sort of thing starts cropping up elsewhere.
> > For that particular function I'd rather add return statements after the
> > BUG()s...
> Yes, this would work, too.
> > As far as I'm aware the current ASoC code is warning free.
> OK, I'll test through all builds again.
Through all builds, I couldn't reproduce the warnings again.
So I reverted the commit. Sorry for noises.
More information about the Alsa-devel