Re: [PATCH] sc1200_wdt: Fix oops

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

 



On Wed, 4 Dec 2013 08:31:18 -0800
Guenter Roeck <linux@xxxxxxxxxxxx> wrote:

> On Wed, Dec 04, 2013 at 03:31:52PM +0000, Alan wrote:
> > If loaded with isapnp = 0 the driver explodes. This is catching
> > people out now and then. What should happen in the working case is
> > a complete mystery and the code appears terminally confused, but we
> 
> Looks like it uses the pnp infrastructure only to auto-detect the
> "io" address, and assumes that pnp_register_driver() calls the
> pnp probe function immediately. Afterwards it unregisters and
> works (or is supposed to work) outside the pnp subsystem. Brr.
> 
> > can at least make the error path work properly.
> > 
> > Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
> > Partially-Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=53991
> 
> Actually, I think it fully resolves it, because the other part (sc520 driver
> not working on SC1100 CPU) isn't a bug. It isn't working on an ARM or PPC CPU
> either, nor on any Intel CPU.

As far as I can tell the fact it fails on the sc1100 is also a bug - but
the mess of idefs around PNP is impossible to figure out without some
actual docs, time and real hardware.

Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux