misc: bh1780gli: add CONFIG_PM_SLEEP to suspend/resume functions
Add CONFIG_PM_SLEEP to suspend/resume functions to fix the following build warning when CONFIG_PM_SLEEP is not selected. This is because sleep PM callbacks defined by SIMPLE_DEV_PM_OPS are only used when the CONFIG_PM_SLEEP is enabled. drivers/misc/bh1780gli.c:200:12: warning: 'bh1780_suspend' defined but not used [-Wunused-function] drivers/misc/bh1780gli.c:222:12: warning: 'bh1780_resume' defined but not used [-Wunused-function] Signed-off-by: Jingoo Han <jg1.han@samsung.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
aa6aef216f
commit
688ef20250
|
@ -196,7 +196,7 @@ static int bh1780_remove(struct i2c_client *client)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef CONFIG_PM
|
#ifdef CONFIG_PM_SLEEP
|
||||||
static int bh1780_suspend(struct device *dev)
|
static int bh1780_suspend(struct device *dev)
|
||||||
{
|
{
|
||||||
struct bh1780_data *ddata;
|
struct bh1780_data *ddata;
|
||||||
|
@ -235,11 +235,9 @@ static int bh1780_resume(struct device *dev)
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
#endif /* CONFIG_PM_SLEEP */
|
||||||
|
|
||||||
static SIMPLE_DEV_PM_OPS(bh1780_pm, bh1780_suspend, bh1780_resume);
|
static SIMPLE_DEV_PM_OPS(bh1780_pm, bh1780_suspend, bh1780_resume);
|
||||||
#define BH1780_PMOPS (&bh1780_pm)
|
|
||||||
#else
|
|
||||||
#define BH1780_PMOPS NULL
|
|
||||||
#endif /* CONFIG_PM */
|
|
||||||
|
|
||||||
static const struct i2c_device_id bh1780_id[] = {
|
static const struct i2c_device_id bh1780_id[] = {
|
||||||
{ "bh1780", 0 },
|
{ "bh1780", 0 },
|
||||||
|
@ -252,7 +250,7 @@ static struct i2c_driver bh1780_driver = {
|
||||||
.id_table = bh1780_id,
|
.id_table = bh1780_id,
|
||||||
.driver = {
|
.driver = {
|
||||||
.name = "bh1780",
|
.name = "bh1780",
|
||||||
.pm = BH1780_PMOPS,
|
.pm = &bh1780_pm,
|
||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue