* [PATCH v2 0/2] dt: serial: Add Renesas RZ/N1 binding documentation
@ 2018-07-13 9:33 Phil Edworthy
2018-07-13 9:33 ` [PATCH v2 1/2] " Phil Edworthy
2018-07-13 9:33 ` [PATCH v2 2/2] serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART Phil Edworthy
0 siblings, 2 replies; 6+ messages in thread
From: Phil Edworthy @ 2018-07-13 9:33 UTC (permalink / raw)
Cc: Geert Uytterhoeven, Simon Horman, linux-serial,
linux-renesas-soc, Phil Edworthy
Just a new compatible string for the Synopsys UART to allow us to add DMA at
some point in the future.
v2:
- Change "renesas,<peripheral>-<family-or-device>" to
"renesas,<family-or-device>-<peripheral>"
Phil Edworthy (2):
dt: serial: Add Renesas RZ/N1 binding documentation
serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART
Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt | 10 ++++++++++
drivers/tty/serial/8250/8250_dw.c | 1 +
2 files changed, 11 insertions(+)
create mode 100644 Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/2] dt: serial: Add Renesas RZ/N1 binding documentation
2018-07-13 9:33 [PATCH v2 0/2] dt: serial: Add Renesas RZ/N1 binding documentation Phil Edworthy
@ 2018-07-13 9:33 ` Phil Edworthy
2018-07-16 17:49 ` Rob Herring
2018-07-17 12:09 ` Simon Horman
2018-07-13 9:33 ` [PATCH v2 2/2] serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART Phil Edworthy
1 sibling, 2 replies; 6+ messages in thread
From: Phil Edworthy @ 2018-07-13 9:33 UTC (permalink / raw)
To: Rob Herring, Greg Kroah-Hartman, Mark Rutland
Cc: Geert Uytterhoeven, Simon Horman, linux-serial,
linux-renesas-soc, Phil Edworthy, devicetree
The RZ/N1 UART is a modified Synopsys DesignWare UART.
The modifications only relate to DMA so you could actually use the
controller with the Synopsys compatible string if you are not using
DMA, but you should not do so.
Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
---
v2:
- Change "renesas,<peripheral>-<family-or-device>" to
"renesas,<family-or-device>-<peripheral>"
---
Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt | 10 ++++++++++
1 file changed, 10 insertions(+)
create mode 100644 Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt
diff --git a/Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt b/Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt
new file mode 100644
index 0000000..8b9e0d4
--- /dev/null
+++ b/Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt
@@ -0,0 +1,10 @@
+Renesas RZ/N1 UART
+
+This controller is based on the Synopsys DesignWare ABP UART and inherits all
+properties defined in snps-dw-apb-uart.txt except for the compatible property.
+
+Required properties:
+- compatible : The device specific string followed by the generic RZ/N1 string.
+ Therefore it must be one of:
+ "renesas,r9a06g032-uart", "renesas,rzn1-uart"
+ "renesas,r9a06g033-uart", "renesas,rzn1-uart"
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/2] serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART
2018-07-13 9:33 [PATCH v2 0/2] dt: serial: Add Renesas RZ/N1 binding documentation Phil Edworthy
2018-07-13 9:33 ` [PATCH v2 1/2] " Phil Edworthy
@ 2018-07-13 9:33 ` Phil Edworthy
2018-07-17 12:09 ` Simon Horman
1 sibling, 1 reply; 6+ messages in thread
From: Phil Edworthy @ 2018-07-13 9:33 UTC (permalink / raw)
To: Andy Shevchenko, Greg Kroah-Hartman
Cc: Geert Uytterhoeven, Simon Horman, linux-serial,
linux-renesas-soc, Phil Edworthy, Jiri Slaby, Joshua Scott,
Ed Blake, Stefan Potyra
The Renesas RZ/N1 UART is based on the Synopsys DW UART, but has additional
registers for DMA. This patch does not address the changes required for DMA
support, it simply adds the compatible string.
Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
---
v2:
- Change "renesas,<peripheral>-<family-or-device>" to
"renesas,<family-or-device>-<peripheral>"
---
drivers/tty/serial/8250/8250_dw.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/tty/serial/8250/8250_dw.c b/drivers/tty/serial/8250/8250_dw.c
index aff04f1..7a7c742 100644
--- a/drivers/tty/serial/8250/8250_dw.c
+++ b/drivers/tty/serial/8250/8250_dw.c
@@ -693,6 +693,7 @@ static const struct of_device_id dw8250_of_match[] = {
{ .compatible = "snps,dw-apb-uart" },
{ .compatible = "cavium,octeon-3860-uart" },
{ .compatible = "marvell,armada-38x-uart" },
+ { .compatible = "renesas,rzn1-uart" },
{ /* Sentinel */ }
};
MODULE_DEVICE_TABLE(of, dw8250_of_match);
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] dt: serial: Add Renesas RZ/N1 binding documentation
2018-07-13 9:33 ` [PATCH v2 1/2] " Phil Edworthy
@ 2018-07-16 17:49 ` Rob Herring
2018-07-17 12:09 ` Simon Horman
1 sibling, 0 replies; 6+ messages in thread
From: Rob Herring @ 2018-07-16 17:49 UTC (permalink / raw)
To: Phil Edworthy
Cc: Greg Kroah-Hartman, Mark Rutland, Geert Uytterhoeven,
Simon Horman, linux-serial, linux-renesas-soc, devicetree
On Fri, Jul 13, 2018 at 10:33:48AM +0100, Phil Edworthy wrote:
> The RZ/N1 UART is a modified Synopsys DesignWare UART.
> The modifications only relate to DMA so you could actually use the
> controller with the Synopsys compatible string if you are not using
> DMA, but you should not do so.
>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> ---
> v2:
> - Change "renesas,<peripheral>-<family-or-device>" to
> "renesas,<family-or-device>-<peripheral>"
> ---
> Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt | 10 ++++++++++
> 1 file changed, 10 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] dt: serial: Add Renesas RZ/N1 binding documentation
2018-07-13 9:33 ` [PATCH v2 1/2] " Phil Edworthy
2018-07-16 17:49 ` Rob Herring
@ 2018-07-17 12:09 ` Simon Horman
1 sibling, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-07-17 12:09 UTC (permalink / raw)
To: Phil Edworthy
Cc: Rob Herring, Greg Kroah-Hartman, Mark Rutland,
Geert Uytterhoeven, linux-serial, linux-renesas-soc, devicetree
On Fri, Jul 13, 2018 at 10:33:48AM +0100, Phil Edworthy wrote:
> The RZ/N1 UART is a modified Synopsys DesignWare UART.
> The modifications only relate to DMA so you could actually use the
> controller with the Synopsys compatible string if you are not using
> DMA, but you should not do so.
>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART
2018-07-13 9:33 ` [PATCH v2 2/2] serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART Phil Edworthy
@ 2018-07-17 12:09 ` Simon Horman
0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-07-17 12:09 UTC (permalink / raw)
To: Phil Edworthy
Cc: Andy Shevchenko, Greg Kroah-Hartman, Geert Uytterhoeven,
linux-serial, linux-renesas-soc, Jiri Slaby, Joshua Scott,
Ed Blake, Stefan Potyra
On Fri, Jul 13, 2018 at 10:33:49AM +0100, Phil Edworthy wrote:
> The Renesas RZ/N1 UART is based on the Synopsys DW UART, but has additional
> registers for DMA. This patch does not address the changes required for DMA
> support, it simply adds the compatible string.
>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-07-17 12:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-13 9:33 [PATCH v2 0/2] dt: serial: Add Renesas RZ/N1 binding documentation Phil Edworthy
2018-07-13 9:33 ` [PATCH v2 1/2] " Phil Edworthy
2018-07-16 17:49 ` Rob Herring
2018-07-17 12:09 ` Simon Horman
2018-07-13 9:33 ` [PATCH v2 2/2] serial: 8250_dw: Add compatible string for Renesas RZ/N1 UART Phil Edworthy
2018-07-17 12:09 ` Simon Horman
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.