All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Chauvet <kwizart@gmail.com>
To: Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Zhang Rui <rui.zhang@intel.com>,
	Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: linux-tegra@vger.kernel.org, linux-pm@vger.kernel.org
Subject: Re: [PATCH v2 0/6] thermal: tegra: soctherm bugfixes
Date: Thu, 19 Nov 2020 12:00:19 +0100	[thread overview]
Message-ID: <CABr+WTnftetrMzC8bq+=RqOxqJ9EvYsU1LEZ5bVv-OptVksbjA@mail.gmail.com> (raw)
In-Reply-To: <20200927150956.34609-1-kwizart@gmail.com>

Le dim. 27 sept. 2020 à 17:10, Nicolas Chauvet <kwizart@gmail.com> a écrit :
>
> When using tegra_soctherm driver on jetson-tk1, the following messages
> can be seen:
> from kernel: tegra_soctherm 700e2000.thermal-sensor:
>  throttle-cfg: heavy: no throt prop or invalid prop
>  soctherm: trip temperature -2147483647 forced to -127000
>  thermtrip: will shut down when cpu reaches 101000 mC
>  soctherm: trip temperature -2147483647 forced to -127000
>  thermtrip: will shut down when gpu reaches 101000 mC
>  soctherm: trip temperature -2147483647 forced to -127000
>  thermtrip: will shut down when pll reaches 103000 mC
>  throttrip: pll: missing hot temperature
>  soctherm: trip temperature -2147483647 forced to -127000
>  thermtrip: will shut down when mem reaches 101000 mC
>  throttrip: mem: missing hot temperature
>  IRQ index 1 not found
>
> This serie fixes two errors and two warnings that are reported in dmesg
> It was compiled and tested at runtime on jetson-tk1 only.
>
>
> v2:
>  * Add missing tegra210 device-tree properties
>  * Add the appropriate prefix for tegra soctherm
>  * Use SoCs condition over of_compatible
>
>
> Nicolas Chauvet (6):
>   ARM: tegra: Add missing gpu-throt-level to tegra124 soctherm
>   ARM: tegra: Add missing hot temperatures to tegra124 thermal-zones
>   arm64: tegra: Add missing hot temperatures to tegra132 thermal-zones
>   arm64: tegra: Add missing gpu-throt-level to tegra210 soctherm
>   arm64: tegra: Add missing hot temperatures to tegra210 thermal-zones
>   thermal: tegra: Avoid setting edp_irq when not relevant
>
>  arch/arm/boot/dts/tegra124.dtsi           | 11 +++++++
>  arch/arm64/boot/dts/nvidia/tegra132.dtsi  | 10 ++++++
>  arch/arm64/boot/dts/nvidia/tegra210.dtsi  | 13 ++++++++
>  drivers/thermal/tegra/soctherm.c          | 38 +++++++++++++----------
>  drivers/thermal/tegra/soctherm.h          |  1 +
>  drivers/thermal/tegra/tegra124-soctherm.c |  1 +
>  drivers/thermal/tegra/tegra132-soctherm.c |  1 +
>  drivers/thermal/tegra/tegra210-soctherm.c |  1 +
>  8 files changed, 60 insertions(+), 16 deletions(-)
>
> --
> 2.25.4

Hello,
This series is still pending review.

Thierry, do you expect some reviewers from the linux-pm (Zhang, Daniel ?)
To me it seems like tegra specific changes, so maybe a Nvidia reviewer
is awaited ?

Thanks.

  parent reply	other threads:[~2020-11-19 11:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-27 15:09 [PATCH v2 0/6] thermal: tegra: soctherm bugfixes Nicolas Chauvet
2020-09-27 15:09 ` [PATCH v2 1/6] ARM: tegra: Add missing gpu-throt-level to tegra124 soctherm Nicolas Chauvet
2020-09-27 15:09 ` [PATCH v2 2/6] ARM: tegra: Add missing hot temperatures to tegra124 thermal-zones Nicolas Chauvet
2020-09-27 15:09 ` [PATCH v2 3/6] arm64: tegra: Add missing hot temperatures to tegra132 thermal-zones Nicolas Chauvet
2020-09-27 15:09 ` [PATCH v2 4/6] arm64: tegra: Add missing gpu-throt-level to tegra210 soctherm Nicolas Chauvet
2020-09-27 15:09 ` [PATCH v2 5/6] arm64: tegra: Add missing hot temperatures to tegra210 thermal-zones Nicolas Chauvet
2020-09-27 15:09 ` [PATCH v2 6/6] thermal: tegra: Avoid setting edp_irq when not relevant Nicolas Chauvet
2020-11-19 19:15   ` Jon Hunter
2020-11-19 11:00 ` Nicolas Chauvet [this message]
2020-11-19 11:04   ` [PATCH v2 0/6] thermal: tegra: soctherm bugfixes Daniel Lezcano
2020-11-20 15:07     ` Thierry Reding
2020-11-20 16:14 ` Thierry Reding

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='CABr+WTnftetrMzC8bq+=RqOxqJ9EvYsU1LEZ5bVv-OptVksbjA@mail.gmail.com' \
    --to=kwizart@gmail.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=jonathanh@nvidia.com \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=rui.zhang@intel.com \
    --cc=thierry.reding@gmail.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.