Commit 6fed0c1e authored by Paul Cercueil's avatar Paul Cercueil Committed by Lee Jones
Browse files

mfd: tc6393xb: Remove #ifdef guards for PM related functions 



Use the new pm_sleep_ptr() macro to handle the .suspend/.resume
callbacks.

This macro allow the suspend and resume functions to be automatically
dropped by the compiler when CONFIG_SUSPEND is disabled, without having
to use #ifdef guards.

This has the advantage of always compiling these functions in,
independently of any Kconfig option. Thanks to that, bugs and other
regressions are subsequently easier to catch.

Signed-off-by: default avatarPaul Cercueil <paul@crapouillou.net>
Signed-off-by: default avatarLee Jones <lee@kernel.org>
parent 02313a90

drivers/mfd/tc6393xb.c

+2 −7
Original line number Diff line number Diff line
@@ -813,7 +813,6 @@ static int tc6393xb_remove(struct platform_device *dev) 
	return 0;

}



#ifdef CONFIG_PM

static int tc6393xb_suspend(struct platform_device *dev, pm_message_t state)

{

	struct tc6393xb_platform_data *tcpd = dev_get_platdata(&dev->dev);
@@ -876,16 +875,12 @@ static int tc6393xb_resume(struct platform_device *dev) 


	return 0;

}

#else

#define tc6393xb_suspend NULL

#define tc6393xb_resume NULL

#endif



static struct platform_driver tc6393xb_driver = {

	.probe = tc6393xb_probe,

	.remove = tc6393xb_remove,

	.suspend = tc6393xb_suspend,

	.resume = tc6393xb_resume,

	.suspend = pm_sleep_ptr(tc6393xb_suspend),

	.resume = pm_sleep_ptr(tc6393xb_resume),



	.driver = {

		.name = "tc6393xb",