| Message ID | 20230206092624.22922-1-krzysztof.kozlowski@linaro.org |
|---|---|
| Headers | show |
| Series | dt-bindings: serial/mtd/mc/ata: use MC peripheral props | expand |
On Mon, 06 Feb 2023 10:26:22 +0100, Krzysztof Kozlowski wrote: > The properties of devices in IXP4xx expansion bus need to be also > applied to actual devices' bindings. Prepare for this by splitting them > to separate intel,ixp4xx-expansion-peripheral-props binding, just like > other memory-controller peripheral properties. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Acked-by: Linus Walleij <linus.walleij@linaro.org> > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > --- > > Changes since v2: > 1. Add tags. > 2. Move intel,ixp4xx-expansion-bus-controller.yaml to memory-controllers (Rob). > 3. Drop | in intel,ixp4xx-expansion-peripheral-props.yaml (Rob). > --- > ...intel,ixp4xx-expansion-bus-controller.yaml | 66 +-------------- > ...tel,ixp4xx-expansion-peripheral-props.yaml | 80 +++++++++++++++++++ > .../mc-peripheral-props.yaml | 1 + > MAINTAINERS | 2 +- > 4 files changed, 84 insertions(+), 65 deletions(-) > rename Documentation/devicetree/bindings/{bus => memory-controllers}/intel,ixp4xx-expansion-bus-controller.yaml (57%) > create mode 100644 Documentation/devicetree/bindings/memory-controllers/intel,ixp4xx-expansion-peripheral-props.yaml > Applied, thanks!
On Mon, 06 Feb 2023 10:26:24 +0100, Krzysztof Kozlowski wrote: > The re-usable serial.yaml schema matches every property with ".*" > pattern, thus any other schema referencing it will not report unknown > (unevaluated) properties. This hides several wrong properties. It is > a limitation of dtschema, thus provide a simple workaround: expect > children to be only of few names matching upstream usage (Bluetooth, > GNSS, GPS and MCU). > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Acked-by: Linus Walleij <linus.walleij@linaro.org> > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > --- > Documentation/devicetree/bindings/serial/serial.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied, thanks!
Hi, Dependencies ============ I think entire patchset should go via one tree (Rob's). Patch #2 depends on patch #1. Patch #3 could go separate tree as long as others are in the linux-next. However for simplicity let's push everything through Rob's DT tree? Changelog ========= Changes since v2: 1. Add tags. 2. Move intel,ixp4xx-expansion-bus-controller.yaml to memory-controllers (Rob). 3. Drop | in intel,ixp4xx-expansion-peripheral-props.yaml (Rob). Changes since v1: 1. Add patches 1 and 2 to fix dt_binding_check and few dtbs_check warnings v1: https://lore.kernel.org/linux-devicetree/20230123151302.368277-11-krzysztof.kozlowski@linaro.org/ Best regards, Krzysztof Krzysztof Kozlowski (3): dt-bindings: intel,ixp4xx-expansion-bus: split out peripheral properties dt-bindings: reference MC peripheral properties in relevant devices dt-bindings: serial: restrict possible child node names .../ata/intel,ixp4xx-compact-flash.yaml | 1 + ...intel,ixp4xx-expansion-bus-controller.yaml | 66 +-------------- ...tel,ixp4xx-expansion-peripheral-props.yaml | 80 +++++++++++++++++++ .../mc-peripheral-props.yaml | 1 + .../devicetree/bindings/mtd/mtd-physmap.yaml | 1 + .../devicetree/bindings/serial/8250.yaml | 1 + .../devicetree/bindings/serial/serial.yaml | 2 +- .../bindings/watchdog/maxim,max63xx.yaml | 1 + MAINTAINERS | 2 +- 9 files changed, 89 insertions(+), 66 deletions(-) rename Documentation/devicetree/bindings/{bus => memory-controllers}/intel,ixp4xx-expansion-bus-controller.yaml (57%) create mode 100644 Documentation/devicetree/bindings/memory-controllers/intel,ixp4xx-expansion-peripheral-props.yaml