| Message ID | 20240518-dt-bindings-ti-soc-mfd-v1-5-b3952f104c9a@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ti: fix dtbs_check for some syscon bindings and DTS node | expand |
On 18/05/2024 23:07, Krzysztof Kozlowski wrote: > SoCs (and associated MCUs) should use dedicated compatibles for their > nodes, not re-use one coming from an another SoC. Using an incorrect > compatible does not allow to properly match/validate children of the > MCU device and causes dtbs_check warnings: > > k3-am62a7-sk.dtb: syscon@43000000: 'syscon@4008', 'syscon@4018' do not match any of the regexes: > '^chipid@[0-9a-f]+$', '^clock-controller@[0-9a-f]+$', '^mux-controller@[0-9a-f]+$', 'phy@[0-9a-f]+$', 'pinctrl-[0-9]+' > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Roger Quadros <rogerq@kernel.org>
diff --git a/arch/arm64/boot/dts/ti/k3-am62a-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-wakeup.dtsi index 98043e9aa316..d12c8c194a46 100644 --- a/arch/arm64/boot/dts/ti/k3-am62a-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-am62a-wakeup.dtsi @@ -7,7 +7,7 @@ &cbass_wakeup { wkup_conf: syscon@43000000 { - compatible = "ti,j721e-system-controller", "syscon", "simple-mfd"; + compatible = "ti,am62a-system-controller", "syscon", "simple-mfd"; reg = <0x00 0x43000000 0x00 0x20000>; #address-cells = <1>; #size-cells = <1>;
SoCs (and associated MCUs) should use dedicated compatibles for their nodes, not re-use one coming from an another SoC. Using an incorrect compatible does not allow to properly match/validate children of the MCU device and causes dtbs_check warnings: k3-am62a7-sk.dtb: syscon@43000000: 'syscon@4008', 'syscon@4018' do not match any of the regexes: '^chipid@[0-9a-f]+$', '^clock-controller@[0-9a-f]+$', '^mux-controller@[0-9a-f]+$', 'phy@[0-9a-f]+$', 'pinctrl-[0-9]+' Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- arch/arm64/boot/dts/ti/k3-am62a-wakeup.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)