Commit 83b50f83 authored by Dexuan Cui's avatar Dexuan Cui Committed by Sasha Levin

Drivers: hv: vmbus: Fix harmless building warnings without CONFIG_PM_SLEEP

If CONFIG_PM_SLEEP is not set, we can comment out these functions to avoid
the below warnings:

drivers/hv/vmbus_drv.c:2208:12: warning: ‘vmbus_bus_resume’ defined but not used [-Wunused-function]
drivers/hv/vmbus_drv.c:2128:12: warning: ‘vmbus_bus_suspend’ defined but not used [-Wunused-function]
drivers/hv/vmbus_drv.c:937:12: warning: ‘vmbus_resume’ defined but not used [-Wunused-function]
drivers/hv/vmbus_drv.c:918:12: warning: ‘vmbus_suspend’ defined but not used [-Wunused-function]

Fixes: 271b2224 ("Drivers: hv: vmbus: Implement suspend/resume for VSC drivers for hibernation")
Fixes: f53335e3 ("Drivers: hv: vmbus: Suspend/resume the vmbus itself for hibernation")
Reported-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarDexuan Cui <decui@microsoft.com>
Reviewed-by: default avatarMichael Kelley <mikelley@microsoft.com>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 6a297c90
...@@ -912,6 +912,7 @@ static void vmbus_shutdown(struct device *child_device) ...@@ -912,6 +912,7 @@ static void vmbus_shutdown(struct device *child_device)
drv->shutdown(dev); drv->shutdown(dev);
} }
#ifdef CONFIG_PM_SLEEP
/* /*
* vmbus_suspend - Suspend a vmbus device * vmbus_suspend - Suspend a vmbus device
*/ */
...@@ -949,6 +950,7 @@ static int vmbus_resume(struct device *child_device) ...@@ -949,6 +950,7 @@ static int vmbus_resume(struct device *child_device)
return drv->resume(dev); return drv->resume(dev);
} }
#endif /* CONFIG_PM_SLEEP */
/* /*
* vmbus_device_release - Final callback release of the vmbus child device * vmbus_device_release - Final callback release of the vmbus child device
...@@ -1070,6 +1072,7 @@ void vmbus_on_msg_dpc(unsigned long data) ...@@ -1070,6 +1072,7 @@ void vmbus_on_msg_dpc(unsigned long data)
vmbus_signal_eom(msg, message_type); vmbus_signal_eom(msg, message_type);
} }
#ifdef CONFIG_PM_SLEEP
/* /*
* Fake RESCIND_CHANNEL messages to clean up hv_sock channels by force for * Fake RESCIND_CHANNEL messages to clean up hv_sock channels by force for
* hibernation, because hv_sock connections can not persist across hibernation. * hibernation, because hv_sock connections can not persist across hibernation.
...@@ -1105,6 +1108,7 @@ static void vmbus_force_channel_rescinded(struct vmbus_channel *channel) ...@@ -1105,6 +1108,7 @@ static void vmbus_force_channel_rescinded(struct vmbus_channel *channel)
vmbus_connection.work_queue, vmbus_connection.work_queue,
&ctx->work); &ctx->work);
} }
#endif /* CONFIG_PM_SLEEP */
/* /*
* Direct callback for channels using other deferred processing * Direct callback for channels using other deferred processing
...@@ -2125,6 +2129,7 @@ static int vmbus_acpi_add(struct acpi_device *device) ...@@ -2125,6 +2129,7 @@ static int vmbus_acpi_add(struct acpi_device *device)
return ret_val; return ret_val;
} }
#ifdef CONFIG_PM_SLEEP
static int vmbus_bus_suspend(struct device *dev) static int vmbus_bus_suspend(struct device *dev)
{ {
struct vmbus_channel *channel, *sc; struct vmbus_channel *channel, *sc;
...@@ -2247,6 +2252,7 @@ static int vmbus_bus_resume(struct device *dev) ...@@ -2247,6 +2252,7 @@ static int vmbus_bus_resume(struct device *dev)
return 0; return 0;
} }
#endif /* CONFIG_PM_SLEEP */
static const struct acpi_device_id vmbus_acpi_device_ids[] = { static const struct acpi_device_id vmbus_acpi_device_ids[] = {
{"VMBUS", 0}, {"VMBUS", 0},
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment