linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: miquel.raynal@free-electrons.com (Miquel RAYNAL)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 09/11] thermal: armada: Wait sensors validity before exiting the init callback
Date: Thu, 14 Dec 2017 12:27:33 +0100	[thread overview]
Message-ID: <20171214122733.5c9264c0@xps13> (raw)
In-Reply-To: <87ind9o8o0.fsf@free-electrons.com>

On Thu, 14 Dec 2017 12:23:11 +0100
Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:

> Hi Miquel,
>  
>  On jeu., d?c. 14 2017, Miquel Raynal
> <miquel.raynal@free-electrons.com> wrote:
> 
> > The thermal core will check for sensors validity right after the
> > initialization callback has returned. As the initialization routine
> > make a reset, the sensors are not ready immediately and the core
> > spawns an error in the dmesg. Avoid this annoying situation by
> > polling on the validity bit before exiting from these routines.
> > This also avoid the use of blind sleeps.  
> 
> You only modified the armada 380 and ap806 init function. Does it
> means that this feature is not supported on the other variant?

I can't tell for other boards, It does not seem they actually need it
so I did not hacked their init callback. Should I? IMOH, if there is
not problem, there is no fix to add neither.

Miqu?l

  reply	other threads:[~2017-12-14 11:27 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-14 10:30 [PATCH v3 00/11] Armada thermal: improvements and A7K/A8K SoCs support Miquel Raynal
2017-12-14 10:30 ` [PATCH v3 01/11] dt-bindings: thermal: Describe Armada AP806 and CP110 Miquel Raynal
2017-12-15  8:27   ` Baruch Siach
2017-12-15  8:32     ` Miquel RAYNAL
2017-12-15  8:44       ` Baruch Siach
2017-12-15  8:44       ` Gregory CLEMENT
2017-12-15 10:52         ` Miquel RAYNAL
2017-12-15 23:28           ` Rob Herring
2017-12-16 12:50             ` Miquel RAYNAL
2017-12-14 10:30 ` [PATCH v3 02/11] thermal: armada: Use msleep for long delays Miquel Raynal
2017-12-14 10:51   ` Gregory CLEMENT
2017-12-14 10:30 ` [PATCH v3 03/11] thermal: armada: Simplify the check of the validity bit Miquel Raynal
2017-12-14 10:52   ` Gregory CLEMENT
2017-12-15  8:33   ` Baruch Siach
2017-12-15 10:56     ` Miquel RAYNAL
2017-12-14 10:30 ` [PATCH v3 04/11] thermal: armada: Rationalize register accesses Miquel Raynal
2017-12-14 10:55   ` Gregory CLEMENT
2017-12-15  8:56   ` Baruch Siach
2017-12-18 13:48     ` Miquel RAYNAL
2017-12-16 22:18   ` Baruch Siach
2017-12-17 22:02     ` Baruch Siach
2017-12-18 12:37       ` Miquel RAYNAL
2017-12-14 10:30 ` [PATCH v3 05/11] thermal: armada: Add support for Armada AP806 Miquel Raynal
2017-12-14 11:05   ` Gregory CLEMENT
2017-12-18  9:41     ` Miquel RAYNAL
2017-12-18 11:11       ` Baruch Siach
2017-12-18 12:25         ` Miquel RAYNAL
2017-12-16 22:22   ` Baruch Siach
2017-12-14 10:30 ` [PATCH v3 06/11] thermal: armada: Add support for Armada CP110 Miquel Raynal
2017-12-14 11:11   ` Gregory CLEMENT
2017-12-14 11:33     ` Miquel RAYNAL
2017-12-14 11:37       ` Gregory CLEMENT
2017-12-14 12:24         ` Miquel RAYNAL
2017-12-14 10:30 ` [PATCH v3 07/11] thermal: armada: Update Kconfig and module description Miquel Raynal
2017-12-14 11:13   ` Gregory CLEMENT
2017-12-14 11:17     ` Miquel RAYNAL
2017-12-14 11:30       ` Gregory CLEMENT
2017-12-14 11:36         ` Miquel RAYNAL
2017-12-14 13:10         ` Thomas Petazzoni
2017-12-14 10:30 ` [PATCH v3 08/11] thermal: armada: Change sensors trim default value Miquel Raynal
2017-12-14 10:30 ` [PATCH v3 09/11] thermal: armada: Wait sensors validity before exiting the init callback Miquel Raynal
2017-12-14 11:23   ` Gregory CLEMENT
2017-12-14 11:27     ` Miquel RAYNAL [this message]
2017-12-14 10:30 ` [PATCH v3 10/11] thermal: armada: Give useful names to the thermal zone Miquel Raynal
2017-12-14 10:30 ` [PATCH v3 11/11] ARM64: dts: marvell: Add thermal support for A7K/A8K Miquel Raynal

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=20171214122733.5c9264c0@xps13 \
    --to=miquel.raynal@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).