All of lore.kernel.org
 help / color / mirror / Atom feed
* Patch "tty: serial: pl011: add ttyAMA for matching pl011 console" has been added to the 4.10-stable tree
@ 2017-05-09  9:21 gregkh
  2017-05-09  9:32 ` Sudeep Holla
  0 siblings, 1 reply; 4+ messages in thread
From: gregkh @ 2017-05-09  9:21 UTC (permalink / raw)
  To: sudeep.holla, aleksey.makarov, gregkh, jslaby, linux, peter
  Cc: stable, stable-commits


This is a note to let you know that I've just added the patch titled

    tty: serial: pl011: add ttyAMA for matching pl011 console

to the 4.10-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     tty-serial-pl011-add-ttyama-for-matching-pl011-console.patch
and it can be found in the queue-4.10 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@vger.kernel.org> know about it.


>From aea9a80ba98a0c9b4de88850260e9fbdcc98360b Mon Sep 17 00:00:00 2001
From: Sudeep Holla <sudeep.holla@arm.com>
Date: Wed, 1 Feb 2017 15:46:38 +0000
Subject: tty: serial: pl011: add ttyAMA for matching pl011 console

From: Sudeep Holla <sudeep.holla@arm.com>

commit aea9a80ba98a0c9b4de88850260e9fbdcc98360b upstream.

Commit c7cef0a84912 ("console: Add extensible console matching") added
match() method to struct console which allows the console to perform
console command line matching instead of (or in addition to) default
console matching (ie., by fixed name and index).

Commit ad1696f6f09d ("ACPI: parse SPCR and enable matching console")
introduced support for SPCR as matching console.

Commit 10879ae5f12e ("serial: pl011: add console matching function")
added the match method for pl011 console which checks for the console
string to be "pl011"

Now on a platform which has both SPCR in the ACPI tables and ttyAMA in
the command line, the ttyAMA is chosen as "selected console" but it
doesn't pass the matching console method which results in CON_CONSDEV
not being set on the "selected console".

As a result of that, the bootconsole(SPCR in the above case) is not
unregistered and all the beginning boot messages are seen twice.

This patch adds "ttyAMA" so that it's considered to match pl011 console.

Fixes: 10879ae5f12e ("serial: pl011: add console matching function")
Cc: Russell King <linux@armlinux.org.uk>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Jiri Slaby <jslaby@suse.com>
Cc: Peter Hurley <peter@hurleysoftware.com>
Cc: Aleksey Makarov <aleksey.makarov@linaro.org>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

---
 drivers/tty/serial/amba-pl011.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/tty/serial/amba-pl011.c
+++ b/drivers/tty/serial/amba-pl011.c
@@ -2340,7 +2340,7 @@ static int __init pl011_console_match(st
 	resource_size_t addr;
 	int i;
 
-	if (strcmp(name, "pl011") != 0)
+	if (strcmp(name, "pl011") != 0 || strcmp(name, "ttyAMA") != 0)
 		return -ENODEV;
 
 	if (uart_parse_earlycon(options, &iotype, &addr, &options))


Patches currently in stable-queue which might be from sudeep.holla@arm.com are

queue-4.10/arm64-remove-wrong-config_proc_sysctl-ifdef.patch
queue-4.10/tty-serial-pl011-add-ttyama-for-matching-pl011-console.patch

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Patch "tty: serial: pl011: add ttyAMA for matching pl011 console" has been added to the 4.10-stable tree
  2017-05-09  9:21 Patch "tty: serial: pl011: add ttyAMA for matching pl011 console" has been added to the 4.10-stable tree gregkh
@ 2017-05-09  9:32 ` Sudeep Holla
  2017-05-09 11:21   ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Sudeep Holla @ 2017-05-09  9:32 UTC (permalink / raw)
  To: gregkh
  Cc: aleksey.makarov, jslaby, linux, peter, Sudeep Holla, stable,
	stable-commits

Hi Greg,

On 09/05/17 10:21, gregkh@linuxfoundation.org wrote:
> 
> This is a note to let you know that I've just added the patch titled
> 
>     tty: serial: pl011: add ttyAMA for matching pl011 console
> 
> to the 4.10-stable tree which can be found at:
>     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> 
> The filename of the patch is:
>      tty-serial-pl011-add-ttyama-for-matching-pl011-console.patch
> and it can be found in the queue-4.10 subdirectory.
> 
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@vger.kernel.org> know about it.
> 
> 
> From aea9a80ba98a0c9b4de88850260e9fbdcc98360b Mon Sep 17 00:00:00 2001
> From: Sudeep Holla <sudeep.holla@arm.com>
> Date: Wed, 1 Feb 2017 15:46:38 +0000
> Subject: tty: serial: pl011: add ttyAMA for matching pl011 console
> 
> From: Sudeep Holla <sudeep.holla@arm.com>
> 
> commit aea9a80ba98a0c9b4de88850260e9fbdcc98360b upstream.
> 

I am not sure on the procedure for stable tree, but thought I will let
you know the information on this.

Commit b767ad726c2a ("Revert "tty: serial: pl011: add ttyAMA for
matching pl011 console"") reverted the above commit(commit in $subject)

It was properly fixed in the following upstream commits:

cf39bf58afda ("printk: fix double printing with earlycon")
ad86ee2b8a47 ("printk: rename selected_console -> preferred_console")
b077bafa2f38 ("printk: fix name/type/scope of preferred_console var")

Sorry for the trouble.

-- 
Regards,
Sudeep

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Patch "tty: serial: pl011: add ttyAMA for matching pl011 console" has been added to the 4.10-stable tree
  2017-05-09  9:32 ` Sudeep Holla
@ 2017-05-09 11:21   ` Greg KH
  2017-05-09 12:10     ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2017-05-09 11:21 UTC (permalink / raw)
  To: Sudeep Holla
  Cc: aleksey.makarov, jslaby, linux, peter, stable, stable-commits

On Tue, May 09, 2017 at 10:32:13AM +0100, Sudeep Holla wrote:
> Hi Greg,
> 
> On 09/05/17 10:21, gregkh@linuxfoundation.org wrote:
> > 
> > This is a note to let you know that I've just added the patch titled
> > 
> >     tty: serial: pl011: add ttyAMA for matching pl011 console
> > 
> > to the 4.10-stable tree which can be found at:
> >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > 
> > The filename of the patch is:
> >      tty-serial-pl011-add-ttyama-for-matching-pl011-console.patch
> > and it can be found in the queue-4.10 subdirectory.
> > 
> > If you, or anyone else, feels it should not be added to the stable tree,
> > please let <stable@vger.kernel.org> know about it.
> > 
> > 
> > From aea9a80ba98a0c9b4de88850260e9fbdcc98360b Mon Sep 17 00:00:00 2001
> > From: Sudeep Holla <sudeep.holla@arm.com>
> > Date: Wed, 1 Feb 2017 15:46:38 +0000
> > Subject: tty: serial: pl011: add ttyAMA for matching pl011 console
> > 
> > From: Sudeep Holla <sudeep.holla@arm.com>
> > 
> > commit aea9a80ba98a0c9b4de88850260e9fbdcc98360b upstream.
> > 
> 
> I am not sure on the procedure for stable tree, but thought I will let
> you know the information on this.
> 
> Commit b767ad726c2a ("Revert "tty: serial: pl011: add ttyAMA for
> matching pl011 console"") reverted the above commit(commit in $subject)
> 
> It was properly fixed in the following upstream commits:
> 
> cf39bf58afda ("printk: fix double printing with earlycon")
> ad86ee2b8a47 ("printk: rename selected_console -> preferred_console")
> b077bafa2f38 ("printk: fix name/type/scope of preferred_console var")
> 
> Sorry for the trouble.

No trouble at all, thanks for letting me know.  I've dropped the tty
patch now and will look at adding the printk ones instead.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Patch "tty: serial: pl011: add ttyAMA for matching pl011 console" has been added to the 4.10-stable tree
  2017-05-09 11:21   ` Greg KH
@ 2017-05-09 12:10     ` Greg KH
  0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2017-05-09 12:10 UTC (permalink / raw)
  To: Sudeep Holla
  Cc: aleksey.makarov, jslaby, linux, peter, stable, stable-commits

On Tue, May 09, 2017 at 01:21:34PM +0200, Greg KH wrote:
> On Tue, May 09, 2017 at 10:32:13AM +0100, Sudeep Holla wrote:
> > Hi Greg,
> > 
> > On 09/05/17 10:21, gregkh@linuxfoundation.org wrote:
> > > 
> > > This is a note to let you know that I've just added the patch titled
> > > 
> > >     tty: serial: pl011: add ttyAMA for matching pl011 console
> > > 
> > > to the 4.10-stable tree which can be found at:
> > >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > > 
> > > The filename of the patch is:
> > >      tty-serial-pl011-add-ttyama-for-matching-pl011-console.patch
> > > and it can be found in the queue-4.10 subdirectory.
> > > 
> > > If you, or anyone else, feels it should not be added to the stable tree,
> > > please let <stable@vger.kernel.org> know about it.
> > > 
> > > 
> > > From aea9a80ba98a0c9b4de88850260e9fbdcc98360b Mon Sep 17 00:00:00 2001
> > > From: Sudeep Holla <sudeep.holla@arm.com>
> > > Date: Wed, 1 Feb 2017 15:46:38 +0000
> > > Subject: tty: serial: pl011: add ttyAMA for matching pl011 console
> > > 
> > > From: Sudeep Holla <sudeep.holla@arm.com>
> > > 
> > > commit aea9a80ba98a0c9b4de88850260e9fbdcc98360b upstream.
> > > 
> > 
> > I am not sure on the procedure for stable tree, but thought I will let
> > you know the information on this.
> > 
> > Commit b767ad726c2a ("Revert "tty: serial: pl011: add ttyAMA for
> > matching pl011 console"") reverted the above commit(commit in $subject)
> > 
> > It was properly fixed in the following upstream commits:
> > 
> > cf39bf58afda ("printk: fix double printing with earlycon")
> > ad86ee2b8a47 ("printk: rename selected_console -> preferred_console")
> > b077bafa2f38 ("printk: fix name/type/scope of preferred_console var")
> > 
> > Sorry for the trouble.
> 
> No trouble at all, thanks for letting me know.  I've dropped the tty
> patch now and will look at adding the printk ones instead.

Hm, as the printk patches will be in 4.12-rc1, I'll wait for that to be
out before considering them.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-05-09 12:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-09  9:21 Patch "tty: serial: pl011: add ttyAMA for matching pl011 console" has been added to the 4.10-stable tree gregkh
2017-05-09  9:32 ` Sudeep Holla
2017-05-09 11:21   ` Greg KH
2017-05-09 12:10     ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.