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 X-Spam-Level: X-Spam-Status: No, score=-14.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C8FFBC433FE for ; Wed, 22 Sep 2021 08:05:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B376C6126A for ; Wed, 22 Sep 2021 08:05:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233491AbhIVIHW (ORCPT ); Wed, 22 Sep 2021 04:07:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51206 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233336AbhIVIHU (ORCPT ); Wed, 22 Sep 2021 04:07:20 -0400 Received: from mail-pg1-x531.google.com (mail-pg1-x531.google.com [IPv6:2607:f8b0:4864:20::531]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4DDA0C061574; Wed, 22 Sep 2021 01:05:51 -0700 (PDT) Received: by mail-pg1-x531.google.com with SMTP id f129so1875358pgc.1; Wed, 22 Sep 2021 01:05:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=3vgj0hW3KSHhJ1FNGvvnjMW3NitG36aDXcJz/7heGcA=; b=FAkBVWugynwfJI5lWaQvYEy5zoZ/PcwxIzBamJjPUP+b2SJ39zuwELnOxZD4mWY7qI VL+NRFb28mlUzCzMNFU8j2NXe+OzTklHCFA1mSE3MXf4GK+Sx65lA1keQKFwGlsrg9tU tRonpRLFulMJRQa9KRODt+Cp+UQlzyTtiUHLtMgCsrKSVqTJbk4CqpICjwqwgiMymCpa zVNq1/Yg3KD7mrkYF4M7le9Q4d3Gn8986+7TMsBcQHgo7Q+GySY6qjBPrwQu5mqiJ3iI EdQMGVC+gVV9oJFAEklO3sPTpktjuuiE5r4+kATExB81e4C/gPw5qi069u7USlKJNEiM KluQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=3vgj0hW3KSHhJ1FNGvvnjMW3NitG36aDXcJz/7heGcA=; b=e8HNrhaTTjfcmwLQai4PLyHPYmWL+nJCRjvEgVrZJ69awBa+Cd/HIaZ16ej5bdrxmy ly3TF8SlPPOii48+fffczdK8Gnak98BlFm2hiRRaO2XUaDvcXpsiVaakaf7iFgplaQc4 q8B1fYFPVFHMx+5YCxQ7/FzKJf609WeSJN9suyrrECibDsdZhXrl83SMiaLM3A5Q5usn OK472jpHgMhFyd8Bu2SUIuG+zt63Zc7oV9xi94/2ocMZXMxDSYstR0LsSE16UyQfMXi5 NYfA33KqGkE1IjFxLDA8s0d6tnjC2pZTyZzSJ9VrJf3oiE8Iu6QmR8Ju+ygWXoEFsPmf a1YQ== X-Gm-Message-State: AOAM532AsEo0jmffvcWWnQhAEiB2z/65MQJLkOSilnrdxzGAx1q9aSdV mtEkO3Df2JcCGVUlBaU5WoQ= X-Google-Smtp-Source: ABdhPJwdylx2gAkFtmU9FHajYgZBLBBYYbPpvbp7sGfAL2p7//GTWekKTYSd4s/OpU5ahmIuNorN4g== X-Received: by 2002:a62:7b14:0:b0:447:c4c9:ce3f with SMTP id w20-20020a627b14000000b00447c4c9ce3fmr12556780pfc.15.1632297950610; Wed, 22 Sep 2021 01:05:50 -0700 (PDT) Received: from [172.30.1.2] ([14.32.163.5]) by smtp.gmail.com with ESMTPSA id c23sm1599245pgb.74.2021.09.22.01.05.45 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 22 Sep 2021 01:05:50 -0700 (PDT) Subject: Re: [PATCH v2 2/3] devfreq: exynos-ppmu: simplify parsing event-type from DT To: Krzysztof Kozlowski , Rob Herring , Chanwoo Choi , MyungJoo Ham , Kyungmin Park , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Cc: Marek Szyprowski , Sylwester Nawrocki , Bartlomiej Zolnierkiewicz References: <20210920071540.38337-1-krzysztof.kozlowski@canonical.com> <20210920071753.38560-2-krzysztof.kozlowski@canonical.com> From: Chanwoo Choi Message-ID: <35ad437f-7527-e9b1-4819-8f4faa765e26@gmail.com> Date: Wed, 22 Sep 2021 17:05:45 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20210920071753.38560-2-krzysztof.kozlowski@canonical.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krzysztof, On 21. 9. 20. 오후 4:17, Krzysztof Kozlowski wrote: > When parsing devicetree, the function of_get_devfreq_events(), for each > device child node, iterates over array of possible events "ppmu_events" > till it finds one matching by node name. When match is found the > ppmu_events[i] points to element having both the name of the event and > the counters ID. > > Each PPMU device child node might have an "event-name" property with the > name of the event, however due to the design of devfreq it must be the > same as the device node name. If it is not the same, the devfreq client > won't be able to use it via devfreq_event_get_edev_by_phandle(). > > Since PPMU device child node name must be equal to the "event-name" > property (event-name == ppmu_events[i].name), there is no need to find > the counters ID by the "event-name". Instead use ppmu_events[i].id > which must be equal to it. > > Signed-off-by: Krzysztof Kozlowski > --- > drivers/devfreq/event/exynos-ppmu.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/devfreq/event/exynos-ppmu.c b/drivers/devfreq/event/exynos-ppmu.c > index 541bd13ab61d..9b849d781116 100644 > --- a/drivers/devfreq/event/exynos-ppmu.c > +++ b/drivers/devfreq/event/exynos-ppmu.c > @@ -566,13 +566,10 @@ static int of_get_devfreq_events(struct device_node *np, > * use default if not. > */ > if (info->ppmu_type == EXYNOS_TYPE_PPMU_V2) { > - int id; > /* Not all registers take the same value for > * read+write data count. > */ > - id = __exynos_ppmu_find_ppmu_id(desc[j].name); > - > - switch (id) { > + switch (ppmu_events[i].id) { > case PPMU_PMNCNT0: > case PPMU_PMNCNT1: > case PPMU_PMNCNT2: > Applied it. Thanks. -- Best Regards, Samsung Electronics Chanwoo Choi 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 X-Spam-Level: X-Spam-Status: No, score=-12.2 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CC405C433EF for ; Wed, 22 Sep 2021 08:08:07 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 9249B61209 for ; Wed, 22 Sep 2021 08:08:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 9249B61209 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=BoP+VCY/OP2M/vWp6SIOkX3b+iF2NxkTyHESNZvzQ+c=; b=hhjls03fvxUYpu4iF6bqu6lkQq 0ANd5e4hySNdHCe/hkiz3tJIS9EDPqEBo5axcNqlBbWaIOnTckB+aC7BwCiVr5zCwfdHk/uMcbC3m +SF5RgenZ48AnQmEHF0pmFYyblaYBj78+qreHJd1fPMtNBrPhEgSQSWkANrss8vN+pHZ6sudEwzqb nF30PgACbGU6hGUzFeb78A2r0lA2dxnQvOKUqH/96T1fmdzAS6IPQ7T13PvgpsrwrLyanS++JMp+/ Vhw2PjWR+225IuH9vqT29hyMjuTu+e7ScX9JZhtFcOkfZq8cw/4PZzmq6QZcTWew770rOWzokwTqX C2JStFKw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mSxGN-007Oz7-RI; Wed, 22 Sep 2021 08:05:56 +0000 Received: from mail-pf1-x42d.google.com ([2607:f8b0:4864:20::42d]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mSxGJ-007OyQ-Vg for linux-arm-kernel@lists.infradead.org; Wed, 22 Sep 2021 08:05:53 +0000 Received: by mail-pf1-x42d.google.com with SMTP id k17so2032192pff.8 for ; Wed, 22 Sep 2021 01:05:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=3vgj0hW3KSHhJ1FNGvvnjMW3NitG36aDXcJz/7heGcA=; b=FAkBVWugynwfJI5lWaQvYEy5zoZ/PcwxIzBamJjPUP+b2SJ39zuwELnOxZD4mWY7qI VL+NRFb28mlUzCzMNFU8j2NXe+OzTklHCFA1mSE3MXf4GK+Sx65lA1keQKFwGlsrg9tU tRonpRLFulMJRQa9KRODt+Cp+UQlzyTtiUHLtMgCsrKSVqTJbk4CqpICjwqwgiMymCpa zVNq1/Yg3KD7mrkYF4M7le9Q4d3Gn8986+7TMsBcQHgo7Q+GySY6qjBPrwQu5mqiJ3iI EdQMGVC+gVV9oJFAEklO3sPTpktjuuiE5r4+kATExB81e4C/gPw5qi069u7USlKJNEiM KluQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=3vgj0hW3KSHhJ1FNGvvnjMW3NitG36aDXcJz/7heGcA=; b=dRN4VJDAqjQqcuvZ4wo8eHxGDDhfT780TYWcyqqVbXRtgSL++s9n8NTZSgXOSvL9zt E+EQRw+Naxqkai1Mt9VoX5kp909dKyP38GwzVUCbFWIJNhkuhjen0jpeq2tjK9EpOlhE jdJO9LICAtewcqqFquAvwVXPZ6o+5j9ELD2sioUUQGCqORc9WQxPbbqFS7Gy0+jrtdRQ +RpSGTksDglTZfDu7LJB8+fj6aeb+nKmn/XJPM47M9nYB2OHIX19xoVx7iYmnxmc1UUp z9RBr3BfCDNxV0c3fPSNGlVc9f0PkR4aED3c+kRExHJNlrulmVUQ8/BLsKUSTiAE+pfF WeSg== X-Gm-Message-State: AOAM531c8E+rCmSaiHr5Dp50AZWGbIz6gzIgoEk6xQqUw3JC9aKcCJtI sM/vXeJ5CIfzsoxajH8BlrDv6ITTZ0s= X-Google-Smtp-Source: ABdhPJwdylx2gAkFtmU9FHajYgZBLBBYYbPpvbp7sGfAL2p7//GTWekKTYSd4s/OpU5ahmIuNorN4g== X-Received: by 2002:a62:7b14:0:b0:447:c4c9:ce3f with SMTP id w20-20020a627b14000000b00447c4c9ce3fmr12556780pfc.15.1632297950610; Wed, 22 Sep 2021 01:05:50 -0700 (PDT) Received: from [172.30.1.2] ([14.32.163.5]) by smtp.gmail.com with ESMTPSA id c23sm1599245pgb.74.2021.09.22.01.05.45 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 22 Sep 2021 01:05:50 -0700 (PDT) Subject: Re: [PATCH v2 2/3] devfreq: exynos-ppmu: simplify parsing event-type from DT To: Krzysztof Kozlowski , Rob Herring , Chanwoo Choi , MyungJoo Ham , Kyungmin Park , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Cc: Marek Szyprowski , Sylwester Nawrocki , Bartlomiej Zolnierkiewicz References: <20210920071540.38337-1-krzysztof.kozlowski@canonical.com> <20210920071753.38560-2-krzysztof.kozlowski@canonical.com> From: Chanwoo Choi Message-ID: <35ad437f-7527-e9b1-4819-8f4faa765e26@gmail.com> Date: Wed, 22 Sep 2021 17:05:45 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20210920071753.38560-2-krzysztof.kozlowski@canonical.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210922_010552_074356_F1F467E8 X-CRM114-Status: GOOD ( 27.27 ) 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-Transfer-Encoding: base64 Content-Type: text/plain; charset="utf-8"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGkgS3J6eXN6dG9mLAoKT24gMjEuIDkuIDIwLiDsmKTtm4QgNDoxNywgS3J6eXN6dG9mIEtvemxv d3NraSB3cm90ZToKPiBXaGVuIHBhcnNpbmcgZGV2aWNldHJlZSwgdGhlIGZ1bmN0aW9uIG9mX2dl dF9kZXZmcmVxX2V2ZW50cygpLCBmb3IgZWFjaAo+IGRldmljZSBjaGlsZCBub2RlLCBpdGVyYXRl cyBvdmVyIGFycmF5IG9mIHBvc3NpYmxlIGV2ZW50cyAicHBtdV9ldmVudHMiCj4gdGlsbCBpdCBm aW5kcyBvbmUgbWF0Y2hpbmcgYnkgbm9kZSBuYW1lLiAgV2hlbiBtYXRjaCBpcyBmb3VuZCB0aGUK PiBwcG11X2V2ZW50c1tpXSBwb2ludHMgdG8gZWxlbWVudCBoYXZpbmcgYm90aCB0aGUgbmFtZSBv ZiB0aGUgZXZlbnQgYW5kCj4gdGhlIGNvdW50ZXJzIElELgo+IAo+IEVhY2ggUFBNVSBkZXZpY2Ug Y2hpbGQgbm9kZSBtaWdodCBoYXZlIGFuICJldmVudC1uYW1lIiBwcm9wZXJ0eSB3aXRoIHRoZQo+ IG5hbWUgb2YgdGhlIGV2ZW50LCBob3dldmVyIGR1ZSB0byB0aGUgZGVzaWduIG9mIGRldmZyZXEg aXQgbXVzdCBiZSB0aGUKPiBzYW1lIGFzIHRoZSBkZXZpY2Ugbm9kZSBuYW1lLiAgSWYgaXQgaXMg bm90IHRoZSBzYW1lLCB0aGUgZGV2ZnJlcSBjbGllbnQKPiB3b24ndCBiZSBhYmxlIHRvIHVzZSBp dCB2aWEgZGV2ZnJlcV9ldmVudF9nZXRfZWRldl9ieV9waGFuZGxlKCkuCj4gCj4gU2luY2UgUFBN VSBkZXZpY2UgY2hpbGQgbm9kZSBuYW1lIG11c3QgYmUgZXF1YWwgdG8gdGhlICJldmVudC1uYW1l Igo+IHByb3BlcnR5IChldmVudC1uYW1lID09IHBwbXVfZXZlbnRzW2ldLm5hbWUpLCB0aGVyZSBp cyBubyBuZWVkIHRvIGZpbmQKPiB0aGUgY291bnRlcnMgSUQgYnkgdGhlICJldmVudC1uYW1lIi4g IEluc3RlYWQgdXNlIHBwbXVfZXZlbnRzW2ldLmlkCj4gd2hpY2ggbXVzdCBiZSBlcXVhbCB0byBp dC4KPiAKPiBTaWduZWQtb2ZmLWJ5OiBLcnp5c3p0b2YgS296bG93c2tpIDxrcnp5c3p0b2Yua296 bG93c2tpQGNhbm9uaWNhbC5jb20+Cj4gLS0tCj4gICBkcml2ZXJzL2RldmZyZXEvZXZlbnQvZXh5 bm9zLXBwbXUuYyB8IDUgKy0tLS0KPiAgIDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwg NCBkZWxldGlvbnMoLSkKPiAKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9kZXZmcmVxL2V2ZW50L2V4 eW5vcy1wcG11LmMgYi9kcml2ZXJzL2RldmZyZXEvZXZlbnQvZXh5bm9zLXBwbXUuYwo+IGluZGV4 IDU0MWJkMTNhYjYxZC4uOWI4NDlkNzgxMTE2IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZGV2ZnJl cS9ldmVudC9leHlub3MtcHBtdS5jCj4gKysrIGIvZHJpdmVycy9kZXZmcmVxL2V2ZW50L2V4eW5v cy1wcG11LmMKPiBAQCAtNTY2LDEzICs1NjYsMTAgQEAgc3RhdGljIGludCBvZl9nZXRfZGV2ZnJl cV9ldmVudHMoc3RydWN0IGRldmljZV9ub2RlICpucCwKPiAgIAkJCSAqIHVzZSBkZWZhdWx0IGlm IG5vdC4KPiAgIAkJCSAqLwo+ICAgCQkJaWYgKGluZm8tPnBwbXVfdHlwZSA9PSBFWFlOT1NfVFlQ RV9QUE1VX1YyKSB7Cj4gLQkJCQlpbnQgaWQ7Cj4gICAJCQkJLyogTm90IGFsbCByZWdpc3RlcnMg dGFrZSB0aGUgc2FtZSB2YWx1ZSBmb3IKPiAgIAkJCQkgKiByZWFkK3dyaXRlIGRhdGEgY291bnQu Cj4gICAJCQkJICovCj4gLQkJCQlpZCA9IF9fZXh5bm9zX3BwbXVfZmluZF9wcG11X2lkKGRlc2Nb al0ubmFtZSk7Cj4gLQo+IC0JCQkJc3dpdGNoIChpZCkgewo+ICsJCQkJc3dpdGNoIChwcG11X2V2 ZW50c1tpXS5pZCkgewo+ICAgCQkJCWNhc2UgUFBNVV9QTU5DTlQwOgo+ICAgCQkJCWNhc2UgUFBN VV9QTU5DTlQxOgo+ICAgCQkJCWNhc2UgUFBNVV9QTU5DTlQyOgo+IAoKQXBwbGllZCBpdC4gVGhh bmtzLgoKCi0tIApCZXN0IFJlZ2FyZHMsClNhbXN1bmcgRWxlY3Ryb25pY3MKQ2hhbndvbyBDaG9p CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1h cm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5v cmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0t a2VybmVsCg==