Re: [PATCH] PM: ARM: locomo: Drop suspend and resume bus type callbacks

From: Andrea Adami
Date: Tue Oct 03 2017 - 10:55:12 EST


On Tue, Oct 3, 2017 at 1:44 PM, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
> On Tuesday, October 3, 2017 10:09:21 AM CEST Ulf Hansson wrote:
>> On 26 September 2017 at 22:45, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
>> > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>> >
>> > None of the locomo drivers in the tree implements the suspend and
>> > resume callbacks from struct locomo_driver, so drop them and drop
>> > the corresponding callbacks from locomo_bus_type.
>> >
>> > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>>
>> FWIW:
>>
>> Reviewed-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
>>
>> Do you also plan to convert the locomo_device_driver (platform_driver)
>> to be converted to the modern PM callbacks for suspend/resume?
>
> Yes, I do, but that later. :-)
>
> Thanks,
> Rafael
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


Thanks for working on this.

Just FYI,
there is a patch-serie for 4.2/locomo ported up to 4.4: never went
mainline unfortunately.

http://git.infradead.org/users/dbaryshkov/zaurus.git

I have just adapted the patches form 4.2 to 4.4 [1] and tested on real hw.
Unfortunately it is not trivial updating it (must be adapted to GPIO
descriptors, etc.).

As far as I see in the new driver there isn't anymore the use of the
bus, so your patch shouldn't harm.
I'll put Dmitry in cc to get his confirmation.

Regards
Andrea

[1] http://cgit.openembedded.org/meta-handheld/tree/recipes-kernel/linux/linux-handheld-4.4/locomo