Re: [PATCH] hda empty modelname comparison

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

 



At Wed, 07 Mar 2007 22:20:20 +0100,
I wrote:
> 
> At Wed, 07 Mar 2007 12:04:30 -0800,
> Tobin Davis wrote:
> > 
> > On Tue, 2007-03-06 at 15:59 +0100, Joachim Deguara wrote:
> > 
> >     This patch prevents a comparison between a possibly empty modelname
> >     string in the hda code.
> >     
> >     -Joachim
> > 
> > Unfortunately, the way the original patch was written, the only options are
> > generic or segfault.  Your patch just removes the segfault, sending all cards
> > to the generic parser.
> > 
> > I've tested a slightly different approach, based on your empty test, just
> > reverse logic.  Test it to confirm that it works.  It works on my system with
> > a Conexant reference board and a Sigmatel STAC9227 (both on the same system).
> > 
> > Signed off by Tobin Davis <tdavis@xxxxxxxxxxxx>                               
> That's my bad.  I haven't tested much, and unfortunately, 1.0.14rc3
> was released while I've been in vacation soon after the wrong patch.
> 
> The fix patch below is already on my local tree, but it couldn't be
> pushed because the ALSA HG tree is currently down (I guess Jaroslav is
> working on his server).  If needed, we'll release the fixed version.
> So stay tuned.

Now fixed on HG tree.


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/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