From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH] sdhci: Forward EPROBE_DEFER on vmmc and vqmmc regulators Date: Mon, 07 Apr 2014 14:51:41 +0200 Message-ID: <4902841.krZFXFDZqb@wuerfel> References: <1395991817-3503-1-git-send-email-mike.looijmans@topic.nl> <10471336.Pb8zVvW3Hp@wuerfel> <53429AD4.7030508@topic.nl> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: Received: from moutng.kundenserver.de ([212.227.126.187]:64366 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754274AbaDGMvt (ORCPT ); Mon, 7 Apr 2014 08:51:49 -0400 In-Reply-To: <53429AD4.7030508@topic.nl> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Mike Looijmans Cc: Ben Dooks , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, cjb@laptop.org, gdjakov@mm-sol.com On Monday 07 April 2014 14:32:20 Mike Looijmans wrote: > On 04/07/2014 02:25 PM, Arnd Bergmann wrote: > > Judging from the kernel output, regulator_get_optional returns -ENODEV if the > supply wasn't found. > > Maybe the API is confusing (or wrong?) here. > > If you change the code as per your suggestion, the SD will not work unless you > explicitly assign supplies. And judging from what I've seen so far, I am the > first to have ever attached a power supply to this controller... It's certainly not very "optional" if it returns an error here. You could try to special-case the "-ENODEV" return here, but I think it would be better to change the regulator interface to be less confusing. Arnd