Re: [PATCH v3 5/9] dt-bindings: pinctrl: realtek: Add Realtek DHC SoC rtd1295
From: Andreas Färber <afaerber@suse.de>
Date: 2020-08-18 10:12:27
Also in:
linux-gpio, lkml
From: Andreas Färber <afaerber@suse.de>
Date: 2020-08-18 10:12:27
Also in:
linux-gpio, lkml
Am 17.08.20 um 22:33 schrieb Rob Herring:
On Thu, 13 Aug 2020 15:49:04 +0800, TY Chang wrote:quoted
Add device tree binding Documentation for rtd1295 pinctrl driver. Signed-off-by: TY Chang <redacted> --- .../pinctrl/realtek,rtd1295-pinctrl.yaml | 192 ++++++++++++++++++ 1 file changed, 192 insertions(+) create mode 100644 Documentation/devicetree/bindings/pinctrl/realtek,rtd1295-pinctrl.yamlPlease add Acked-by/Reviewed-by tags when posting new versions. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for acks received on the version they apply. If a tag was not added on purpose, please state why and what changed.
The thing really missing here is a per-patch change log. Things were added here that I'm sure you would not give your Reviewed-by for, in particular new properties prefixed with unregistered rtk prefix instead of the registered realtek prefix. @TY, hiding such changes in a big previously reviewed patch without any mention is problematic - please rather do smaller follow-up patches to not invalidate previous reviews with new features. Thanks, Andreas -- SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer HRB 36809 (AG Nürnberg)