[PATCH 0/3] staging: mt7621-pci: extract PCI PHY part into a new 'pci-mt7621-phy' driver
Sergio Paracuellos
sergio.paracuellos at gmail.com
Wed Jan 2 10:31:08 UTC 2019
On Mon, Dec 24, 2018 at 11:45 AM Sergio Paracuellos
<sergio.paracuellos at gmail.com> wrote:
>
> There is a lot of code related with PHY part of the mt7621 PCI driver
> that can be extracted to a new PHY generic driver to do the same function.
>
> This patch series add a new 'mt7621-pci-phy' driver to handle those
> and updates the mt7621-pci driver code to use generic kernel's PHY
> API. This makes the code much better and readable.
>
> Device tree has been also updated with new related bindinds for the
> PCI PHY part.
>
> There is one issue which makes the new driver not as clean as I would
> like to be which is the fact that pcie0_port and pcie1_port share the
> same address space for its registers and only differ en enable bit for
> the phy. Because of this, we cannot remap the register address space twice.
> The function 'probe' of the driver take this into consideration to not
> fail if port is '1' which is not a good way to go, but I cannot find similar
> sample or another way to go. It would be nice if someone could point me
> out in the correct way.
>
> This changes are only compile-tested.
>
> Hope this helps.
>
> Thanks in advance.
>
> Best regards,
> Sergio Paracuellos
>
> Sergio Paracuellos (3):
> staging: mt7621-pci-phy: add new driver for phy part of mt7621-pci
> staging: mt7621-pci: handle phy using new generic phy 'pci-mt7621-phy'
> driver
> staging: mt7621-dts: add pci-phy related bindings to board's device
> tree
>
> drivers/staging/Kconfig | 2 +
> drivers/staging/Makefile | 1 +
> drivers/staging/mt7621-dts/mt7621.dtsi | 40 ++
> drivers/staging/mt7621-pci-phy/Kconfig | 7 +
> drivers/staging/mt7621-pci-phy/Makefile | 1 +
> drivers/staging/mt7621-pci-phy/TODO | 4 +
> .../staging/mt7621-pci-phy/pci-mt7621-phy.c | 422 ++++++++++++++++++
> drivers/staging/mt7621-pci/pci-mt7621.c | 250 ++---------
> 8 files changed, 509 insertions(+), 218 deletions(-)
> create mode 100644 drivers/staging/mt7621-pci-phy/Kconfig
> create mode 100644 drivers/staging/mt7621-pci-phy/Makefile
> create mode 100644 drivers/staging/mt7621-pci-phy/TODO
> create mode 100644 drivers/staging/mt7621-pci-phy/pci-mt7621-phy.c
>
> --
> 2.19.1
>
V2 of this series has been sent already. Forget about this v1.
Thanks in advance.
Best regards,
Sergio Paracuellos
More information about the devel
mailing list