[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <bf10026f-13bb-c1c6-2787-d8c9520f8401@redhat.com>
Date: Thu, 11 Feb 2021 16:45:18 +0100
From: Hans de Goede <hdegoede@...hat.com>
To: "Rafael J. Wysocki" <rafael@...nel.org>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>,
Masahiro Yamada <yamada.masahiro@...ionext.com>,
Mika Westerberg <mika.westerberg@...ux.intel.com>,
Lee Jones <lee.jones@...aro.org>,
Linus Walleij <linus.walleij@...aro.org>,
Thomas Gleixner <tglx@...utronix.de>,
Mike Rapoport <rppt@...nel.org>, Wolfram Sang <wsa@...nel.org>,
Sumit Gupta <sumitg@...dia.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Linux PCI <linux-pci@...r.kernel.org>,
Platform Driver <platform-driver-x86@...r.kernel.org>,
the arch/x86 maintainers <x86@...nel.org>,
Linux PM <linux-pm@...r.kernel.org>,
Linux Media Mailing List <linux-media@...r.kernel.org>,
devel@...verdev.osuosl.org, Ingo Molnar <mingo@...hat.com>,
Borislav Petkov <bp@...en8.de>,
"H. Peter Anvin" <hpa@...or.com>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Darren Hart <dvhart@...radead.org>,
Andy Shevchenko <andy@...radead.org>,
"Rafael J. Wysocki" <rjw@...ysocki.net>,
Viresh Kumar <viresh.kumar@...aro.org>,
Mark Gross <mgross@...ux.intel.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
ACPI Devel Maling List <linux-acpi@...r.kernel.org>
Subject: Re: [PATCH v1 0/9] x86/platform: Remove SFI framework and users
Hi,
On 2/11/21 4:24 PM, Rafael J. Wysocki wrote:
> On Thu, Feb 11, 2021 at 2:50 PM Andy Shevchenko
> <andriy.shevchenko@...ux.intel.com> wrote:
>>
>> This is last part of Intel MID (SFI based) removal. We have no more users of it
>> in the kernel and since SFI has been marked Obsolete for a few years already,
>> Remove all the stuff altogether.
>>
>> Note, the more recent platforms (Intel Merrifield and Moorefield) still work as
>> long as they provide correct ACPI tables.
>>
>> The series requires two prerequisite branches to be pulled first, i.e.
>> - one form Rafael's PM tree (currently bleeding-edge)
>> - one form TIP tree (x86/platform), actually only one patch is needed from it
>>
>> Due to above it's convenient to proceed all of these via Rafael's PM tree,
>>
>> Note, atomisp change is tagged by Sakari on behalf of media tree maintainers.
>>
>> Andy Shevchenko (9):
>> media: atomisp: Remove unused header
>> cpufreq: sfi-cpufreq: Remove driver for deprecated firmware
>> sfi: Remove framework for deprecated firmware
>> x86/PCI: Get rid of custom x86 model comparison
>> x86/PCI: Describe @reg for type1_access_ok()
>> x86/platform/intel-mid: Get rid of intel_scu_ipc_legacy.h
>> x86/platform/intel-mid: Drop unused __intel_mid_cpu_chip and Co.
>> x86/platform/intel-mid: Remove unused header inclusion in intel-mid.h
>> x86/platform/intel-mid: Update Copyright year and drop file names
>>
>> Documentation/ABI/testing/sysfs-firmware-sfi | 15 -
>> Documentation/ABI/testing/sysfs-platform-kim | 2 +-
>> MAINTAINERS | 7 -
>> arch/x86/Kconfig | 7 +-
>> arch/x86/include/asm/intel-mid.h | 65 +--
>> arch/x86/include/asm/intel_scu_ipc.h | 2 -
>> arch/x86/include/asm/intel_scu_ipc_legacy.h | 74 ---
>> arch/x86/include/asm/platform_sst_audio.h | 2 -
>> arch/x86/kernel/apic/io_apic.c | 4 +-
>> arch/x86/kernel/setup.c | 2 -
>> arch/x86/pci/intel_mid_pci.c | 18 +-
>> arch/x86/pci/mmconfig-shared.c | 6 +-
>> arch/x86/platform/Makefile | 1 -
>> arch/x86/platform/intel-mid/Makefile | 5 -
>> .../platform/intel-mid/device_libs/Makefile | 23 -
>> .../intel-mid/device_libs/platform_bcm43xx.c | 101 ----
>> .../intel-mid/device_libs/platform_bma023.c | 16 -
>> .../intel-mid/device_libs/platform_bt.c | 101 ----
>> .../intel-mid/device_libs/platform_emc1403.c | 39 --
>> .../device_libs/platform_gpio_keys.c | 81 ---
>> .../intel-mid/device_libs/platform_lis331.c | 37 --
>> .../intel-mid/device_libs/platform_max7315.c | 77 ---
>> .../intel-mid/device_libs/platform_mpu3050.c | 32 --
>> .../device_libs/platform_mrfld_pinctrl.c | 39 --
>> .../device_libs/platform_mrfld_rtc.c | 44 --
>> .../intel-mid/device_libs/platform_mrfld_sd.c | 43 --
>> .../device_libs/platform_mrfld_spidev.c | 50 --
>> .../device_libs/platform_pcal9555a.c | 95 ----
>> .../intel-mid/device_libs/platform_tc35876x.c | 42 --
>> .../intel-mid/device_libs/platform_tca6416.c | 53 --
>> arch/x86/platform/intel-mid/intel-mid.c | 27 +-
>> arch/x86/platform/intel-mid/sfi.c | 419 --------------
>> arch/x86/platform/sfi/Makefile | 2 -
>> arch/x86/platform/sfi/sfi.c | 100 ----
>> drivers/Makefile | 2 +-
>> drivers/cpufreq/Kconfig.x86 | 10 -
>> drivers/cpufreq/Makefile | 1 -
>> drivers/cpufreq/sfi-cpufreq.c | 127 -----
>> drivers/platform/x86/intel_scu_pcidrv.c | 22 +-
>> drivers/sfi/Kconfig | 18 -
>> drivers/sfi/Makefile | 4 -
>> drivers/sfi/sfi_acpi.c | 214 -------
>> drivers/sfi/sfi_core.c | 522 ------------------
>> drivers/sfi/sfi_core.h | 81 ---
>> .../atomisp/include/linux/atomisp_platform.h | 1 -
>> include/linux/sfi.h | 210 -------
>> include/linux/sfi_acpi.h | 93 ----
>> init/main.c | 2 -
>> 48 files changed, 37 insertions(+), 2901 deletions(-)
>> delete mode 100644 Documentation/ABI/testing/sysfs-firmware-sfi
>> delete mode 100644 arch/x86/include/asm/intel_scu_ipc_legacy.h
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/Makefile
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_bcm43xx.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_bma023.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_bt.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_emc1403.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_gpio_keys.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_lis331.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_max7315.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_mpu3050.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_mrfld_pinctrl.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_mrfld_rtc.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_mrfld_sd.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_mrfld_spidev.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_pcal9555a.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_tc35876x.c
>> delete mode 100644 arch/x86/platform/intel-mid/device_libs/platform_tca6416.c
>> delete mode 100644 arch/x86/platform/intel-mid/sfi.c
>> delete mode 100644 arch/x86/platform/sfi/Makefile
>> delete mode 100644 arch/x86/platform/sfi/sfi.c
>> delete mode 100644 drivers/cpufreq/sfi-cpufreq.c
>> delete mode 100644 drivers/sfi/Kconfig
>> delete mode 100644 drivers/sfi/Makefile
>> delete mode 100644 drivers/sfi/sfi_acpi.c
>> delete mode 100644 drivers/sfi/sfi_core.c
>> delete mode 100644 drivers/sfi/sfi_core.h
>> delete mode 100644 include/linux/sfi.h
>> delete mode 100644 include/linux/sfi_acpi.h
>>
>> --
>
> All of this looks good to me, so I'm going to queue it up for 5.12
> unless there are objections against doing that.
That is fine by me (for the drivers/platform/x86 bits) :
Acked-by: Hans de Goede <hdegoede@...hat.com>
Regards,
Hans
Powered by blists - more mailing lists