Re: [PATCH v2] net: enetc: fix the netc-lib driver build dependency

[Vladimir Oltean <vladimir.oltean@nxp.com>]

On Thu, Jun 05, 2025 at 02:05:47PM +0300, Wei Fang wrote:
> > I think you slightly misunderstood Arnd's suggestion. NXP_NTMP was named
> > "NXP_NETC_NTMP" in his proposal, and it meant "does FSL_ENETC_CORE need
> > the functionality from NXP_NETC_LIB?".
> > 
> > The switch driver shouldn't need to select NXP_NTMP. Just NXP_NETC_LIB.
> > 
> 
> For the case :NXP_ENET4=n , FSL_ENETC=y and NXP_NETC_SWITCH=m, if the
> switch driver only selects NXP_NETC_LIB, then the netc-lib driver will be compiled
> as a module. So the issue will be reported again. And Arnd also said "The switch
> module can then equally enable bool symbol."
> 
> > I don't agree with removing "NETC" from NXP_NETC_NTMP, I think it helps
> > clarify that the option pertains just to the NETC drivers.
> 
> NTMP means " NETC Table Management Protocol", I don't know why we need
> to add a 'NETC' again, it does not make sense to me.

Understood, thanks for the example. It was me who misunderstood. In that case

Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>