Re: [PATCH v3] Drivers: hv: vmbus: Fix harmless building warnings without CONFIG_PM_SLEEP

[Sasha Levin]

On Wed, Sep 25, 2019 at 06:43:57PM +0000, Dexuan Cui wrote:
> 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: 271b2224d42f ("Drivers: hv: vmbus: Implement suspend/resume for VSC drivers for hibernation")
> Fixes: f53335e3289f ("Drivers: hv: vmbus: Suspend/resume the vmbus itself for hibernation")
> Reported-by: Arnd Bergmann <arnd@xxxxxxxx>
> Reviewed-by: Michael Kelley <mikelley@xxxxxxxxxxxxx>
> Signed-off-by: Dexuan Cui <decui@xxxxxxxxxxxxx>

I've queued this up for hyperv-fixes.

> In v3:
> 	Add Michael's Reviewed-by.
> 	No other change.

There's really no need to re-spin just to add tags :)

--
Thanks,
Sasha