* [PATCH] tty/early: make serial8250_early_{in,out} static again
@ 2015-06-24 11:26 Vineet Gupta
2015-07-09 6:20 ` [PATCH REBASED RESEND] " Vineet Gupta
0 siblings, 1 reply; 4+ messages in thread
From: Vineet Gupta @ 2015-06-24 11:26 UTC (permalink / raw)
To: linux-serial
Cc: Vineet Gupta, linux-kernel, Greg Kroah-Hartman, Jiri Slaby,
Peter Hurley, Rob Herring, Kevin Cernekee, Noam Camus
Commit ed71871bed719 ("tty/8250_early: Turn serial_in/serial_out into
weak symbols") made these routines weak to allow platform specific
Big endian override
However recent updates to core, specifically ebc5e20082 ("serial:
of_serial: Support big-endian register accesses") and 6e63be3fee14
("serial: earlycon: Add support for big-endian MMIO accesses") means
that round about way to overide the early serial accessors is no longer
needed.
Cc: <linux-serial@vger.kernel.org>
Cc: <linux-kernel@vger.kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Jiri Slaby <jslaby@suse.cz>
Cc: Peter Hurley <peter@hurleysoftware.com>
Cc: Rob Herring <robh@kernel.org>
Cc: Kevin Cernekee <cernekee@gmail.com>
Cc: Noam Camus <noamc@ezchip.com>
Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
---
drivers/tty/serial/8250/8250_early.c | 4 ++--
include/linux/serial_8250.h | 2 --
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/tty/serial/8250/8250_early.c b/drivers/tty/serial/8250/8250_early.c
index 771dda29a0f8..faed05f25bc2 100644
--- a/drivers/tty/serial/8250/8250_early.c
+++ b/drivers/tty/serial/8250/8250_early.c
@@ -35,7 +35,7 @@
#include <asm/io.h>
#include <asm/serial.h>
-unsigned int __weak __init serial8250_early_in(struct uart_port *port, int offset)
+static unsigned int __init serial8250_early_in(struct uart_port *port, int offset)
{
switch (port->iotype) {
case UPIO_MEM:
@@ -51,7 +51,7 @@ unsigned int __weak __init serial8250_early_in(struct uart_port *port, int offse
}
}
-void __weak __init serial8250_early_out(struct uart_port *port, int offset, int value)
+static void __init serial8250_early_out(struct uart_port *port, int offset, int value)
{
switch (port->iotype) {
case UPIO_MEM:
diff --git a/include/linux/serial_8250.h b/include/linux/serial_8250.h
index ba82c07feb95..9075712db8d8 100644
--- a/include/linux/serial_8250.h
+++ b/include/linux/serial_8250.h
@@ -136,8 +136,6 @@ void serial8250_resume_port(int line);
extern int early_serial_setup(struct uart_port *port);
-extern unsigned int serial8250_early_in(struct uart_port *port, int offset);
-extern void serial8250_early_out(struct uart_port *port, int offset, int value);
extern int early_serial8250_setup(struct earlycon_device *device,
const char *options);
extern void serial8250_do_set_termios(struct uart_port *port,
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH REBASED RESEND] tty/early: make serial8250_early_{in,out} static again
2015-06-24 11:26 [PATCH] tty/early: make serial8250_early_{in,out} static again Vineet Gupta
@ 2015-07-09 6:20 ` Vineet Gupta
2015-07-09 7:31 ` Noam Camus
2015-07-10 17:42 ` Peter Hurley
0 siblings, 2 replies; 4+ messages in thread
From: Vineet Gupta @ 2015-07-09 6:20 UTC (permalink / raw)
To: linux-serial
Cc: Vineet Gupta, linux-kernel, Greg Kroah-Hartman, Jiri Slaby,
Peter Hurley, Rob Herring, Kevin Cernekee, Noam Camus
Commit ed71871bed719 ("tty/8250_early: Turn serial_in/serial_out into
weak symbols") made these routines weak to allow platform specific
Big endian override
However recent updates to core, specifically ebc5e20082 ("serial:
of_serial: Support big-endian register accesses") and 6e63be3fee14
("serial: earlycon: Add support for big-endian MMIO accesses") means
that round about way to overide the early serial accessors is no longer
needed.
Cc: <linux-serial@vger.kernel.org>
Cc: <linux-kernel@vger.kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Jiri Slaby <jslaby@suse.cz>
Cc: Peter Hurley <peter@hurleysoftware.com>
Cc: Rob Herring <robh@kernel.org>
Cc: Kevin Cernekee <cernekee@gmail.com>
Cc: Noam Camus <noamc@ezchip.com>
Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
---
drivers/tty/serial/8250/8250_early.c | 4 ++--
include/linux/serial_8250.h | 2 --
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/tty/serial/8250/8250_early.c b/drivers/tty/serial/8250/8250_early.c
index 771dda29a0f8..faed05f25bc2 100644
--- a/drivers/tty/serial/8250/8250_early.c
+++ b/drivers/tty/serial/8250/8250_early.c
@@ -35,7 +35,7 @@
#include <asm/io.h>
#include <asm/serial.h>
-unsigned int __weak __init serial8250_early_in(struct uart_port *port, int offset)
+static unsigned int __init serial8250_early_in(struct uart_port *port, int offset)
{
switch (port->iotype) {
case UPIO_MEM:
@@ -51,7 +51,7 @@ unsigned int __weak __init serial8250_early_in(struct uart_port *port, int offse
}
}
-void __weak __init serial8250_early_out(struct uart_port *port, int offset, int value)
+static void __init serial8250_early_out(struct uart_port *port, int offset, int value)
{
switch (port->iotype) {
case UPIO_MEM:
diff --git a/include/linux/serial_8250.h b/include/linux/serial_8250.h
index ba82c07feb95..9075712db8d8 100644
--- a/include/linux/serial_8250.h
+++ b/include/linux/serial_8250.h
@@ -136,8 +136,6 @@ void serial8250_resume_port(int line);
extern int early_serial_setup(struct uart_port *port);
-extern unsigned int serial8250_early_in(struct uart_port *port, int offset);
-extern void serial8250_early_out(struct uart_port *port, int offset, int value);
extern int early_serial8250_setup(struct earlycon_device *device,
const char *options);
extern void serial8250_do_set_termios(struct uart_port *port,
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* RE: [PATCH REBASED RESEND] tty/early: make serial8250_early_{in,out} static again
2015-07-09 6:20 ` [PATCH REBASED RESEND] " Vineet Gupta
@ 2015-07-09 7:31 ` Noam Camus
2015-07-10 17:42 ` Peter Hurley
1 sibling, 0 replies; 4+ messages in thread
From: Noam Camus @ 2015-07-09 7:31 UTC (permalink / raw)
To: Vineet Gupta, linux-serial@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman, Jiri Slaby,
Peter Hurley, Rob Herring, Kevin Cernekee
> From: Vineet Gupta [mailto:Vineet.Gupta1@synopsys.com]
> Sent: Thursday, July 09, 2015 9:21 AM
>
> Commit ed71871bed719 ("tty/8250_early: Turn serial_in/serial_out into weak symbols") made these
> routines weak to allow platform specific Big endian override
>
> However recent updates to core, specifically ebc5e20082 ("serial:
> of_serial: Support big-endian register accesses") and 6e63be3fee14
> ("serial: earlycon: Add support for big-endian MMIO accesses") means that round about way to override
> the early serial accessors is no longer needed.
As author of commit ed71871bed719 my original problem is now solved in more generic way and there is no more need for the __weak attribute on these two early serial accessors.
ACK
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH REBASED RESEND] tty/early: make serial8250_early_{in,out} static again
2015-07-09 6:20 ` [PATCH REBASED RESEND] " Vineet Gupta
2015-07-09 7:31 ` Noam Camus
@ 2015-07-10 17:42 ` Peter Hurley
1 sibling, 0 replies; 4+ messages in thread
From: Peter Hurley @ 2015-07-10 17:42 UTC (permalink / raw)
To: Vineet Gupta, linux-serial
Cc: linux-kernel, Greg Kroah-Hartman, Jiri Slaby, Rob Herring,
Kevin Cernekee, Noam Camus
On 07/09/2015 02:20 AM, Vineet Gupta wrote:
> Commit ed71871bed719 ("tty/8250_early: Turn serial_in/serial_out into
> weak symbols") made these routines weak to allow platform specific
> Big endian override
>
> However recent updates to core, specifically ebc5e20082 ("serial:
> of_serial: Support big-endian register accesses") and 6e63be3fee14
> ("serial: earlycon: Add support for big-endian MMIO accesses") means
> that round about way to overide the early serial accessors is no longer
> needed.
Reviewed-by: Peter Hurley <peter@hurleysoftware.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-07-10 17:42 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-24 11:26 [PATCH] tty/early: make serial8250_early_{in,out} static again Vineet Gupta
2015-07-09 6:20 ` [PATCH REBASED RESEND] " Vineet Gupta
2015-07-09 7:31 ` Noam Camus
2015-07-10 17:42 ` Peter Hurley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).