Re: [PATCH] wss_lib: snd_wss_calibrate_mute improvement

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxx>
> > 
> > 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@xxxxx>
> 
> Grmmbbled-over-by: Rene Herman <rene.herman@xxxxxxxxx>
> 
> 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
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux