On 06/07/2022 12:14, Christian Marangi wrote: > On Wed, Jul 06, 2022 at 10:39:16AM +0200, Krzysztof Kozlowski wrote: >> On 05/07/2022 15:39, Christian Marangi wrote: >>> Add missing smem node for ipq8064. >>> >>> Signed-off-by: Christian Marangi <ansuelsmth@xxxxxxxxx> >>> Tested-by: Jonathan McDowell <noodles@xxxxxxxx> >>> --- >>> arch/arm/boot/dts/qcom-ipq8064.dtsi | 18 ++++++++++++++++++ >>> 1 file changed, 18 insertions(+) >>> >>> diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi >>> index b5aede3d7ccf..98527a7d885e 100644 >>> --- a/arch/arm/boot/dts/qcom-ipq8064.dtsi >>> +++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi >>> @@ -908,6 +908,11 @@ lcc: clock-controller@28000000 { >>> #reset-cells = <1>; >>> }; >>> >>> + sfpb_mutex_block: syscon@1200600 { >>> + compatible = "syscon"; >> >> syscon alone is not allowed. >> > > Mh... This is problematic. How this should be handled? As usual - add specific compatible. Maybe I was not specific enough, but my commit was under compatible, not under device node, so it seems obvious. Just in case it is not obvious - you cannot have "syscon" compatible alone. Best regards, Krzysztof