Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754789AbdGSVHG convert rfc822-to-8bit (ORCPT ); Wed, 19 Jul 2017 17:07:06 -0400 Received: from gloria.sntech.de ([95.129.55.99]:55148 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752172AbdGSVHD (ORCPT ); Wed, 19 Jul 2017 17:07:03 -0400 From: Heiko Stuebner To: Caesar Wang Cc: Rob Herring , rocky.hao@rock-chips.com, Douglas Anderson , devicetree@vger.kernel.org, Brian Norris , David Airlie , Catalin Marinas , Shawn Lin , Elaine Zhang , Will Deacon , Kever Yang , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, Matthias Kaehlcke , Klaus Goger , Mark Rutland , Roger Chen , William wu , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399 Date: Wed, 19 Jul 2017 23:06:53 +0200 Message-ID: <3610458.AeGaZ0pxhP@phil> User-Agent: KMail/5.2.3 (Linux/4.9.0-2-amd64; KDE/5.28.0; x86_64; ; ) In-Reply-To: <65759378-509b-3287-fd78-02361e89429b@rock-chips.com> References: <1500279271-15249-1-git-send-email-wxt@rock-chips.com> <65759378-509b-3287-fd78-02361e89429b@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4539 Lines: 111 Am Mittwoch, 19. Juli 2017, 17:06:59 CEST schrieb Caesar Wang: > Hi Rob & Heiko, > > Do we have the chance to merge these patches? >From the mali-thermal thingy, it doesn't look like Rob likes that IPA stuff. So I'll most likely pick the other patches that rely on approved bindings after I return from my vacation and leave the thermal issue until it can be resolved. Heiko > > I'm try to bring up the display and run webgl for testing with my github on > https://github.com/Caesar-github/rockchip/commits/gru/next-stable-chromeos > > I believe the Rocky@RK will post patches to support the other SoCs after. > > -Caesar > > 在 2017年07月17日 16:14, Caesar Wang 写道: > > This series patches supported the mail in devicetree and used the > > thermal IPA by default. > > > > Verified with kernel is based on Linus's master branch and Heiko's > > v4.14-armsoc-tmp/dts64 branch. ( The Linux version 4.12.0 for now). > > > > The most rockchip SoCs will be supported with IPA mode for thermal > > in later. > > > > --- > > > > History version: > > 1. The first version found on > > https://www.spinics.net/lists/arm-kernel/msg593118.html > > > > Tested on Kevin board with bringing up ChromeOS. > > OS VERSION: > > CHROMEOS_RELEASE_DESCRIPTION=9693.1.0 (Official Build) dev-channel kevin test > > > > BIOS VERSION: > > Google_Kevin.8785.211.2017_06_20_1043 > > > > EC VERSION: > > Build info: kevin_v1.10.217-24514961d 2017-07-03 07:46:36 wxt@nb > > > > With the ARM's lastest mali driver TX011-SW-99002-r18p0-01rel0 on > > https://developer.arm.com/products/software/mali-drivers/midgard-kernel > > > > From the bootup log: > > localhost devfreq0 # dmesg |grep mali > > [ 0.933334] mali ff9a0000.gpu: GPU identified as 0x0860 r2p0 status 0 > > [ 0.940830] mali ff9a0000.gpu: Protected mode not available > > [ 0.947334] mali ff9a0000.gpu: Using configured power model mali-simple-power-model, and fallback mali-simple-power-model > > [ 0.960083] mali ff9a0000.gpu: Probed as mali0 > > > > localhost devfreq0 # pwd > > /sys/devices/platform/ff9a0000.gpu/devfreq/devfreq0 > > localhost devfreq0 # ls > > available_frequencies device min_freq subsystem uevent > > available_governors governor polling_interval target_freq userspace > > cur_freq max_freq power trans_stat > > > > localhost ff9a0000.gpu # ls > > core_availability_policy gpuinfo modalias soft_job_timeout > > core_mask js_scheduling_period of_node subsystem > > devfreq js_timeouts pm_poweroff uevent > > driver mem_pool_max_size power > > driver_override mem_pool_size power_policy > > dvfs_period misc reset_timeout > > --- > > > > And for thermal with IPA. > > Try to run 'md5sum /dev/zero &' and octane/benchmark scripts to go up the temperature. > > From the scripts to have a look at the actual control. > > " > > while true; do grep "" /sys/class/thermal/thermal_zone[0-1]/temp /sys/devices/system/cpu/cpu[0-5]/cpufreq/scaling_cur_freq /sys/devices/platform/ff9a0000.gpu/devfreq/devfreq0/cur_freq;date;sleep .5; done & > > " > > > > -Caesar > > > > > > Changes in v2: > > As Heiko comments on https://patchwork.kernel.org/patch/9835939/ > > - interrupt-name use the lower case. > > - use the correct compatible "arm,mali-t860" > > - remove the clock name, since the mali only have one input clock. > > - foo@ will produce warnings when used without reg property. > > - update the commit to explain the two passive trip points changed. > > > > Caesar Wang (5): > > dt-bindings: gpu: add the RK3399 mali for rockchip specifics > > dt-bindings: gpu: add a power_model optional properties for MALI > > arm64: dts: rockchip: add ARM Mali GPU node for RK3399 SoCs > > arm64: dts: rockchip: enable the GPU for RK3399-GRU > > arm64: dts: rockchip: update the thermal zones for RK3399 SoCs > > > > .../devicetree/bindings/gpu/arm,mali-midgard.txt | 13 ++++ > > arch/arm64/boot/dts/rockchip/rk3399-gru.dtsi | 5 ++ > > arch/arm64/boot/dts/rockchip/rk3399-op1-opp.dtsi | 33 ++++++++++ > > arch/arm64/boot/dts/rockchip/rk3399-opp.dtsi | 33 ++++++++++ > > arch/arm64/boot/dts/rockchip/rk3399.dtsi | 74 ++++++++++++---------- > > 5 files changed, 125 insertions(+), 33 deletions(-) > > > > > >