lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 15 Feb 2017 16:40:25 -0600 From: Rob Herring <robh@...nel.org> To: Chris Packham <chris.packham@...iedtelesis.co.nz> Cc: linux-arm-kernel@...ts.infradead.org, Mark Rutland <mark.rutland@....com>, Jason Cooper <jason@...edaemon.net>, Andrew Lunn <andrew@...n.ch>, Gregory Clement <gregory.clement@...e-electrons.com>, Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>, Russell King <linux@...linux.org.uk>, netdev@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v2 2/6] ARM: dts: armada-xp-98dx3236: combine dfx server nodes On Wed, Feb 08, 2017 at 09:28:11AM +1300, Chris Packham wrote: > Rather than having a separate node for the dfx server add a reg property > to the parent node. This give somes compatibility with the Marvell > supplied SDK. But breaks upstream compatibility. Need to explain why that's okay. > > Signed-off-by: Chris Packham <chris.packham@...iedtelesis.co.nz> > --- > > Notes: > Changes in v2: > - none > > Documentation/devicetree/bindings/net/marvell,prestera.txt | 13 +++++-------- > arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 8 ++------ > 2 files changed, 7 insertions(+), 14 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/marvell,prestera.txt b/Documentation/devicetree/bindings/net/marvell,prestera.txt > index 5fbab29718e8..c329608fa887 100644 > --- a/Documentation/devicetree/bindings/net/marvell,prestera.txt > +++ b/Documentation/devicetree/bindings/net/marvell,prestera.txt > @@ -32,19 +32,16 @@ DFX Server bindings > ------------------- > > Required properties: > -- compatible: must be "marvell,dfx-server" > +- compatible: must be "marvell,dfx-server", "simple-bus" > +- ranges: describes the address mapping of a memory-mapped bus. > - reg: address and length of the register set for the device. > > Example: > > -dfx-registers { > - compatible = "simple-bus"; > +dfx-server { > + compatible = "marvell,dfx-server", "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > ranges = <0 MBUS_ID(0x08, 0x00) 0 0x100000>; > - > - dfx: dfx@0 { > - compatible = "marvell,dfx-server"; > - reg = <0 0x100000>; > - }; > + reg = <MBUS_ID(0x08, 0x00) 0 0x100000>; > }; > diff --git a/arch/arm/boot/dts/armada-xp-98dx3236.dtsi b/arch/arm/boot/dts/armada-xp-98dx3236.dtsi > index f6a03dcee5ef..bd8261fdec81 100644 > --- a/arch/arm/boot/dts/armada-xp-98dx3236.dtsi > +++ b/arch/arm/boot/dts/armada-xp-98dx3236.dtsi > @@ -195,11 +195,12 @@ > }; > }; > > - dfxr: dfx-registers@...00000 { > + dfx: dfx-server@...00000 { > compatible = "simple-bus"; Missing marvell,dfx-server? > #address-cells = <1>; > #size-cells = <1>; > ranges = <0 MBUS_ID(0x08, 0x00) 0 0x100000>; > + reg = <MBUS_ID(0x08, 0x00) 0 0x100000>; > > dfx_coredivclk: corediv-clock@...68 { > compatible = "marvell,mv98dx3236-corediv-clock"; > @@ -208,11 +209,6 @@ > clocks = <&mainpll>; > clock-output-names = "nand"; > }; > - > - dfx: dfx@0 { > - compatible = "marvell,dfx-server"; > - reg = <0 0x100000>; > - }; > }; > > switch: switch@...00000 { > -- > 2.11.0.24.ge6920cf >
Powered by blists - more mailing lists