From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Kaehlcke Subject: Re: [PATCH v3 16/16] arm64: dts: sdm845: enable tsens thermal zones Date: Thu, 25 Oct 2018 13:37:52 -0700 Message-ID: <20181025203752.GA22824@google.com> References: <5e2d5fc463a5f5623c4fc1b18ccdf62adc954758.1536744310.git.amit.kucheria@linaro.org> <20181025192501.GZ22824@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Amit Kucheria Cc: Linux Kernel Mailing List , Rajendra Nayak , linux-arm-msm , Bjorn Andersson , Eduardo Valentin , smohanad@codeaurora.org, Andy Gross , Doug Anderson , David Brown , Rob Herring , Mark Rutland , "open list:ARM/QUALCOMM SUPPORT" , DTML List-Id: devicetree@vger.kernel.org On Fri, Oct 26, 2018 at 01:33:40AM +0530, Amit Kucheria wrote: > On Fri, Oct 26, 2018 at 12:55 AM Matthias Kaehlcke wrote: > > > > Hi Amit, > > > > vaguely related question, since you are working on SDM845 thermal > > stuff: Do you have plans to add CPU cooling devices? I raised the > > point during the review of the CPUfreq HW, however there was no > > clear answer: https://lore.kernel.org/patchwork/patch/966833/#1164748 > > The driver is still under development/review, but I wonder what the > > path forward is. > > > > Hi Matthias, > > I plan to Great! > but the OSM driver doesn't work well enough for me to send > out that patch yet. I responded with my findings on the v9 posting > where it seems to get stuck at the high OPP. It'd be nice if somebody > else could confirm (or refute) this behaviour. Ok, we aren't using the latest version yet, I'll try to repro and report back. Cheers Matthias