All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Robert Marko <robimarko@gmail.com>
Cc: sjg@chromium.org, pali@kernel.org, u-boot@lists.denx.de,
	luka.perkov@sartura.hr, Robert Marko <robert.marko@sartura.hr>
Subject: Re: [PATCH v3 1/4] cmd: add temperature command
Date: Tue, 11 Oct 2022 17:37:03 -0400	[thread overview]
Message-ID: <20221011213703.GR2020586@bill-the-cat> (raw)
In-Reply-To: <20220906113036.1768055-1-robert.marko@sartura.hr>

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

On Tue, Sep 06, 2022 at 01:30:33PM +0200, Robert Marko wrote:

> Currently, there is no way for users to check the readings from thermal
> sensors from U-boot console, only some boards print it during boot.
> 
> So, lets add a simple "temperature" command that allows listing thermal
> uclass devices and getting their value.
> 
> Note that the thermal devices are intenionally probed if list is used as
> almost always they will not get probed otherwise and there is no way for
> users to manually call probe on a certain device from console.
> 
> Assumption is made that temperature is returned in degrees C and not
> milidegrees like in Linux as this is what most drivers seem to return.
> 
> Signed-off-by: Robert Marko <robert.marko@sartura.hr>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  parent reply	other threads:[~2022-10-11 21:38 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-06 11:30 [PATCH v3 1/4] cmd: add temperature command Robert Marko
2022-09-06 11:30 ` [PATCH v3 2/4] doc: cmd: temperature: add documentation Robert Marko
2022-10-11 21:37   ` Tom Rini
2022-09-06 11:30 ` [PATCH v3 3/4] thermal: add sandbox driver Robert Marko
2022-10-11 21:37   ` Tom Rini
2022-09-06 11:30 ` [PATCH v3 4/4] test: cmd: add test for temperature command Robert Marko
2022-10-11 21:37   ` Tom Rini
2022-10-11 21:37 ` Tom Rini [this message]
  -- strict thread matches above, loose matches on Subject: below --
2022-09-06 11:31 [PATCH v3 1/4] cmd: add " Robert Marko

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=20221011213703.GR2020586@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=luka.perkov@sartura.hr \
    --cc=pali@kernel.org \
    --cc=robert.marko@sartura.hr \
    --cc=robimarko@gmail.com \
    --cc=sjg@chromium.org \
    --cc=u-boot@lists.denx.de \
    /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.