All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Orzel, Michal" <michal.orzel@amd.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: "Andrew Cooper" <andrew.cooper3@citrix.com>,
	"Anthony PERARD" <anthony.perard@vates.tech>,
	"Julien Grall" <julien@xen.org>,
	"Roger Pau Monné" <roger@xenproject.org>,
	"Stefano Stabellini" <sstabellini@kernel.org>,
	xen-devel@lists.xenproject.org
Subject: Re: [PATCH 1/3] cmdline: Document console=dtuart option
Date: Wed, 2 Sep 2026 13:12:13 +0200	[thread overview]
Message-ID: <37875a01-a4fd-4c2a-8ae0-df88c769b9f1@amd.com> (raw)
In-Reply-To: <37872f45-8078-44ef-9db8-a9868c9b79aa@suse.com>



On 02-Sep-26 10:55, Jan Beulich wrote:
> On 02.09.2026 09:36, Michal Orzel wrote:
>> Document the default console= option on Arm and RISC-V that is
>> "dtuart" indicating a generic UART parsed from a device tree or
>> ACPI SPCR table. This serial utilizes SERHND_DTUART handle.
>>
>> Take the opportunity to:
>>  - fix documented x86 default to vga to match OPT_CONSOLE_STR,
> 
> I first wanted to object to this part, as I was sure this used to be
> "com1,vga". Yet indeed it's been almost 19 years ago when this was
> changed (92877a1f9ab2 ["x86: Auto-probe the serial port baud rate if
> 'com1' or 'com2' is"]).
> 
>>  - mark dtuart option as available on RISC-V,
>>  - document fall back to /chosen/stdout-path.
>>
>> Signed-off-by: Michal Orzel <michal.orzel@amd.com>
> 
> Acked-by: Jan Beulich <jbeulich@suse.com>
> 
>> --- a/docs/misc/xen-command-line.pandoc
>> +++ b/docs/misc/xen-command-line.pandoc
>> @@ -430,9 +430,10 @@ The following are examples of correct specifications:
>>  Specify the size of the console ring buffer.
>>  
>>  ### console
>> -> `= List of [ vga | com1[H,L] | com2[H,L] | pv | dbgp | ehci | xhci | none ]`
>> +> `= List of [ vga | com1[H,L] | com2[H,L] | pv | dbgp | ehci | xhci | dtuart | none ]`
>>  
>> -> Default: `console=com1,vga`
>> +> Default (x86): `console=vga`
>> +> Default (Arm, RISC-V): `console=dtuart`
> 
> Any reason to not also cover PPC here?
While PPC sets OPT_CONSOLE_STR to "dtuart", it does not select
CONFIG_GENERIC_UART_INIT that compiles in the uart-init.c which is what really
matters to decide whether dtuart is supported or not.

~Michal



  reply	other threads:[~2026-09-02 11:12 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-02  7:36 [PATCH 0/3] dtuart fixes Michal Orzel
2026-09-02  7:36 ` [PATCH 1/3] cmdline: Document console=dtuart option Michal Orzel
2026-09-02  8:55   ` Jan Beulich
2026-09-02 11:12     ` Orzel, Michal [this message]
2026-09-02  7:36 ` [PATCH 2/3] drivers/char: Check if console=dtuart for ACPI SPCR serial bring up Michal Orzel
2026-09-02  8:56   ` Jan Beulich
2026-09-02  7:36 ` [PATCH 3/3] drivers/char: Panic when the requested UART fails to initialise Michal Orzel
2026-09-02  9:44   ` Halder, Ayan Kumar
2026-09-08 10:15     ` Halder, Ayan Kumar

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=37875a01-a4fd-4c2a-8ae0-df88c769b9f1@amd.com \
    --to=michal.orzel@amd.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=anthony.perard@vates.tech \
    --cc=jbeulich@suse.com \
    --cc=julien@xen.org \
    --cc=roger@xenproject.org \
    --cc=sstabellini@kernel.org \
    --cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.