Thanks for the patch, It looks ok to me. Acked-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxx> On 27/02/14 11:43, Jingoo Han wrote: > Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler. > > Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx> > --- > drivers/tty/serial/st-asc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/tty/serial/st-asc.c b/drivers/tty/serial/st-asc.c > index 21e6e84..93e27b9 100644 > --- a/drivers/tty/serial/st-asc.c > +++ b/drivers/tty/serial/st-asc.c > @@ -883,9 +883,8 @@ static struct uart_driver asc_uart_driver = { > .cons = ASC_SERIAL_CONSOLE, > }; > > -static const struct dev_pm_ops asc_serial_pm_ops = { > - SET_SYSTEM_SLEEP_PM_OPS(asc_serial_suspend, asc_serial_resume) > -}; > +static SIMPLE_DEV_PM_OPS(asc_serial_pm_ops, asc_serial_suspend, > + asc_serial_resume); > > static struct platform_driver asc_serial_driver = { > .probe = asc_serial_probe, > -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html