Re: [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
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> > 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.
Re: [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
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> 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 -- > ge...@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
Re: [PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
Hi Jacopo, On Mon, Feb 26, 2018 at 6:57 PM, Jacopo Mondiwrote: > 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 -- ge...@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
[PATCH 0/8] R-Car M3-N: Enable EtherAVB device node
Hi Simon, 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. This series adds the phy-mode property to salvator-common.dtsi and reset the one for R8A77965/R8A7796/R8A77995 to "rgmii". 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). Series based on what you already applied to your development tree (M3-N initial support - EtherAVB + DTS fixes). A branch for testing is available at git://jmondi.org/linux m3-n/renesas-drivers-2018-02-13-v4.16-rc1/v2-simon Thanks j Jacopo Mondi (8): arm64: dts: renesas: salvator-common: Override EtherAVB phy-mode arm64: dts: renesas: r8a7796: Set EtherAVB phy mode to "rgmii" arm64: dts: renesas: r8a7795: Set EtherAVB phy mode to "rgmii" arm64: dts: renesas: r8a77995: Set EtherAVB phy mode to "rgmii" dt-bindings: iommu/ipmmu-vmsa: Add R-Car M3-N (R8A77965) iommu/ipmmu-vmsa: Hook up R8A77965 DT matching code arm64: dts: renesas: r8a77965: Add IPMMU mm and ds0 blocks arm64: dts: renesas: r8a77965: Add EtherAVB device node .../bindings/iommu/renesas,ipmmu-vmsa.txt | 1 + arch/arm64/boot/dts/renesas/r8a7795.dtsi | 2 +- arch/arm64/boot/dts/renesas/r8a7796.dtsi | 2 +- arch/arm64/boot/dts/renesas/r8a77965.dtsi | 61 -- arch/arm64/boot/dts/renesas/r8a77995.dtsi | 2 +- arch/arm64/boot/dts/renesas/salvator-common.dtsi | 1 + drivers/iommu/ipmmu-vmsa.c | 5 ++ 7 files changed, 68 insertions(+), 6 deletions(-) -- 2.7.4