[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220908185952.GA3343467@roeck-us.net>
Date: Thu, 8 Sep 2022 11:59:52 -0700
From: Guenter Roeck <linux@...ck-us.net>
To: Eliav Farber <farbere@...zon.com>
Cc: jdelvare@...e.com, robh+dt@...nel.org, p.zabel@...gutronix.de,
rtanwar@...linear.com, andriy.shevchenko@...el.com,
linux-hwmon@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, hhhawa@...zon.com, jonnyc@...zon.com,
Rob Herring <robh@...nel.org>
Subject: Re: [PATCH v5 01/21] dt-bindings: hwmon: (mr75203) fix
"intel,vm-map" property to be optional
On Thu, Sep 08, 2022 at 03:24:29PM +0000, Eliav Farber wrote:
> Change "intel,vm-map" property to be optional instead of required.
>
> The driver implementation indicates it is not mandatory to have
> "intel,vm-map" in the device tree:
> - probe doesn't fail in case it is absent.
> - explicit comment in code - "Incase intel,vm-map property is not
> defined, we assume incremental channel numbers".
>
> Fixes: 748022ef093f ("hwmon: Add DT bindings schema for PVT controller")
> Signed-off-by: Eliav Farber <farbere@...zon.com>
> Acked-by: Rob Herring <robh@...nel.org>
Applied, together with patches 2..6 of the series (all fixes).
Thanks,
Guenter
> ---
> V3 -> V2:
> - Change this patch to be first in the series.
> - Add explanation why "intel,vm-map" is not required.
>
> Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml b/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml
> index b79f069a04c2..8ea97e774364 100644
> --- a/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml
> @@ -48,7 +48,6 @@ required:
> - compatible
> - reg
> - reg-names
> - - intel,vm-map
> - clocks
> - resets
> - "#thermal-sensor-cells"
Powered by blists - more mailing lists