mbox series

[net-next,v3,0/3] net: phy: mscc-miim: add MDIO bus frequency support

Message ID 20220405120951.4044875-1-michael@walle.cc
Headers show
Series net: phy: mscc-miim: add MDIO bus frequency support | expand

Message

Michael Walle April 5, 2022, 12:09 p.m. UTC
Introduce MDIO bus frequency support. This way the board can have a
faster (or maybe slower) bus frequency than the hardware default.

changes since v2:
 - resend, no RFC anymore, because net-next is open again

changes since v1:
 - fail probe if clock-frequency is set, but not clock is given
 - rename clk_freq to bus_freq
 - add maxItems to interrupts property
 - put compatible and reg first in the example

Michael Walle (3):
  dt-bindings: net: convert mscc-miim to YAML format
  dt-bindings: net: mscc-miim: add clock and clock-frequency
  net: phy: mscc-miim: add support to set MDIO bus frequency

 .../devicetree/bindings/net/mscc,miim.yaml    | 61 +++++++++++++++++++
 .../devicetree/bindings/net/mscc-miim.txt     | 26 --------
 drivers/net/mdio/mdio-mscc-miim.c             | 58 +++++++++++++++++-
 3 files changed, 117 insertions(+), 28 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/net/mscc,miim.yaml
 delete mode 100644 Documentation/devicetree/bindings/net/mscc-miim.txt

Comments

Andrew Lunn April 5, 2022, 12:19 p.m. UTC | #1
On Tue, Apr 05, 2022 at 02:09:49PM +0200, Michael Walle wrote:
> Convert the mscc-miim device tree binding to the new YAML format.
> 
> The original binding don't mention if the interrupt property is optional
> or not. But on the SparX-5 SoC, for example, the interrupt property isn't
> used, thus in the new binding that property is optional. FWIW the driver
> doesn't use interrupts at all.
> 
> Signed-off-by: Michael Walle <michael@walle.cc>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
Andrew Lunn April 5, 2022, 12:20 p.m. UTC | #2
On Tue, Apr 05, 2022 at 02:09:50PM +0200, Michael Walle wrote:
> Add the (optional) clock input of the MDIO controller and indicate that
> the common clock-frequency property is supported. The driver can use it
> to set the desired MDIO bus frequency.
> 
> Signed-off-by: Michael Walle <michael@walle.cc>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
patchwork-bot+netdevbpf@kernel.org April 6, 2022, 1:30 p.m. UTC | #3
Hello:

This series was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Tue,  5 Apr 2022 14:09:48 +0200 you wrote:
> Introduce MDIO bus frequency support. This way the board can have a
> faster (or maybe slower) bus frequency than the hardware default.
> 
> changes since v2:
>  - resend, no RFC anymore, because net-next is open again
> 
> changes since v1:
>  - fail probe if clock-frequency is set, but not clock is given
>  - rename clk_freq to bus_freq
>  - add maxItems to interrupts property
>  - put compatible and reg first in the example
> 
> [...]

Here is the summary with links:
  - [net-next,v3,1/3] dt-bindings: net: convert mscc-miim to YAML format
    https://git.kernel.org/netdev/net-next/c/ed941f65da81
  - [net-next,v3,2/3] dt-bindings: net: mscc-miim: add clock and clock-frequency
    https://git.kernel.org/netdev/net-next/c/b0385d4c1fff
  - [net-next,v3,3/3] net: phy: mscc-miim: add support to set MDIO bus frequency
    https://git.kernel.org/netdev/net-next/c/bb2a1934ca01

You are awesome, thank you!