Re: [PATCH v2 7/8] nvme: hwmon: switch to use <linux/temperature.h> helpers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Akinobu Mita <akinobu.mita@xxxxxxxxx>
Subject: Re: [PATCH v2 7/8] nvme: hwmon: switch to use <linux/temperature.h> helpers
From: Keith Busch <kbusch@xxxxxxxxxx>
Date: Thu, 12 Dec 2019 03:33:35 +0900
Cc: linux-nvme@xxxxxxxxxxxxxxxxxxx, linux-hwmon@xxxxxxxxxxxxxxx, linux-pm@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Sujith Thomas <sujith.thomas@xxxxxxxxx>, Darren Hart <dvhart@xxxxxxxxxxxxx>, Andy Shevchenko <andy@xxxxxxxxxxxxx>, Zhang Rui <rui.zhang@xxxxxxxxx>, Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>, Amit Kucheria <amit.kucheria@xxxxxxxxxxxxx>, Jean Delvare <jdelvare@xxxxxxxx>, Guenter Roeck <linux@xxxxxxxxxxxx>, Jens Axboe <axboe@xxxxxx>, Christoph Hellwig <hch@xxxxxx>, Sagi Grimberg <sagi@xxxxxxxxxxx>
In-reply-to: <1574952879-7200-8-git-send-email-akinobu.mita@gmail.com>
References: <1574952879-7200-1-git-send-email-akinobu.mita@gmail.com> <1574952879-7200-8-git-send-email-akinobu.mita@gmail.com>
User-agent: Mutt/1.12.1 (2019-06-15)

On Thu, Nov 28, 2019 at 11:54:38PM +0900, Akinobu Mita wrote:
> This switches the nvme driver to use kelvin_to_millicelsius() and
> millicelsius_to_kelvin() in <linux/temperature.h>.

nvme change looks fine to me.

Reviewed-by: Keith Busch <kbusch@xxxxxxxxxx>

References:
- [PATCH v2 0/8] add header file for kelvin to/from Celsius conversion helpers
  - From: Akinobu Mita
- [PATCH v2 7/8] nvme: hwmon: switch to use <linux/temperature.h> helpers
  - From: Akinobu Mita

Prev by Date: Re: [RFC] hwmon: add support for IT8686E to it87.c
Next by Date: [PATCH v11 00/14] PECI device driver introduction
Previous by thread: Re: [PATCH v2 7/8] nvme: hwmon: switch to use <linux/temperature.h> helpers
Next by thread: [PATCH v2 8/8] thermal: remove kelvin to/from Celsius conversion helpers from <linux/thermal.h>
Index(es):
- Date
- Thread

[Index of Archives] [LM Sensors] [Linux Sound] [ALSA Users] [ALSA Devel] [Linux Audio Users] [Linux Media] [Kernel] [Gimp] [Yosemite News] [Linux Media]