Re: [PATCH net-next v3 1/3] dt-bindings: net: fec: add pps channel property

From: Conor Dooley
Date: Fri Aug 09 2024 - 10:27:57 EST


On Fri, Aug 09, 2024 at 11:48:02AM +0200, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
>
> Add fsl,pps-channel property to specify to which timer instance the PPS
> channel is connected to.

In the driver patch you say "depending on the soc ... might be routed to
different timer instances", why is a soc-specific compatible
insufficient to determine which timer instance is in use?
I think I know what you mean, but I'm not 100%.

That said, the explanation in the driver patch is better than the one
here, so a commit message improvement is required.

Cheers,
Conor.

>
> Signed-off-by: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> ---
> v3: added net-next subject prefix
> v2: no changes
> ---
> Documentation/devicetree/bindings/net/fsl,fec.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/fsl,fec.yaml b/Documentation/devicetree/bindings/net/fsl,fec.yaml
> index 5536c06139ca..24e863fdbdab 100644
> --- a/Documentation/devicetree/bindings/net/fsl,fec.yaml
> +++ b/Documentation/devicetree/bindings/net/fsl,fec.yaml
> @@ -183,6 +183,13 @@ properties:
> description:
> Register bits of stop mode control, the format is <&gpr req_gpr req_bit>.
>
> + fsl,pps-channel:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + default: 0
> + description:
> + Specifies to which timer instance the PPS signal is routed.
> + enum: [0, 1, 2, 3]
> +
> mdio:
> $ref: mdio.yaml#
> unevaluatedProperties: false
> --
> 2.39.2
>

Attachment: signature.asc
Description: PGP signature