Re: [PATCH] backlight: da903x_bl: switch to using SIMPLE_DEV_PM_OPS

2013-03-12 Thread Jingoo Han
On Wednesday, March 13, 2013 4:03 AM, Lars-Peter Clausen wrote: > > On 03/12/2013 06:48 AM, Jingoo Han wrote: > > This reduces #ifdefs in the code. Also, CONFIG_PM_SLEEP is used > > to avoid warnings of unused functions if CONFIG_PM_SLEEP is not > > defined. > > > > Signed-off-by: Jingoo Han > >

Re: [PATCH] backlight: da903x_bl: switch to using SIMPLE_DEV_PM_OPS

2013-03-12 Thread Lars-Peter Clausen
On 03/12/2013 06:48 AM, Jingoo Han wrote: > This reduces #ifdefs in the code. Also, CONFIG_PM_SLEEP is used > to avoid warnings of unused functions if CONFIG_PM_SLEEP is not > defined. > > Signed-off-by: Jingoo Han I think you can eliminate the suspend/resume functions all together by using the

[PATCH] backlight: da903x_bl: switch to using SIMPLE_DEV_PM_OPS

2013-03-11 Thread Jingoo Han
This reduces #ifdefs in the code. Also, CONFIG_PM_SLEEP is used to avoid warnings of unused functions if CONFIG_PM_SLEEP is not defined. Signed-off-by: Jingoo Han --- drivers/video/backlight/da903x_bl.c | 12 1 files changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/v