Re:Re: [PATCH v5 02/12] dt-bindings: mfd: syscon: add Ambarella CV75 secure scratchpad
zl020895 <[email protected]>
| Newsgroups | gmane.linux.kernel.gpio,gmane.linux.ports.arm.kernel,gmane.linux.drivers.devicetree,gmane.linux.kernel.clk,gmane.linux.serial,gmane.linux.kernel |
|---|---|
| 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