From: Rob Herring <robh@kernel.org> To: Lubomir Rintel <lkundrak@v3.sk> Cc: Linus Walleij <linus.walleij@linaro.org>, Bartosz Golaszewski <bgolaszewski@baylibre.com>, Thomas Gleixner <tglx@linutronix.de>, Jason Cooper <jason@lakedaemon.net>, Marc Zyngier <maz@kernel.org>, Mauro Carvalho Chehab <mchehab@kernel.org>, Ulf Hansson <ulf.hansson@linaro.org>, Kishon Vijay Abraham I <kishon@ti.com>, Alessandro Zummo <a.zummo@towertech.it>, Alexandre Belloni <alexandre.belloni@bootlin.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Mark Brown <broonie@kernel.org>, Daniel Lezcano <daniel.lezcano@linaro.org>, Andrew Lunn <andrew@lunn.ch>, Gregory Clement <gregory.clement@bootlin.com>, Daniel Mack <daniel@zonque.org>, Haojian Zhuang <haojian.zhuang@gmail.com>, Robert Jarzmik <robert.jarzmik@free.fr>, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, linux-i2c@vger.kernel.org, linux-media@vger.kernel.org, linux-mmc@vger.kernel.org, linux-rtc@vger.kernel.org, linux-serial@vger.kernel.org, linux-spi@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH 14/28] dt-bindings: arm: l2x0: Tauros 3 is PL310 compatible Date: Fri, 27 Mar 2020 13:03:04 -0600 [thread overview] Message-ID: <20200327190304.GA27639@bogus> (raw) In-Reply-To: <20200317093922.20785-15-lkundrak@v3.sk> On Tue, Mar 17, 2020 at 10:39:08AM +0100, Lubomir Rintel wrote: > The validation is unhappy about mmp3-dell-ariel declaring its > marvell,tauros3-cache node to be compatible with arm,pl310-cache: > > mmp3-dell-ariel.dt.yaml: cache-controller@d0020000: compatible: > Additional items are not allowed ('arm,pl310-cache' was unexpected) > mmp3-dell-ariel.dt.yaml: cache-controller@d0020000: compatible: > ['marvell,tauros3-cache', 'arm,pl310-cache'] is too long > > Let's allow this -- Tauros 3 is designed to be compatible with PL310. > > Signed-off-by: Lubomir Rintel <lkundrak@v3.sk> > --- > .../devicetree/bindings/arm/l2c2x0.yaml | 45 ++++++++++--------- > 1 file changed, 24 insertions(+), 21 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/l2c2x0.yaml b/Documentation/devicetree/bindings/arm/l2c2x0.yaml > index 913a8cd8b2c00..7e39088a9bed2 100644 > --- a/Documentation/devicetree/bindings/arm/l2c2x0.yaml > +++ b/Documentation/devicetree/bindings/arm/l2c2x0.yaml > @@ -29,27 +29,30 @@ allOf: > > properties: > compatible: > - enum: > - - arm,pl310-cache > - - arm,l220-cache > - - arm,l210-cache > - # DEPRECATED by "brcm,bcm11351-a2-pl310-cache" > - - bcm,bcm11351-a2-pl310-cache > - # For Broadcom bcm11351 chipset where an > - # offset needs to be added to the address before passing down to the L2 > - # cache controller > - - brcm,bcm11351-a2-pl310-cache > - # Marvell Controller designed to be > - # compatible with the ARM one, with system cache mode (meaning > - # maintenance operations on L1 are broadcasted to the L2 and L2 > - # performs the same operation). > - - marvell,aurora-system-cache > - # Marvell Controller designed to be > - # compatible with the ARM one with outer cache mode. > - - marvell,aurora-outer-cache > - # Marvell Tauros3 cache controller, compatible > - # with arm,pl310-cache controller. > - - marvell,tauros3-cache > + oneOf: > + - enum: > + - arm,pl310-cache The list should be indented 2 more spaces. I'll fixup when applying. > + - arm,l220-cache > + - arm,l210-cache > + # DEPRECATED by "brcm,bcm11351-a2-pl310-cache" > + - bcm,bcm11351-a2-pl310-cache > + # For Broadcom bcm11351 chipset where an > + # offset needs to be added to the address before passing down to the L2 > + # cache controller > + - brcm,bcm11351-a2-pl310-cache > + # Marvell Controller designed to be > + # compatible with the ARM one, with system cache mode (meaning > + # maintenance operations on L1 are broadcasted to the L2 and L2 > + # performs the same operation). > + - marvell,aurora-system-cache > + # Marvell Controller designed to be > + # compatible with the ARM one with outer cache mode. > + - marvell,aurora-outer-cache > + - items: > + # Marvell Tauros3 cache controller, compatible > + # with arm,pl310-cache controller. > + - const: marvell,tauros3-cache > + - const: arm,pl310-cache > > cache-level: > const: 2 > -- > 2.25.1 >
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org> To: Lubomir Rintel <lkundrak@v3.sk> Cc: Linus Walleij <linus.walleij@linaro.org>, Bartosz Golaszewski <bgolaszewski@baylibre.com>, Thomas Gleixner <tglx@linutronix.de>, Jason Cooper <jason@lakedaemon.net>, Marc Zyngier <maz@kernel.org>, Mauro Carvalho Chehab <mchehab@kernel.org>, Ulf Hansson <ulf.hansson@linaro.org>, Kishon Vijay Abraham I <kishon@ti.com>, Alessandro Zummo <a.zummo@towertech.it>, Alexandre Belloni <alexandre.belloni@bootlin.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Mark Brown <broonie@kernel.org>, Daniel Lezcano <daniel.lezcano@linaro.org>, Andrew Lunn <andrew@lunn.ch>, Gregory Clement <gregory.clement@bootlin.com>, Daniel Mack <daniel@zonque.org>, Haojian Zhuang <haojian.zhuang@gmail.com>, Robert Jarzmik <robert.jarzmik@free.fr>, devicetree@vger.kerne Subject: Re: [PATCH 14/28] dt-bindings: arm: l2x0: Tauros 3 is PL310 compatible Date: Fri, 27 Mar 2020 13:03:04 -0600 [thread overview] Message-ID: <20200327190304.GA27639@bogus> (raw) In-Reply-To: <20200317093922.20785-15-lkundrak@v3.sk> On Tue, Mar 17, 2020 at 10:39:08AM +0100, Lubomir Rintel wrote: > The validation is unhappy about mmp3-dell-ariel declaring its > marvell,tauros3-cache node to be compatible with arm,pl310-cache: > > mmp3-dell-ariel.dt.yaml: cache-controller@d0020000: compatible: > Additional items are not allowed ('arm,pl310-cache' was unexpected) > mmp3-dell-ariel.dt.yaml: cache-controller@d0020000: compatible: > ['marvell,tauros3-cache', 'arm,pl310-cache'] is too long > > Let's allow this -- Tauros 3 is designed to be compatible with PL310. > > Signed-off-by: Lubomir Rintel <lkundrak@v3.sk> > --- > .../devicetree/bindings/arm/l2c2x0.yaml | 45 ++++++++++--------- > 1 file changed, 24 insertions(+), 21 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/l2c2x0.yaml b/Documentation/devicetree/bindings/arm/l2c2x0.yaml > index 913a8cd8b2c00..7e39088a9bed2 100644 > --- a/Documentation/devicetree/bindings/arm/l2c2x0.yaml > +++ b/Documentation/devicetree/bindings/arm/l2c2x0.yaml > @@ -29,27 +29,30 @@ allOf: > > properties: > compatible: > - enum: > - - arm,pl310-cache > - - arm,l220-cache > - - arm,l210-cache > - # DEPRECATED by "brcm,bcm11351-a2-pl310-cache" > - - bcm,bcm11351-a2-pl310-cache > - # For Broadcom bcm11351 chipset where an > - # offset needs to be added to the address before passing down to the L2 > - # cache controller > - - brcm,bcm11351-a2-pl310-cache > - # Marvell Controller designed to be > - # compatible with the ARM one, with system cache mode (meaning > - # maintenance operations on L1 are broadcasted to the L2 and L2 > - # performs the same operation). > - - marvell,aurora-system-cache > - # Marvell Controller designed to be > - # compatible with the ARM one with outer cache mode. > - - marvell,aurora-outer-cache > - # Marvell Tauros3 cache controller, compatible > - # with arm,pl310-cache controller. > - - marvell,tauros3-cache > + oneOf: > + - enum: > + - arm,pl310-cache The list should be indented 2 more spaces. I'll fixup when applying. > + - arm,l220-cache > + - arm,l210-cache > + # DEPRECATED by "brcm,bcm11351-a2-pl310-cache" > + - bcm,bcm11351-a2-pl310-cache > + # For Broadcom bcm11351 chipset where an > + # offset needs to be added to the address before passing down to the L2 > + # cache controller > + - brcm,bcm11351-a2-pl310-cache > + # Marvell Controller designed to be > + # compatible with the ARM one, with system cache mode (meaning > + # maintenance operations on L1 are broadcasted to the L2 and L2 > + # performs the same operation). > + - marvell,aurora-system-cache > + # Marvell Controller designed to be > + # compatible with the ARM one with outer cache mode. > + - marvell,aurora-outer-cache > + - items: > + # Marvell Tauros3 cache controller, compatible > + # with arm,pl310-cache controller. > + - const: marvell,tauros3-cache > + - const: arm,pl310-cache > > cache-level: > const: 2 > -- > 2.25.1 >
next prev parent reply other threads:[~2020-03-27 19:03 UTC|newest] Thread overview: 155+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-03-17 9:38 [PATCH 00/28] DT: Improve validation for Marvell SoCs Lubomir Rintel 2020-03-17 9:38 ` Lubomir Rintel 2020-03-17 9:38 ` [PATCH 01/28] ARM: dts: kirkwood: Fix interrupt controller node name Lubomir Rintel 2020-03-17 9:38 ` Lubomir Rintel 2020-03-17 13:20 ` Andrew Lunn 2020-03-17 13:20 ` Andrew Lunn 2020-03-17 9:38 ` [PATCH 02/28] ARM: dts: dove: " Lubomir Rintel 2020-03-17 9:38 ` Lubomir Rintel 2020-03-17 13:20 ` Andrew Lunn 2020-03-17 13:20 ` Andrew Lunn 2020-03-17 9:38 ` [PATCH 03/28] ARM: dts: pxa168: Add missing address/size cells to i2c nodes Lubomir Rintel 2020-03-17 9:38 ` Lubomir Rintel 2020-03-17 13:21 ` Andrew Lunn 2020-03-17 13:21 ` Andrew Lunn 2020-03-17 9:38 ` [PATCH 04/28] ARM: dts: pxa168: Fix the gpio interrupt cell number Lubomir Rintel 2020-03-17 9:38 ` Lubomir Rintel 2020-03-17 13:27 ` Andrew Lunn 2020-03-17 13:27 ` Andrew Lunn 2020-03-17 9:38 ` [PATCH 05/28] ARM: dts: pxa3xx: Fix up encoding of the /gpio interrupts property Lubomir Rintel 2020-03-17 9:38 ` Lubomir Rintel 2020-03-17 13:28 ` Andrew Lunn 2020-03-17 13:28 ` Andrew Lunn 2020-03-21 7:36 ` Robert Jarzmik 2020-03-21 7:36 ` Robert Jarzmik 2020-03-21 7:36 ` Robert Jarzmik 2020-03-17 9:39 ` [PATCH 06/28] ARM: dts: pxa910: Fix the gpio interrupt cell number Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:29 ` Andrew Lunn 2020-03-17 13:29 ` Andrew Lunn 2020-03-17 9:39 ` [PATCH 07/28] ARM: dts: pxa*: Fix up encoding of the /rtc interrupts property Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:29 ` Andrew Lunn 2020-03-17 13:29 ` Andrew Lunn 2020-03-17 9:39 ` [PATCH 08/28] ARM: dts: mmp*: " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:30 ` Andrew Lunn 2020-03-17 13:30 ` Andrew Lunn 2020-03-17 13:30 ` Andrew Lunn 2020-03-17 13:30 ` Andrew Lunn 2020-03-17 9:39 ` [PATCH 09/28] ARM: dts: mmp3: fix L2 cache controller node name Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:31 ` Andrew Lunn 2020-03-17 13:31 ` Andrew Lunn 2020-03-17 9:39 ` [PATCH 10/28] ARM: dts: mmp3: fix USB & USB PHY node names Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:32 ` Andrew Lunn 2020-03-17 13:32 ` Andrew Lunn 2020-03-17 9:39 ` [PATCH 11/28] ARM: dts: berlin*: Fix up the SDHCI " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:34 ` Andrew Lunn 2020-03-17 13:34 ` Andrew Lunn 2020-03-17 9:39 ` [PATCH 12/28] spi: dt-bindings: spi-controller: Slaves have no address cells Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-19 16:51 ` Rob Herring 2020-03-19 16:51 ` Rob Herring 2020-03-17 9:39 ` [PATCH 13/28] dt-bindings: serial: move Marvell compatible string to 8250 binding doc Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:48 ` Andrew Lunn 2020-03-17 13:48 ` Andrew Lunn 2020-03-17 19:09 ` Lubomir Rintel 2020-03-17 19:09 ` Lubomir Rintel 2020-03-19 16:09 ` Rob Herring 2020-03-19 16:09 ` Rob Herring 2020-03-19 16:11 ` Rob Herring 2020-03-19 16:11 ` Rob Herring 2020-03-20 9:24 ` Lubomir Rintel 2020-03-20 9:24 ` Lubomir Rintel 2020-03-17 9:39 ` [PATCH 14/28] dt-bindings: arm: l2x0: Tauros 3 is PL310 compatible Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:03 ` Rob Herring [this message] 2020-03-27 19:03 ` Rob Herring 2020-03-27 19:04 ` Rob Herring 2020-03-27 19:04 ` Rob Herring 2020-03-17 9:39 ` [PATCH 15/28] dt-bindings: arm: mrvl: Add missing compatible strings Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 13:54 ` Andrew Lunn 2020-03-17 13:54 ` Andrew Lunn 2020-03-27 19:06 ` Rob Herring 2020-03-27 19:06 ` Rob Herring 2020-03-17 9:39 ` [PATCH 16/28] dt-bindings: Add "mrvl", a legacy vendor prefix for Marvell Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:06 ` Rob Herring 2020-03-27 19:06 ` Rob Herring 2020-03-17 9:39 ` [PATCH 17/28] dt-bindings: mmc: Fix up clk-phase-sd-hs in an example Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-17 10:04 ` Sergei Shtylyov 2020-03-17 10:04 ` Sergei Shtylyov 2020-03-27 19:08 ` Rob Herring 2020-03-27 19:08 ` Rob Herring 2020-03-17 9:39 ` [PATCH 18/28] dt-bindings: mmc: Fix node name " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-26 14:28 ` Ulf Hansson 2020-03-26 14:28 ` Ulf Hansson 2020-03-17 9:39 ` [PATCH 19/28] dt-bindings: mmc: Convert sdhci-pxa to json-schema Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-18 22:14 ` Rob Herring 2020-03-18 22:14 ` Rob Herring 2020-03-18 22:37 ` Rob Herring 2020-03-18 22:37 ` Rob Herring 2020-05-17 18:35 ` Lubomir Rintel 2020-03-17 9:39 ` [PATCH 20/28] dt-bindings: phy: Convert phy-mmp3-usb " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:09 ` Rob Herring 2020-03-27 19:09 ` Rob Herring 2020-03-17 9:39 ` [PATCH 21/28] dt-bindings: gpio: Convert mrvl-gpio " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 10:04 ` Linus Walleij 2020-03-27 10:04 ` Linus Walleij 2020-03-27 19:29 ` Rob Herring 2020-03-27 19:29 ` Rob Herring 2020-03-27 19:28 ` Rob Herring 2020-03-27 19:28 ` Rob Herring 2020-05-17 19:00 ` Lubomir Rintel 2020-03-17 9:39 ` [PATCH 22/28] dt-bindings: i2c: Convert i2c-pxa " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-06-03 20:41 ` Wolfram Sang 2020-03-17 9:39 ` [PATCH 23/28] dt-bindings: interrupt-controller: Convert mrvl,intc " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:42 ` Rob Herring 2020-03-27 19:42 ` Rob Herring 2020-05-17 19:11 ` Lubomir Rintel 2020-03-17 9:39 ` [PATCH 24/28] dt-bindings: media: Convert marvell,mmp2-ccic " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:46 ` Rob Herring 2020-03-27 19:46 ` Rob Herring 2020-05-17 19:21 ` Lubomir Rintel 2020-03-17 9:39 ` [PATCH 25/28] dt-bindings: rtc: Convert sa1100-rtc " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:49 ` Rob Herring 2020-03-27 19:49 ` Rob Herring 2020-05-17 19:25 ` Lubomir Rintel 2020-05-18 15:12 ` Rob Herring 2020-03-17 9:39 ` [PATCH 26/28] dt-bindings: spi: Convert spi-pxa2xx " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:52 ` Rob Herring 2020-03-27 19:52 ` Rob Herring 2020-05-17 19:30 ` Lubomir Rintel 2020-03-17 9:39 ` [PATCH 27/28] dt-bindings: timer: Convert mrvl,mmp-timer " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:53 ` Rob Herring 2020-03-27 19:53 ` Rob Herring 2020-03-17 9:39 ` [PATCH 28/28] dt-bindings: usb: Convert ehci-mv " Lubomir Rintel 2020-03-17 9:39 ` Lubomir Rintel 2020-03-27 19:55 ` Rob Herring 2020-03-27 19:55 ` Rob Herring 2020-05-17 19:34 ` Lubomir Rintel 2020-05-18 15:20 ` Rob Herring 2020-03-17 13:19 ` [PATCH 00/28] DT: Improve validation for Marvell SoCs Mark Brown 2020-03-17 13:19 ` Mark Brown 2020-03-17 13:46 ` Andrew Lunn 2020-03-17 13:46 ` Andrew Lunn 2020-03-17 13:55 ` Alexandre Belloni 2020-03-17 13:55 ` Alexandre Belloni 2020-03-17 13:59 ` Andrew Lunn 2020-03-17 13:59 ` Andrew Lunn
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200327190304.GA27639@bogus \ --to=robh@kernel.org \ --cc=a.zummo@towertech.it \ --cc=alexandre.belloni@bootlin.com \ --cc=andrew@lunn.ch \ --cc=bgolaszewski@baylibre.com \ --cc=broonie@kernel.org \ --cc=daniel.lezcano@linaro.org \ --cc=daniel@zonque.org \ --cc=devicetree@vger.kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=gregory.clement@bootlin.com \ --cc=haojian.zhuang@gmail.com \ --cc=jason@lakedaemon.net \ --cc=kishon@ti.com \ --cc=linus.walleij@linaro.org \ --cc=linux-gpio@vger.kernel.org \ --cc=linux-i2c@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-rtc@vger.kernel.org \ --cc=linux-serial@vger.kernel.org \ --cc=linux-spi@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=lkundrak@v3.sk \ --cc=maz@kernel.org \ --cc=mchehab@kernel.org \ --cc=robert.jarzmik@free.fr \ --cc=tglx@linutronix.de \ --cc=ulf.hansson@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.