lkml.org 
[lkml]   [2004]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] smsc-ircc2: Add PnP support.
From
On Thu, Nov 18, 2004 at 10:41:45PM +0200, Ville Syrjälä wrote:
> On Thu, Nov 18, 2004 at 08:49:50PM +0100, matthieu castet wrote:
> > Jean Tourrilhes wrote:
> > >On Thu, Nov 18, 2004 at 07:42:07PM +0100, matthieu castet wrote:
> > >>>>On3) If the ressources are markes as disabled, you just quit
> > >>>>with an error. Compouded with (2), this makes me doubly
> > >>>>nervous. Wouldn't it be possible to forcefully enable those
> > >>
> > >>ressources ?
> > >>pnp should call automatiquely pnp_activate_dev() before probing the
> > >>driver, so the resource should be activated. Have you got an example
> > >>where the resource wheren't activated ?
> > >
> > >
> > > No, it was more that I don't understand what PnP does for
> > >us. I don't have a SMS chipset to test on. Also, I would like to know
> > >if it remove the need of smcinit.
> > >
> > PnP is easy to understand ;)
> > When you probe a device, it will activate a device with the best
> > configuration available.
>
> So can we just remove the IORESOURCE_DISABLED tests?
>
> And what about the pnp_*_valid() tests?
>
> parport_pc (which I used as a guide) does both tests but 8250_pnp doesn't
> do either.

Parport_pc uses them because resources could potentially be disabled, but the
parport could still be functional. Therefore, it must check. It is probably
ok to not check on most hardware. Nonetheless, it's best to play it safe, and
always verify the resource configuration.

Thanks,
Adam
-
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/

\
 
 \ /
  Last update: 2005-03-22 14:08    [W:0.097 / U:7.460 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site