[PATCH 1/2] [media] adv7180: Simplify PM hooks

2014-12-16 Thread Fabio Estevam
The macro SIMPLE_DEV_PM_OPS already takes care of the CONFIG_PM_SLEEP=n case, so move it out of the CONFIG_PM_SLEEP 'if' block and remove the unneeded ADV7180_PM_OPS definition to make the code simpler. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- drivers/media/i2c/adv7180.c | 9

Re: [PATCH 1/2] [media] adv7180: Simplify PM hooks

2014-12-16 Thread Lars-Peter Clausen
On 12/16/2014 05:49 PM, Fabio Estevam wrote: The macro SIMPLE_DEV_PM_OPS already takes care of the CONFIG_PM_SLEEP=n case. I guess that's kind of debatable. The purpose of ifdef-ing stuff out is to decrease the driver size if PM support is disabled. With this change you are adding a rather