| Message ID | 20230127093217.60818-3-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | dt-bindings: serial/mtd/mc/ata: use MC peripheral props | expand |
On Fri, Jan 27, 2023 at 10:32 AM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > Several devices can be attached to memory controllers (or memory-mapped > buses), thus they can come with additional controller-specific > properties, e.g. devices wired under Intel IXP4XX bus: cfi-flash, > intel,ixp4xx-compact-flash, NS8250 serial and MAX6369 watchdog. > > Referencing Memory Controller or IXP4XX bus peripheral properties fixes > few dtbs_check warnings like: > > intel-ixp42x-gateworks-gw2348.dtb: ide@1,0: Unevaluated properties are not allowed > ('intel,ixp4xx-eb-ahb-split-transfers', 'intel,ixp4xx-eb-byte-access', ... ' were unexpected) > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Acked-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij
On 1/27/23 18:32, Krzysztof Kozlowski wrote: > Several devices can be attached to memory controllers (or memory-mapped > buses), thus they can come with additional controller-specific > properties, e.g. devices wired under Intel IXP4XX bus: cfi-flash, > intel,ixp4xx-compact-flash, NS8250 serial and MAX6369 watchdog. > > Referencing Memory Controller or IXP4XX bus peripheral properties fixes > few dtbs_check warnings like: > > intel-ixp42x-gateworks-gw2348.dtb: ide@1,0: Unevaluated properties are not allowed > ('intel,ixp4xx-eb-ahb-split-transfers', 'intel,ixp4xx-eb-byte-access', ... ' were unexpected) > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Acked-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Hi Krzysztof, krzysztof.kozlowski@linaro.org wrote on Fri, 27 Jan 2023 10:32:16 +0100: > Several devices can be attached to memory controllers (or memory-mapped > buses), thus they can come with additional controller-specific > properties, e.g. devices wired under Intel IXP4XX bus: cfi-flash, > intel,ixp4xx-compact-flash, NS8250 serial and MAX6369 watchdog. > > Referencing Memory Controller or IXP4XX bus peripheral properties fixes > few dtbs_check warnings like: > > intel-ixp42x-gateworks-gw2348.dtb: ide@1,0: Unevaluated properties are not allowed > ('intel,ixp4xx-eb-ahb-split-transfers', 'intel,ixp4xx-eb-byte-access', ... ' were unexpected) > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com> Thanks, Miquèl
diff --git a/Documentation/devicetree/bindings/ata/intel,ixp4xx-compact-flash.yaml b/Documentation/devicetree/bindings/ata/intel,ixp4xx-compact-flash.yaml index 52e18600ecff..378692010c56 100644 --- a/Documentation/devicetree/bindings/ata/intel,ixp4xx-compact-flash.yaml +++ b/Documentation/devicetree/bindings/ata/intel,ixp4xx-compact-flash.yaml @@ -35,6 +35,7 @@ required: allOf: - $ref: pata-common.yaml# + - $ref: /schemas/memory-controllers/intel,ixp4xx-expansion-peripheral-props.yaml# unevaluatedProperties: false diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml index 5df94953c34e..44cd4476d1d3 100644 --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml @@ -15,6 +15,7 @@ description: | allOf: - $ref: "mtd.yaml#" + - $ref: /schemas/memory-controllers/mc-peripheral-props.yaml# properties: compatible: diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml index 2d0cac642427..4aeecd0abb68 100644 --- a/Documentation/devicetree/bindings/serial/8250.yaml +++ b/Documentation/devicetree/bindings/serial/8250.yaml @@ -11,6 +11,7 @@ maintainers: allOf: - $ref: serial.yaml# + - $ref: /schemas/memory-controllers/mc-peripheral-props.yaml# - if: anyOf: - required: diff --git a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml index ab9641e845db..38079e1b6a44 100644 --- a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml +++ b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml @@ -8,6 +8,7 @@ title: Maxim 63xx Watchdog Timers allOf: - $ref: "watchdog.yaml#" + - $ref: /schemas/memory-controllers/mc-peripheral-props.yaml# maintainers: - Marc Zyngier <maz@kernel.org>
Several devices can be attached to memory controllers (or memory-mapped buses), thus they can come with additional controller-specific properties, e.g. devices wired under Intel IXP4XX bus: cfi-flash, intel,ixp4xx-compact-flash, NS8250 serial and MAX6369 watchdog. Referencing Memory Controller or IXP4XX bus peripheral properties fixes few dtbs_check warnings like: intel-ixp42x-gateworks-gw2348.dtb: ide@1,0: Unevaluated properties are not allowed ('intel,ixp4xx-eb-ahb-split-transfers', 'intel,ixp4xx-eb-byte-access', ... ' were unexpected) Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/ata/intel,ixp4xx-compact-flash.yaml | 1 + Documentation/devicetree/bindings/mtd/mtd-physmap.yaml | 1 + Documentation/devicetree/bindings/serial/8250.yaml | 1 + Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml | 1 + 4 files changed, 4 insertions(+)