* Re: linux-next: Tree for Dec 3 (drivers/ptp/ptp_clockmatrix) [not found] <20191203155405.31404722@canb.auug.org.au> @ 2019-12-03 16:52 ` Randy Dunlap 2019-12-03 23:43 ` linux-next: Tree for Dec 3 (switchdev & TI_CPSW_SWITCHDEV) Randy Dunlap 1 sibling, 0 replies; 4+ messages in thread From: Randy Dunlap @ 2019-12-03 16:52 UTC (permalink / raw) To: Stephen Rothwell, Linux Next Mailing List Cc: Linux Kernel Mailing List, Richard Cochran, netdev On 12/2/19 8:54 PM, Stephen Rothwell wrote: > Hi all, > > Please do not add any material for v5.6 to your linux-next included > trees until after v5.5-rc1 has been released. > > Changes since 20191202: > on x86_64: CONFIG_I2C=m ld: drivers/ptp/ptp_clockmatrix.o: in function `idtcm_xfer': ptp_clockmatrix.c:(.text+0x135): undefined reference to `i2c_transfer' ld: drivers/ptp/ptp_clockmatrix.o: in function `idtcm_driver_init': ptp_clockmatrix.c:(.init.text+0x14): undefined reference to `i2c_register_driver' ld: drivers/ptp/ptp_clockmatrix.o: in function `idtcm_driver_exit': ptp_clockmatrix.c:(.exit.text+0xd): undefined reference to `i2c_del_driver' -- ~Randy Reported-by: Randy Dunlap <rdunlap@infradead.org> ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: linux-next: Tree for Dec 3 (switchdev & TI_CPSW_SWITCHDEV) [not found] <20191203155405.31404722@canb.auug.org.au> 2019-12-03 16:52 ` linux-next: Tree for Dec 3 (drivers/ptp/ptp_clockmatrix) Randy Dunlap @ 2019-12-03 23:43 ` Randy Dunlap 2019-12-04 15:35 ` Grygorii Strashko 1 sibling, 1 reply; 4+ messages in thread From: Randy Dunlap @ 2019-12-03 23:43 UTC (permalink / raw) To: Stephen Rothwell, Linux Next Mailing List Cc: Linux Kernel Mailing List, netdev, Jiri Pirko, Ivan Vecera, Grygorii Strashko, linux-omap On 12/2/19 8:54 PM, Stephen Rothwell wrote: > Hi all, > > Please do not add any material for v5.6 to your linux-next included > trees until after v5.5-rc1 has been released. > > Changes since 20191202: I am seeing this (happens to be on i386; I doubt that it matters): CONFIG_COMPILE_TEST=y WARNING: unmet direct dependencies detected for NET_SWITCHDEV Depends on [n]: NET [=y] && INET [=n] Selected by [y]: - TI_CPSW_SWITCHDEV [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_TI [=y] && (ARCH_DAVINCI || ARCH_OMAP2PLUS || COMPILE_TEST [=y]) because TI_CPSW_SWITCHDEV blindly selects NET_SWITCHDEV even though INET is not set/enabled, while NET_SWITCHDEV depends on INET. However, the build succeeds, including net/switchdev/*. So why does NET_SWITCHDEV depend on INET? It looks like TI_CPSW_SWITCHDEV should depend on INET (based on the Kconfig rules), but in practice it doesn't seem to matter to the build. thanks. -- ~Randy ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: linux-next: Tree for Dec 3 (switchdev & TI_CPSW_SWITCHDEV) 2019-12-03 23:43 ` linux-next: Tree for Dec 3 (switchdev & TI_CPSW_SWITCHDEV) Randy Dunlap @ 2019-12-04 15:35 ` Grygorii Strashko 2019-12-04 16:11 ` Randy Dunlap 0 siblings, 1 reply; 4+ messages in thread From: Grygorii Strashko @ 2019-12-04 15:35 UTC (permalink / raw) To: Randy Dunlap, Stephen Rothwell, Linux Next Mailing List Cc: Linux Kernel Mailing List, netdev, Jiri Pirko, Ivan Vecera, linux-omap On 04/12/2019 01:43, Randy Dunlap wrote: > On 12/2/19 8:54 PM, Stephen Rothwell wrote: >> Hi all, >> >> Please do not add any material for v5.6 to your linux-next included >> trees until after v5.5-rc1 has been released. >> >> Changes since 20191202: > > I am seeing this (happens to be on i386; I doubt that it matters): > CONFIG_COMPILE_TEST=y > > > WARNING: unmet direct dependencies detected for NET_SWITCHDEV > Depends on [n]: NET [=y] && INET [=n] > Selected by [y]: > - TI_CPSW_SWITCHDEV [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_TI [=y] && (ARCH_DAVINCI || ARCH_OMAP2PLUS || COMPILE_TEST [=y]) > > because TI_CPSW_SWITCHDEV blindly selects NET_SWITCHDEV even though > INET is not set/enabled, while NET_SWITCHDEV depends on INET. > > However, the build succeeds, including net/switchdev/*. > > So why does NET_SWITCHDEV depend on INET? > > It looks like TI_CPSW_SWITCHDEV should depend on INET (based on the > Kconfig rules), but in practice it doesn't seem to matter to the build. > Thanks for reporting this. I'd like to ask for some advice of how to proceed? a) change it to "depends on NET_SWITCHDEV" (as it's done in other drivers), but this will require to add NET_SWITCHDEV in defconfig b) change it to "imply NET_SWITCHDEV", but then NET_SWITCHDEV can be switched off manually or by random build and cause build failure of cpsw_new. To fix build below diff can be used, but TI_CPSW_SWITCHDEV will not be functional --- diff --git a/drivers/net/ethernet/ti/cpsw_new.c b/drivers/net/ethernet/ti/cpsw_new.c index 71215db7934b..22e8fc548d48 100644 --- a/drivers/net/ethernet/ti/cpsw_new.c +++ b/drivers/net/ethernet/ti/cpsw_new.c @@ -368,8 +368,9 @@ static void cpsw_rx_handler(void *token, int len, int status) page_pool_recycle_direct(pool, page); goto requeue; } - +#ifdef CONFIG_NET_SWITCHDEV skb->offload_fwd_mark = priv->offload_fwd_mark; +#endif skb_reserve(skb, headroom); skb_put(skb, len); skb->dev = ndev; Thank you. -- Best regards, grygorii ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: linux-next: Tree for Dec 3 (switchdev & TI_CPSW_SWITCHDEV) 2019-12-04 15:35 ` Grygorii Strashko @ 2019-12-04 16:11 ` Randy Dunlap 0 siblings, 0 replies; 4+ messages in thread From: Randy Dunlap @ 2019-12-04 16:11 UTC (permalink / raw) To: Grygorii Strashko, Stephen Rothwell, Linux Next Mailing List Cc: Linux Kernel Mailing List, netdev, Jiri Pirko, Ivan Vecera, linux-omap On 12/4/19 7:35 AM, Grygorii Strashko wrote: > > > On 04/12/2019 01:43, Randy Dunlap wrote: >> On 12/2/19 8:54 PM, Stephen Rothwell wrote: >>> Hi all, >>> >>> Please do not add any material for v5.6 to your linux-next included >>> trees until after v5.5-rc1 has been released. >>> >>> Changes since 20191202: >> >> I am seeing this (happens to be on i386; I doubt that it matters): >> CONFIG_COMPILE_TEST=y >> >> >> WARNING: unmet direct dependencies detected for NET_SWITCHDEV >> Depends on [n]: NET [=y] && INET [=n] >> Selected by [y]: >> - TI_CPSW_SWITCHDEV [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_TI [=y] && (ARCH_DAVINCI || ARCH_OMAP2PLUS || COMPILE_TEST [=y]) >> >> because TI_CPSW_SWITCHDEV blindly selects NET_SWITCHDEV even though >> INET is not set/enabled, while NET_SWITCHDEV depends on INET. >> >> However, the build succeeds, including net/switchdev/*. >> >> So why does NET_SWITCHDEV depend on INET? >> >> It looks like TI_CPSW_SWITCHDEV should depend on INET (based on the >> Kconfig rules), but in practice it doesn't seem to matter to the build. >> > > Thanks for reporting this. I'd like to ask for some advice of how to proceed? > a) change it to "depends on NET_SWITCHDEV" (as it's done in other drivers), > but this will require to add NET_SWITCHDEV in defconfig IMO TI_CPSW_SWITCHDEV should depend on NET_SWITCHDEV, as other drivers do that. That will require to add NET_SWITCHDEV in what defconfig? To me, it just means that whoever is doing the kernel config must enable/set NET_SWITCHDEV first, same as other drivers that depend on NET_SWITCHDEV. > b) change it to "imply NET_SWITCHDEV", but then NET_SWITCHDEV can be switched off > manually or by random build and cause build failure of cpsw_new. > To fix build below diff can be used, but TI_CPSW_SWITCHDEV will not be functional > > --- > diff --git a/drivers/net/ethernet/ti/cpsw_new.c b/drivers/net/ethernet/ti/cpsw_new.c > index 71215db7934b..22e8fc548d48 100644 > --- a/drivers/net/ethernet/ti/cpsw_new.c > +++ b/drivers/net/ethernet/ti/cpsw_new.c > @@ -368,8 +368,9 @@ static void cpsw_rx_handler(void *token, int len, int status) > page_pool_recycle_direct(pool, page); > goto requeue; > } > - > +#ifdef CONFIG_NET_SWITCHDEV > skb->offload_fwd_mark = priv->offload_fwd_mark; > +#endif > skb_reserve(skb, headroom); > skb_put(skb, len); > skb->dev = ndev; > > Thank you. > -- ~Randy Reported-by: Randy Dunlap <rdunlap@infradead.org> ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-12-04 16:11 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20191203155405.31404722@canb.auug.org.au> 2019-12-03 16:52 ` linux-next: Tree for Dec 3 (drivers/ptp/ptp_clockmatrix) Randy Dunlap 2019-12-03 23:43 ` linux-next: Tree for Dec 3 (switchdev & TI_CPSW_SWITCHDEV) Randy Dunlap 2019-12-04 15:35 ` Grygorii Strashko 2019-12-04 16:11 ` Randy Dunlap
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).