Re: [PATCH] dt-bindings: crypto: qcom,inline-crypto-engine: Document Nord ICE

Krzysztof Kozlowski
2026-04-20T11:56:54Z

On 20/04/2026 10:39, Shawn Guo wrote:
> On Mon, Apr 20, 2026 at 10:27:56AM +0200, Krzysztof Kozlowski wrote:
>> On 20/04/2026 09:33, Shawn Guo wrote:
>>> Add compatible for Inline Crypto Engine (ICE) on Qualcomm Nord SoC
>>> witha fallback on qcom,inline-crypto-engine.
>>
>> Don't explain what the diff is doing. Explain why. Why do you use fallback?
>>
>> What is Nord? It's nowhere explained. First posting was 1.5 months ago
>> and it did not provide any explanation. I don't see any information
>> being posted in the series sent now.
>
> I'm still checking internally to see how we can get the best socinfo
> patch describing Nord which is a SoC family covering both SA8997P and
> IQ10 variant. Hopefully I will get it soon.

I found the DTS on:
https://github.com/qualcomm-linux/kernel-topics/commits/early/hwe/nord/
so it should be mentioned somewhere, which I kind of asked when we
discussed about adding compatibles used by that DTS. You would solve
yourself all my questions from three threads.

Best regards,
Krzysztof