diff mbox

[U-Boot] x86: conga-qeval20-qa3: Add README to explain the console UART options

Message ID 20160905090326.9881-1-sr@denx.de
State Superseded
Delegated to: Bin Meng
Headers show

Commit Message

Stefan Roese Sept. 5, 2016, 9:03 a.m. UTC
This patch adds a small README to explain the 2 defconfig files and its
usage for the different console UART options.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Bin Meng <bmeng.cn@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
---
 board/congatec/conga-qeval20-qa3-e3845/README | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100644 board/congatec/conga-qeval20-qa3-e3845/README

Comments

Bin Meng Sept. 6, 2016, 1:57 a.m. UTC | #1
Hi Stefan,

On Mon, Sep 5, 2016 at 5:03 PM, Stefan Roese <sr@denx.de> wrote:
> This patch adds a small README to explain the 2 defconfig files and its
> usage for the different console UART options.
>
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Bin Meng <bmeng.cn@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
> ---
>  board/congatec/conga-qeval20-qa3-e3845/README | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>  create mode 100644 board/congatec/conga-qeval20-qa3-e3845/README
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

One nits below:

> diff --git a/board/congatec/conga-qeval20-qa3-e3845/README b/board/congatec/conga-qeval20-qa3-e3845/README
> new file mode 100644
> index 0000000..a784f3c
> --- /dev/null
> +++ b/board/congatec/conga-qeval20-qa3-e3845/README
> @@ -0,0 +1,23 @@
> +------------------------------
> +U-Boot console UART selection:
> +------------------------------
> +
> +The U-Boot port for this congatec board currently supports two different
> +configurations (defconfig files). The only difference is the UART that
> +is used as the U-Boot console UART. The default defconfig file:
> +
> +conga-qeval20-qa3-e3845_defconfig
> +
> +provides this console on the UART0 which is provided via a Winbond
> +Super-IO chip connected on the congatec Qseven 2.0 evaluation carrier
> +board (conga-QEVAL). This UART is the one provided with a SubD9
> +connector on the mainboard (the low one). The 2nd defconfig file:
> +
> +conga-qeval20-qa3-e3845-internal-uart_defconfig
> +
> +provides the U-Boot console on the BayTrail internal legacy UART.

nits: the ending period should be ,

> +Which is routed from the QSeven SoM to the X300 connector on the

nits: Which -> which

> +baseboard. Here is called COM2. The baseboard already provides the
> +RS232 level shifters. So a TTL-USB UART adapter does not work in
> +this case. The signals need to get connected directly to the
> +RS232 level signals of the PC UART via some adapter cable.
> --

Regards,
Bin
diff mbox

Patch

diff --git a/board/congatec/conga-qeval20-qa3-e3845/README b/board/congatec/conga-qeval20-qa3-e3845/README
new file mode 100644
index 0000000..a784f3c
--- /dev/null
+++ b/board/congatec/conga-qeval20-qa3-e3845/README
@@ -0,0 +1,23 @@ 
+------------------------------
+U-Boot console UART selection:
+------------------------------
+
+The U-Boot port for this congatec board currently supports two different
+configurations (defconfig files). The only difference is the UART that
+is used as the U-Boot console UART. The default defconfig file:
+
+conga-qeval20-qa3-e3845_defconfig
+
+provides this console on the UART0 which is provided via a Winbond
+Super-IO chip connected on the congatec Qseven 2.0 evaluation carrier
+board (conga-QEVAL). This UART is the one provided with a SubD9
+connector on the mainboard (the low one). The 2nd defconfig file:
+
+conga-qeval20-qa3-e3845-internal-uart_defconfig
+
+provides the U-Boot console on the BayTrail internal legacy UART.
+Which is routed from the QSeven SoM to the X300 connector on the
+baseboard. Here is called COM2. The baseboard already provides the
+RS232 level shifters. So a TTL-USB UART adapter does not work in
+this case. The signals need to get connected directly to the
+RS232 level signals of the PC UART via some adapter cable.