CIP-dev Archive on lore.kernel.org
 help / color / Atom feed
From: "Nobuhiro Iwamatsu" <nobuhiro1.iwamatsu@toshiba.co.jp>
To: <wens@csie.org>, <pavel@denx.de>
Cc: <cip-dev@lists.cip-project.org>, <JohnsonCH.Chen@moxa.com>
Subject: Re: [cip-dev] [PATCH RESEND 4.4.y-cip 00/15] PM / OPP v2 & cpufreq backports part 1
Date: Thu, 4 Jun 2020 06:55:38 +0000
Message-ID: <OSBPR01MB29831550F3393A10777EF21D92890@OSBPR01MB2983.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <20200604041745.28886-1-wens@csie.org>


[-- Attachment #1: Type: text/plain, Size: 4370 bytes --]

Hi Wens,

Thanks for your patch!

> -----Original Message-----
> From: Chen-Yu Tsai (Moxa) [mailto:wens@csie.org]
> Sent: Thursday, June 4, 2020 1:18 PM
> To: iwamatsu nobuhiro(岩松 信洋 □SWC◯ACT) <nobuhiro1.iwamatsu@toshiba.co.jp>; pavel@denx.de
> Cc: cip-dev@lists.cip-project.org; JohnsonCH.Chen@moxa.com
> Subject: [PATCH RESEND 4.4.y-cip 00/15] PM / OPP v2 & cpufreq backports part 1
> 
> Hi everyone,
> 
> This is part 1 of MOXA's PM / OPP / cpufreq backport series. The whole
> series aims to backport patches related to PM / OPPv2 and cpufreq
> which were included in the v4.4 kernel from TI's SDK. The end goal is
> to include cpufreq-ti and convert am33xx to using it and OPPv2.
> 
> Part 1 here includes patches from the v4.5 cycle, listed in topological
> order:
> 
>     deaa51465105 PM / OPP: Add debugfs support
>     1c4d12de2719 PM / OPP: Add "opp-supported-hw" binding
>     ffdb8cc7a27c PM / OPP: Add {opp-microvolt|opp-microamp}-<name> binding
>     af87a39a5f7c PM / OPP: Remove 'operating-points-names' binding
>     754dcf35f346 PM / OPP: Rename OPP nodes as opp@<opp-hz>
>     dc4e7b1fa20a PM / OPP: Add missing doc comments
>     7de36b0aa51a PM / OPP: Parse 'opp-supported-hw' binding
>     01fb4d3c39d3 PM / OPP: Parse 'opp-<prop>-<name>' bindings
>     fd8d8e63467c PM / OPP: Fix parsing of opp-microvolt and opp-microamp properties
>     d9de19b1cc01 PM / OPP: Set cpu_dev->id in cpumask first
>     5ff24d601092 PM / OPP: Use snprintf() instead of sprintf()
>     3be3f8f36e73 devicetree: bindings: Add optional dynamic-power-coefficient property
>     f8fa8ae06b8c cpufreq-dt: Supply power coefficient when registering cooling devices
>     929ca89c305a cpufreq-dt: fix handling regulator_get_voltage() result
>     b331bc20d928 cpufreq: cpufreq-dt: avoid uninitialized variable warnings:
> 
> Of these,
> 
>     dc4e7b1fa20a PM / OPP: Add missing doc comments
> 
> was identified as a patch dependency, but otherwise serves no purpose. And
> 
>     d9de19b1cc01 PM / OPP: Set cpu_dev->id in cpumask first
> 
> was found while looking through git logs. Last,
> 
>     3be3f8f36e73 devicetree: bindings: Add optional dynamic-power-coefficient property
> 
> is related to
> 
>     f8fa8ae06b8c cpufreq-dt: Supply power coefficient when registering cooling devices
> 
> All other patches were included from TI's SDK.
> 
> The patches apply cleanly on top of linux-4.4.y-cip. Please have a look.
> 

I have nothing to point out other than the comment on "PM / OPP: Set cpu_dev->id in cpumask first".

> 
> Regards
> ChenYu
> 

Best regards,
  Nobuhiro

> 
> Andrzej Hajda (1):
>   cpufreq-dt: fix handling regulator_get_voltage() result
> 
> Arnd Bergmann (1):
>   cpufreq: cpufreq-dt: avoid uninitialized variable warnings:
> 
> Bartlomiej Zolnierkiewicz (1):
>   PM / OPP: Fix parsing of opp-microvolt and opp-microamp properties
> 
> Pi-Cheng Chen (1):
>   PM / OPP: Set cpu_dev->id in cpumask first
> 
> Punit Agrawal (2):
>   devicetree: bindings: Add optional dynamic-power-coefficient property
>   cpufreq-dt: Supply power coefficient when registering cooling devices
> 
> Viresh Kumar (9):
>   PM / OPP: Add debugfs support
>   PM / OPP: Add "opp-supported-hw" binding
>   PM / OPP: Add {opp-microvolt|opp-microamp}-<name> binding
>   PM / OPP: Remove 'operating-points-names' binding
>   PM / OPP: Rename OPP nodes as opp@<opp-hz>
>   PM / OPP: Add missing doc comments
>   PM / OPP: Parse 'opp-supported-hw' binding
>   PM / OPP: Parse 'opp-<prop>-<name>' bindings
>   PM / OPP: Use snprintf() instead of sprintf()
> 
>  .../devicetree/bindings/arm/cpus.txt          |  17 +
>  Documentation/devicetree/bindings/opp/opp.txt | 132 +++++--
>  drivers/base/power/opp/Makefile               |   1 +
>  drivers/base/power/opp/core.c                 | 336 +++++++++++++++++-
>  drivers/base/power/opp/cpu.c                  |   3 +-
>  drivers/base/power/opp/debugfs.c              | 219 ++++++++++++
>  drivers/base/power/opp/opp.h                  |  53 ++-
>  drivers/cpufreq/cpufreq-dt.c                  |  29 +-
>  include/linux/pm_opp.h                        |  22 ++
>  9 files changed, 742 insertions(+), 70 deletions(-)
>  create mode 100644 drivers/base/power/opp/debugfs.c
> 
> --
> 2.27.0.rc0


[-- Attachment #2: Type: text/plain, Size: 419 bytes --]

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#4693): https://lists.cip-project.org/g/cip-dev/message/4693
Mute This Topic: https://lists.cip-project.org/mt/74665632/4520388
Group Owner: cip-dev+owner@lists.cip-project.org
Unsubscribe: https://lists.cip-project.org/g/cip-dev/leave/8129055/727948398/xyzzy  [cip-dev@archiver.kernel.org]
-=-=-=-=-=-=-=-=-=-=-=-

      parent reply index

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-04  4:17 Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 01/15] PM / OPP: Add debugfs support Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 02/15] PM / OPP: Add "opp-supported-hw" binding Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 03/15] PM / OPP: Add {opp-microvolt|opp-microamp}-<name> binding Chen-Yu Tsai (Moxa)
2020-06-04  8:46   ` Pavel Machek
2020-06-04  9:03     ` Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 04/15] PM / OPP: Remove 'operating-points-names' binding Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 05/15] PM / OPP: Rename OPP nodes as opp@<opp-hz> Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 06/15] PM / OPP: Add missing doc comments Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 07/15] PM / OPP: Parse 'opp-supported-hw' binding Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 08/15] PM / OPP: Parse 'opp-<prop>-<name>' bindings Chen-Yu Tsai (Moxa)
2020-06-04  8:56   ` Pavel Machek
2020-06-04  9:03     ` Pavel Machek
2020-06-04  9:21       ` Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 09/15] PM / OPP: Fix parsing of opp-microvolt and opp-microamp properties Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 10/15] PM / OPP: Set cpu_dev->id in cpumask first Chen-Yu Tsai (Moxa)
2020-06-04  6:42   ` Nobuhiro Iwamatsu
2020-06-04  6:54     ` Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 11/15] PM / OPP: Use snprintf() instead of sprintf() Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 12/15] devicetree: bindings: Add optional dynamic-power-coefficient property Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 13/15] cpufreq-dt: Supply power coefficient when registering cooling devices Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 14/15] cpufreq-dt: fix handling regulator_get_voltage() result Chen-Yu Tsai (Moxa)
2020-06-04  4:17 ` [cip-dev] [PATCH RESEND 4.4.y-cip 15/15] cpufreq: cpufreq-dt: avoid uninitialized variable warnings: Chen-Yu Tsai (Moxa)
2020-06-04  6:55 ` Nobuhiro Iwamatsu [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=OSBPR01MB29831550F3393A10777EF21D92890@OSBPR01MB2983.jpnprd01.prod.outlook.com \
    --to=nobuhiro1.iwamatsu@toshiba.co.jp \
    --cc=JohnsonCH.Chen@moxa.com \
    --cc=cip-dev@lists.cip-project.org \
    --cc=pavel@denx.de \
    --cc=wens@csie.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

CIP-dev Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/cip-dev/0 cip-dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 cip-dev cip-dev/ https://lore.kernel.org/cip-dev \
		cip-dev@lists.cip-project.org
	public-inbox-index cip-dev

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.cip-project.lists.cip-dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git