lkml.org 
[lkml]   [2023]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH net-next] dt-bindings: net: ti: k3-am654-cpsw-nuss: Document Serdes PHY
On Mon, Mar 06, 2023 at 03:17:50PM +0530, Siddharth Vadapalli wrote:
> Update bindings to include Serdes PHY as an optional PHY, in addition to
> the existing CPSW MAC's PHY. The CPSW MAC's PHY is required while the
> Serdes PHY is optional. The Serdes PHY handle has to be provided only
> when the Serdes is being configured in a Single-Link protocol. Using the
> name "serdes-phy" to represent the Serdes PHY handle, the am65-cpsw-nuss
> driver can obtain the Serdes PHY and request the Serdes to be
> configured.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
>
> Hello,
>
> This patch corresponds to the Serdes PHY bindings that were missed out in
> the series at:
> Link: https://lore.kernel.org/r/20230104103432.1126403-1-s-vadapalli@ti.com/
> This was pointed out at:
> https://lore.kernel.org/r/CAMuHMdW5atq-FuLEL3htuE3t2uO86anLL3zeY7n1RqqMP_rH1g@mail.gmail.com/
>
> .../bindings/net/ti,k3-am654-cpsw-nuss.yaml | 21 +++++++++++++++++--
> 1 file changed, 19 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml b/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml
> index 900063411a20..fab7df437dcc 100644
> --- a/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml
> +++ b/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml
> @@ -126,8 +126,25 @@ properties:
> description: CPSW port number
>
> phys:
> - maxItems: 1
> - description: phandle on phy-gmii-sel PHY
> + minItems: 1
> + maxItems: 2
> + description:
> + phandle(s) on CPSW MAC's PHY (Required) and the Serdes
> + PHY (Optional). phandle to the Serdes PHY is required
> + when the Serdes has to be configured in Single-Link
> + configuration.

Like this:

minItems: 1
items:
- description: CPSW MAC's PHY
- description: Serdes PHY. Serdes PHY is required
when the Serdes has to be configured in Single-Link

> +
> + phy-names:
> + oneOf:
> + - items:
> + - const: mac-phy
> + - const: serdes-phy
> + - items:
> + - const: mac-phy

Drop this and use minItems in 1st 'items' entry.

> + description:
> + Identifiers for the CPSW MAC's PHY and the Serdes PHY.
> + CPSW MAC's PHY is required and therefore "mac-phy" is
> + required, while "serdes-phy" is optional.

No need to state in plain text what the schema already says.

Rob

\
 
 \ /
  Last update: 2023-03-27 00:47    [W:0.078 / U:0.472 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site