From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757944AbcJXWs5 (ORCPT ); Mon, 24 Oct 2016 18:48:57 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:54239 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757620AbcJXWsz (ORCPT ); Mon, 24 Oct 2016 18:48:55 -0400 DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org 5A78F61860 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=pass smtp.mailfrom=sboyd@codeaurora.org Date: Mon, 24 Oct 2016 15:48:52 -0700 From: Stephen Boyd To: Viresh Kumar Cc: Rafael Wysocki , nm@ti.com, Viresh Kumar , linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Vincent Guittot , robh@kernel.org, d-gerlach@ti.com, broonie@kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH V2 1/8] PM / OPP: Reword binding supporting multiple regulators per device Message-ID: <20161024224852.GQ26139@codeaurora.org> References: <891ef8705e13dff331a3135647f4c18f88402a12.1476952750.git.viresh.kumar@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <891ef8705e13dff331a3135647f4c18f88402a12.1476952750.git.viresh.kumar@linaro.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/20, Viresh Kumar wrote: > On certain platforms (like TI), DVFS for a single device (CPU) requires > configuring multiple power supplies. > > The OPP bindings already contains binding and example to explain this > case, but it isn't sufficient. For example, there is no way for the code > parsing these bindings to know which voltage values belong to which > power supply. Also its not possible to know the order in which the > supplies need to be configured while switching OPPs. > > This patch tries to clarify on those details and does some minor changes > as well. > > Note that the bindings do not specify the order in which the regulators > need to be programmed and the order in which the entries are added for > the supplies. > > The user of the bindings (like the kernel) shall know these details > already and the DT is responsible to supply only the readings for the > regulators. > > Cc: Mark Brown > Cc: devicetree@vger.kernel.org > Signed-off-by: Viresh Kumar > Acked-by: Rob Herring > --- Reviewed-by: Stephen Boyd -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project