mbox series

[net-next:,v3,0/6] ACPI MDIO support for Marvell controllers

Message ID 20210621173028.3541424-1-mw@semihalf.com (mailing list archive)
Headers show
Series ACPI MDIO support for Marvell controllers | expand

Message

Marcin Wojtas June 21, 2021, 5:30 p.m. UTC
Hi,

The third version of the patchset main change is
dropping a clock handling optimisation patch
for mvmdio driver. Other than that it sets
explicit dependency on FWNODE_MDIO for CONFIG_FSL_XGMAC_MDIO
and applies minor cosmetic improvements (please see the
'Changelog' below).

The firmware ACPI description is exposed in the public github branch:
https://github.com/semihalf-wojtas-marcin/edk2-platforms/commits/acpi-mdio-r20210613
There is also MacchiatoBin firmware binary available for testing:
https://drive.google.com/file/d/1eigP_aeM4wYQpEaLAlQzs3IN_w1-kQr0

I'm looking forward to the comments or remarks.

Best regards,
Marcin

Changelog:
v2->v3
* Rebase on top of net-next/master.
* Drop "net: mvmdio: simplify clock handling" patch.
* 1/6 - fix code block comments.
* 2/6 - unchanged
* 3/6 - add "depends on FWNODE_MDIO" for CONFIG_FSL_XGMAC_MDIO
* 4/6 - drop mention about the clocks from the commit message.
* 5/6 - unchanged
* 6/6 - add Andrew's RB.

v1->v2
* 1/7 - new patch
* 2/7 - new patch
* 3/7 - new patch
* 4/7 - new patch
* 5/7 - remove unnecessary `if (has_acpi_companion())` and rebase onto
        the new clock handling
* 6/7 - remove deprecated comment
* 7/7 - no changes

Marcin Wojtas (6):
  Documentation: ACPI: DSD: describe additional MAC configuration
  net: mdiobus: Introduce fwnode_mdbiobus_register()
  net/fsl: switch to fwnode_mdiobus_register
  net: mvmdio: add ACPI support
  net: mvpp2: enable using phylink with ACPI
  net: mvpp2: remove unused 'has_phy' field

 drivers/net/ethernet/marvell/mvpp2/mvpp2.h      |  3 -
 include/linux/fwnode_mdio.h                     | 12 ++++
 drivers/net/ethernet/freescale/xgmac_mdio.c     | 11 +---
 drivers/net/ethernet/marvell/mvmdio.c           | 14 ++++-
 drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c | 23 ++++++--
 drivers/net/mdio/fwnode_mdio.c                  | 22 ++++++++
 Documentation/firmware-guide/acpi/dsd/phy.rst   | 59 ++++++++++++++++++++
 drivers/net/ethernet/freescale/Kconfig          |  4 +-
 8 files changed, 125 insertions(+), 23 deletions(-)

Comments

Marcin Wojtas June 23, 2021, 11:56 a.m. UTC | #1
Hi,

pon., 21 cze 2021 o 19:31 Marcin Wojtas <mw@semihalf.com> napisaƂ(a):
>
> Hi,
>
> The third version of the patchset main change is
> dropping a clock handling optimisation patch
> for mvmdio driver. Other than that it sets
> explicit dependency on FWNODE_MDIO for CONFIG_FSL_XGMAC_MDIO
> and applies minor cosmetic improvements (please see the
> 'Changelog' below).
>
> The firmware ACPI description is exposed in the public github branch:
> https://github.com/semihalf-wojtas-marcin/edk2-platforms/commits/acpi-mdio-r20210613
> There is also MacchiatoBin firmware binary available for testing:
> https://drive.google.com/file/d/1eigP_aeM4wYQpEaLAlQzs3IN_w1-kQr0
>
> I'm looking forward to the comments or remarks.
>

I would really appreciate if this third revision can possibly get
attention some in coming days. There's still some buffer for
improvements, so that in case of no objections, it can be queued for
v5.14 via net-next tree.

Thanks,
Marcin

Marcin

>
> Changelog:
> v2->v3
> * Rebase on top of net-next/master.
> * Drop "net: mvmdio: simplify clock handling" patch.
> * 1/6 - fix code block comments.
> * 2/6 - unchanged
> * 3/6 - add "depends on FWNODE_MDIO" for CONFIG_FSL_XGMAC_MDIO
> * 4/6 - drop mention about the clocks from the commit message.
> * 5/6 - unchanged
> * 6/6 - add Andrew's RB.
>
> v1->v2
> * 1/7 - new patch
> * 2/7 - new patch
> * 3/7 - new patch
> * 4/7 - new patch
> * 5/7 - remove unnecessary `if (has_acpi_companion())` and rebase onto
>         the new clock handling
> * 6/7 - remove deprecated comment
> * 7/7 - no changes
>
> Marcin Wojtas (6):
>   Documentation: ACPI: DSD: describe additional MAC configuration
>   net: mdiobus: Introduce fwnode_mdbiobus_register()
>   net/fsl: switch to fwnode_mdiobus_register
>   net: mvmdio: add ACPI support
>   net: mvpp2: enable using phylink with ACPI
>   net: mvpp2: remove unused 'has_phy' field
>
>  drivers/net/ethernet/marvell/mvpp2/mvpp2.h      |  3 -
>  include/linux/fwnode_mdio.h                     | 12 ++++
>  drivers/net/ethernet/freescale/xgmac_mdio.c     | 11 +---
>  drivers/net/ethernet/marvell/mvmdio.c           | 14 ++++-
>  drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c | 23 ++++++--
>  drivers/net/mdio/fwnode_mdio.c                  | 22 ++++++++
>  Documentation/firmware-guide/acpi/dsd/phy.rst   | 59 ++++++++++++++++++++
>  drivers/net/ethernet/freescale/Kconfig          |  4 +-
>  8 files changed, 125 insertions(+), 23 deletions(-)
>
> --
> 2.29.0
>