Re: [PATCH 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 7/8] nvme: hwmon: switch to use <linux/temperature.h> helpers
From: Christoph Hellwig <hch@xxxxxx>
Date: Mon, 25 Nov 2019 07:59:07 +0100
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>, Eduardo Valentin <edubezval@xxxxxxxxx>, Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>, Amit Kucheria <amit.kucheria@xxxxxxxxxxxxx>, Jean Delvare <jdelvare@xxxxxxxx>, Guenter Roeck <linux@xxxxxxxxxxxx>, Keith Busch <kbusch@xxxxxxxxxx>, Jens Axboe <axboe@xxxxxx>, Christoph Hellwig <hch@xxxxxx>, Sagi Grimberg <sagi@xxxxxxxxxxx>
In-reply-to: <1574604530-9024-8-git-send-email-akinobu.mita@gmail.com>
References: <1574604530-9024-1-git-send-email-akinobu.mita@gmail.com> <1574604530-9024-8-git-send-email-akinobu.mita@gmail.com>
User-agent: Mutt/1.5.17 (2007-11-01)

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

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

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

Prev by Date: [PATCH 2/2] hwmon: (w83627ehf) convert to with_info interface
Next by Date: hwmon: (nct7802) buggy VSEN1/2/3 alarm
Previous by thread: [PATCH 7/8] nvme: hwmon: switch to use <linux/temperature.h> helpers
Next by thread: [PATCH 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]