Re: [PATCH v8 2/6] dt-bindings: pinctrl: mediatek,mt6779-pinctrl: Improve pinctrl subnode and property descriptions

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Commit message inaccuracy regarding capitalization of the pinmux description.
--

commit a7cfb3754d286ad6671b0ee2b11ae49fbb660dff
Author: Yassine Oudjana <y.oudjana@protonmail.com>

dt-bindings: pinctrl: mediatek,mt6779-pinctrl: Improve pinctrl subnode and property descriptions

This commit updates the Mediatek MT6779 pinctrl binding descriptions to fix
grammar, wrap lines at 80 characters, and remove unneeded literal style
indicators.

> Change "subnodes" to "subnode" in subnode description for better
> grammatical accuracy, capitalize pinmux description, wrap all descriptions
> at 80 characters, and remove literal style indicators from descriptions
> that don't need their new lines preserved.

[Severity: Low]
The commit message mentions it will "capitalize pinmux description". Is this
intended change missing from the patch? 

Looking at the diff for the pinmux description, the text was wrapped to 80
characters, but terms like "gpio" were left uncapitalized.

> diff --git a/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml
> --- a/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml

[ ... ]

>              description:
>                Integer array, represents gpio pin number and mux setting.
>                Supported pin number and mux varies for different SoCs, and are
> -              defined as macros in dt-bindings/pinctrl/<soc>-pinfunc.h directly.
> +              defined as macros in dt-bindings/pinctrl/<soc>-pinfunc.h
> +              directly.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260530145800.1029920-1-y.oudjana@protonmail.com?part=2