lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 20 Oct 2008 13:11:05 -0600 From: Bjorn Helgaas <bjorn.helgaas@...com> To: n0ano@...no.com Cc: Lennart Sorensen <lsorense@...lub.uwaterloo.ca>, Alan Cox <alan@...rguk.ukuu.org.uk>, donald.d.dugger@...el.com, linux-kernel@...r.kernel.org Subject: Re: [PATCH] Enable console on PCI serial devices On Monday 20 October 2008 11:43:51 am n0ano@...no.com wrote: > On Mon, Oct 20, 2008 at 11:11:07AM -0600, Bjorn Helgaas wrote: > >... > > It doesn't bother me that the number is large. It's only a factor > > of 16 bigger than the baud rate. It is sort of magic, but on the > > other hand, the number is often printed on a part on the board, so > > it's easy to determine, and fairly easy to document. If you've > > lost the box with the marketing messages on it, I'd argue that it's > > easier to determine the clock rate than the fastest baud rate. > > > > To me it seems more confusing to specify two baud rates: the one > > you want to use, and the fastest one the UART supports. > > Well, how about consistency with the current serial driver. The > current driver uses base baud, as evidenced by this code from > `drivers/serial/8250_pci.c': > > static struct pciserial_board pci_boards[] __devinitdata = { > [pbn_default] = { > .flags = FL_BASE0, > .num_ports = 1, > .base_baud = 115200, > .uart_offset = 8, > }, > > I think it's even more important to follow what the driver is > doing. (If I'd lost the bix I would get the PCI IDs and see > what the driver is doing for my unknown card :-) Ideally we could put something in Documentation/kernel-parameters.txt that's useful for people who can't or don't want to look at the source. Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists