Hi, Dependency ========== Depends on https://lore.kernel.org/r/20240510123018.3902184-1-robh@xxxxxxxxxx because it touches the same lines. Merging ======= 1. Bindings: preferrably everything via MFD tree (file/context dependencies). 2. DTS: via subarch maintainers (no dependencies). Description/problem =================== Simple syscon nodes can be documented in common syscon.yaml, however devices with simple-mfd compatible, thus some children, should have their own schema listing these children. Such listing makes the binding specific, allows better validation (so the incorrect child would not appear in the simple-mfd node) and actually enforces repeated rule for simple-mfd devices: "simple-mfd" is only for simple devices, where the children do not depend on the parent. Currently the syscon+simple-mfd binding is quite broad and allows any child or property, thus above rule cannot be enforced. Solution ======== 1. Split the syscon.yaml binding into common syscon properties, used potentially by many bindings, and only simple syscon devices (NO simple-mfd!). 2. Move some known simple-mfd bindings from syscon.yaml to dedicated files. This patchset might introduce new dtbs_check warnings for devices having simple-mfd and being part of syscon.yaml previously. I fixed some of them, but probably not all. Best regards, Krzysztof To: Lee Jones <lee@xxxxxxxxxx> To: Rob Herring <robh@xxxxxxxxxx> To: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx> To: Conor Dooley <conor+dt@xxxxxxxxxx> To: Lars Povlsen <lars.povlsen@xxxxxxxxxxxxx> To: Steen Hegelund <Steen.Hegelund@xxxxxxxxxxxxx> To: Daniel Machon <daniel.machon@xxxxxxxxxxxxx> To: UNGLinuxDriver@xxxxxxxxxxxxx To: Nishanth Menon <nm@xxxxxx> To: Matthias Brugger <matthias.bgg@xxxxxxxxx> To: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx> Cc: devicetree@xxxxxxxxxxxxxxx Cc: linux-kernel@xxxxxxxxxxxxxxx Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx Cc: linux-mediatek@xxxxxxxxxxxxxxxxxxx --- Krzysztof Kozlowski (8): dt-bindings: mfd: syscon: Drop hwlocks dt-bindings: soc: sprd: sc9863a-glbregs: Document SC9863A syscon dt-bindings: soc: intel: lgm-syscon: Move to dedicated schema dt-bindings: soc: microchip: sparx5-cpu-syscon: Move to dedicated schema dt-bindings: soc: ti: am654-serdes-ctrl: Move to dedicated schema dt-bindings: mfd: syscon: Split and enforce documenting MFD children dt-bindings: mfd: syscon: Add APM poweroff mailbox arm64: dts: apm: Add dedicated syscon poweroff compatibles .../devicetree/bindings/mfd/syscon-common.yaml | 72 +++++++ Documentation/devicetree/bindings/mfd/syscon.yaml | 222 +++++++++------------ .../bindings/soc/intel/intel,lgm-syscon.yaml | 53 +++++ .../soc/microchip/microchip,sparx5-cpu-syscon.yaml | 49 +++++ .../bindings/soc/sprd/sprd,sc9863a-glbregs.yaml | 55 +++++ .../bindings/soc/ti/ti,am654-serdes-ctrl.yaml | 42 ++++ arch/arm64/boot/dts/apm/apm-merlin.dts | 2 +- arch/arm64/boot/dts/apm/apm-mustang.dts | 2 +- 8 files changed, 365 insertions(+), 132 deletions(-) --- base-commit: ca42fd4fd365cbaf798f17f655aee3d8192829f3 change-id: 20240517-dt-bindings-mfd-syscon-split-37e23996523d Best regards, -- Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
