[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <cf67db7e-f707-2448-4648-3cffa459355a@amazon.com>
Date: Mon, 29 Aug 2022 21:59:05 +0300
From: "Farber, Eliav" <farbere@...zon.com>
To: Guenter Roeck <linux@...ck-us.net>
CC: <jdelvare@...e.com>, <robh+dt@...nel.org>, <mark.rutland@....com>,
<linux-hwmon@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <talel@...zon.com>,
<hhhawa@...zon.com>, <jonnyc@...zon.com>, <hanochu@...zon.com>,
<ronenk@...zon.com>, <itamark@...zon.com>, <shellykz@...zon.com>,
<shorer@...zon.com>, <amitlavi@...zon.com>, <almogbs@...zon.com>,
<dwmw@...zon.co.uk>, <rtanwar@...linear.com>,
"Farber, Eliav" <farbere@...zon.com>
Subject: Re: [PATCH v2 14/16] hwmon: (mr75203) parse thermal coefficients from
device-tree
On 8/22/2022 7:31 PM, Guenter Roeck wrote:
> It should not be necessary to provide explicit default values for any
> of the
> series. Yes, default values can be overwritten with explicit coefficient
> properties, but it should not be necessary to provide those if the
> defaults
> are used. So I would expect separate compatible properties for each of
> the
> supported series plus separate coefficient properties.
I added a "moortec,ts-series" so that user will not need to provide all
4 coefficients.
The values of the "moortec,ts-series" can be 5 (default) or 6.
I didn't do it as a compatible property because the the driver is for
the Moortec controller (mr75203) while series 5 or 6 are only relevant
for the thermal sensor (mr74137).
--
Thanks, Eliav
Powered by blists - more mailing lists