Re: [PATCH v4 1/4] dt-bindings: soc: Add bindings for Airoha SCU Serdes lines

From: Conor Dooley <conor@kernel.org>
To: Christian Marangi <ansuelsmth@gmail.com>
Cc: Vinod Koul <vkoul@kernel.org>,
	Kishon Vijay Abraham I <kishon@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Lorenzo Bianconi <lorenzo@kernel.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-phy@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 1/4] dt-bindings: soc: Add bindings for Airoha SCU Serdes lines
Date: Fri, 7 Nov 2025 18:14:31 +0000	[thread overview]
Message-ID: <20251107-crisping-doable-365d6b7d60a6@spud> (raw)
In-Reply-To: <20251107160251.2307088-2-ansuelsmth@gmail.com>

[-- Attachment #1.1: Type: text/plain, Size: 1469 bytes --]

On Fri, Nov 07, 2025 at 05:02:43PM +0100, Christian Marangi wrote:
> The Airoha AN7581 SoC can configure the SCU serdes lines for multiple
> purpose. For example the Serdes for the USB1 port can be both
> used for USB 3.0 operation or for Ethernet. Or the USB2 serdes can both
> used for USB 3.0 operation or for PCIe.
> 
> The PCIe Serdes can be both used for PCIe operation or for Ethernet.
> 
> Add bindings to permit correct reference of the different ports in DT,
> mostly to differentiate the different supported modes internally to the
> drivers.
> 
> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> ---
>  include/dt-bindings/soc/airoha,scu-ssr.h | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>  create mode 100644 include/dt-bindings/soc/airoha,scu-ssr.h
> 
> diff --git a/include/dt-bindings/soc/airoha,scu-ssr.h b/include/dt-bindings/soc/airoha,scu-ssr.h
> new file mode 100644
> index 000000000000..a14cef465dad
> --- /dev/null
> +++ b/include/dt-bindings/soc/airoha,scu-ssr.h
> @@ -0,0 +1,11 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +
> +#ifndef __DT_BINDINGS_AIROHA_SCU_SSR_H
> +#define __DT_BINDINGS_AIROHA_SCU_SSR_H
> +
> +#define AIROHA_SCU_SERDES_PCIE1		0
> +#define AIROHA_SCU_SERDES_PCIE2		1
> +#define AIROHA_SCU_SERDES_USB1		0
> +#define AIROHA_SCU_SERDES_USB2		1

I'm going to assume that 01 01 is correct here.
Acked-by: Conor Dooley <conor.dooley@microchip.com>

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 112 bytes --]

-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy