Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759977Ab3FDHUA (ORCPT ); Tue, 4 Jun 2013 03:20:00 -0400 Received: from mail-oa0-f46.google.com ([209.85.219.46]:51822 "EHLO mail-oa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759355Ab3FDHT5 (ORCPT ); Tue, 4 Jun 2013 03:19:57 -0400 MIME-Version: 1.0 In-Reply-To: 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:19:57 +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: 1659 Lines: 42 On Tue, Jun 4, 2013 at 9:11 AM, Linus Walleij wrote: > 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 So please consider my other option given in patch 2 instead. 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/