Re: [PATCH v4 0/5] support ipq5332 platform

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


Hi Luo,

I have a few questions regarding the high level design of implementation. I hope that clarifying these topics will help us to find a good model for the case and finally merge a supporting code. Please find the questions below.

On 25.12.2023 10:44, Luo Jie wrote:
For IPQ5332 platform, there are two MAC PCSs, and qca8084 is
connected with one of them.

1. The Ethernet LDO needs to be enabled to make the PHY GPIO
    reset taking effect, which uses the MDIO bus level reset.

2. The SoC GCC uniphy AHB and SYS clocks need to be enabled
    to make the ethernet PHY device accessible.

3. To provide the clock to the ethernet, the CMN clock needs
    to be initialized for selecting reference clock and enabling
    the output clock.

4. Support optional MDIO clock frequency config.

5. Update dt-bindings doc for the new added properties.

Changes in v2:
	* remove the PHY related features such as PHY address
	  program and clock initialization.
	* leverage the MDIO level GPIO reset for qca8084 PHY.

Changes in v3:
	* fix the christmas-tree format issue.
	* improve the dt-binding changes.

Changes in v4:
	* improve the CMN PLL reference clock config.
	* improve the dt-binding changes.

Luo Jie (5):
   net: mdio: ipq4019: move eth_ldo_rdy before MDIO bus register
   net: mdio: ipq4019: enable the SoC uniphy clocks for ipq5332 platform
   net: mdio: ipq4019: configure CMN PLL clock for ipq5332
   net: mdio: ipq4019: support MDIO clock frequency divider
   dt-bindings: net: ipq4019-mdio: Document ipq5332 platform

  .../bindings/net/qcom,ipq4019-mdio.yaml       | 141 ++++++++-
  drivers/net/mdio/mdio-ipq4019.c               | 288 ++++++++++++++++--
  2 files changed, 399 insertions(+), 30 deletions(-)

I'm asking these questions because after checking the patches and following the earlier discussion, the series is looks like an overloading of the MDIO driver with somehow but not strictly related functionality.

First, let me summarize the case. Feel free to correct me if I took something wrong. So, we have:
- a reference design contains IPQ5332 SoC + QCA8084 switch/Phy;
- QCA8084 requires a reference clock for normal functionality;
- IPQ5332, as a chip, is able to provide a set of reference clocks for external devices;
- you want to configure IPQ5332 to provide the reference clock for QCA8084.

So, the high level questions are:
1. Is QCA8084 capable to consume the clock from some other generator? Is it possible to clock QCA8084 from external XO/PLL/whatever?
2. Is IPQ5332 capable to provide reference clock to another switch model?
3. Is the reference clock generation subsystem part of the MDIO block of IPQ5332?

And there are some tiny questions to make sure that we are on the same page:
a. What is the mentioned Ethernet LDO? AFAIK, LDO is some kind of gate (or switch) that enables clock output through an IPQ5332 pin. Isn't it? And if it's true, then can you clarify, what exactly clock is outputted? b. Is the Ethernet LDO part of the MDIO block of IPQ5332? According to iomem addresses that was used in the example reg property, the Ethernet LDO is not part of MDIO. c. Is the CMN PLL part of the MDIO block of IPQ5332? Again, according to iomem address, the CMN PLL is not part of MDIO. d. Are GCC AHB & SYS clocks really consumed by MDIO itself? Or are they need for the external reference clock generation?

Please answer these questions one by one and we will have a good basis to move forward.

To speed up the discussion, let me share my user's view of the reference clocks modeling. I would like to join the option that has already been suggested by the maintainers. It is better to implement reference clocks handling using the clocks API, and the clock subsystem will take care of enabling and configuring them.

And considering the expected answers to the above questions, I would like to suggest to implement the clock handling using a dedicated clock controlling driver. Or even using several of such tiny dedicated drivers. So DTS will become like this:

  ext_ref_clock: ext_ref_clock {
    compatible = "fixed-clock";
    clock-frequency = <48000000>;

  eth_cmn_pll: clock-controller@9b000 {
    compatible = "qcom,eth-cmn-pll-ipq5223";
    reg = <0x9b000 0x800>;
    clocks = <&ext_ref_clock>; /* use external 48MHz clock */

  phy0_ext_clk: clock-controller@7a00610 {
    compatible = "qcom,ipq-eth-ldo";
    reg = <0x7a00610 0x4>;
    clocks = <&eth_cmn_pll>;

  mdio@90000 {
    compatible = "qcom,ipq4019-mdio";
    reg = <0x90000 0x64>;
    clocks = <&gcc GCC_MDIO_AHB_CLK>;

    ethernet-phy@1 {
      compatible = "...";
      reg = <1>;
      clocks = <&phy0_ext_clk>;
      reset-gpios = <&gcc ...>;


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux