From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sibi Sankar Subject: Re: [PATCH v4 0/3] Introduce Bandwidth OPPs for interconnects Date: Tue, 30 Jul 2019 22:13:34 +0530 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; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Saravana Kannan 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 7/30/19 11:23 AM, Saravana Kannan wrote: > 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. Sure I'll do that. > > -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 -- Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc, is a member of Code Aurora Forum, a Linux Foundation Collaborative Project