mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Bjorn Helgaas <bjorn.helgaas@hp.com>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: mingo@redhat.com, linux-kernel@vger.kernel.org,
	yinghai@kernel.org, just.for.lkml@googlemail.com,
	tglx@linutronix.de, hpa@linux.intel.com,
	Feng Tang <feng.tang@intel.com>, Alan Cox <alan@linux.intel.com>,
	greg@kroah.com
Subject: Re: Early console confusion
Date: Wed, 22 Sep 2010 10:44:58 -0600	[thread overview]
Message-ID: <201009221044.58735.bjorn.helgaas@hp.com> (raw)
In-Reply-To: <4C9A271F.30608@zytor.com>

On Wednesday, September 22, 2010 09:56:15 am H. Peter Anvin wrote:
> On 09/22/2010 08:12 AM, Bjorn Helgaas wrote:
> > Can somebody explain the differences between
> > 
> > 	earlycon=uart,...
> > 	earlycon=uart8250,...
> > 	earlyprintk=serial,...
> > 	console=uart,...
> > 	console=uart8250,...
> > 
> > I'm not really convinced we need them all, and it's getting a
> > little confusing.
> 
> Some of them come from the embedded industry, where it's felt that you
> have to be able to specify the exact parameters because there are no
> standards, and some come from the PC world where "serial port 0" is a
> well-defined concept and noone wants to bloody call it "uart8250,0x3f8".

OK, my fault for not asking a specific question :-)  Let me try again.

For "earlycon=" and "console=", we support both "uart" and "uart8250".
I see the value of "uart8250", because we might want to support a
non-8250 UART someday.  But keeping "uart" just seems confusing.

We support both of the following, but I can't tell what the difference
is or why we need both:

    earlycon=uart8250,io,0x3f8,9600n8
    console=uart8250,io,0x3f8,9600n8

We support both of the following (per kernel-parameters.txt, anyway),
but again, I don't know what the difference is:

    earlyprintk=serial,ttyS0
    earlyprintk=ttyS0

If we don't like "uart8250,io,0x3f8" (which I agree is awfully clunky),
we could easily add something like "console=com1" which encapsulated
the well-defined PC concept.  It would be similar to "earlyprintk=ttyS0",
and if it could be made *enough* like it, maybe "earlyprintk=ttyS0"
could be dropped.

I also see the recent "earlyprintk=hsu" patch from  Feng Tang, and I
wonder whether that could be recast in the "console=hsu" style.  Like
"com1", "hsu" is apparently enough to specifically identify a device.

I think it's nicer to extend "console=" in ways that allow early
devices, because then users don't have to mess with both "console="
and "earlyprintk=".  The 8250_early driver has a reasonable model
of switching from the early device to the real one automatically,
so maybe something like this is all we would need:

    console=ttyS0  (late only, use whatever device becomes ttyS0)
    console=com1   (early and late, use I/O port 0x3f8 on PCs)
    console=hsu    (early and late, use Medfield UART)
    console=uart8250,mmio,...  (early and late, for MMIO UARTs)
    console=uartXYZ,mmio,...   (early and late, for non-8250 UARTs)
    console=vga    (early and late, use legacy VGA (switch to tty0 late))
    console=tty0   (late only)

Bjorn

  reply	other threads:[~2010-09-22 16:45 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-29 17:25 Linux 2.6.36-rc3 Linus Torvalds
2010-08-29 19:24 ` Chris Ball
2010-08-29 19:24 ` Torsten Kaiser
2010-08-29 22:27   ` Yinghai Lu
2010-08-29 22:36     ` Yinghai Lu
2010-08-29 22:53   ` Yinghai Lu
2010-08-30  4:50     ` Torsten Kaiser
2010-08-30  1:12   ` [PATCH] x86, setup: fix earlyprintk=serial,ttyS0,115200 Yinghai Lu
2010-08-30  1:13     ` [PATCH] x86, setup: fix earlyprintk=serial,0x3f8,115200 Yinghai Lu
2010-09-08 21:28       ` H. Peter Anvin
2010-09-09  7:23         ` Yinghai Lu
2010-09-21 19:58       ` [tip:x86/urgent] x86, setup: Fix earlyprintk=serial,0x3f8,115200 tip-bot for Yinghai Lu
2010-09-21 19:58     ` [tip:x86/urgent] x86, setup: Fix earlyprintk=serial,ttyS0,115200 tip-bot for Yinghai Lu
2010-09-22 15:12       ` Early console confusion Bjorn Helgaas
2010-09-22 15:56         ` H. Peter Anvin
2010-09-22 16:44           ` Bjorn Helgaas [this message]
2010-09-22 16:53             ` H. Peter Anvin
2010-09-22 18:21               ` Bjorn Helgaas
2010-09-22 18:26                 ` H. Peter Anvin
2010-08-29 22:40 ` Linux 2.6.36-rc3 Jean-Denis Girard
2010-08-30  5:23   ` Borislav Petkov
2010-08-30  8:00     ` Jean-Denis Girard
2010-08-30  8:47 ` Miguel Calleja
2010-08-30 17:00   ` Suresh Jayaraman
2010-08-30 17:14     ` Shirish Pargaonkar
2010-08-30 17:25       ` Randy Dunlap
2010-08-30 21:27         ` Steve French
2010-08-30 19:28 ` Piotr Hosowicz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201009221044.58735.bjorn.helgaas@hp.com \
    --to=bjorn.helgaas@hp.com \
    --cc=alan@linux.intel.com \
    --cc=feng.tang@intel.com \
    --cc=greg@kroah.com \
    --cc=hpa@linux.intel.com \
    --cc=hpa@zytor.com \
    --cc=just.for.lkml@googlemail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=tglx@linutronix.de \
    --cc=yinghai@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®