[PATCH v2] serial: xuartps: Enable OF earlycon support

[PATCH v2] serial: xuartps: Enable OF earlycon support

[Michal Simek]

Support early console setup via DT for all listed compatible strings.
Remove EARLYCON_DECLARE which was done by:
"Use common framework for earlycon declarations"
(sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
when OF_EARLYCON_DECLARE is defined.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

Changes in v2:
- Separate serial patch from others
- Remove EARLYCON_DECLARE and use the same earlycon name with different
  compatible string as was pointed by Peter Hurley
- Extend commit message

 drivers/tty/serial/xilinx_uartps.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 131a3117fbbb..cd46e64c4255 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1077,7 +1077,9 @@ static int __init cdns_early_console_setup(struct earlycon_device *device,
 
 	return 0;
 }
-EARLYCON_DECLARE(cdns, cdns_early_console_setup);
+OF_EARLYCON_DECLARE(cdns, "xlnx,xuartps", cdns_early_console_setup);
+OF_EARLYCON_DECLARE(cdns, "cdns,uart-r1p8", cdns_early_console_setup);
+OF_EARLYCON_DECLARE(cdns, "cdns,uart-r1p12", cdns_early_console_setup);
 
 /**
  * cdns_uart_console_write - perform write operation
-- 
1.9.1

[PATCH v2] serial: xuartps: Enable OF earlycon support

[Michal Simek]

Support early console setup via DT for all listed compatible strings.
Remove EARLYCON_DECLARE which was done by:
"Use common framework for earlycon declarations"
(sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
when OF_EARLYCON_DECLARE is defined.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

Changes in v2:
- Separate serial patch from others
- Remove EARLYCON_DECLARE and use the same earlycon name with different
  compatible string as was pointed by Peter Hurley
- Extend commit message

 drivers/tty/serial/xilinx_uartps.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 131a3117fbbb..cd46e64c4255 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1077,7 +1077,9 @@ static int __init cdns_early_console_setup(struct earlycon_device *device,
 
 	return 0;
 }
-EARLYCON_DECLARE(cdns, cdns_early_console_setup);
+OF_EARLYCON_DECLARE(cdns, "xlnx,xuartps", cdns_early_console_setup);
+OF_EARLYCON_DECLARE(cdns, "cdns,uart-r1p8", cdns_early_console_setup);
+OF_EARLYCON_DECLARE(cdns, "cdns,uart-r1p12", cdns_early_console_setup);
 
 /**
  * cdns_uart_console_write - perform write operation
-- 
1.9.1

Re: [PATCH v2] serial: xuartps: Enable OF earlycon support

[Peter Hurley]

On 02/17/2016 11:57 PM, Michal Simek wrote:
> Support early console setup via DT for all listed compatible strings.
> Remove EARLYCON_DECLARE which was done by:
> "Use common framework for earlycon declarations"
> (sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
> when OF_EARLYCON_DECLARE is defined.

Reviewed-by: Peter Hurley <peter@hurleysoftware.com>

[PATCH v2] serial: xuartps: Enable OF earlycon support

[Peter Hurley]

On 02/17/2016 11:57 PM, Michal Simek wrote:
> Support early console setup via DT for all listed compatible strings.
> Remove EARLYCON_DECLARE which was done by:
> "Use common framework for earlycon declarations"
> (sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
> when OF_EARLYCON_DECLARE is defined.

Reviewed-by: Peter Hurley <peter@hurleysoftware.com>

Re: [PATCH v2] serial: xuartps: Enable OF earlycon support

[Michal Simek]

[-- Attachment #1.1: Type: text/plain, Size: 907 bytes --]

Hi Greg,

On 18.2.2016 17:50, Peter Hurley wrote:
> On 02/17/2016 11:57 PM, Michal Simek wrote:
>> Support early console setup via DT for all listed compatible strings.
>> Remove EARLYCON_DECLARE which was done by:
>> "Use common framework for earlycon declarations"
>> (sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
>> when OF_EARLYCON_DECLARE is defined.
> 
> Reviewed-by: Peter Hurley <peter@hurleysoftware.com>

Can you please add it your queue for 4.6?
Or can we merge it via arm-soc tree as was suggested by Arnd here?
https://lkml.org/lkml/2016/2/16/315

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[PATCH v2] serial: xuartps: Enable OF earlycon support

[Michal Simek]

Hi Greg,

On 18.2.2016 17:50, Peter Hurley wrote:
> On 02/17/2016 11:57 PM, Michal Simek wrote:
>> Support early console setup via DT for all listed compatible strings.
>> Remove EARLYCON_DECLARE which was done by:
>> "Use common framework for earlycon declarations"
>> (sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
>> when OF_EARLYCON_DECLARE is defined.
> 
> Reviewed-by: Peter Hurley <peter@hurleysoftware.com>

Can you please add it your queue for 4.6?
Or can we merge it via arm-soc tree as was suggested by Arnd here?
https://lkml.org/lkml/2016/2/16/315

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160225/a8164c32/attachment.sig>

Re: [PATCH v2] serial: xuartps: Enable OF earlycon support

[Greg Kroah-Hartman]

On Thu, Feb 25, 2016 at 02:04:23PM +0100, Michal Simek wrote:
> Hi Greg,
> 
> On 18.2.2016 17:50, Peter Hurley wrote:
> > On 02/17/2016 11:57 PM, Michal Simek wrote:
> >> Support early console setup via DT for all listed compatible strings.
> >> Remove EARLYCON_DECLARE which was done by:
> >> "Use common framework for earlycon declarations"
> >> (sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
> >> when OF_EARLYCON_DECLARE is defined.
> > 
> > Reviewed-by: Peter Hurley <peter@hurleysoftware.com>
> 
> Can you please add it your queue for 4.6?
> Or can we merge it via arm-soc tree as was suggested by Arnd here?
> https://lkml.org/lkml/2016/2/16/315

I can take it, thanks.

greg k-h

[PATCH v2] serial: xuartps: Enable OF earlycon support

[Greg Kroah-Hartman]

On Thu, Feb 25, 2016 at 02:04:23PM +0100, Michal Simek wrote:
> Hi Greg,
> 
> On 18.2.2016 17:50, Peter Hurley wrote:
> > On 02/17/2016 11:57 PM, Michal Simek wrote:
> >> Support early console setup via DT for all listed compatible strings.
> >> Remove EARLYCON_DECLARE which was done by:
> >> "Use common framework for earlycon declarations"
> >> (sha1: 2eaa790989e03900298ad24f77f1086dbbc1aebd)
> >> when OF_EARLYCON_DECLARE is defined.
> > 
> > Reviewed-by: Peter Hurley <peter@hurleysoftware.com>
> 
> Can you please add it your queue for 4.6?
> Or can we merge it via arm-soc tree as was suggested by Arnd here?
> https://lkml.org/lkml/2016/2/16/315

I can take it, thanks.

greg k-h