linux-kernel - Re: [PATCH] Enable console on PCI serial devices

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 20 Oct 2008 11:43:51 -0600
From:	n0ano@...no.com
To:	Bjorn Helgaas <bjorn.helgaas@...com>
Cc:	Lennart Sorensen <lsorense@...lub.uwaterloo.ca>, n0ano@...no.com,
	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 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 :-)

-- 
Don Dugger
"Censeo Toto nos in Kansa esse decisse." - D. Gale
n0ano@...no.com
Ph: 303/443-3786
--
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/