On Sun, 22 May 2005 11:58:23 -0700 David Brownell <[EMAIL PROTECTED]> wrote:
> > From a quick glance it seems that the data/addr resources are > > requested twice, both in sl811_cs via platform_device_register() and > > then in the probe function of the hcd driver but the release order is > > different, > > The release order of those resources doesn't matter; they're > independent of each other. > ... > Resources are branches in a tree. And that's where the problem lies I believe. __release_resource() does WARN_ON(old->child). Since the hcd-driver's registered resource is the child of the one registered by platform_device_register(), WARN_ON is triggered because the latter is being released first. But I might be misunderstanding something. > And I think the issue there is that the platform device isn't > getting unregistered at the right spot. well, yes ;) It should be done after sl811h_remove is called I believe. Now how do you do that? > > The second issue is some irq problem that the new changes in > > sl811-hcd.c introduced. > > When plugging in a usb device, it is recognized only after a few > > seconds of delay and unplugging goes unnoticed frequently, resulting > > in various errors. See the attached log snippet. > > That's pretty wierd. Maybe one of the relevant boards is wiring the > IRQ line up through an inverter or FPGA or something, and others aren't. > (I saw that recently with smc91x ...) How does IRQT_FALLING act? Without touching the irq with irq_set_type, it works fine. A few weeks ago when I started playing with the sl811_cs code, I was told on #irc that the sharp pcmcia/scoop interface needs to be setup in order to be able to access the CF card otherwise it won't work. This driver being my first shot at kernel hacking (well it's obvious;), not to mention arm/embedded, I was happy that I didn't need to do anything with the scoop/pcmcia stuff and the driver worked. But the irq setup needs to be done there probably. For now we could use #ifndef CONFIG_PXA_SHARPSL for the Zaurus until this issue is fixed in the right place. I wonder about the rest of the PDAs though. ------------------------------------------------------- This SF.Net email is sponsored by Oracle Space Sweepstakes Want to be the first software developer in space? Enter now for the Oracle Space Sweepstakes! http://ads.osdn.com/?ad_id=7412&alloc_id=16344&op=click _______________________________________________ [email protected] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
