Re:Re: [PATCH v5 02/12] dt-bindings: mfd: syscon: add Ambarella CV75 secure scratchpad

[zl020895]

Resend this due to wrong HTML format before.

Hi Krzysztof,

Sorry for resending without an on-list reply.

I treated the cover letter as enough: v2 asked to drop the separate
binding and put the compatible in syscon.yaml, which v3/v5 already
did. Sashiko then asked to also add it to a select enum. I thought
that was a false positive — current syscon.yaml has no such select;
matching is in syscon-common.yaml via the "syscon" fallback — so I
did not reply.

I am new to this: if I think a Sashiko comment is a false positive,
should I reply on the patch thread with the reason and wait, rather
than only mentioning it in the cover letter?

If I still missed something, I will fix it in v6.

Best regards,
Long Zhao

At 2026-08-19 03:27:07, "Krzysztof Kozlowski" <krzk@xxxxxxxxxx> wrote:
>On 18/08/2026 12:31, Long Zhao via B4 Relay wrote:
>> From: Long Zhao <longzhao@xxxxxxxxxxxxx>
>> 
>> Allow ambarella,cv75-sec-scratchpad as a syscon for the shared secure
>> scratchpad registers on CV75.
>> 
>> Signed-off-by: Long Zhao <longzhao@xxxxxxxxxxxxx>
>
>You already received review at v2 which you did not respond to and did
>not implement.
>
>Best regards,
>Krzysztof