Re: [PATCH V1 1/4] dt-bindings: sram: qcom,imem: Add Boot Stat region within IMEM

From: Krzysztof Kozlowski
Date: Wed Mar 22 2023 - 12:27:54 EST


On 22/03/2023 14:34, Souradeep Chowdhury wrote:
>>
>>> + - const: qcom,imem-boot_stats
>>
>> No underscores in compatibles. Why this is not SoC specific compatible?
>
> Ack. The boot_stats module is not specific to a device. It is written to
> read some values from this imem region which is present for almost all
> QCOM SoCs. So SoC specific compatible is not given in this case.

Yeah, but the generic rule is that we always want SoC specific
compatibles. If this is not specific to a device, then you do not need
anything in DT and just instantiate it from some soc-driver...

Best regards,
Krzysztof