diff mbox series

[v3] xen/arm: Add Amlogic Meson SoCs earlyprintk support

Message ID 1555223611-10263-1-git-send-email-amittomer25@gmail.com (mailing list archive)
State Superseded
Headers show
Series [v3] xen/arm: Add Amlogic Meson SoCs earlyprintk support | expand

Commit Message

Amit Tomer April 14, 2019, 6:33 a.m. UTC
This patch adds earlyprintk support for Amlogic Meson SoC based
boards.

It relies on firmware[1] to initialize the UART.

Tested With:
 http://wiki.friendlyarm.com/wiki/index.php/NanoPi_K2

[1]: https://github.com/ARM-software/arm-trusted-firmware/blob/master/drivers/meson/console/aarch64/meson_console.S#L92

Signed-off-by: Amit Singh Tomar <amittomer25@gmail.com>
---
Changes since v2:

        * Added the commit message that talks about
          uart initialization.
    
        * Added dummy early_uart_init function.

Changes since v1:

        * Fixed coding style issue.
        * Undone changes in early-printk.txt.

Changes since RFC:

        * Replaced LDRH with LDR, with this there
          is no scattered output on console now.
        * Used tbnz instead of tst and b.ne.
        * Used AML_ prefix against register names.
---
 xen/arch/arm/arm64/debug-meson.inc | 55 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 55 insertions(+)
 create mode 100644 xen/arch/arm/arm64/debug-meson.inc

Comments

Julien Grall April 14, 2019, 4:10 p.m. UTC | #1
Hi Amit,

On 4/14/19 7:33 AM, Amit Singh Tomar wrote:
> This patch adds earlyprintk support for Amlogic Meson SoC based
> boards.
> 
> It relies on firmware[1] to initialize the UART.

Do all the firmware existing for this platform initialize the UART?

Cheers,
Amit Tomer April 14, 2019, 4:35 p.m. UTC | #2
Hello,

> Do all the firmware existing for this platform initialize the UART?

U-boot does also initialize the uart for this platform.
https://github.com/u-boot/u-boot/blob/master/drivers/serial/serial_meson.c#L44

Thanks,
-Amit
Julien Grall April 14, 2019, 4:39 p.m. UTC | #3
On 4/14/19 5:35 PM, Amit Tomer wrote:
> Hello,

Hi,

>> Do all the firmware existing for this platform initialize the UART?
> 
> U-boot does also initialize the uart for this platform.
> https://github.com/u-boot/u-boot/blob/master/drivers/serial/serial_meson.c#L44

How about replacing "It relies on firmware..." by "ATF and U-boot 
already initialize the UART for us. So no need to do it again."

So the user at least know what has been verified.

Cheers,
diff mbox series

Patch

diff --git a/xen/arch/arm/arm64/debug-meson.inc b/xen/arch/arm/arm64/debug-meson.inc
new file mode 100644
index 0000000..e1a8fa8
--- /dev/null
+++ b/xen/arch/arm/arm64/debug-meson.inc
@@ -0,0 +1,55 @@ 
+/*
+ * xen/arch/arm/arm64/debug-meson.inc
+ *
+ * MESON specific debug code.
+ *
+ * Copyright (c) 2019, Amit Singh Tomar <amittomer25@gmail.com>.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms and conditions of the GNU General Public
+ * License, version 2, as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public
+ * License along with this program; If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#define AML_UART_WFIFO_REG      0x00
+#define AML_UART_STATUS_REG     0x0c
+
+#define AML_UART_TX_FIFO_FULL   21
+
+.macro early_uart_init xb, c
+/* Uart has already been initialized by Firmware, for instance by TF-A */
+.endm
+
+/*
+ * MESON UART wait UART to be ready to transmit
+ * xb: register which contains the UART base address
+ * c: scratch register
+ */
+.macro early_uart_ready xb c
+1:
+        ldr     w\c, [\xb, #AML_UART_STATUS_REG]        /* status register */
+        tbnz    w\c, #AML_UART_TX_FIFO_FULL, 1b         /* Check TXFIFO FULL bit */
+.endm
+
+/*
+ * MESON UART transmit character
+ * xb: register which contains the UART base address
+ * wt: register which contains the character to transmit
+ */
+.macro early_uart_transmit xb wt
+        str    \wt, [\xb, #AML_UART_WFIFO_REG]
+.endm
+
+/*
+ * Local variables:
+ * mode: ASM
+ * indent-tabs-mode: nil
+ * End:
+ */