3a9b20b
From patchwork Wed Mar  1 15:23:02 2017
3a9b20b
Content-Type: text/plain; charset="utf-8"
3a9b20b
MIME-Version: 1.0
3a9b20b
Content-Transfer-Encoding: 7bit
3a9b20b
Subject: [v3] Revert "tty: serial: pl011: add ttyAMA for matching pl011
3a9b20b
 console"
3a9b20b
From: Aleksey Makarov <aleksey.makarov@linaro.org>
3a9b20b
X-Patchwork-Id: 9598601
3a9b20b
Message-Id: <20170301152304.29635-1-aleksey.makarov@linaro.org>
3a9b20b
To: linux-serial@vger.kernel.org
3a9b20b
Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
3a9b20b
 Aleksey Makarov <aleksey.makarov@linaro.org>,
3a9b20b
 Sudeep Holla <sudeep.holla@arm.com>,
3a9b20b
 Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
3a9b20b
 Peter Hurley <peter@hurleysoftware.com>,
3a9b20b
 Russell King <linux@armlinux.org.uk>, Jiri Slaby <jslaby@suse.com>,
3a9b20b
 Robin Murphy <robin.murphy@arm.com>
3a9b20b
Date: Wed,  1 Mar 2017 18:23:02 +0300
3a9b20b
3a9b20b
The original patch makes the condition always true, so it is wrong.
3a9b20b
3a9b20b
It masks (but not fixes) the bug described in the commit message
3a9b20b
but introduces a regression (no console is selected by SPCR)
3a9b20b
in regular (no 'console=ttyAMA') case.
3a9b20b
3a9b20b
s/||/&&/ would not fix the problem as the root cause was identified
3a9b20b
incorrectly.
3a9b20b
3a9b20b
This reverts commit aea9a80ba98a0c9b4de88850260e9fbdcc98360b.
3a9b20b
3a9b20b
Signed-off-by: Aleksey Makarov <aleksey.makarov@linaro.org>
3a9b20b
---
3a9b20b
3a9b20b
v3: fix commit message (Robin Murphy)
3a9b20b
v2: add Signed-off-by:
3a9b20b
3a9b20b
 drivers/tty/serial/amba-pl011.c | 2 +-
3a9b20b
 1 file changed, 1 insertion(+), 1 deletion(-)
3a9b20b
3a9b20b
diff --git a/drivers/tty/serial/amba-pl011.c b/drivers/tty/serial/amba-pl011.c
3a9b20b
index 8789ea423ccf..56f92d7348bf 100644
3a9b20b
--- a/drivers/tty/serial/amba-pl011.c
3a9b20b
+++ b/drivers/tty/serial/amba-pl011.c
3a9b20b
@@ -2373,7 +2373,7 @@ static int __init pl011_console_match(struct console *co, char *name, int idx,
3a9b20b
 	if (strcmp(name, "qdf2400_e44") == 0) {
3a9b20b
 		pr_info_once("UART: Working around QDF2400 SoC erratum 44");
3a9b20b
 		qdf2400_e44_present = true;
3a9b20b
-	} else if (strcmp(name, "pl011") != 0 || strcmp(name, "ttyAMA") != 0) {
3a9b20b
+	} else if (strcmp(name, "pl011") != 0) {
3a9b20b
 		return -ENODEV;
3a9b20b
 	}
3a9b20b