On Wed, 28 Aug 2024, Xi Pardee wrote: In the subject: h file -> header > telemetry.h header file is not needed in arl.c or mtl.c. Remove > them to avoid confusion. I'd also note this is cross directory/driver include, so perhaps something along these lines: telemetry.h from PMT is not needed in arl.c or mtl.c so remove the cross-driver include. Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx> -- i. > Signed-off-by: Xi Pardee <xi.pardee@xxxxxxxxxxxxxxx> > --- > drivers/platform/x86/intel/pmc/arl.c | 1 - > drivers/platform/x86/intel/pmc/mtl.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git a/drivers/platform/x86/intel/pmc/arl.c b/drivers/platform/x86/intel/pmc/arl.c > index 870da98ceb41..0460715c58f4 100644 > --- a/drivers/platform/x86/intel/pmc/arl.c > +++ b/drivers/platform/x86/intel/pmc/arl.c > @@ -10,7 +10,6 @@ > > #include <linux/pci.h> > #include "core.h" > -#include "../pmt/telemetry.h" > > /* PMC SSRAM PMT Telemetry GUID */ > #define IOEP_LPM_REQ_GUID 0x5077612 > diff --git a/drivers/platform/x86/intel/pmc/mtl.c b/drivers/platform/x86/intel/pmc/mtl.c > index 908b5f8bb6e5..e7f5b650902d 100644 > --- a/drivers/platform/x86/intel/pmc/mtl.c > +++ b/drivers/platform/x86/intel/pmc/mtl.c > @@ -10,7 +10,6 @@ > > #include <linux/pci.h> > #include "core.h" > -#include "../pmt/telemetry.h" > > /* PMC SSRAM PMT Telemetry GUIDS */ > #define SOCP_LPM_REQ_GUID 0x2625030 >
