On 05/16/2018 11:17 AM, David Miller wrote: > From: David Miller <da...@davemloft.net> > Date: Wed, 16 May 2018 14:15:14 -0400 (EDT) > >> From: Florian Fainelli <f.faine...@gmail.com> >> Date: Tue, 15 May 2018 16:48:22 -0700 >> >>> This patch series includes more drivers to be build tested with COMPILE_TEST >>> enabled. This helps cover some of the issues I just ran into with missing >>> a driver *sigh*. >> >> Thank you for doing this. >> >> Series applied. > > Actually, I have to revert. > > The FEC driver cannot be built tested without the appropriate platform > specific register set defines and x86-64 for example won't provide that.
Ah, yes, it built okay for ARM 32-bit, let me fix that. > > drivers/net/ethernet/freescale/fec_main.c: In function ‘fec_restart’: > drivers/net/ethernet/freescale/fec_main.c:959:26: error: ‘FEC_RACC’ > undeclared (first use in this function); did you mean ‘FEC_RXIC0’? > val = readl(fep->hwp + FEC_RACC); > ^~~~~~~~ > FEC_RXIC0 > drivers/net/ethernet/freescale/fec_main.c:959:26: note: each undeclared > identifier is reported only once for each function it appears in > drivers/net/ethernet/freescale/fec_main.c:968:38: error: ‘FEC_FTRL’ > undeclared (first use in this function); did you mean ‘FEC_ECNTRL’? > writel(PKT_MAXBUF_SIZE, fep->hwp + FEC_FTRL); > ^~~~~~~~ > FEC_ECNTRL > ... > -- Florian
