[Intel-wired-lan] [PATCH] ice: fix build issue if CONFIG_1588_PTP_CLOCk=m

Keller, Jacob E jacob.e.keller at intel.com
Thu Jul 29 22:13:34 UTC 2021



> -----Original Message-----
> From: G, GurucharanX <gurucharanx.g at intel.com>
> Sent: Thursday, July 29, 2021 8:57 AM
> To: intel-wired-lan at osuosl.org; intel-wired-lan at lists.osuosl.org; Keller, Jacob E
> <jacob.e.keller at intel.com>
> Subject: RE: [Intel-wired-lan] [PATCH] ice: fix build issue if
> CONFIG_1588_PTP_CLOCk=m
> 
> 
> > -----Original Message-----
> > From: Jacob Keller <jacob.e.keller at intel.com>
> > To: Intel Wired LAN <intel-wired-lan at lists.osuosl.org>
> > Subject: [Intel-wired-lan] [PATCH] ice: fix build issue if
> > 	CONFIG_1588_PTP_CLOCk=m
> > Message-ID: <20210709210726.224848-1-jacob.e.keller at intel.com>
> >
> > If the CONFIG_1588_PTP_CLOCK symbol is set to m, then the driver will
> > not compile ice_ptp.o and ice_ptp_hw.o because it adds them to ice-m
> > instead of ice-y, and nothing includes ice-m into the final kernel
> > objects.
> >
> > Fix this by using the ':m=y' so that we always compile the files and
> > link them to ice.o when CONFIG_1588_PTP_CLOCK is a module or a builtin.
> >
> > Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> > ---
> > This fixes the compilation errors that have appeared from lkp on Tony's
> > dev-queue. The lkp complains about other patches, but the root cause is this
> > issue. If we apply this, it should fix all of the lkp issues I've seen so
> > far.
> >
> >  drivers/net/ethernet/intel/ice/Makefile | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> 
> Tested-by: Gurucharan  G <Gurucharanx.g at intel.com> (A Contingent Worker at
> Intel)

FWIW, this should probably be superseded with a v2 of Arnd's fix to modify the Kbuild instead.

Thanks,
Jake


More information about the Intel-wired-lan mailing list