Re: [PATCH v6 04/17] pinctrl: airoha: an7583: there is no mux to enable the second i2c bus

From: Krzysztof Kozlowski

Date: Mon Jun 29 2026 - 02:43:36 EST


On Sun, Jun 28, 2026 at 05:37:20PM +0300, Mikhail Kshevetskiy wrote:
> In the AN7583 case there is no mux bit to enable the second i2c bus,
> so remove it.

But you do not remove second i2c bus, but the i2c bus.

> Also remove i2c pin function from device tree binding schema.

Why?

>
> Fixes: 3ffeb17a9a27 ("pinctrl: airoha: add support for Airoha AN7583 PINs")
> Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevetskiy@xxxxxxxxx>
> ---
> .../bindings/pinctrl/airoha,an7583-pinctrl.yaml | 10 +---------
> drivers/pinctrl/airoha/pinctrl-airoha.c | 1 -
> 2 files changed, 1 insertion(+), 10 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/airoha,an7583-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/airoha,an7583-pinctrl.yaml
> index f52802a3ad79..8d8ec6adb97d 100644
> --- a/Documentation/devicetree/bindings/pinctrl/airoha,an7583-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/airoha,an7583-pinctrl.yaml
> @@ -60,7 +60,7 @@ patternProperties:
> function:
> description:
> A string containing the name of the function to mux to the group.
> - enum: [pon, tod_1pps, sipo, mdio, uart, i2c, jtag, pcm, spi,
> + enum: [pon, tod_1pps, sipo, mdio, uart, jtag, pcm, spi,
> pcm_spi, emmc, pnand, pcie_reset, pwm, phy1_led0,
> phy2_led0, phy3_led0, phy4_led0, phy1_led1, phy2_led1,
> phy3_led1, phy4_led1]
> @@ -117,14 +117,6 @@ patternProperties:
> enum: [uart2, uart2_cts_rts, hsuart, hsuart_cts_rts,
> uart4, uart5]

Best regards,
Krzysztof