From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 115E7C67871 for ; Tue, 10 Jan 2023 08:47:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Okst2HqM2SuWB9IuouAKP+W27V2PCLKyYPV30JFyiys=; b=4wVPw0Ypyp0TE3 ZRVOnMGiLykfgmgdfgSjeJIa6IdctCChfuLYunhIyRZPJknfMXsIgJ6wpKAJNIGxIXKsLEBcnDolR MJmsHV0lmYitQMiRyquNfbB4YMmM9DhmGJ4Q2lqCjGQqOCZ/d02tweMeoZ+5fYI67FJm151Q0nvBs aHa/cAtJNWFYVMpYujyEP0lUywEmyf22jJ/HydiMC3UIbPAopQ8q3vV11k5lzzPwOvkedX6foVEEh r9QEB54+zRWaMaV/b2B1oOUgVYwS4o5wwtnTQqGWOdf8AHh1ufockdzXL7hCFIV9M+i+35FZ6X0jS WiBzG6v6BKcRKB6t6tLA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFAGt-005tDi-Rq; Tue, 10 Jan 2023 08:46:15 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFAGq-005tCG-S1 for linux-arm-kernel@lists.infradead.org; Tue, 10 Jan 2023 08:46:14 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BA62FAD7; Tue, 10 Jan 2023 00:46:52 -0800 (PST) Received: from [10.162.42.6] (unknown [10.162.42.6]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B47043F587; Tue, 10 Jan 2023 00:46:04 -0800 (PST) Message-ID: Date: Tue, 10 Jan 2023 14:16:01 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: [PATCH v4 5/8] perf: arm_spe: Use new PMSIDR_EL1 register enums Content-Language: en-US To: Rob Herring , Peter Zijlstra , Will Deacon , Mark Rutland , Catalin Marinas , Marc Zyngier , James Morse , Alexandru Elisei , Suzuki K Poulose , Oliver Upton , Ingo Molnar , Arnaldo Carvalho de Melo , Alexander Shishkin , Jiri Olsa , Namhyung Kim Cc: Mark Brown , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, kvmarm@lists.linux.dev, linux-perf-users@vger.kernel.org, James Clark References: <20220825-arm-spe-v8-7-v4-0-327f860daf28@kernel.org> <20220825-arm-spe-v8-7-v4-5-327f860daf28@kernel.org> From: Anshuman Khandual In-Reply-To: <20220825-arm-spe-v8-7-v4-5-327f860daf28@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230110_004613_005510_E6904841 X-CRM114-Status: GOOD ( 16.24 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 1/10/23 00:56, Rob Herring wrote: > Now that the SPE register definitions include enums for some PMSIDR_EL1 > fields, use them in the driver in place of magic values. > > Signed-off-by: Rob Herring Reviewed-by: Anshuman Khandual > --- > v4: > - Rebase on v6.2-rc1 > v3: New patch > --- > drivers/perf/arm_spe_pmu.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/drivers/perf/arm_spe_pmu.c b/drivers/perf/arm_spe_pmu.c > index 9b4bd72087ea..af6d3867c3e7 100644 > --- a/drivers/perf/arm_spe_pmu.c > +++ b/drivers/perf/arm_spe_pmu.c > @@ -1006,32 +1006,32 @@ static void __arm_spe_pmu_dev_probe(void *info) > /* This field has a spaced out encoding, so just use a look-up */ > fld = FIELD_GET(PMSIDR_EL1_INTERVAL, reg); > switch (fld) { > - case 0: > + case PMSIDR_EL1_INTERVAL_256: > spe_pmu->min_period = 256; > break; > - case 2: > + case PMSIDR_EL1_INTERVAL_512: > spe_pmu->min_period = 512; > break; > - case 3: > + case PMSIDR_EL1_INTERVAL_768: > spe_pmu->min_period = 768; > break; > - case 4: > + case PMSIDR_EL1_INTERVAL_1024: > spe_pmu->min_period = 1024; > break; > - case 5: > + case PMSIDR_EL1_INTERVAL_1536: > spe_pmu->min_period = 1536; > break; > - case 6: > + case PMSIDR_EL1_INTERVAL_2048: > spe_pmu->min_period = 2048; > break; > - case 7: > + case PMSIDR_EL1_INTERVAL_3072: > spe_pmu->min_period = 3072; > break; > default: > dev_warn(dev, "unknown PMSIDR_EL1.Interval [%d]; assuming 8\n", > fld); > fallthrough; > - case 8: > + case PMSIDR_EL1_INTERVAL_4096: > spe_pmu->min_period = 4096; > } > > @@ -1050,10 +1050,10 @@ static void __arm_spe_pmu_dev_probe(void *info) > dev_warn(dev, "unknown PMSIDR_EL1.CountSize [%d]; assuming 2\n", > fld); > fallthrough; > - case 2: > + case PMSIDR_EL1_COUNTSIZE_12_BIT_SAT: > spe_pmu->counter_sz = 12; > break; > - case 3: > + case PMSIDR_EL1_COUNTSIZE_16_BIT_SAT: > spe_pmu->counter_sz = 16; > } > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel