Re: [PATCH v4 12/21] dt-bindings: hwmon: (mr75203) add "moortec, vm-active-channels" property

[Farber, Eliav]

On 9/6/2022 8:08 PM, Guenter Roeck wrote:
> On Tue, Sep 06, 2022 at 08:33:47AM +0000, Eliav Farber wrote:
> > Add optional "moortec,vm-active-channels" property to define the number
> > of active channels per VM.
> >
> > This shall be useful to avoid exposing sysfs for reading inputs that are
> > not connected to any voltage source.
> >
> > Signed-off-by: Eliav Farber <farbere@xxxxxxxxxx>
> > ---
> > V4 -> V3:
> > - Fix DT checker errors.
> >
> > V3 -> V2:
> > - Add "moortec" prefix to property name.
> > - Add explanation why this change is needed.
> >
> >  .../devicetree/bindings/hwmon/moortec,mr75203.yaml     | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> >
> > diff --git 
> > a/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml 
> > b/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml
> > index 9454576ebb73..2aa4c3618596 100644
> > --- a/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml
> > +++ b/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml
> > @@ -70,6 +70,15 @@ properties:
> >    "#thermal-sensor-cells":
> >      const: 1
> >
> > +  moortec,vm-active-channels:
> > +    description:
> > +      Defines the number of channels per VM that are actually used 
> > and are
> > +      connected to some input source.
> > +      Maximum number of items - number of VMs.
> > +      Maximum value of each item - number of channels.
> > +      Minimum value of each item - 0 (which means entire VM sensor 
> > is nou used).
>
> s/nou/not/


Typo fixed in v5.

--
Thanks, Eliav