| Message ID | 20220908152449.35457-2-farbere@amazon.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | Variety of fixes and new features for mr75203 driver | expand |
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@amazon.com> > Acked-by: Rob Herring <robh@kernel.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"
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"
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@amazon.com> Acked-by: Rob Herring <robh@kernel.org> --- 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(-)