Hi Vinod, On Thu, Nov 19, 2020 at 4:43 PM Sergio Paracuellos <sergio.paracuellos@xxxxxxxxx> wrote: > > This series adds support for the PCIe PHY found in the Mediatek > MT7621 SoC. > > There is also a 'mt7621-pci' driver which is the controller part > which is still in staging and is a client of this phy. > > Both drivers have been tested together in a gnubee1 board. > > This series are rebased on the top of linux-next: > commit 4e78c578cb98 ("Add linux-next specific files for 20201030") > > Changes in v5: > - PATCH 1/4: Recollect Rob's Reviewed-by of bindings. > - PATCH 4/4: Recollect Greg's Acked-by for removing stuff from > staging area. > - Make Vinod's review comments changes in [0]: > * Use FIELD_GET and FIELD_PREP apis and avoid multiple *_VAL and > *_SHIFT custom definitions. > * Remove phy-read and phy-write internal functions and directly > call regmap_read and regmap_write in 'mt7621_phy_rmw'. > * Change some traces from info to debug log level. > * Note that I have maintained 'mt7621_phy_rmw' instead of use > 'regmap_update_bits'. This is because in order to get a reliable > boot registers must be written event the contained value in > that register is the same. I have preferred doing in this way > instead of using 'regmap_update_bits_base' passing 'false' for > async and 'true' for the force write. If this way of using > 'regmap_update_bits_base' is preferred just let me know. I notice we already have 'regmap_write_bits' function. I will use this and avoid mt7621_phy_rmw and send v6 of this series. Also, do you have any preference of where you want this series to be rebased on? Best regards, Sergio Paracuellos > > Changes in v4: > - Bindings moved from txt to yaml so previous Rob's Reviewed-by > is not in the new patch with the yaml file. > - 'phy-cells' property means now if phy is dual-ported. > - Avoid custom 'xlate' function and properly set registers > when the phy is dual ported. > - Add use of 'builtin_platform_driver'. > - Added a patch including myself as maintainer in the > MAINTAINERS file. > - Add a patch removing patch from staging to make easier > the complete inclusion and avoid possible problems might > appear in 'linux-next' if the series are included. > > Changes in v3: > - Recollect Rob's Reviewed-by of bindings. > - Make Kishon Vijay suggested changes in v2: > (See https://lkml.org/lkml/2019/4/17/53) > - Kconfig: > * Add depends on COMPILE_TEST > * Select REGMAP_MMIO > - Make use of 'soc_device_attribute' and 'soc_device_match' > - Use regmap mmio API instead of directly 'readl' and 'writel'. > - Use 'platform_get_resource' instead of 'of_address_to_resource'. > > Changes in v2: > - Reorder patches to get bindings first in the series. > - Don't use child nodes in the device tree. Use #phy-cells=1 instead. > - Update driver code with new 'xlate' function for the new device tree. > - Minor changes in driver's macros changing some spaces to tabs. > > Thanks in advance for your time. > > Best regards, > Sergio Paracuellos > > [0]: http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2020-November/148864.html > > Sergio Paracuellos (4): > dt-bindings: phy: Add binding for Mediatek MT7621 PCIe PHY > phy: ralink: Add PHY driver for MT7621 PCIe PHY > MAINTAINERS: add MT7621 PHY PCI maintainer > staging: mt7621-pci-phy: remove driver from staging > > .../phy}/mediatek,mt7621-pci-phy.yaml | 0 > MAINTAINERS | 6 + > drivers/phy/ralink/Kconfig | 8 ++ > drivers/phy/ralink/Makefile | 1 + > .../ralink/phy-mt7621-pci.c} | 103 +++++++----------- > drivers/staging/Kconfig | 2 - > drivers/staging/Makefile | 1 - > drivers/staging/mt7621-pci-phy/Kconfig | 8 -- > drivers/staging/mt7621-pci-phy/Makefile | 2 - > drivers/staging/mt7621-pci-phy/TODO | 4 - > 10 files changed, 53 insertions(+), 82 deletions(-) > rename {drivers/staging/mt7621-pci-phy => Documentation/devicetree/bindings/phy}/mediatek,mt7621-pci-phy.yaml (100%) > rename drivers/{staging/mt7621-pci-phy/pci-mt7621-phy.c => phy/ralink/phy-mt7621-pci.c} (76%) > delete mode 100644 drivers/staging/mt7621-pci-phy/Kconfig > delete mode 100644 drivers/staging/mt7621-pci-phy/Makefile > delete mode 100644 drivers/staging/mt7621-pci-phy/TODO > > -- > 2.25.1 > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel