Re: Current saa7134 driver breaks KNC One Tv-Station DVR (card=24)

From: Johannes Stezenbach (js_at_linuxtv.org)
Date: 12/31/04

  • Next message: Jesper Juhl: "Re: [patch] add loglevel to a few printk's in arch/m32r/kernel/ptrace.c"
    Date:	Fri, 31 Dec 2004 03:08:14 +0100
    To: linux-kernel@vger.kernel.org
    
    

    Bernhard Rosenkraenzer wrote:
    > On Tuesday 28 December 2004 18:21, Bernhard Rosenkraenzer wrote:
    > > Trying to modprobe saa7134 with this card results in a hanging modprobe
    > > process.
    >
    > Some more debug info (with 2.6.10-ac1):
    >
    > saa7130/34: v4l2 driver version 0.2.12 loaded
    > ACPI: PCI interrupt 0000:00:06.0[A] -> GSI 17 (level, low) -> IRQ 177
    > saa7134[0]: found at 0000:00:06.0, rev: 1, irq: 177, latency: 64, mmio: 0xdfff9c00
    > saa7134[0]: subsystem: 1894:a006, board: KNC One TV-Station DVR [card=24,autodetected]
    ...
    > saa7134: Loading i2c helpers<7>saa7134[0]: i2c xfer: < c0 >
    > tuner: chip found at addr 0xc0 i2c-bus saa7134[0]
    > tuner: type set to 38 (Philips PAL/SECAM multi (FM1216ME MK3)) by saa7134[0]
    ...
    > tda9885/6/7: chip found @ 0x86
    ...
    > saa7134[0]/irq: looping -- clearing PE (parity error!) enable bit
    > saa7134[0]/irq[0,-227141]: r=0x20 s=0x00 PE
    > [modprobe hangs]
    >
    > Looks like it's hanging at request_module("saa7134-empress");

    I've reported this before:
    http://lkml.org/lkml/2004/11/17/275

    The patch that Rusty posted doesn't seem to have it made into
    module-init-tools-3.1. Bummer :-(

    The Bug in saa7134-core.c doesn't seem to have been addressed yet, so
    I suggest the following patch as a temporary measure:

    --- linux-2.6.10/drivers/media/video/saa7134/saa7134-core.c.orig 2004-12-25 19:22:55.000000000 +0100
    +++ linux-2.6.10/drivers/media/video/saa7134/saa7134-core.c 2004-12-25 19:23:31.000000000 +0100
    @@ -940,12 +940,12 @@ static int __devinit saa7134_initdev(str
                     request_module("tuner");
             if (dev->tda9887_conf)
                     request_module("tda9887");
    - if (card_is_empress(dev)) {
    - request_module("saa7134-empress");
    + if (card_is_empress(dev)) {
    +// request_module("saa7134-empress");
                     request_module("saa6752hs");
             }
    - if (card_is_dvb(dev))
    - request_module("saa7134-dvb");
    +// if (card_is_dvb(dev))
    +// request_module("saa7134-dvb");
     
             v4l2_prio_init(&dev->prio);
     

    Johannes
    -
    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: Jesper Juhl: "Re: [patch] add loglevel to a few printk's in arch/m32r/kernel/ptrace.c"

    Relevant Pages

    • Re: make checkstack on m68k
      ... here is a patch against pristine 2.6.7: ... when I'm talking to journalists I just say "programmer" or something like that. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: 2.6.7-mm1
      ... > Bummer. ... Does this reversion fix it? ... With this patch it works, but it works in that way, that the wrong ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: 2.6.10-rc1-mm3: modprobe oopses [x86]
      ... Bernhard Rosenkraenzer writes: ... Apply this patch. ... send the line "unsubscribe linux-kernel" in ... Please read the FAQ at http://www.tux.org/lkml/ ...
      (Linux-Kernel)
    • Re: [parisc-linux] Re: [PATCH 3/9] mm: parisc pte atomicity
      ... using your own tmpalias area sounds much better than getting ... I've simply not wrapped my head around the races, ... it looks like we agree that my patch is necessary and valid as is; ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: keyboard - was: Re: Linux 2.6.0-test4
      ... >> I was able to get the key unstuck by switching back and forth between ... I rebuild my kernel including your patch; ... I'll get back to you once I verify that the problem doesn't occur ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)