[alsa-devel] [PATCH] wss_lib: snd_wss_calibrate_mute improvement
Takashi Iwai
tiwai at suse.de
Mon Aug 18 12:16:02 CEST 2008
At Mon, 18 Aug 2008 10:50:15 +0200,
Rene Herman wrote:
>
> On 17-08-08 22:38, Krzysztof Helt wrote:
>
> > From: Krzysztof Helt <krzysztof.h1 at wp.pl>
> >
> > Mute sound by setting mute bit without
> > setting volume to 0. It makes both source code
> > and binary shorter.
> >
> > Signed-off-by: Krzysztof Helt <krzysztof.h1 at wp.pl>
>
> Grmmbbled-over-by: Rene Herman <rene.herman at gmail.com>
>
> This should be safe and the current function indeed generates crap code
> but in the sense of always expecting the worst broken hardware variants
> to be out there I must say I'd rather have seen that GCC were beter. But
> yes, sure.
Yeah, the slightest concern is the intention of using 0x80 there.
If this patch was already tested on some real hardwares (and I guess
it should work), I'm willing to apply it. Krzystof, was this tested
on your machines?
Takashi
More information about the Alsa-devel
mailing list