Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH 0/3] PM / devfreq: Add devfreq_frequency tracepoint to
 track frequency change
To:     linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     mka@chromium.org, rostedt@goodmis.org, mingo@redhat.com,
        chanwoo@kernel.org, myungjoo.ham@samsung.com,
        kyungmin.park@samsung.com
From:   Chanwoo Choi <cw00.choi@samsung.com>
Organization: Samsung Electronics
Message-ID: <1763ff8e-e7c7-af36-13b2-1794d109f206@samsung.com>
Date:   Mon, 19 Oct 2020 20:04:16 +0900
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101
        Thunderbird/59.0
MIME-Version: 1.0
In-Reply-To: <20201008075403.26181-1-cw00.choi@samsung.com>
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 101P
DLP-Filter: Pass
References: <CGME20201008074041epcas1p14111dc3070c66fce8d775e2fbae39f15@epcas1p1.samsung.com>
        <20201008075403.26181-1-cw00.choi@samsung.com>
Precedence: bulk

On 10/8/20 4:54 PM, Chanwoo Choi wrote:
> Add devfreq_tracepoint to track the correct timing of frequency change
> with following information:
> - device name
> - current frequency
> - previous frequency
> - load when change frequency
> - tracepoint path : /sys/kernel/debug/tracing/events/devfreq_frequency
> 
> And add devfreq_update_target() function to unify the frequency change code
> on both devfreq core and devfreq passive governor because there are redundant
> duplicate code. Lastly, Use fixed indentation size to improve readability
> for 'devfreq_monitor' tracepoint.
> 
> Matthias already sent the patch[1]. Make patch3 by editing patch[1].
> [1]https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg2108015.html
> 
> Chanwoo Choi (2):
>   trace: events: devfreq: Use fixed indentation size to improve readability
>   PM / devfreq: Unify frequency change to devfreq_update_target func
> 
> Matthias Kaehlcke (1):
>   PM / devfreq: Add tracepoint for frequency changes
> 
>  drivers/devfreq/devfreq.c          | 37 +++++++++++++++++++++-----
>  drivers/devfreq/governor.h         |  1 +
>  drivers/devfreq/governor_passive.c | 42 +++++++-----------------------
>  include/trace/events/devfreq.h     | 30 ++++++++++++++++++++-
>  4 files changed, 70 insertions(+), 40 deletions(-)
> 

Applied them. Thanks.

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics