[PATCH v6 06/17] pinctrl: airoha: an7583: fix muxing of non-gpio default pins
Krzysztof Kozlowski
krzk at kernel.org
Sun Jun 28 23:41:42 PDT 2026
On Sun, Jun 28, 2026 at 05:37:22PM +0300, Mikhail Kshevetskiy wrote:
> Current an7583 pinmux implementation have following issues:
> * pins 51 and 52 can't be set as pcie_reset, current pcie_reset code
> will sets pins to gpio mode instead.
> * there is no proper way to set pins 41--54 to gpio mode.
> * pins 41--53 can't be actually set as pwm pins. These pins must be
> muxed to gpio mode as well.
>
> This patch fixes above issues. Also device tree binding schema
> was updated.
Also was updated? "Also" is a sign you are mixing two commits into one.
Describe WHY you are doing things, not what you did. And describe in
details, becayse "binding was updated" brings no information.
>
> Fixes: 3ffeb17a9a27 ("pinctrl: airoha: add support for Airoha AN7583 PINs")
> Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevetskiy at iopsys.eu>
> ---
> .../pinctrl/airoha,an7583-pinctrl.yaml | 14 ++-
> drivers/pinctrl/airoha/pinctrl-airoha.c | 99 ++++++++++++++++---
Please run scripts/checkpatch.pl on the patches and fix reported
warnings. After that, run also 'scripts/checkpatch.pl --strict' on the
patches and (probably) fix more warnings. Some warnings can be ignored,
especially from --strict run, but the code here looks like it needs a
fix. Feel free to get in touch if the warning is not clear.
Best regards,
Krzysztof
More information about the Linux-mediatek
mailing list