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

zl020895 <[email protected]>
Newsgroups org.kernel.vger.linux-gpio,dev.linux.lists.mfd,dev.linux.lists.soc,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-clk,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-serial
Message-ID <[email protected]>
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" <[email protected]> wrote:
>On 18/08/2026 12:31, Long Zhao via B4 Relay wrote:
>> From: Long Zhao <[email protected]>
>> 
>> Allow ambarella,cv75-sec-scratchpad as a syscon for the shared secure
>> scratchpad registers on CV75.
>> 
>> Signed-off-by: Long Zhao <[email protected]>
>
>You already received review at v2 which you did not respond to and did
>not implement.
>
>Best regards,
>Krzysztof
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.