--- Lothar Wassmann <[EMAIL PROTECTED]> wrote:
> Hi Michael,
>
> Michael writes:
> > I would disagree. I added a printk at the top of the
> > isp1362_irq function, as you can see in the bootup messages.
> > However, it doesn't show up with the "nobody cared" messages.
> > Perhaps this is a hint in itself.
> >
> Ah. Take a look at drivers/usb/core/hcd.c:
> |irqreturn_t usb_hcd_irq (int irq, void *__hcd, struct pt_regs *
> r)
> |{
> | struct usb_hcd *hcd = __hcd;
> | int start = hcd->state;
> |
> | if (start == USB_STATE_HALT)
> | return IRQ_NONE;
> | if (hcd->driver->irq (hcd, r) == IRQ_NONE)
> | return IRQ_NONE;
> |
> | hcd->saw_irq = 1;
> | if (hcd->state != start && hcd->state == USB_STATE_HALT)
> | usb_hc_died (hcd);
> | return IRQ_HANDLED;
> |}
> >
> This is the 'real' IRQ handler that is registered with
> request_irq()
> by the hcd core and in turn calls the chip specific handler.
>
> Thus your controller obviously generates interrupts while it is
> in the USB_STATE_HALT state. This is rather strange since after
> calling isp1362_hc_start() the HCD should be in
USB_STATE_RUNNING.
>
> You should check, who messes with hcd->state after it has been
> initialized in isp1362_hc_start().
>
..
> You should enable the kernel configuration option CONFIG_KALLSYMS
> to get symbolic stack traces.
>
> Do you have a special reason for building the driver into the
> kernel instead of using a module which wouldn't require a reboot
> for every code change?
>
No, I suppose I don't have a special reason. I'm running my kernel
without any modules at all, so I was sticking to that. I have it
automated to be easy enough to boot with a new kernel. (As easy as
transferring a new module over and trying it out).
Could a module mess with the state so that things don't work as
well as a fresh bootup? I do agree that in most cases developing
with modules makes more sense.
I'll check on your suggestions. Stay tuned.
Thanks a bunch!
Mike
-------------------------------------------------------
SF.Net email is sponsored by: Tell us your software development plans!
Take this survey and enter to win a one-year sub to SourceForge.net
Plus IDC's 2005 look-ahead and a copy of this survey
Click here to start! http://www.idcswdc.com/cgi-bin/survey?id=105hix
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
