Re: [PATCH] cs4236: add missing pnp_request_card_device()

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

 



On Tue, 22 Jul 2008 23:51:00 +0200
Krzysztof Helt <krzysztof.h1@xxxxxxxxx> wrote:

> From: Krzysztof Helt <krzysztof.h1@xxxxx>
> 
> There is a missing pnp_request_card_device() call
> for cs4232 card. The acard->wss pointer is left
> uninitialized which leads to crash during detection.
> 
> Signed-off-by: Krzysztof Helt <krzysztof.h1@xxxxx>
> ---
> 
> This bug was found in the 2.6.26-git9 kernel.
> 

The patch below is incorrect. Please wait for another one.

Regards,
Krzysztof

> Please compare the snd_card_cs4232_pnp() and
> the snd_card_cs423x_pnpc().
> 
> --- linux-2.6.26/sound/isa/cs423x/cs4236.c~	2008-07-13 23:51:29.000000000 +0200
> +++ linux-2.6.26/sound/isa/cs423x/cs4236.c	2008-07-22 23:41:56.010942922 +0200
> @@ -325,6 +325,9 @@ static int __devinit snd_cs423x_pnp_init
>  static int __devinit snd_card_cs4232_pnp(int dev, struct snd_card_cs4236 *acard,
>  					 struct pnp_dev *pdev)
>  {
> +	acard->wss = pnp_request_card_device(card, id->devs[0].id, NULL);
> +	if (acard->wss == NULL)
> +		return -EBUSY;
>  	if (snd_cs423x_pnp_init_wss(dev, acard->wss) < 0)
>  		return -EBUSY;
>  	cport[dev] = -1;
> 
> ----------------------------------------------------------------------
> Czy Twoj mezczyzna gotuje dla Ciebie? 
> Wygraj nagrody >>> http://link.interia.pl/f1e86 
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 

----------------------------------------------------------------------
Galeria absurdow.
zobacz >>> http://link.interia.pl/f1e5e

_______________________________________________
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