From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srinivas Kandagatla Subject: Re: [PATCH v4 1/3] bus: simple-pm: add support to pm clocks Date: Wed, 16 Nov 2016 16:33:06 +0000 Message-ID: <5c02f6bc-59dc-2c5e-d8e5-0d2e2d1d0fce@linaro.org> References: <1479122155-13393-1-git-send-email-srinivas.kandagatla@linaro.org> <1479122155-13393-2-git-send-email-srinivas.kandagatla@linaro.org> <20161114221447.GH9868@bhelgaas-glaptop.roam.corp.google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org To: "Nayak, Rajendra" , Geert Uytterhoeven , Bjorn Helgaas Cc: svarbanov@mm-sol.com, linux-pci , Bjorn Helgaas , Rob Herring , "linux-arm-msm@vger.kernel.org" , "devicetree@vger.kernel.org" , Geert Uytterhoeven , Kevin Hilman , Simon Horman , Linux PM list , "Nayak, Rajendra" List-Id: devicetree@vger.kernel.org Thanks Rajendra for pointing to the patch On 16/11/16 15:50, Nayak, Rajendra wrote: >> >> >> Rajendra, >> Looks like qcom gdsc pm domain provider driver does not handle bus clks >> along with power-domain, Is this something we should do? Or the bus >> driver take care of it? > > I did post some patches to support handling of clocks associated with > gdscs [1], but it got dropped at that point since there wasn't a > real user, besides there were some open issues wrt the handling of > !CONFIG_PM cases etc. > I will revive and repost those patches again now based on the > discussions last time around. > > [1] > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-August/362492.html This patch looks exactly like the one useful in pcie case, i would be interesting to see the final patch on how we handle clocks which are both related and not related to power domain. thanks, srini >