From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754389AbcIDWyX (ORCPT ); Sun, 4 Sep 2016 18:54:23 -0400 Received: from regular1.263xmail.com ([211.150.99.132]:36474 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753071AbcIDWyU (ORCPT ); Sun, 4 Sep 2016 18:54:20 -0400 X-Greylist: delayed 365 seconds by postgrey-1.27 at vger.kernel.org; Sun, 04 Sep 2016 18:54:20 EDT X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-ADDR-CHECKED: 0 X-RL-SENDER: wxt@rock-chips.com X-FST-TO: heiko@sntech.de X-SENDER-IP: 104.132.1.81 X-LOGIN-NAME: wxt@rock-chips.com X-UNIQUE-TAG: X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Message-ID: <57CCA49D.6060907@rock-chips.com> Date: Mon, 05 Sep 2016 06:47:57 +0800 From: Caesar Wang User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Heiko Stuebner CC: Caesar Wang , mark.rutland@arm.com, huangtao@rock-chips.com, Xing Zheng , devicetree@vger.kernel.org, Marc Zyngier , Catalin Marinas , Brian Norris , Will Deacon , dianders@chromium.org, linux-kernel@vger.kernel.org, Masahiro Yamada , Rob Herring , David Wu , cf@rock-chips.com, linux-rockchip@lists.infradead.org, Jianqun Xu , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 0/2] arm64: dts: rockchip: Support PMU for rk3399 SoCs References: <1467792357-9749-1-git-send-email-wxt@rock-chips.com> <57CA159D.8050103@rock-chips.com> <2592055.1Nh3JWbsOA@phil> In-Reply-To: <2592055.1Nh3JWbsOA@phil> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016年09月05日 06:09, Heiko Stuebner wrote: > Hi Caesar, > > Am Samstag, 3. September 2016, 08:13:17 CEST schrieb Caesar Wang: >> Heiko, >> >> What do you think of it? >> Maybe I need re-update these patches on next kernel, and re-test them. > I checked linux-next and the underlying changes seem to all have gone in now, > so we should be fine. > > Updating the patches would be cool. I tried applying them to my dts64 branch > just now, but they don't apply cleanly anymore due to other changes going in > before. > > When you check against linux-next, please take into account that there is no > new linux-next since 20160825 and 20160905 is the next planned linux-next > release (probably Stephen being on holiday). I will update/resend them base on your v4.9-armsoc/dts64 branch. (Wait a moment to retest them) Thanks. > > > Thanks > Heiko > >> On 2016年07月06日 16:05, Caesar Wang wrote: >>> Hello Heiko, Marc & ARM guys >>> >>> When Jay first submitted the rk3399.dtsi upstream >>> he had the PMU node in >>> there, >>> but then took it out because the upstream binding wasn't done yet. >>> It looks as if the upstream stuff has landed, since in linux/master I see: >>> 287e9357abcc DT/arm,gic-v3: Documment PPI partition support >>> e3825ba1af3a irqchip/gic-v3: Add support for partitioned PPIs >>> 9e2c986cb460 irqchip: Add per-cpu interrupt partitioning library >>> 222df54fd8b7 genirq: Allow the affinity of a percpu interrupt to be >>> set/retrieved 651e8b54abde irqdomain: Allow domain matching on irq_fwspec >>> >>> This series patches add to support the rk3399 SoCs PMU. >>> I pick up the https://patchwork.kernel.org/patch/9209369/. >>> >>> As do some tests with ChromeOs for my rk3399 board. >>> Tested with linus master 4.7-rc6 kernel on rk3399 board. >>> https://github.com/Caesar-github/rockchip/tree/rk3399/pmu-upstream >>> >>> localhost / # perf list >>> >>> List of pre-defined events (to be used in -e): >>> cpu-cycles OR cycles [Hardware event] >>> instructions [Hardware event] >>> cache-references [Hardware event] >>> cache-misses [Hardware event] >>> branch-instructions OR branches [Hardware event] >>> branch-misses [Hardware event] >>> bus-cycles [Hardware event] >>> ... >>> >>> perf stat --cpu 0/1/2/3..... to minitor >>> e.g. cpu0; >>> >>> localhost / # perf stat --cpu 0 >>> >>> Performance counter stats for 'CPU(s) 0': >>> >>> 3374.917571 task-clock (msec) # 1.001 CPUs utilized [100.00%] >>> 20 context-switches # 0.006 K/sec [100.00%] >>> 2 cpu-migrations # 0.001 K/sec [100.00%] >>> 55 page-faults # 0.016 K/sec >>> 7151843 cycles # 0.002 GHz [100.00%] >>> stalled-cycles-frontend >>> stalled-cycles-backend >>> 4272536 instructions # 0.60 insns per cycle [100.00%] >>> 568406 branches # 0.168 M/sec [100.00%] >>> 65652 branch-misses # 11.55% of all branches >>> >>> Also, 'perf top' to monitor the PMU interrupts from cpus >>> >>> -Caesar >>> >>> >>> Changes in v2: >>> - AS Mark comments on https://patchwork.kernel.org/patch/9209369/ >>> >>> remove the interrupt-affinity property, we need depend on Marc' perf >>> code on https://patchwork.kernel.org/patch/9209369/. >>> >>> Caesar Wang (2): >>> arm64: dts: rockchip: change all interrupts cells for 4 on rk3399 SoCs >>> arm64: dts: rockchip: support the pmu node for rk3399 >>> >>> arch/arm64/boot/dts/rockchip/rk3399.dtsi | 118 >>> ++++++++++++++++++------------- 1 file changed, 69 insertions(+), 49 >>> deletions(-) > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip -- caesar wang | software engineer | wxt@rock-chip.com