Re: [PATCH] serial: do not add port that is not initialized

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

 



On Wed, Jul 11, 2007 at 10:05:53AM -0600, Bjorn Helgaas wrote:
> On Wednesday 11 July 2007 01:17:42 am Russell King wrote:
> > On Tue, Jul 10, 2007 at 05:03:35PM -0700, Yinghai Lu wrote:
> > > [PATCH] serial: do not add port that is not initialized
> > > 
> > > if the port is not initialized with correct iobase, and membase, we don't
> > > need to add that port.
> > > for x86, when pnpacpi is enabled, we will not get extra ttyS1/ttyS2/ttyS3 in
> > > /sys/devices/platform/serial8250/tty
> > > 
> > > Sign-off-by: Yinghai Lu <[email protected]>
> > 
> > What if someone wants to run setserial on /dev/ttyS1 to set it's base
> > address, irq and other parameters?  People still do this.
> 
> I agree that people still use setserial that way, and we shouldn't
> make this change because it would break things.
> 
> But I also agree with Yinghai that these extra phantom devices are
> just weird.  It would be a lot nicer if there were a way to say,
> "please, Mr. Serial Driver, create me a new ttyS device with these
> parameters."
> 
> But no, I'm not volunteering to do that.

Indeed, and since distros still haven't picked up my setserial v3,
I see little chance of that ever being practical.  Not that I've
really tried very hard to get them to - no real idea who to contact
to make that happen, and of course I've no interest in making it
happen anymore.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux