From: Christian Marangi <ansuelsmth@gmail.com>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: Lee Jones <lee@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Heiner Kallweit <hkallweit1@gmail.com>,
Russell King <linux@armlinux.org.uk>,
Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org, netdev@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
upstream@airoha.com
Subject: Re: [net-next PATCH v11 3/9] dt-bindings: net: dsa: Document support for Airoha AN8855 DSA Switch
Date: Tue, 10 Dec 2024 23:26:17 +0100 [thread overview]
Message-ID: <6758c00c.050a0220.1dd14c.63cd@mx.google.com> (raw)
In-Reply-To: <20241210221602.ohyzlic2x3pflmrg@skbuf>
On Wed, Dec 11, 2024 at 12:16:02AM +0200, Vladimir Oltean wrote:
> On Tue, Dec 10, 2024 at 09:59:04PM +0100, Christian Marangi wrote:
> > > > + airoha,ext-surge:
> > > > + $ref: /schemas/types.yaml#/definitions/flag
> > > > + description:
> > > > + Calibrate the internal PHY with the calibration values stored in EFUSE
> > > > + for the r50Ohm values.
> > >
> > > Doesn't seem that this pertains to the switch.
> >
> > Do you think this should be placed in each PHY node?
>
> Logically speaking, that's where it belongs.
>
> > I wanted to prevent having to define a schema also for PHY if possible
> > given how integrated these are. (originally it was defined in DT node
> > to follow how it was done in Airoha SDK)
>
> Does compatibility with the Airoha SDK dt-bindings matter in any way?
No it doesn't, the requirement for nvmem already deviates a lot so changes
are needed anyway.
--
Ansuel
next prev parent reply other threads:[~2024-12-10 22:27 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-09 13:44 [net-next PATCH v11 0/9] net: dsa: Add Airoha AN8855 support Christian Marangi
2024-12-09 13:44 ` [net-next PATCH v11 1/9] dt-bindings: nvmem: Document support for Airoha AN8855 Switch EFUSE Christian Marangi
2024-12-17 13:18 ` Rob Herring (Arm)
2024-12-09 13:44 ` [net-next PATCH v11 2/9] dt-bindings: net: Document support for Airoha AN8855 Switch Virtual MDIO Christian Marangi
2024-12-17 13:28 ` Rob Herring (Arm)
2024-12-09 13:44 ` [net-next PATCH v11 3/9] dt-bindings: net: dsa: Document support for Airoha AN8855 DSA Switch Christian Marangi
2024-12-10 20:48 ` Vladimir Oltean
2024-12-10 20:59 ` Christian Marangi
2024-12-10 22:16 ` Vladimir Oltean
2024-12-10 22:26 ` Christian Marangi [this message]
2024-12-09 13:44 ` [net-next PATCH v11 4/9] dt-bindings: mfd: Document support for Airoha AN8855 Switch SoC Christian Marangi
2024-12-10 20:55 ` Vladimir Oltean
2024-12-10 21:03 ` Christian Marangi
2024-12-09 13:44 ` [net-next PATCH v11 5/9] mfd: an8855: Add support for Airoha AN8855 Switch MFD Christian Marangi
2024-12-09 23:18 ` Jakub Kicinski
2024-12-09 23:22 ` Christian Marangi
2024-12-09 23:40 ` Jakub Kicinski
2024-12-09 23:48 ` Christian Marangi
2024-12-10 21:05 ` Vladimir Oltean
2024-12-10 21:15 ` Vladimir Oltean
2024-12-10 22:32 ` Christian Marangi
2024-12-10 23:11 ` Andrew Lunn
2024-12-10 23:48 ` Vladimir Oltean
2024-12-14 15:11 ` Christian Marangi
2024-12-17 15:13 ` Vladimir Oltean
2024-12-17 15:18 ` Vladimir Oltean
2024-12-09 13:44 ` [net-next PATCH v11 6/9] net: mdio: Add Airoha AN8855 Switch MDIO Passtrough Christian Marangi
2024-12-10 1:53 ` Andrew Lunn
2024-12-10 12:06 ` Christian Marangi
2024-12-10 13:38 ` Andrew Lunn
2024-12-09 13:44 ` [net-next PATCH v11 7/9] nvmem: an8855: Add support for Airoha AN8855 Switch EFUSE Christian Marangi
2024-12-09 13:44 ` [net-next PATCH v11 8/9] net: dsa: Add Airoha AN8855 5-Port Gigabit DSA Switch driver Christian Marangi
2024-12-10 22:12 ` Vladimir Oltean
2024-12-09 13:44 ` [net-next PATCH v11 9/9] net: phy: Add Airoha AN8855 Internal Switch Gigabit PHY Christian Marangi
2024-12-10 1:36 ` Andrew Lunn
2024-12-10 12:10 ` Christian Marangi
2024-12-10 13:39 ` Andrew Lunn
2024-12-10 20:20 ` [net-next PATCH v11 0/9] net: dsa: Add Airoha AN8855 support Vladimir Oltean
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=6758c00c.050a0220.1dd14c.63cd@mx.google.com \
--to=ansuelsmth@gmail.com \
--cc=andrew+netdev@lunn.ch \
--cc=angelogioacchino.delregno@collabora.com \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=hkallweit1@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=lee@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux@armlinux.org.uk \
--cc=matthias.bgg@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=olteanv@gmail.com \
--cc=pabeni@redhat.com \
--cc=robh@kernel.org \
--cc=srinivas.kandagatla@linaro.org \
--cc=upstream@airoha.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.