From: Romain Gantois <romain.gantois@bootlin.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Romain Gantois <romain.gantois@bootlin.com>,
Rob Herring <robh@kernel.org>,
Luka Perkov <luka.perkov@sartura.hr>,
Konrad Dybcio <konrad.dybcio@somainline.org>,
Rob Herring <robh+dt@kernel.org>,
netdev@vger.kernel.org, Jakub Kicinski <kuba@kernel.org>,
Maxime Chevallier <maxime.chevallier@bootlin.com>,
Russell King <linux@armlinux.org.uk>,
Andy Gross <agross@kernel.org>,
davem@davemloft.net, thomas.petazzoni@bootlin.com,
Paolo Abeni <pabeni@redhat.com>,
devicetree@vger.kernel.org,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Florian Fainelli <f.fainelli@gmail.com>,
linux-kernel@vger.kernel.org, Eric Dumazet <edumazet@google.com>,
Bjorn Andersson <andersson@kernel.org>,
linux-arm-kernel@lists.infradead.org,
Robert Marko <robert.marko@sartura.hr>,
Vladimir Oltean <vladimir.oltean@nxp.com>,
Andrew Lunn <andrew@lunn.ch>,
Heiner Kallweit <hkallweit1@gmail.com>
Subject: Re: [PATCH net-next 1/5] net: dt-bindings: Introduce the Qualcomm IPQESS Ethernet switch
Date: Tue, 24 Oct 2023 14:13:17 +0200 (CEST) [thread overview]
Message-ID: <0389bcc9-b74e-5394-d15a-17914ec3c1a9@bootlin.com> (raw)
In-Reply-To: <7018bf8b-1f89-408e-8649-3788a28f3b1a@linaro.org>
On Tue, 24 Oct 2023, Krzysztof Kozlowski wrote:
> Rob's bot might be using not-yet-released dtschema from main branch,
> thus the error. However the error is true: you added a custom field
> without type. That's why I asked: where is it defined?
>
I didn't define it anywhere, that's an oversight on my part. The psgmii_ethphy
property is a handle to an MDIO device, which I thought was integrated to the
PSGMII bus in the IPQ4019. However, I just learned from Robert Marko that this
MDIO device corresponds to a SoC-facing PHY integrated in the external QCA807x
IP. Therefore, I'm not convinced that this MDIO device should be handled by
the ESS driver.
I'm going to have to consider refactoring the psgmii_ethphy handling out of
the IPQESS driver, which would make this device tree property unnecessary.
Best Regards,
Romain
next prev parent reply other threads:[~2023-10-24 12:13 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-23 15:50 [PATCH net-next 0/5] net: ipqess: introduce Qualcomm IPQESS driver Romain Gantois
2023-10-23 15:50 ` [PATCH net-next 1/5] net: dt-bindings: Introduce the Qualcomm IPQESS Ethernet switch Romain Gantois
2023-10-23 17:37 ` Krzysztof Kozlowski
2023-10-24 8:01 ` Romain Gantois
2023-10-23 17:40 ` Rob Herring
2023-10-24 9:54 ` Romain Gantois
2023-10-24 9:56 ` Krzysztof Kozlowski
2023-10-24 10:05 ` Romain Gantois
2023-10-24 10:54 ` Krzysztof Kozlowski
2023-10-24 12:13 ` Romain Gantois [this message]
2023-10-23 15:50 ` [PATCH net-next 2/5] net: dsa: qca: Make the QCA8K hardware library available globally Romain Gantois
2023-10-23 16:40 ` Andrew Lunn
2023-10-23 15:50 ` [PATCH net-next 3/5] net: ipqess: introduce the Qualcomm IPQESS driver Romain Gantois
2023-10-23 17:50 ` Andrew Lunn
2023-10-24 9:16 ` Romain Gantois
2023-10-24 14:08 ` Andrew Lunn
2023-10-24 16:23 ` Robert Marko
2023-10-24 16:48 ` Andrew Lunn
2023-10-24 11:44 ` Wojciech Drewek
2023-10-24 15:55 ` kernel test robot
2023-10-23 15:50 ` [PATCH net-next 4/5] net: ipqess: add a PSGMII calibration procedure to the " Romain Gantois
2023-10-23 18:01 ` Andrew Lunn
2023-10-24 9:03 ` Romain Gantois
2023-10-24 14:13 ` Andrew Lunn
2023-10-24 16:49 ` Robert Marko
2023-10-23 15:50 ` [PATCH net-next 5/5] dts: qcom: ipq4019: Add description for the IPQ4019 ESS EDMA and switch Romain Gantois
2023-10-23 17:39 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0389bcc9-b74e-5394-d15a-17914ec3c1a9@bootlin.com \
--to=romain.gantois@bootlin.com \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=andrew@lunn.ch \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--cc=konrad.dybcio@somainline.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=luka.perkov@sartura.hr \
--cc=maxime.chevallier@bootlin.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=robert.marko@sartura.hr \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=thomas.petazzoni@bootlin.com \
--cc=vladimir.oltean@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox