Re: [PATCH 3/3] spi: sirf: Use SIMPLE_DEV_PM_OPS macro

[Barry Song <baohua@xxxxxxxxxx>]

2014-02-26 16:55 GMT+08:00 Jingoo Han <jg1.han@xxxxxxxxxxx>:
> On Wednesday, February 26, 2014 5:32 PM, Barry Song wrote:
>> 2014-02-26 9:32 GMT+08:00 Jingoo Han <jg1.han@xxxxxxxxxxx>:
>> > Use SIMPLE_DEV_PM_OPS macro in order to make the code simpler.
>> >
>> > Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx>
>>
>> thanks, but this has been done and merged.
>
> Thank you for your comment. However I have a question. :-)
>
> My patch replaces 'SET_SYSTEM_SLEEP_PM_OPS' with 'SIMPLE_DEV_PM_OPS'.
> However, I cannot find the same patch in SPI mailing-list.
> Would you let me know how to find the same patch that was submitted
> earlier?

sorry, i misunderstood your patch.

Acked-by: Barry Song <Baohua.Song@xxxxxxx>

thanks a lot!

>
> Best regards,
> Jingoo Han
>
>>
>> > ---
>> >  drivers/spi/spi-sirf.c |    5 ++---
>> >  1 file changed, 2 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/drivers/spi/spi-sirf.c b/drivers/spi/spi-sirf.c
>> > index 5210b94..dc962a1 100644
>> > --- a/drivers/spi/spi-sirf.c
>> > +++ b/drivers/spi/spi-sirf.c
>> > @@ -718,9 +718,8 @@ static int spi_sirfsoc_resume(struct device *dev)
>> >  }
>> >  #endif
>> >
>> > -static const struct dev_pm_ops spi_sirfsoc_pm_ops = {
>> > -       SET_SYSTEM_SLEEP_PM_OPS(spi_sirfsoc_suspend, spi_sirfsoc_resume)
>> > -};
>> > +static SIMPLE_DEV_PM_OPS(spi_sirfsoc_pm_ops, spi_sirfsoc_suspend,
>> > +                        spi_sirfsoc_resume);
>> >
>> >  static const struct of_device_id spi_sirfsoc_of_match[] = {
>> >         { .compatible = "sirf,prima2-spi", },
>> > --
>> > 1.7.10.4
>> >
>> >
>> -barry


-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html