Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754928Ab2KNI57 (ORCPT ); Wed, 14 Nov 2012 03:57:59 -0500 Received: from mail-wi0-f172.google.com ([209.85.212.172]:45359 "EHLO mail-wi0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751820Ab2KNI56 (ORCPT ); Wed, 14 Nov 2012 03:57:58 -0500 MIME-Version: 1.0 In-Reply-To: <20121114084105.GG7407@opensource.wolfsonmicro.com> References: <25B60CDC2F704E4E9D88FFD52780CB4C060FBEA29A@SC-VEXCH1.marvell.com> <20121114072855.GF7407@opensource.wolfsonmicro.com> <20121114084105.GG7407@opensource.wolfsonmicro.com> Date: Wed, 14 Nov 2012 16:57:56 +0800 Message-ID: Subject: Re: FW: [PATCH v2] mmc: sdhci: apply voltage range check only for non-fixed regulators From: Kevin Liu To: Mark Brown Cc: m.szyprowski@samsung.com, Chris Ball , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, kyungmin.park@samsung.com, lrg@ti.com, Philip Rakity 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: 1155 Lines: 26 2012/11/14 Mark Brown : > On Wed, Nov 14, 2012 at 04:36:28PM +0800, Kevin Liu wrote: >> 2012/11/14 Mark Brown : > >> > Should this be regulator_set_voltage_tol()? Otherwise it'd be good to >> > explain where the numbers come from. > >> In SD physical layer spec 3.01 chapter 6.6.1, the threshold level for >> voltage range is defined as below: >> Vdd(min) = 2.7V while Vdd(max) = 3.6V. >> The card should work within the voltage range. > >> If you are afraid the voltage value is too aggressive, maybe we can >> use regulator_set_voltage_tol() to set a smaller range. >> But which range should be reasonable? > > The above makes total sense - thanks! I just wasn't aware that the > range was specified in this fashion in the spec. Might be worth a > comment in the code if you need to respin. Sure, I will update the patch. Thanks! -- 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/