Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
Message-ID: <trinity-5eab4708-85af-4a10-911f-86b9afc2c0cc-1528711303734@3c-app-gmx-bs27>
From:   st6589@public-files.de
To:     linux-kernel@vger.kernel.org
Cc:     linux-amlogic@lists.infradead.org
Subject: [BUG] Pinning in pinctrl-meson-gxbb.c for uart_C is wrong
Content-Type: multipart/mixed;
 boundary=refeik-b4421615-3be7-45eb-a938-7f1e82e91fd3
Date:   Mon, 11 Jun 2018 12:01:43 +0200
Importance: normal
Sensitivity: Normal
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

--refeik-b4421615-3be7-45eb-a938-7f1e82e91fd3
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable


Hello with the actuel Pinning definition in=C2=A0 pinctrl-meson-gxbb=2Ec f=
or the uart_c
the system crashes when activated=2E The seams to be a depency with mmc dr=
iver for SD cards=2E
=C2=A0
according to the comment here: https://forum=2Eodroid=2Ecom/viewtopic=2Eph=
p?f=3D137&t=3D25373#p177855
the seams to be an error in the amlogic datasheets regarding uart_c
=C2=A0
with the attached patch everything works as it should=2E
=C2=A0
Best regards
=C2=A0
=C2=A0
##
#    https://forum=2Eodroid=2Ecom/viewtopic=2Ephp?f=3D137&t=3D25373#p17785=
5
#
--- linux-next-20180608/drivers/pinctrl/meson/pinctrl-meson-gxbb=2Ec=2Eori=
g 2018-06-09 12:03:58=2E921540445 +0200
+++ linux-next-20180608/drivers/pinctrl/meson/pinctrl-meson-gxbb=2Ec      =
2018-06-10 18:51:52=2E080665271 +0200
@@ -198,8 +198,8 @@
=20
 static const unsigned int uart_tx_c_pins[]     =3D { GPIOY_13 };
 static const unsigned int uart_rx_c_pins[]     =3D { GPIOY_14 };
-static const unsigned int uart_cts_c_pins[]    =3D { GPIOX_11 };
-static const unsigned int uart_rts_c_pins[]    =3D { GPIOX_12 };
+static const unsigned int uart_cts_c_pins[]    =3D { GPIOY_11 };
+static const unsigned int uart_rts_c_pins[]    =3D { GPIOY_12 };
=20
 static const unsigned int i2c_sck_a_pins[]     =3D { GPIODV_25 };
 static const unsigned int i2c_sda_a_pins[]     =3D { GPIODV_24 };
@@ -443,10 +443,10 @@
        GROUP(pwm_f_x,          3,      18),
=20
        /* Bank Y */
-       GROUP(uart_cts_c,       1,      19),
-       GROUP(uart_rts_c,       1,      18),
-       GROUP(uart_tx_c,        1,      17),
-       GROUP(uart_rx_c,        1,      16),
+       GROUP(uart_cts_c,       1,      17),
+       GROUP(uart_rts_c,       1,      16),
+       GROUP(uart_tx_c,        1,      19),
+       GROUP(uart_rx_c,        1,      18),
        GROUP(pwm_a_y,          1,      21),
        GROUP(pwm_f_y,          1,      20),
        GROUP(i2s_out_ch23_y,   1,      5),

--refeik-b4421615-3be7-45eb-a938-7f1e82e91fd3
Content-Type: text/x-patch
Content-Disposition: attachment; filename=my-diff-4.17-uac-pin.diff

##
#    https://forum.odroid.com/viewtopic.php?f=137&t=25373#p177855
#
--- linux-next-20180608/drivers/pinctrl/meson/pinctrl-meson-gxbb.c.orig	2018-06-09 12:03:58.921540445 +0200
+++ linux-next-20180608/drivers/pinctrl/meson/pinctrl-meson-gxbb.c	2018-06-10 18:51:52.080665271 +0200
@@ -198,8 +198,8 @@
 
 static const unsigned int uart_tx_c_pins[]	= { GPIOY_13 };
 static const unsigned int uart_rx_c_pins[]	= { GPIOY_14 };
-static const unsigned int uart_cts_c_pins[]	= { GPIOX_11 };
-static const unsigned int uart_rts_c_pins[]	= { GPIOX_12 };
+static const unsigned int uart_cts_c_pins[]	= { GPIOY_11 };
+static const unsigned int uart_rts_c_pins[]	= { GPIOY_12 };
 
 static const unsigned int i2c_sck_a_pins[]	= { GPIODV_25 };
 static const unsigned int i2c_sda_a_pins[]	= { GPIODV_24 };
@@ -443,10 +443,10 @@
 	GROUP(pwm_f_x,		3,	18),
 
 	/* Bank Y */
-	GROUP(uart_cts_c,	1,	19),
-	GROUP(uart_rts_c,	1,	18),
-	GROUP(uart_tx_c,	1,	17),
-	GROUP(uart_rx_c,	1,	16),
+	GROUP(uart_cts_c,	1,	17),
+	GROUP(uart_rts_c,	1,	16),
+	GROUP(uart_tx_c,	1,	19),
+	GROUP(uart_rx_c,	1,	18),
 	GROUP(pwm_a_y,		1,	21),
 	GROUP(pwm_f_y,		1,	20),
 	GROUP(i2s_out_ch23_y,	1,	5),

--refeik-b4421615-3be7-45eb-a938-7f1e82e91fd3--