From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH 3/3] arm64: dts: apq8016-sbc: Set up LDO2, LDO6 and LDO17 regulator voltage ranges Date: Wed, 31 Aug 2016 13:54:26 -0700 Message-ID: References: <1472640730-24326-1-git-send-email-architt@codeaurora.org> <1472640730-24326-4-git-send-email-architt@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:37714 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760326AbcHaUyc (ORCPT ); Wed, 31 Aug 2016 16:54:32 -0400 In-Reply-To: <1472640730-24326-4-git-send-email-architt@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Archit Taneja , andy.gross@linaro.org, robh@kernel.org Cc: linux-arm-msm@vger.kernel.org On 08/31/2016 03:52 AM, Archit Taneja wrote: > On the APQ8016 SBC, the LDO2 PM8916 regulator feeds 1.2V to the following: > > - VDDA_1P2_MIPI_DSI and VDDA_MIPI_CSI pins on APQ8016. > - VCCCAD pins on the LPDDR3 chip. > - VDDPX_1 pins on APQ8016. > > The LDO6 regulator feeds 1.8V to: > - VDAA_MIPI_DSI0_PLL pin on APQ8016. > - QFPROM_BLOW_VDD pin on PM8916. > - The AVDD, A2VDD and DVDD pins on ADV7533 bridge. > > The LDO17 regulator feeds 3.3V to: > - The V3P3 pin on ADV7533 bridge. > > Currently, the regulator min/max voltages for all the LDOs are set to the > range of what the PMIC supports. Set the ranges for L2, L6 and L17 to what > we need, i.e. 1.2V, 1.8V and 3.3V respectively. Why doesn't the consuming driver request a particular voltage on these LDOs? It doesn't seem correct to modify board constraints to satisfy a consumer (even if that consumer is on the board). -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project