Hi, This is another independent cleanup series that I split out from my SCU/PMC IPC rework patch set [1] as suggested by Greg. This removes code that is not used anywhere and makes certain functions static as they are not called outside of the driver. We also make the driver to use driver->dev_groups to expose sysfs attributes. [1] https://lkml.org/lkml/2020/1/21/678 Mika Westerberg (9): platform/x86: intel_pmc_ipc: Make intel_pmc_gcr_update() static platform/x86: intel_pmc_ipc: Make intel_pmc_ipc_simple_command() static platform/x86: intel_pmc_ipc: Make intel_pmc_ipc_raw_cmd() static platform/x86: intel_pmc_ipc: Drop intel_pmc_gcr_read() and intel_pmc_gcr_write() platform/x86: intel_pmc_ipc: Drop ipc_data_readb() platform/x86: intel_pmc_ipc: Get rid of unnecessary includes platform/x86: intel_pmc_ipc: Use octal permissions in sysfs attributes platform/x86: intel_pmc_ipc: Propagate error from kstrtoul() platform/x86: intel_pmc_ipc: Switch to use driver->dev_groups arch/x86/include/asm/intel_pmc_ipc.h | 32 -------- drivers/platform/x86/intel_pmc_ipc.c | 114 ++++----------------------- 2 files changed, 16 insertions(+), 130 deletions(-) -- 2.24.1
