Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759827Ab3FDHLD (ORCPT ); Tue, 4 Jun 2013 03:11:03 -0400 Received: from mail-ob0-f182.google.com ([209.85.214.182]:39629 "EHLO mail-ob0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759255Ab3FDHLA (ORCPT ); Tue, 4 Jun 2013 03:11:00 -0400 MIME-Version: 1.0 In-Reply-To: <1369995191-20855-10-git-send-email-gururaja.hebbar@ti.com> References: <1369995191-20855-1-git-send-email-gururaja.hebbar@ti.com> <1369995191-20855-10-git-send-email-gururaja.hebbar@ti.com> Date: Tue, 4 Jun 2013 09:11:00 +0200 Message-ID: Subject: Re: [PATCH 09/11] mmc: omap_hsmmc: enhance pinctrl support From: Linus Walleij To: Hebbar Gururaja Cc: Kevin Hilman , Grant Likely , Rob Herring , "davinci-linux-open-source@linux.davincidsp.com" , "devicetree-discuss@lists.ozlabs.org" , "linux-arm-kernel@lists.infradead.org" , Russell King - ARM Linux , "linux-kernel@vger.kernel.org" , Vaibhav Bedia , sudhakar.raj@ti.com, Balaji T K , Chris Ball , "linux-mmc@vger.kernel.org" , Linux-OMAP Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1684 Lines: 43 On Fri, May 31, 2013 at 12:13 PM, Hebbar Gururaja wrote: > Amend the hsmmc controller to optionally take a pin control handle and > set the state of the pins to: > > - "default" on boot, resume and before performing a mmc transfer > - "idle" after initial default, after resume default, and after each > mmc/sd card access > - "sleep" on suspend() > > By optionally putting the pins into sleep state in the suspend callback > we can accomplish two things. > - One is to minimize current leakage from pins and thus save power, > - second, we can prevent the IP from driving pins output in an > uncontrolled manner, which may happen if the power domain drops the > domain regulator. > > If any of the above pin states are missing in dt, a warning message > about the missing state is displayed. > If certain pin-states are not available, to remove this warning message > pass respective state name with null phandler. > > Signed-off-by: Hebbar Gururaja > Cc: Balaji T K > Cc: Chris Ball > Cc: linux-mmc@vger.kernel.org > Cc: linux-omap@vger.kernel.org This is perfectly correct. Acked-by: Linus Walleij As the PM code seems to be similar across platforms I have had loose plans to move this to the device core as well, but right now I'm too busy with other things, and it can surely be refactored later. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/