From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-qy0-f181.google.com ([209.85.216.181]:34460 "EHLO mail-qy0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750911Ab1A0WS2 (ORCPT ); Thu, 27 Jan 2011 17:18:28 -0500 MIME-Version: 1.0 In-Reply-To: <87tyguje7j.fsf@ti.com> References: <87tyguje7j.fsf@ti.com> Date: Thu, 27 Jan 2011 23:18:27 +0100 Message-ID: Subject: Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions From: Vitaly Wool To: Kevin Hilman Cc: Alan Stern , linux-wireless@vger.kernel.org, linux-mmc@vger.kernel.org, Ido Yariv , linux-pm@lists.linux-foundation.org, Johannes Berg Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, Jan 27, 2011 at 9:15 PM, Kevin Hilman wrote: > >> If it is, then you can call omap_pm_runtime_suspend() directly instead >> of calling dev->bus->pm->runtime_suspend(). > > Personally, I prefer going through dev->bus as we try to avoid SoC > specific calls in the driver. > > This same HW block might be re-used on non-OMAP SoCs (e.g. TI DaVinci) > that would have different PM at the susbystem level. > > So, to summarize, as long as folks are OK with drivers directly calling > the subsystem runtime PM callbacks, I'll go this route. I personally think it's okay for the moment. Generally speaking, SD bus driver might not have runtime PM support so it's better to have this explicitly called and not compiling for other platforms rather than have it compiling but working not the way it's expected to. ~Vitaly From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vitaly Wool Subject: Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions Date: Thu, 27 Jan 2011 23:18:27 +0100 Message-ID: References: <87tyguje7j.fsf@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: In-Reply-To: <87tyguje7j.fsf-l0cyMroinI0@public.gmane.org> Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Kevin Hilman Cc: Alan Stern , linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ido Yariv , linux-pm-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, Johannes Berg List-Id: linux-mmc@vger.kernel.org On Thu, Jan 27, 2011 at 9:15 PM, Kevin Hilman wrote: > >> If it is, then you can call omap_pm_runtime_suspend() directly instead >> of calling dev->bus->pm->runtime_suspend(). > > Personally, I prefer going through dev->bus as we try to avoid SoC > specific calls in the driver. > > This same HW block might be re-used on non-OMAP SoCs (e.g. TI DaVinci) > that would have different PM at the susbystem level. > > So, to summarize, as long as folks are OK with drivers directly calling > the subsystem runtime PM callbacks, I'll go this route. I personally think it's okay for the moment. Generally speaking, SD bus driver might not have runtime PM support so it's better to have this explicitly called and not compiling for other platforms rather than have it compiling but working not the way it's expected to. ~Vitaly -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html