* [PATCH 1/8] arm64: dts: renesas: salvator-common: Override EtherAVB phy-mode
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-2-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
2018-02-26 17:57 ` [PATCH 2/8] arm64: dts: renesas: r8a7796: Set EtherAVB phy mode to "rgmii" Jacopo Mondi
` (7 subsequent siblings)
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
As the PHY interface installed on the Salvator-X[S] board, provides TX
channel delay, make the "phy-mode" property a board-specific one, meant
to override the one specified in the SoC DTSI.
Follow up patches will reset the SoC DTSI to use "rgmii" mode and let
the board file override that.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
arch/arm64/boot/dts/renesas/salvator-common.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
index 8e8ec30..c725f9b 100644
--- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
+++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
@@ -279,6 +279,7 @@
pinctrl-0 = <&avb_pins>;
pinctrl-names = "default";
phy-handle = <&phy0>;
+ phy-mode = "rgmii-txid";
status = "okay";
phy0: ethernet-phy@0 {
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 2/8] arm64: dts: renesas: r8a7796: Set EtherAVB phy mode to "rgmii"
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
2018-02-26 17:57 ` [PATCH 1/8] arm64: dts: renesas: salvator-common: Override EtherAVB phy-mode Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-3-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
2018-02-26 17:57 ` [PATCH 3/8] arm64: dts: renesas: r8a7795: " Jacopo Mondi
` (6 subsequent siblings)
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
Set the "phy-mode" property of EtherAVB device to "rgmii" and let board
files override it if the installed PHY layer provides delays for the
RX/TX channels.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
arch/arm64/boot/dts/renesas/r8a7796.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
index 6075511..a99b0b2 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
@@ -974,7 +974,7 @@
clocks = <&cpg CPG_MOD 812>;
power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
resets = <&cpg 812>;
- phy-mode = "rgmii-txid";
+ phy-mode = "rgmii";
iommus = <&ipmmu_ds0 16>;
#address-cells = <1>;
#size-cells = <0>;
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 3/8] arm64: dts: renesas: r8a7795: Set EtherAVB phy mode to "rgmii"
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
2018-02-26 17:57 ` [PATCH 1/8] arm64: dts: renesas: salvator-common: Override EtherAVB phy-mode Jacopo Mondi
2018-02-26 17:57 ` [PATCH 2/8] arm64: dts: renesas: r8a7796: Set EtherAVB phy mode to "rgmii" Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-4-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
2018-02-26 17:57 ` [PATCH 4/8] arm64: dts: renesas: r8a77995: " Jacopo Mondi
` (5 subsequent siblings)
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
Set the "phy-mode" property of EtherAVB device to "rgmii" and let board
files override it if the installed PHY layer provides delays for the
RX/TX channels.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
arch/arm64/boot/dts/renesas/r8a7795.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index 1f32340..87327eb 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -873,7 +873,7 @@
clocks = <&cpg CPG_MOD 812>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
resets = <&cpg 812>;
- phy-mode = "rgmii-txid";
+ phy-mode = "rgmii";
iommus = <&ipmmu_ds0 16>;
#address-cells = <1>;
#size-cells = <0>;
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 4/8] arm64: dts: renesas: r8a77995: Set EtherAVB phy mode to "rgmii"
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
` (2 preceding siblings ...)
2018-02-26 17:57 ` [PATCH 3/8] arm64: dts: renesas: r8a7795: " Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-5-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
2018-02-26 17:57 ` [PATCH 5/8] dt-bindings: iommu/ipmmu-vmsa: Add R-Car M3-N (R8A77965) Jacopo Mondi
` (4 subsequent siblings)
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: devicetree, netdev, linux-kernel, linux-renesas-soc, iommu,
Jacopo Mondi, linux-arm-kernel
Set the "phy-mode" property of EtherAVB device to "rgmii" and let board
files override it if the installed PHY layer provides delays for the
RX/TX channels.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
arch/arm64/boot/dts/renesas/r8a77995.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
index cd3c6a3..fc48677 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
@@ -512,7 +512,7 @@
clocks = <&cpg CPG_MOD 812>;
power-domains = <&sysc R8A77995_PD_ALWAYS_ON>;
resets = <&cpg 812>;
- phy-mode = "rgmii-txid";
+ phy-mode = "rgmii";
iommus = <&ipmmu_ds0 16>;
#address-cells = <1>;
#size-cells = <0>;
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 5/8] dt-bindings: iommu/ipmmu-vmsa: Add R-Car M3-N (R8A77965)
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
` (3 preceding siblings ...)
2018-02-26 17:57 ` [PATCH 4/8] arm64: dts: renesas: r8a77995: " Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-6-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
2018-02-26 17:57 ` [PATCH 6/8] iommu/ipmmu-vmsa: Hook up R8A77965 DT matching code Jacopo Mondi
` (3 subsequent siblings)
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
Add Renesas R-Car M3-N (R8A77965) compat string to IPMMU DT bindings
documentation.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt b/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt
index 1fd5d69..0143fd4 100644
--- a/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt
+++ b/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt
@@ -17,6 +17,7 @@ Required Properties:
- "renesas,ipmmu-r8a7794" for the R8A7794 (R-Car E2) IPMMU.
- "renesas,ipmmu-r8a7795" for the R8A7795 (R-Car H3) IPMMU.
- "renesas,ipmmu-r8a7796" for the R8A7796 (R-Car M3-W) IPMMU.
+ - "renesas,ipmmu-r8a77965" for the R8A77965 (R-Car M3-N) IPMMU.
- "renesas,ipmmu-r8a77970" for the R8A77970 (R-Car V3M) IPMMU.
- "renesas,ipmmu-r8a77995" for the R8A77995 (R-Car D3) IPMMU.
- "renesas,ipmmu-vmsa" for generic R-Car Gen2 VMSA-compatible IPMMU.
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 6/8] iommu/ipmmu-vmsa: Hook up R8A77965 DT matching code
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
` (4 preceding siblings ...)
2018-02-26 17:57 ` [PATCH 5/8] dt-bindings: iommu/ipmmu-vmsa: Add R-Car M3-N (R8A77965) Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-7-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
2018-02-26 17:57 ` [PATCH 7/8] arm64: dts: renesas: r8a77965: Add IPMMU mm and ds0 blocks Jacopo Mondi
` (2 subsequent siblings)
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
Add support for R-Car M3-N (R8A77965) SoC IPMMUs.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
drivers/iommu/ipmmu-vmsa.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/iommu/ipmmu-vmsa.c b/drivers/iommu/ipmmu-vmsa.c
index 933a3da..6a0e714 100644
--- a/drivers/iommu/ipmmu-vmsa.c
+++ b/drivers/iommu/ipmmu-vmsa.c
@@ -763,6 +763,7 @@ static bool ipmmu_slave_whitelist(struct device *dev)
static const struct soc_device_attribute soc_rcar_gen3[] = {
{ .soc_id = "r8a7795", },
{ .soc_id = "r8a7796", },
+ { .soc_id = "r8a77965", },
{ .soc_id = "r8a77970", },
{ .soc_id = "r8a77995", },
{ /* sentinel */ }
@@ -945,6 +946,9 @@ static const struct of_device_id ipmmu_of_ids[] = {
.compatible = "renesas,ipmmu-r8a7796",
.data = &ipmmu_features_rcar_gen3,
}, {
+ .compatible = "renesas,ipmmu-r8a77965",
+ .data = &ipmmu_features_rcar_gen3,
+ }, {
.compatible = "renesas,ipmmu-r8a77970",
.data = &ipmmu_features_rcar_gen3,
}, {
@@ -1127,6 +1131,7 @@ module_exit(ipmmu_exit);
IOMMU_OF_DECLARE(ipmmu_vmsa_iommu_of, "renesas,ipmmu-vmsa");
IOMMU_OF_DECLARE(ipmmu_r8a7795_iommu_of, "renesas,ipmmu-r8a7795");
IOMMU_OF_DECLARE(ipmmu_r8a7796_iommu_of, "renesas,ipmmu-r8a7796");
+IOMMU_OF_DECLARE(ipmmu_r8a77965_iommu_of, "renesas,ipmmu-r8a77965");
IOMMU_OF_DECLARE(ipmmu_r8a77970_iommu_of, "renesas,ipmmu-r8a77970");
IOMMU_OF_DECLARE(ipmmu_r8a77995_iommu_of, "renesas,ipmmu-r8a77995");
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 7/8] arm64: dts: renesas: r8a77965: Add IPMMU mm and ds0 blocks
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
` (5 preceding siblings ...)
2018-02-26 17:57 ` [PATCH 6/8] iommu/ipmmu-vmsa: Hook up R8A77965 DT matching code Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
2018-02-26 17:57 ` [PATCH 8/8] arm64: dts: renesas: r8a77965: Add EtherAVB device node Jacopo Mondi
[not found] ` <1519667836-31330-1-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
8 siblings, 0 replies; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
Add IPMMU device nodes for mm and ds0 domains. "ipmmu_ds0" is a
dependency for EtherAVB enablement and it has "ipmmu_mm" as it main
ipmmu.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
arch/arm64/boot/dts/renesas/r8a77965.dtsi | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
index 8c9648a..b3c0be8 100644
--- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
@@ -169,6 +169,23 @@
reg = <0 0xe6060000 0 0x50c>;
};
+ ipmmu_ds0: mmu@e6740000 {
+ compatible = "renesas,ipmmu-r8a77965";
+ reg = <0 0xe6740000 0 0x1000>;
+ renesas,ipmmu-main = <&ipmmu_mm 0>;
+ power-domains = <&sysc 32>;
+ #iommu-cells = <1>;
+ };
+
+ ipmmu_mm: mmu@e67b0000 {
+ compatible = "renesas,ipmmu-r8a77965";
+ reg = <0 0xe67b0000 0 0x1000>;
+ interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 197 IRQ_TYPE_LEVEL_HIGH>;
+ power-domains = <&sysc 32>;
+ #iommu-cells = <1>;
+ };
+
cpg: clock-controller@e6150000 {
compatible = "renesas,r8a77965-cpg-mssr";
reg = <0 0xe6150000 0 0x1000>;
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH 8/8] arm64: dts: renesas: r8a77965: Add EtherAVB device node
2018-02-26 17:57 [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node Jacopo Mondi
` (6 preceding siblings ...)
2018-02-26 17:57 ` [PATCH 7/8] arm64: dts: renesas: r8a77965: Add IPMMU mm and ds0 blocks Jacopo Mondi
@ 2018-02-26 17:57 ` Jacopo Mondi
[not found] ` <1519667836-31330-9-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
[not found] ` <1519667836-31330-1-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
8 siblings, 1 reply; 20+ messages in thread
From: Jacopo Mondi @ 2018-02-26 17:57 UTC (permalink / raw)
To: geert, horms, sergei.shtylyov, magnus.damm, robh+dt, mark.rutland
Cc: Jacopo Mondi, devicetree, linux-arm-kernel, iommu, netdev,
linux-renesas-soc, linux-kernel
Populate the ethernet@e6800000 device node to enable Ethernet interface
for R-Car M3-N (R8A77965) SoC.
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v1 -> v2:
- Replace ALWAYS_ON power area identifier with numeric constant
---
arch/arm64/boot/dts/renesas/r8a77965.dtsi | 44 ++++++++++++++++++++++++++++---
1 file changed, 41 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
index b3c0be8..37aac63 100644
--- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
@@ -539,11 +539,49 @@
};
avb: ethernet@e6800000 {
+ compatible = "renesas,etheravb-r8a77965",
+ "renesas,etheravb-rcar-gen3";
+ reg = <0 0xe6800000 0 0x800>, <0 0xe6a00000 0 0x10000>;
+ interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 41 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 46 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 52 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 53 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 54 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 55 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 59 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 63 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-names = "ch0", "ch1", "ch2", "ch3",
+ "ch4", "ch5", "ch6", "ch7",
+ "ch8", "ch9", "ch10", "ch11",
+ "ch12", "ch13", "ch14", "ch15",
+ "ch16", "ch17", "ch18", "ch19",
+ "ch20", "ch21", "ch22", "ch23",
+ "ch24";
+ clocks = <&cpg CPG_MOD 812>;
+ power-domains = <&sysc 32>;
+ resets = <&cpg 812>;
+ phy-mode = "rgmii";
+ iommus = <&ipmmu_ds0 16>;
#address-cells = <1>;
#size-cells = <0>;
-
- reg = <0 0xe6800000 0 0x800>, <0 0xe6a00000 0 0x10000>;
- /* placeholder */
+ status = "disabled";
};
csi20: csi2@fea80000 {
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread[parent not found: <1519667836-31330-1-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
[not found] ` <1519667836-31330-1-git-send-email-jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org>
@ 2018-02-26 18:28 ` Geert Uytterhoeven
2018-02-27 8:28 ` jacopo mondi
0 siblings, 1 reply; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-26 18:28 UTC (permalink / raw)
To: Jacopo Mondi
Cc: Mark Rutland,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Sergei Shtylyov, Simon Horman, netdev, Magnus Damm,
Linux Kernel Mailing List, Linux-Renesas,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA, Rob Herring,
Linux ARM
Hi Jacopo,
On Mon, Feb 26, 2018 at 6:57 PM, Jacopo Mondi <jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org> wrote:
> as discussed with you Sergei and Geert, in order to enable EtherAVB for
> R8A77965 we first wanted to make the phy-mode "rgmii-txid" a board specific
> property for all other SoCs.
Thanks for your series!
> This series adds the phy-mode property to salvator-common.dtsi and reset the
> one for R8A77965/R8A7796/R8A77995 to "rgmii".
I forgot that r8a7795.dtsi and r8a7796.dtsi are used for the ULCB boards, too.
So to avoid regressions, you need to make a similar change to ulcb.dtsi like you
made for salvator-common.dtsi.
In addition, r8a77995.dtsi is only used for the Draak board.
So you have to update r8a7795-draak.dtsi first, too.
> Then, in order to enable EtherAVB for R-Car M3-N, iommu nodes had to be added
> as they are referenced by the "avb" device node (CC the iommu list for the
> series for that reason).
I don't think we need the iommu properties and nodes at this early stage.
Ethernet works fine without them. Simon, do you agree?
P.S. scripts/dtc/dtx_diff is a great tool to compare DTBs before and after your
changes. It would have revealed the changes for ULCB and Draak.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
2018-02-26 18:28 ` [PATCH 0/8] R-Car M3-N: Enable " Geert Uytterhoeven
@ 2018-02-27 8:28 ` jacopo mondi
2018-02-27 8:35 ` Simon Horman
0 siblings, 1 reply; 20+ messages in thread
From: jacopo mondi @ 2018-02-27 8:28 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Jacopo Mondi, Simon Horman, Sergei Shtylyov, Magnus Damm,
Rob Herring, Mark Rutland,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Linux ARM, iommu, netdev, Linux-Renesas,
Linux Kernel Mailing List
Hi Geert,
On Mon, Feb 26, 2018 at 07:28:47PM +0100, Geert Uytterhoeven wrote:
> Hi Jacopo,
>
> On Mon, Feb 26, 2018 at 6:57 PM, Jacopo Mondi <jacopo+renesas@jmondi.org> wrote:
> > as discussed with you Sergei and Geert, in order to enable EtherAVB for
> > R8A77965 we first wanted to make the phy-mode "rgmii-txid" a board specific
> > property for all other SoCs.
>
> Thanks for your series!
>
> > This series adds the phy-mode property to salvator-common.dtsi and reset the
> > one for R8A77965/R8A7796/R8A77995 to "rgmii".
>
> I forgot that r8a7795.dtsi and r8a7796.dtsi are used for the ULCB boards, too.
> So to avoid regressions, you need to make a similar change to ulcb.dtsi like you
> made for salvator-common.dtsi.
>
> In addition, r8a77995.dtsi is only used for the Draak board.
> So you have to update r8a7795-draak.dtsi first, too.
Of course! I forgot about ULCB (and had a patch for Draak I lost while
rebasing :/ )
>
> > Then, in order to enable EtherAVB for R-Car M3-N, iommu nodes had to be added
> > as they are referenced by the "avb" device node (CC the iommu list for the
> > series for that reason).
>
> I don't think we need the iommu properties and nodes at this early stage.
> Ethernet works fine without them. Simon, do you agree?
I'll wait for Simon reply and then resend, possibly without iommu and
ULCB and Draak patches!
Thanks
j
>
> P.S. scripts/dtc/dtx_diff is a great tool to compare DTBs before and after your
> changes. It would have revealed the changes for ULCB and Draak.
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
2018-02-27 8:28 ` jacopo mondi
@ 2018-02-27 8:35 ` Simon Horman
0 siblings, 0 replies; 20+ messages in thread
From: Simon Horman @ 2018-02-27 8:35 UTC (permalink / raw)
To: jacopo mondi
Cc: Mark Rutland,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Sergei Shtylyov, netdev, Magnus Damm, Linux Kernel Mailing List,
Linux-Renesas, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
Rob Herring, Jacopo Mondi, Geert Uytterhoeven, Linux ARM
On Tue, Feb 27, 2018 at 09:28:38AM +0100, jacopo mondi wrote:
> Hi Geert,
>
> On Mon, Feb 26, 2018 at 07:28:47PM +0100, Geert Uytterhoeven wrote:
> > Hi Jacopo,
> >
> > On Mon, Feb 26, 2018 at 6:57 PM, Jacopo Mondi <jacopo+renesas-AW8dsiIh9cEdnm+yROfE0A@public.gmane.org> wrote:
> > > as discussed with you Sergei and Geert, in order to enable EtherAVB for
> > > R8A77965 we first wanted to make the phy-mode "rgmii-txid" a board specific
> > > property for all other SoCs.
> >
> > Thanks for your series!
> >
> > > This series adds the phy-mode property to salvator-common.dtsi and reset the
> > > one for R8A77965/R8A7796/R8A77995 to "rgmii".
> >
> > I forgot that r8a7795.dtsi and r8a7796.dtsi are used for the ULCB boards, too.
> > So to avoid regressions, you need to make a similar change to ulcb.dtsi like you
> > made for salvator-common.dtsi.
> >
> > In addition, r8a77995.dtsi is only used for the Draak board.
> > So you have to update r8a7795-draak.dtsi first, too.
>
> Of course! I forgot about ULCB (and had a patch for Draak I lost while
> rebasing :/ )
> >
> > > Then, in order to enable EtherAVB for R-Car M3-N, iommu nodes had to be added
> > > as they are referenced by the "avb" device node (CC the iommu list for the
> > > series for that reason).
> >
> > I don't think we need the iommu properties and nodes at this early stage.
> > Ethernet works fine without them. Simon, do you agree?
>
> I'll wait for Simon reply and then resend, possibly without iommu and
> ULCB and Draak patches!
I don't feel strongly about this but I don't think iommu is a strict
dependency of enabling Ethernet and I think its good not to include extra
dependencies - if it was me I'd try to get Ethernet accepted then follow-up
on iommu.
^ permalink raw reply [flat|nested] 20+ messages in thread