[net-next,v2,0/7] Marvell nvmem mac addresses support

Message

Miquel Raynal Nov. 24, 2022, 11:15 a.m. UTC

Hello,

Now that we are aligned on how to make information available from static
storage media to drivers like Ethernet controller drivers or switch
drivers by using nvmem cells and going through the whole nvmem
infrastructure, here are two driver updates to reflect these changes.

Prior to the driver updates, I propose:
* Reverting binding changes which should have never been accepted like
  that.
* A conversion of the (old) Prestera and DFX server bindings (optional,
  can be dropped if not considered necessary).
* A better description of the more recent Prestera PCI switch.

Please mind that this series cannot break anything since retrieving the
MAC address Prestera driver has never worked upstream, because the (ONIE
tlv) driver supposed to export the MAC address has not been accepted in
its original form and has been updated to the nvmem-layout
infrastructure (bindings have been merged, the code remains to be
applied).

Thanks,
Miquèl

Changes in v2:
* Bindings:
  - Add Rob's tags.
  - Use "additionalProperties: \n type: object" in both yaml files.
  - Add missing PCI properties to the prestera example as suggested.
  - Added a limitation over the number of register entries in the
    dfx-server description.
* Drivers:
  - Export the of_ helper retrieving MAC addresses from nvmem cells.
  - Add a comment in mvpp2 to say that the nvmem lookup only is valid on
    OF enabled platforms as suggested by Marcin.

Miquel Raynal (7):
  Revert "dt-bindings: marvell,prestera: Add description for device-tree
    bindings"
  dt-bindings: net: marvell,dfx-server: Convert to yaml
  dt-bindings: net: marvell,prestera: Convert to yaml
  dt-bindings: net: marvell,prestera: Describe PCI devices of the
    prestera family
  of: net: export of_get_mac_address_nvmem()
  net: marvell: prestera: Avoid unnecessary DT lookups
  net: mvpp2: Consider NVMEM cells as possible MAC address source

 .../bindings/net/marvell,dfx-server.yaml      | 62 +++++++++++++
 .../bindings/net/marvell,prestera.txt         | 81 -----------------
 .../bindings/net/marvell,prestera.yaml        | 91 +++++++++++++++++++
 .../net/ethernet/marvell/mvpp2/mvpp2_main.c   |  7 ++
 .../ethernet/marvell/prestera/prestera_main.c | 15 +--
 include/linux/of_net.h                        |  6 ++
 net/core/of_net.c                             |  5 +-
 7 files changed, 173 insertions(+), 94 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/net/marvell,dfx-server.yaml
 delete mode 100644 Documentation/devicetree/bindings/net/marvell,prestera.txt
 create mode 100644 Documentation/devicetree/bindings/net/marvell,prestera.yaml

Comments

[e-mail] Nov. 29, 2022, 10:10 a.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (master)
by Paolo Abeni <[e-mail]>:

On Thu, 24 Nov 2022 12:15:49 +0100 you wrote:
> Hello,
> 
> Now that we are aligned on how to make information available from static
> storage media to drivers like Ethernet controller drivers or switch
> drivers by using nvmem cells and going through the whole nvmem
> infrastructure, here are two driver updates to reflect these changes.
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/7] Revert "dt-bindings: marvell,prestera: Add description for device-tree bindings"
    https://git.kernel.org/netdev/net-next/c/98eb05dc99fd
  - [net-next,v2,2/7] dt-bindings: net: marvell,dfx-server: Convert to yaml
    https://git.kernel.org/netdev/net-next/c/63b956f99175
  - [net-next,v2,3/7] dt-bindings: net: marvell,prestera: Convert to yaml
    https://git.kernel.org/netdev/net-next/c/a429ab01163c
  - [net-next,v2,4/7] dt-bindings: net: marvell,prestera: Describe PCI devices of the prestera family
    https://git.kernel.org/netdev/net-next/c/39d103862015
  - [net-next,v2,5/7] of: net: export of_get_mac_address_nvmem()
    https://git.kernel.org/netdev/net-next/c/4c47867bc789
  - [net-next,v2,6/7] net: marvell: prestera: Avoid unnecessary DT lookups
    https://git.kernel.org/netdev/net-next/c/a48acad789ff
  - [net-next,v2,7/7] net: mvpp2: Consider NVMEM cells as possible MAC address source
    https://git.kernel.org/netdev/net-next/c/7a74c1265ab4

You are awesome, thank you!