From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753219AbdIDFrt (ORCPT ); Mon, 4 Sep 2017 01:47:49 -0400 Received: from fllnx210.ext.ti.com ([198.47.19.17]:13170 "EHLO fllnx210.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750943AbdIDFrq (ORCPT ); Mon, 4 Sep 2017 01:47:46 -0400 Subject: Re: Applied "regulator: pbias: Select voltage table based on max-voltage" to the regulator tree To: Mark Brown , Ulf Hansson References: <64d9c623-456c-9142-9c5b-a94593628643@ti.com> <20170831135050.adiek6iqqzhndgci@sirena.org.uk> <20170901162322.xrjnim4khn3wfslj@sirena.org.uk> CC: Ravikumar Kattekola , Sekhar Nori , Adrian Hunter , Rob Herring , Mark Rutland , Tony Lindgren , Liam Girdwood , Russell King , "linux-mmc@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-omap , "linux-arm-kernel@lists.infradead.org" From: Kishon Vijay Abraham I Message-ID: Date: Mon, 4 Sep 2017 11:16:54 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <20170901162322.xrjnim4khn3wfslj@sirena.org.uk> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, On Friday 01 September 2017 09:53 PM, Mark Brown wrote: > On Fri, Sep 01, 2017 at 03:18:03PM +0200, Ulf Hansson wrote: >> On 31 August 2017 at 15:50, Mark Brown wrote: >>> On Thu, Aug 31, 2017 at 05:37:34PM +0530, Kishon Vijay Abraham I wrote: > >>>> This patch should be merged along with the 1st patch of the series "mmc: host: >>>> omap_hsmmc: Remove setting PBIAS voltage". Or else it'll break MMC with >>>> omap_hsmmc driver. > > Hang on, how can this break anything? If it breaks things shouldn't > there be bugfixes for incorrect constraints somewhere? omap_hsmmc sets the pbias voltage to 3V. But here we program the volt table to support either 1.8V or 3.3V (Initially the volt table was programmed to support 3V because of a bug in TRM). So set_voltage of pbias voltage in omap_hsmmc will fail. Thanks Kishon