Re: [oops] # rmmod snd_nm256

From: Takashi Iwai (tiwai_at_suse.de)
Date: 12/30/04

  • Next message: Dave Dillow: "Re: [RFC 2.6.10 1/22] xfrm: Add direction information to xfrm_state"
    Date:	Thu, 30 Dec 2004 17:12:24 +0100
    To: Paweł Sikora <pluto@pld-linux.org>
    
    

    At Thu, 30 Dec 2004 00:59:42 +0100,
    Paweł Sikora wrote:
    >
    > # rmmod snd_nm256
    > Oops: 0000 [#1]

    Does the patch below fix the problem?

    Takashi

    --- linux/sound/pci/nm256/nm256.c 27 Dec 2004 13:32:57 -0000 1.61
    +++ linux/sound/pci/nm256/nm256.c 30 Dec 2004 16:10:00 -0000
    @@ -1486,12 +1486,6 @@
     
             snd_nm256_init_chip(chip);
     
    - if ((err = snd_nm256_pcm(chip, 0)) < 0)
    - goto __error;
    -
    - if ((err = snd_nm256_mixer(chip)) < 0)
    - goto __error;
    -
             // pci_set_master(pci); /* needed? */
             
             snd_card_set_pm_callback(card, nm256_suspend, nm256_resume, chip);
    @@ -1612,6 +1606,12 @@
                     chip->reset_workaround = 1;
             }
     
    + if ((err = snd_nm256_pcm(chip, 0)) < 0 ||
    + (err = snd_nm256_mixer(chip)) < 0) {
    + snd_card_free(card);
    + return err;
    + }
    +
             sprintf(card->shortname, "NeoMagic %s", card->driver);
             sprintf(card->longname, "%s at 0x%lx & 0x%lx, irq %d",
                     card->shortname,
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/


  • Next message: Dave Dillow: "Re: [RFC 2.6.10 1/22] xfrm: Add direction information to xfrm_state"

    Relevant Pages

    • Re: BIOS overwritten during resume (was: Re: Asus L5D resume on battery power)
      ... >> and you probably need to fix those, ... I think I'll just port the Nigel's patch to x86-64. ... People were complaining that M$ turns users into beta-testers... ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] fix readahead breakage for sequential after random reads
      ... > yes I will run all my standard testsuites before we take this patch. ... I have also enclosed a patch that partially backs off Miklos's fix. ... ra->average value to max/2 when we move from readahead-off mode to ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: IA32 - 27 New warnings
      ... > | needed to confirm the patch and preferably some brave soul with Promise ... > Bart has already posted patches for this. ... I'm off looking for more resonably simple stuff to fix. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • 2.6.0-test7 byteorder.h ( as you go AGAIN! )
      ... seen many patches to attempt to fix this issue. ... Attached is my proposed patch to correct this error. ... all Linux CD-ROM application programs will use this ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: ALPS psmouse_reset on reconnect confusing Tecra M2
      ... > It's hard to believe your patch ... > can fix it, because the ALPS_DUALPOINT constant doesn't affect the ... > initalization behavior at all, only the way how TouchPoint data are ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)