| Message ID | 20240305161539.1364717-3-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | platform/x86: Remove unused intel-mid.h | expand |
On 3/5/24 8:14 AM, Andy Shevchenko wrote: > intel-mid.h is providing some core parts of the South Complex PM, > which are usually are not used by individual drivers. In particular, > this driver doesn't use it, so simply remove the unused header. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- Looks fine to me. Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com> > drivers/platform/x86/intel_scu_pcidrv.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/platform/x86/intel_scu_pcidrv.c b/drivers/platform/x86/intel_scu_pcidrv.c > index d904fad499aa..dbf0310448da 100644 > --- a/drivers/platform/x86/intel_scu_pcidrv.c > +++ b/drivers/platform/x86/intel_scu_pcidrv.c > @@ -11,7 +11,6 @@ > #include <linux/init.h> > #include <linux/pci.h> > > -#include <asm/intel-mid.h> > #include <asm/intel_scu_ipc.h> > > static int intel_scu_pci_probe(struct pci_dev *pdev,
On Tue, 5 Mar 2024, Andy Shevchenko wrote: > intel-mid.h is providing some core parts of the South Complex PM, > which are usually are not used by individual drivers. In particular, Both applied to review-ilpo. I fixed that double "are" for you while applying.
On Wed, Mar 06, 2024 at 12:44:03PM +0200, Ilpo Järvinen wrote: > On Tue, 5 Mar 2024, Andy Shevchenko wrote: > > > intel-mid.h is providing some core parts of the South Complex PM, > > which are usually are not used by individual drivers. In particular, > > Both applied to review-ilpo. I fixed that double "are" for you while > applying. Thank you!
diff --git a/drivers/platform/x86/intel_scu_pcidrv.c b/drivers/platform/x86/intel_scu_pcidrv.c index d904fad499aa..dbf0310448da 100644 --- a/drivers/platform/x86/intel_scu_pcidrv.c +++ b/drivers/platform/x86/intel_scu_pcidrv.c @@ -11,7 +11,6 @@ #include <linux/init.h> #include <linux/pci.h> -#include <asm/intel-mid.h> #include <asm/intel_scu_ipc.h> static int intel_scu_pci_probe(struct pci_dev *pdev,
intel-mid.h is providing some core parts of the South Complex PM, which are usually are not used by individual drivers. In particular, this driver doesn't use it, so simply remove the unused header. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- drivers/platform/x86/intel_scu_pcidrv.c | 1 - 1 file changed, 1 deletion(-)