From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750935AbaKURmN (ORCPT ); Fri, 21 Nov 2014 12:42:13 -0500 Received: from mail-vc0-f180.google.com ([209.85.220.180]:61899 "EHLO mail-vc0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750733AbaKURmJ (ORCPT ); Fri, 21 Nov 2014 12:42:09 -0500 MIME-Version: 1.0 In-Reply-To: References: <1415109789-7046-1-git-send-email-addy.ke@rock-chips.com> <1415678573-6093-1-git-send-email-addy.ke@rock-chips.com> <5464152E.7040209@rock-chips.com> Date: Fri, 21 Nov 2014 09:42:08 -0800 X-Google-Sender-Auth: JNJuzB8oeNJx0rmMWJue8RPWdH8 Message-ID: Subject: Re: [PATCH] mmc: dw_mmc: try pick the exact same voltage as vmmc for vqmmc From: Doug Anderson To: Ulf Hansson Cc: addy ke , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Randy Dunlap , "tgih.jun@samsung.com" , Jaehoon Chung , Chris Ball , Dinh Nguyen , =?UTF-8?Q?Heiko_St=C3=BCbner?= , Olof Johansson , Sonny Rao , Alexandru Stan , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-mmc , "linux-arm-kernel@lists.infradead.org" , "open list:ARM/Rockchip SoC..." , "zhenfu.fang" , Eddie Cai , lintao , chenfen , zyf , Jianqun Xu , Tao Huang , Chris Zhong , =?UTF-8?B?5aea5pm65oOF?= , han jiang , Kever Yang , zhangqing , Lin Huang Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Ulf, On Fri, Nov 21, 2014 at 4:06 AM, Ulf Hansson wrote: > [...] > >> Sure >> If the first card is sd2.0 since startup, dw_mci_switch_voltage will not be called, > > That can't be right. mmc_power_up() should trigger > dw_mci_switch_voltage() to be invoked. Hmmm, I think you're right. Addy: can you double check if it's only the 2nd card for you? I was thinking that if a regulator is currently 3.3V and you request 2.7 - 3.3V the regulator framework will treat that as a noop. ...but that definitely doesn't appear to be the case. When I boot up the first time even with no SD card plugged in, I see this at bootup: [ 3.042234] vccio_sd: 1800 <--> 3300 mV at 3300 mV ...showing that it started at 3.3V. Then I see: $ grep "" /sys/class/regulator/regulator.16/{name,microvolts} /sys/class/regulator/regulator.16/name:vccio_sd /sys/class/regulator/regulator.16/microvolts:2700000 ...so it is certainly getting changed even with no card plugged in. BTW: I don't actually have one of these failing cards--all of mine work. Addy, do you know the make and model of the card you have that fails? -Doug