RE: Re: [PATCH v4 1/4] dt-bindings: pinctrl: add Ambarella CV75 pinctrl
Long Zhao <[email protected]>
| Newsgroups | org.kernel.vger.linux-gpio,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <DS0PR19MB7925EFC4D705CAD8DBD1C300DAA72@DS0PR19MB7925.namprd19.prod.outlook.com> |
Hi Krzysztof, Thanks for the review. Agreed — the redundant $ref will be dropped. I'll fix this in the upcoming unified v5 series. Best regards, Long Zhao Ambarella Shanghai Co., Ltd -----Original Message----- From: Krzysztof Kozlowski <[email protected]> Sent: Monday, August 17, 2026 14:44 To: Long Zhao <[email protected]> Cc: Linus Walleij <[email protected]>; Rob Herring <[email protected]>; Krzysztof Kozlowski <[email protected]>; Conor Dooley <[email protected]>; Bartosz Golaszewski <[email protected]>; [email protected]; [email protected]; [email protected]; [email protected] Subject: [EXT] Re: [PATCH v4 1/4] dt-bindings: pinctrl: add Ambarella CV75 pinctrl On Fri, Aug 14, 2026 at 02:25:41PM +0800, Long Zhao wrote: > + - vin_master_sync > + - vsync0 > + - vsync1 > + - vsync2 > + - vsync3 > + - wdt > + > + groups: > + $ref: /schemas/types.yaml#/definitions/string-array Why are you redefining existing property from pincfg bindings? Since this cannot be tested by automation, I will not provide final reviews. Best regards, Krzysztof ###################################################################### This EXTERNAL email has been scanned by Proofpoint Email Protect service. ********************************************************************** This email and attachments contain Ambarella Proprietary and/or Confidential Information and is intended solely for the use of the individual(s) to whom it is addressed. Any unauthorized review, use, disclosure, distribute, copy, or print is prohibited. If you are not an intended recipient, please contact the sender by reply email and destroy all copies of the original message. Thank you.