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