From: Christian Marangi <ansuelsmth@gmail.com>
To: "Russell King (Oracle)" <linux@armlinux.org.uk>
Cc: Andrew Lunn <andrew@lunn.ch>,
Heiner Kallweit <hkallweit1@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Daniel Golle <daniel@makrotopia.org>,
Qingfang Deng <dqfext@gmail.com>,
SkyLake Huang <SkyLake.Huang@mediatek.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
Randy Dunlap <rdunlap@infradead.org>,
Simon Horman <horms@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.or
Subject: Re: [net-next PATCH 2/2] net: phy: mediatek: add Airoha PHY ID to SoC driver
Date: Tue, 8 Apr 2025 19:17:43 +0200 [thread overview]
Message-ID: <67f55a3d.050a0220.f3f61.8550@mx.google.com> (raw)
In-Reply-To: <Z_VYz6InC1p4vwku@shell.armlinux.org.uk>
On Tue, Apr 08, 2025 at 06:11:43PM +0100, Russell King (Oracle) wrote:
> On Tue, Apr 08, 2025 at 05:53:14PM +0200, Christian Marangi wrote:
> > config MEDIATEK_GE_SOC_PHY
> > tristate "MediaTek SoC Ethernet PHYs"
> > - depends on (ARM64 && ARCH_MEDIATEK) || COMPILE_TEST
> > - depends on NVMEM_MTK_EFUSE || COMPILE_TEST
> > + depends on (ARM64 && (ARCH_MEDIATEK || ARCH_AIROHA)) || COMPILE_TEST
> > + depends on (ARCH_MEDIATEK && NVMEM_MTK_EFUSE) || ARCH_AIROHA || COMPILE_TEST
>
> So...
> COMPILE_TEST ARM64 ARCH_AIROHA ARCH_MEDIATEK NVMEM_MTK_EFUSE result
> N N x x x N
> N Y N N x N
> N Y N Y N N
> N Y N Y Y Y
> N Y Y x x Y
> Y x x x x Y
>
> Hence this simplifies to:
>
> depends on ARM64 || COMPILE_TEST
> depends on ARCH_AIROHA || (ARCH_MEDIATEK && NVMEM_MTK_EFUSE) || \
> COMPILE_TEST
>
Right, I will update in v2.
Hope Andrew is OK with keeping the review tag.
--
Ansuel
next prev parent reply other threads:[~2025-04-08 18:57 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-08 15:53 [net-next PATCH 1/2] net: phy: mediatek: permit to compile test GE SOC PHY driver Christian Marangi
2025-04-08 15:53 ` [net-next PATCH 2/2] net: phy: mediatek: add Airoha PHY ID to SoC driver Christian Marangi
2025-04-08 17:08 ` Andrew Lunn
2025-04-08 17:11 ` Russell King (Oracle)
2025-04-08 17:17 ` Christian Marangi [this message]
2025-04-08 15:58 ` [net-next PATCH 1/2] net: phy: mediatek: permit to compile test GE SOC PHY driver Daniel Golle
2025-04-08 17:04 ` Andrew Lunn
2025-04-08 17:57 ` Russell King (Oracle)
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=67f55a3d.050a0220.f3f61.8550@mx.google.com \
--to=ansuelsmth@gmail.com \
--cc=SkyLake.Huang@mediatek.com \
--cc=andrew@lunn.ch \
--cc=angelogioacchino.delregno@collabora.com \
--cc=arnd@arndb.de \
--cc=daniel@makrotopia.org \
--cc=davem@davemloft.net \
--cc=dqfext@gmail.com \
--cc=edumazet@google.com \
--cc=hkallweit1@gmail.com \
--cc=horms@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.or \
--cc=linux@armlinux.org.uk \
--cc=matthias.bgg@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=rdunlap@infradead.org \
/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.