From mboxrd@z Thu Jan 1 00:00:00 1970 From: Saravana Kannan Subject: Re: [PATCH v4 0/3] Introduce Bandwidth OPPs for interconnects Date: Mon, 29 Jul 2019 22:53:12 -0700 Message-ID: References: <20190726231558.175130-1-saravanak@google.com> <20190729093545.kvnqxjkyx4nogddk@vireshk-i7> <20190730024640.xk27jgdfl2j6ucx7@vireshk-i7> <361effba-4433-24d9-243c-201af39214cc@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <361effba-4433-24d9-243c-201af39214cc@codeaurora.org> Sender: linux-kernel-owner@vger.kernel.org To: Sibi Sankar Cc: Viresh Kumar , Georgi Djakov , Rob Herring , Mark Rutland , Viresh Kumar , Nishanth Menon , Stephen Boyd , "Rafael J. Wysocki" , Vincent Guittot , "Sweeney, Sean" , David Dai , adharmap@codeaurora.org, Rajendra Nayak , Bjorn Andersson , Evan Green , Android Kernel Team , Linux PM , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , LKML List-Id: devicetree@vger.kernel.org On Mon, Jul 29, 2019 at 10:28 PM Sibi Sankar wrote: > > Hey Viresh, > > On 7/30/19 8:16 AM, Viresh Kumar wrote: > > On 29-07-19, 13:16, Saravana Kannan wrote: > >> Sibi might be working on doing that for the SDM845 CPUfreq driver. > >> Georgi could also change his GPU driver use case to use this BW OPP > >> table and required-opps. > >> > >> The problem is that people don't want to start using this until we > >> decide on the DT representation. So it's like a chicken and egg > >> situation. > > > > Yeah, I agree to that. > > > > @Georgi and @Sibi: This is your chance to speak up about the proposal > > from Saravana and if you find anything wrong with them. And specially > > that it is mostly about interconnects here, I would like to have an > > explicit Ack from Georgi on this. > > > > And if you guys are all okay about this then please at least commit > > that you will convert your stuff based on this in coming days. > > I've been using both Saravana's and Georgi's series for a while > now to scale DDR and L3 on SDM845. There is currently no consensus > as to where the votes are to be actuated from, hence couldn't post > anything out. > > DCVS based on Saravana's series + passive governor: > https://github.com/QuinAsura/linux/tree/lnext-072619-SK-series Thanks Sibi! You might want to convert your patches so that until the passive governor is ready, you just look up the required opps and vote for BW directly from the cpufreq driver. Once devfreq governor is ready, you can switch to it. -Saravana > > DCVS based on Georgi's series: (I had already posted this out) > https://github.com/QuinAsura/linux/tree/lnext-072619-GJ-series > > -- > Qualcomm Innovation Center, Inc. > Qualcomm Innovation Center, Inc, is a member of Code Aurora Forum, > a Linux Foundation Collaborative Project