From: Sascha Hauer <s.hauer@pengutronix.de>
To: Peter Rosin <peda@axentia.se>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH] at91: serial: pullup RX pins, do not pullup TX pins
Date: Tue, 14 Feb 2017 08:55:56 +0100 [thread overview]
Message-ID: <20170214075556.gzafi4wt2xqu7whd@pengutronix.de> (raw)
In-Reply-To: <1487005616-19189-1-git-send-email-peda@axentia.se>
On Mon, Feb 13, 2017 at 06:06:56PM +0100, Peter Rosin wrote:
> We have a number of sama5d3 devices that sometimes hangs at the
> barebox prompt during boot due to floating RX pins. This patch
> fixes the problem for us (and probably others). It is similar in
> nature to linux kernel commit 138c2b2f175b ("ARM: dts: at91: fixes
> dbgu pinctrl, set pullup on rx, clear pullup on tx")
>
> While at it, remove pointless waste of power that the pullup of
> the TX pins causes and fix the signal comments for SAMA5D4.
>
> Signed-off-by: Peter Rosin <peda@axentia.se>
> ---
> arch/arm/mach-at91/at91rm9200_devices.c | 20 ++++++++++----------
> arch/arm/mach-at91/at91sam9260_devices.c | 28 ++++++++++++++--------------
> arch/arm/mach-at91/at91sam9261_devices.c | 16 ++++++++--------
> arch/arm/mach-at91/at91sam9263_devices.c | 16 ++++++++--------
> arch/arm/mach-at91/at91sam9g45_devices.c | 20 ++++++++++----------
> arch/arm/mach-at91/at91sam9n12_devices.c | 20 ++++++++++----------
> arch/arm/mach-at91/at91sam9x5_devices.c | 20 ++++++++++----------
> arch/arm/mach-at91/sama5d3_devices.c | 12 ++++++------
> arch/arm/mach-at91/sama5d4_devices.c | 32 ++++++++++++++++----------------
> 9 files changed, 92 insertions(+), 92 deletions(-)
>
> This is resend with the correct barebox ML address (hopefully). Sorry about
> that...
Appearantly it was the correct address ;)
Applied, thanks
Sascha
>
> Cheers,
> peda
>
> diff --git a/arch/arm/mach-at91/at91rm9200_devices.c b/arch/arm/mach-at91/at91rm9200_devices.c
> index 7b68cda..a110ee3 100644
> --- a/arch/arm/mach-at91/at91rm9200_devices.c
> +++ b/arch/arm/mach-at91/at91rm9200_devices.c
> @@ -250,16 +250,16 @@ void __init at91_add_device_spi(int spi_id, struct at91_spi_platform_data *pdata
>
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PA30, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PA31, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PA30, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PA31, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA17, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PA18, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PA17, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PA18, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_CTS)
> at91_set_A_periph(AT91_PIN_PA20, 0); /* CTS0 */
> @@ -277,8 +277,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB20, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PB21, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PB20, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PB21, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RI)
> at91_set_A_periph(AT91_PIN_PB18, 0); /* RI1 */
> @@ -298,8 +298,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA22, 0); /* RXD2 */
> - at91_set_A_periph(AT91_PIN_PA23, 1); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PA22, 1); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PA23, 0); /* TXD2 */
>
> if (pins & ATMEL_UART_CTS)
> at91_set_B_periph(AT91_PIN_PA30, 0); /* CTS2 */
> @@ -311,8 +311,8 @@ resource_size_t __init at91_configure_usart2(unsigned pins)
>
> resource_size_t __init at91_configure_usart3(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PA5, 1); /* TXD3 */
> - at91_set_B_periph(AT91_PIN_PA6, 0); /* RXD3 */
> + at91_set_B_periph(AT91_PIN_PA5, 0); /* TXD3 */
> + at91_set_B_periph(AT91_PIN_PA6, 1); /* RXD3 */
>
> if (pins & ATMEL_UART_CTS)
> at91_set_B_periph(AT91_PIN_PB1, 0); /* CTS3 */
> diff --git a/arch/arm/mach-at91/at91sam9260_devices.c b/arch/arm/mach-at91/at91sam9260_devices.c
> index 67c4ea8..99919b3 100644
> --- a/arch/arm/mach-at91/at91sam9260_devices.c
> +++ b/arch/arm/mach-at91/at91sam9260_devices.c
> @@ -261,16 +261,16 @@ void __init at91_add_device_spi(int spi_id, struct at91_spi_platform_data *pdata
>
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PB14, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PB15, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PB14, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PB15, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB4, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PB5, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PB4, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PB5, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PB26, 0); /* RTS0 */
> @@ -290,8 +290,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB6, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PB7, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PB6, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PB7, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PB28, 0); /* RTS1 */
> @@ -303,8 +303,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB8, 1); /* TXD2 */
> - at91_set_A_periph(AT91_PIN_PB9, 0); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PB8, 0); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PB9, 1); /* RXD2 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PA4, 0); /* RTS2 */
> @@ -316,8 +316,8 @@ resource_size_t __init at91_configure_usart2(unsigned pins)
>
> resource_size_t __init at91_configure_usart3(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB10, 1); /* TXD3 */
> - at91_set_A_periph(AT91_PIN_PB11, 0); /* RXD3 */
> + at91_set_A_periph(AT91_PIN_PB10, 0); /* TXD3 */
> + at91_set_A_periph(AT91_PIN_PB11, 1); /* RXD3 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PC8, 0); /* RTS3 */
> @@ -329,16 +329,16 @@ resource_size_t __init at91_configure_usart3(unsigned pins)
>
> resource_size_t __init at91_configure_usart4(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PA31, 1); /* TXD4 */
> - at91_set_B_periph(AT91_PIN_PA30, 0); /* RXD4 */
> + at91_set_B_periph(AT91_PIN_PA31, 0); /* TXD4 */
> + at91_set_B_periph(AT91_PIN_PA30, 1); /* RXD4 */
>
> return AT91SAM9260_BASE_US4;
> }
>
> resource_size_t __init at91_configure_usart5(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB12, 1); /* TXD5 */
> - at91_set_A_periph(AT91_PIN_PB13, 0); /* RXD5 */
> + at91_set_A_periph(AT91_PIN_PB12, 0); /* TXD5 */
> + at91_set_A_periph(AT91_PIN_PB13, 1); /* RXD5 */
>
> return AT91SAM9260_BASE_US5;
> }
> diff --git a/arch/arm/mach-at91/at91sam9261_devices.c b/arch/arm/mach-at91/at91sam9261_devices.c
> index 620ed65..e63e0e7 100644
> --- a/arch/arm/mach-at91/at91sam9261_devices.c
> +++ b/arch/arm/mach-at91/at91sam9261_devices.c
> @@ -265,16 +265,16 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_platform_data *data) {}
>
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PA9, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PA10, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PA9, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PA10, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PC8, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PC9, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PC8, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PC9, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PC10, 0); /* RTS0 */
> @@ -286,8 +286,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PC12, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PC13, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PC12, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PC13, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PA12, 0); /* RTS1 */
> @@ -299,8 +299,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PC15, 0); /* RXD2 */
> - at91_set_A_periph(AT91_PIN_PC14, 1); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PC15, 1); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PC14, 0); /* TXD2 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PA15, 0); /* RTS2*/
> diff --git a/arch/arm/mach-at91/at91sam9263_devices.c b/arch/arm/mach-at91/at91sam9263_devices.c
> index a3683e5..559b77e 100644
> --- a/arch/arm/mach-at91/at91sam9263_devices.c
> +++ b/arch/arm/mach-at91/at91sam9263_devices.c
> @@ -296,16 +296,16 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_platform_data *data) {}
>
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PC30, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PC31, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PC30, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PC31, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA26, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PA27, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PA26, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PA27, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PA28, 0); /* RTS0 */
> @@ -317,8 +317,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PD0, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PD1, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PD0, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PD1, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PD7, 0); /* RTS1 */
> @@ -330,8 +330,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PD2, 1); /* TXD2 */
> - at91_set_A_periph(AT91_PIN_PD3, 0); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PD2, 0); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PD3, 1); /* RXD2 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PD5, 0); /* RTS2 */
> diff --git a/arch/arm/mach-at91/at91sam9g45_devices.c b/arch/arm/mach-at91/at91sam9g45_devices.c
> index bad7f9c..bc41320 100644
> --- a/arch/arm/mach-at91/at91sam9g45_devices.c
> +++ b/arch/arm/mach-at91/at91sam9g45_devices.c
> @@ -213,16 +213,16 @@ void at91_add_device_i2c(short i2c_id, struct i2c_board_info *devices, int nr_de
>
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PB12, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PB13, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PB12, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PB13, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB19, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PB18, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PB19, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PB18, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PB17, 0); /* RTS0 */
> @@ -234,8 +234,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB4, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PB5, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PB4, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PB5, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PD16, 0); /* RTS1 */
> @@ -247,8 +247,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB6, 1); /* TXD2 */
> - at91_set_A_periph(AT91_PIN_PB7, 0); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PB6, 0); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PB7, 1); /* RXD2 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PC9, 0); /* RTS2 */
> @@ -260,8 +260,8 @@ resource_size_t __init at91_configure_usart2(unsigned pins)
>
> resource_size_t __init at91_configure_usart3(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB8, 1); /* TXD3 */
> - at91_set_A_periph(AT91_PIN_PB9, 0); /* RXD3 */
> + at91_set_A_periph(AT91_PIN_PB8, 0); /* TXD3 */
> + at91_set_A_periph(AT91_PIN_PB9, 1); /* RXD3 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PA23, 0); /* RTS3 */
> diff --git a/arch/arm/mach-at91/at91sam9n12_devices.c b/arch/arm/mach-at91/at91sam9n12_devices.c
> index bac023f..84c871c 100644
> --- a/arch/arm/mach-at91/at91sam9n12_devices.c
> +++ b/arch/arm/mach-at91/at91sam9n12_devices.c
> @@ -370,16 +370,16 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_platform_data *data) {}
> #if defined(CONFIG_DRIVER_SERIAL_ATMEL)
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PA9, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PA10, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PA9, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PA10, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA0, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PA1, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PA0, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PA1, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PA2, 0); /* RTS0 */
> @@ -391,8 +391,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA5, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PA6, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PA5, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PA6, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_C_periph(AT91_PIN_PC27, 0); /* RTS1 */
> @@ -404,8 +404,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA7, 1); /* TXD2 */
> - at91_set_A_periph(AT91_PIN_PA8, 0); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PA7, 0); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PA8, 1); /* RXD2 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PB0, 0); /* RTS2 */
> @@ -417,8 +417,8 @@ resource_size_t __init at91_configure_usart2(unsigned pins)
>
> resource_size_t __init at91_configure_usart3(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PC22, 1); /* TXD3 */
> - at91_set_B_periph(AT91_PIN_PC23, 0); /* RXD3 */
> + at91_set_B_periph(AT91_PIN_PC22, 0); /* TXD3 */
> + at91_set_B_periph(AT91_PIN_PC23, 1); /* RXD3 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PC24, 0); /* RTS3 */
> diff --git a/arch/arm/mach-at91/at91sam9x5_devices.c b/arch/arm/mach-at91/at91sam9x5_devices.c
> index 34537d8..d7ddda4 100644
> --- a/arch/arm/mach-at91/at91sam9x5_devices.c
> +++ b/arch/arm/mach-at91/at91sam9x5_devices.c
> @@ -453,16 +453,16 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_platform_data *data) {}
> #if defined(CONFIG_DRIVER_SERIAL_ATMEL)
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PA9, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PA10, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PA9, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PA10, 0); /* DTXD */
>
> return AT91_BASE_SYS + AT91_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA0, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PA1, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PA0, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PA1, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PA2, 0); /* RTS0 */
> @@ -474,8 +474,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA5, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PA6, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PA5, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PA6, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_C_periph(AT91_PIN_PC27, 0); /* RTS1 */
> @@ -487,8 +487,8 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PA7, 1); /* TXD2 */
> - at91_set_A_periph(AT91_PIN_PA8, 0); /* RXD2 */
> + at91_set_A_periph(AT91_PIN_PA7, 0); /* TXD2 */
> + at91_set_A_periph(AT91_PIN_PA8, 1); /* RXD2 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PB0, 0); /* RTS2 */
> @@ -500,8 +500,8 @@ resource_size_t __init at91_configure_usart2(unsigned pins)
>
> resource_size_t __init at91_configure_usart3(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PC22, 1); /* TXD3 */
> - at91_set_B_periph(AT91_PIN_PC23, 0); /* RXD3 */
> + at91_set_B_periph(AT91_PIN_PC22, 0); /* TXD3 */
> + at91_set_B_periph(AT91_PIN_PC23, 1); /* RXD3 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_B_periph(AT91_PIN_PC24, 0); /* RTS3 */
> diff --git a/arch/arm/mach-at91/sama5d3_devices.c b/arch/arm/mach-at91/sama5d3_devices.c
> index 3fdfca8..c6f5e3a 100644
> --- a/arch/arm/mach-at91/sama5d3_devices.c
> +++ b/arch/arm/mach-at91/sama5d3_devices.c
> @@ -464,16 +464,16 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_platform_data *data) {}
> #if defined(CONFIG_DRIVER_SERIAL_ATMEL)
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PB30, 0); /* DRXD */
> - at91_set_A_periph(AT91_PIN_PB31, 1); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PB30, 1); /* DRXD */
> + at91_set_A_periph(AT91_PIN_PB31, 0); /* DTXD */
>
> return AT91_BASE_DBGU1;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PD18, 1); /* TXD0 */
> - at91_set_A_periph(AT91_PIN_PD17, 0); /* RXD0 */
> + at91_set_A_periph(AT91_PIN_PD18, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PD17, 1); /* RXD0 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PD16, 0); /* RTS0 */
> @@ -485,8 +485,8 @@ resource_size_t __init at91_configure_usart0(unsigned pins)
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PB29, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PB28, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PB29, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PB28, 1); /* RXD1 */
>
> if (pins & ATMEL_UART_RTS)
> at91_set_A_periph(AT91_PIN_PB27, 0); /* RTS1 */
> diff --git a/arch/arm/mach-at91/sama5d4_devices.c b/arch/arm/mach-at91/sama5d4_devices.c
> index e1b0a64..c2f171a 100644
> --- a/arch/arm/mach-at91/sama5d4_devices.c
> +++ b/arch/arm/mach-at91/sama5d4_devices.c
> @@ -430,24 +430,24 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_platform_data *data) {}
> #if defined(CONFIG_DRIVER_SERIAL_ATMEL)
> resource_size_t __init at91_configure_dbgu(void)
> {
> - at91_set_A_periph(AT91_PIN_PB25, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PB24, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PB25, 0); /* DTXD */
> + at91_set_A_periph(AT91_PIN_PB24, 1); /* DRXD */
>
> return SAMA5D4_BASE_DBGU;
> }
>
> resource_size_t __init at91_configure_usart0(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PD13, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PD12, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PD13, 0); /* TXD0 */
> + at91_set_A_periph(AT91_PIN_PD12, 1); /* RXD0 */
>
> return SAMA5D4_BASE_USART0;
> }
>
> resource_size_t __init at91_configure_usart1(unsigned pins)
> {
> - at91_set_A_periph(AT91_PIN_PD17, 1); /* TXD1 */
> - at91_set_A_periph(AT91_PIN_PD16, 0); /* RXD1 */
> + at91_set_A_periph(AT91_PIN_PD17, 0); /* TXD1 */
> + at91_set_A_periph(AT91_PIN_PD16, 1); /* RXD1 */
>
> return SAMA5D4_BASE_USART1;
> }
> @@ -455,40 +455,40 @@ resource_size_t __init at91_configure_usart1(unsigned pins)
>
> resource_size_t __init at91_configure_usart2(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PB5, 1); /* TXD1 */
> - at91_set_B_periph(AT91_PIN_PB4, 0); /* RXD1 */
> + at91_set_B_periph(AT91_PIN_PB5, 0); /* TXD2 */
> + at91_set_B_periph(AT91_PIN_PB4, 1); /* RXD2 */
>
> return SAMA5D4_BASE_USART2;
> }
>
> resource_size_t __init at91_configure_usart3(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PE17, 1); /* TXD1 */
> - at91_set_B_periph(AT91_PIN_PE16, 0); /* RXD1 */
> + at91_set_B_periph(AT91_PIN_PE17, 0); /* TXD3 */
> + at91_set_B_periph(AT91_PIN_PE16, 1); /* RXD3 */
>
> return SAMA5D4_BASE_USART3;
> }
>
> resource_size_t __init at91_configure_usart4(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PE27, 1); /* TXD1 */
> - at91_set_B_periph(AT91_PIN_PE26, 0); /* RXD1 */
> + at91_set_B_periph(AT91_PIN_PE27, 0); /* TXD4 */
> + at91_set_B_periph(AT91_PIN_PE26, 1); /* RXD4 */
>
> return SAMA5D4_BASE_USART4;
> }
>
> resource_size_t __init at91_configure_usart5(unsigned pins)
> {
> - at91_set_B_periph(AT91_PIN_PE30, 1); /* TXD1 */
> - at91_set_B_periph(AT91_PIN_PE29, 0); /* RXD1 */
> + at91_set_B_periph(AT91_PIN_PE30, 0); /* UTXD0 */
> + at91_set_B_periph(AT91_PIN_PE29, 1); /* URXD0 */
>
> return SAMA5D4_BASE_UART0;
> }
>
> resource_size_t __init at91_configure_usart6(unsigned pins)
> {
> - at91_set_C_periph(AT91_PIN_PC26, 1); /* TXD1 */
> - at91_set_C_periph(AT91_PIN_PC25, 0); /* RXD1 */
> + at91_set_C_periph(AT91_PIN_PC26, 0); /* UTXD1 */
> + at91_set_C_periph(AT91_PIN_PC25, 1); /* URXD1 */
>
> return SAMA5D4_BASE_UART1;
> }
> --
> 2.1.4
>
>
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
prev parent reply other threads:[~2017-02-14 7:56 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-13 17:06 Peter Rosin
2017-02-14 7:55 ` Sascha Hauer [this message]
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=20170214075556.gzafi4wt2xqu7whd@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=peda@axentia.se \
/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