From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754649AbdJRNh7 (ORCPT ); Wed, 18 Oct 2017 09:37:59 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:8945 "EHLO szxga04-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754777AbdJRNh4 (ORCPT ); Wed, 18 Oct 2017 09:37:56 -0400 Subject: Re: [PATCH v5 5/6] perf: hisi: Add support for HiSilicon SoC DDRC PMU driver To: Mark Rutland References: <1503389277-134131-1-git-send-email-zhangshaokun@hisilicon.com> <1503389277-134131-6-git-send-email-zhangshaokun@hisilicon.com> <20171017152109.cuslnq3jsp6kx4au@lakrids.cambridge.arm.com> CC: , , , , , , Anurup M From: Zhangshaokun Message-ID: <1ce6d265-44cd-abc9-e32e-74a3afa0f162@hisilicon.com> Date: Wed, 18 Oct 2017 21:37:40 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <20171017152109.cuslnq3jsp6kx4au@lakrids.cambridge.arm.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.74.221.148] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A090204.59E7592E.0031,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 73531a6c19bd8bfc966327c2f033ba15 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, On 2017/10/17 23:21, Mark Rutland wrote: > On Tue, Aug 22, 2017 at 04:07:56PM +0800, Shaokun Zhang wrote: >> This patch adds support for DDRC PMU driver in HiSilicon SoC chip, Each >> DDRC has own control, counter and interrupt registers and is an separate >> PMU. For each DDRC PMU, it has 8-fixed-purpose counters which have been >> mapped to 8-events by hardware, it assumes that counter index is equal >> to event code (0 - 7) in DDRC PMU driver. Interrupt is supported to >> handle counter (32-bits) overflow. >> >> Reviewed-by: Jonathan Cameron >> Signed-off-by: Shaokun Zhang >> Signed-off-by: Anurup M > > I have the same comments for this case as for the other two PMU drivers. > Sure. Thanks, Shaokun > Thanks, > Mark. > > . >