Drivers such as intel_pmc_ipc.c can be unloaded as well so in order to support those in this driver add a new function that can be called to remove the SCU IPC if the driver is unloaded. Signed-off-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> --- arch/x86/include/asm/intel_scu_ipc.h | 1 + drivers/platform/x86/intel_scu_ipc.c | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+) diff --git a/arch/x86/include/asm/intel_scu_ipc.h b/arch/x86/include/asm/intel_scu_ipc.h index 9895b60386c5..250127eb1e38 100644 --- a/arch/x86/include/asm/intel_scu_ipc.h +++ b/arch/x86/include/asm/intel_scu_ipc.h @@ -32,6 +32,7 @@ struct intel_scu_ipc_pdata { struct intel_scu_ipc_dev * intel_scu_ipc_probe(struct device *dev, const struct intel_scu_ipc_pdata *pdata); +void intel_scu_ipc_remove(struct intel_scu_ipc_dev *scu); struct intel_scu_ipc_dev *intel_scu_ipc_dev_get(void); void intel_scu_ipc_dev_put(struct intel_scu_ipc_dev *scu); diff --git a/drivers/platform/x86/intel_scu_ipc.c b/drivers/platform/x86/intel_scu_ipc.c index cc29f504adcf..9fa0ea95198b 100644 --- a/drivers/platform/x86/intel_scu_ipc.c +++ b/drivers/platform/x86/intel_scu_ipc.c @@ -566,3 +566,21 @@ intel_scu_ipc_probe(struct device *dev, const struct intel_scu_ipc_pdata *pdata) return scu; } EXPORT_SYMBOL_GPL(intel_scu_ipc_probe); + +/** + * intel_scu_ipc_remove() - Remove SCU IPC + * @scu: SCU IPC handle + * + * This unregisters the SCU IPC device and releases the interrupt. + */ +void intel_scu_ipc_remove(struct intel_scu_ipc_dev *scu) +{ + mutex_lock(&ipclock); + if (!WARN_ON(!scu->dev)) { + if (scu->irq > 0) + devm_free_irq(scu->dev, scu->irq, scu); + scu->dev = NULL; + } + mutex_unlock(&ipclock); +} +EXPORT_SYMBOL_GPL(intel_scu_ipc_remove); -- 2.24.1