From: Michal Orzel <michal.orzel@amd.com>
To: <xen-devel@lists.xenproject.org>
Cc: Michal Orzel <michal.orzel@amd.com>,
Stefano Stabellini <sstabellini@kernel.org>,
Julien Grall <julien@xen.org>,
Bertrand Marquis <bertrand.marquis@arm.com>,
Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
Subject: [PATCH 0/4] xen/arm: pl011: Use correct accessors
Date: Wed, 7 Jun 2023 11:27:23 +0200 [thread overview]
Message-ID: <20230607092727.19654-1-michal.orzel@amd.com> (raw)
At the moment we have a real mix of accessors used for PL011 registers across
early printk and runtime driver resulting in a) not spec compliant behavior
and b) inconsistency. This series:
- switches to use the 8/16-bit accessors in normal case,
- adds support to use 32-bit only accessors.
The behavior is now the same as in Linux.
The last patch adding support for SBSA UART in dt boot was added to make
the merge process easier. Also, arm,sbsa-uart property is usually present
next to arm,pl011 which would casue the pl011 driver to believe it is driving
PL011 and not SBSA, therefore resulting in accessing the registers that shall
not be touched (as oppose to just failing).
Discussion:
https://lore.kernel.org/xen-devel/b31a9f06-1ad8-b882-2fb0-84a84a1accb8@xen.org/T/#
Michal Orzel (4):
xen/arm: debug-pl011: Use correct accessors
xen/arm: debug-pl011: Add support for 32-bit only MMIO
xen/arm: pl011: Use correct accessors
xen/arm: pl011: Add SBSA UART device-tree support
docs/misc/arm/early-printk.txt | 3 ++
xen/arch/arm/Kconfig.debug | 7 ++++
xen/arch/arm/arm32/debug-pl011.inc | 12 +++---
xen/arch/arm/arm64/debug-pl011.inc | 12 +++---
xen/arch/arm/include/asm/pl011-uart.h | 19 +++++++++
xen/drivers/char/pl011.c | 57 ++++++++++++++++++++++++---
6 files changed, 92 insertions(+), 18 deletions(-)
--
2.25.1
next reply other threads:[~2023-06-07 9:28 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-07 9:27 Michal Orzel [this message]
2023-06-07 9:27 ` [PATCH 1/4] xen/arm: debug-pl011: Use correct accessors Michal Orzel
2023-06-13 2:59 ` Henry Wang
2023-06-15 1:36 ` Stefano Stabellini
2023-06-15 6:41 ` Michal Orzel
2023-06-15 21:03 ` Stefano Stabellini
2023-06-07 9:27 ` [PATCH 2/4] xen/arm: debug-pl011: Add support for 32-bit only MMIO Michal Orzel
2023-06-13 2:59 ` Henry Wang
2023-06-15 1:42 ` Stefano Stabellini
2023-06-07 9:27 ` [PATCH 3/4] xen/arm: pl011: Use correct accessors Michal Orzel
2023-06-13 2:59 ` Henry Wang
2023-06-15 1:49 ` Stefano Stabellini
2023-06-07 9:27 ` [PATCH 4/4] xen/arm: pl011: Add SBSA UART device-tree support Michal Orzel
2023-06-13 3:00 ` Henry Wang
2023-06-15 1:50 ` Stefano Stabellini
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=20230607092727.19654-1-michal.orzel@amd.com \
--to=michal.orzel@amd.com \
--cc=Volodymyr_Babchuk@epam.com \
--cc=bertrand.marquis@arm.com \
--cc=julien@xen.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.