On Mon, Jul 13, 2020 at 2:39 PM Rob Herring <robh@xxxxxxxxxx> wrote: > > On Mon, Jun 15, 2020 at 06:51:11AM -0400, Kevin P. Fleming wrote: > > Add a property to allow control of the autocalibration filter > > capacitor. > > > > Signed-off-by: Kevin P. Fleming <kevin+linux@xxxxxxx> > > Cc: Alessandro Zummo <a.zummo@xxxxxxxxxxxx> > > Cc: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx> > > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > > To: linux-rtc@xxxxxxxxxxxxxxx > > To: devicetree@xxxxxxxxxxxxxxx > > --- > > v3: corrected whitespace > > Documentation/devicetree/bindings/rtc/abracon,abx80x.txt | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/rtc/abracon,abx80x.txt b/Documentation/devicetree/bindings/rtc/abracon,abx80x.txt > > index 2405e35a1bc0f..1b606e33d1a83 100644 > > --- a/Documentation/devicetree/bindings/rtc/abracon,abx80x.txt > > +++ b/Documentation/devicetree/bindings/rtc/abracon,abx80x.txt > > @@ -29,3 +29,11 @@ and valid to enable charging: > > - "abracon,tc-diode": should be "standard" (0.6V) or "schottky" (0.3V) > > - "abracon,tc-resistor": should be <0>, <3>, <6> or <11>. 0 disables the output > > resistor, the other values are in kOhm. > > + > > +All of the devices can have a 47pf capacitor attached to increase the > > +autocalibration accuracy of their RC oscillators. To enable or disable usage > > +of the capacitor the following property can be defined: > > + > > + - "abracon,autocal-filter": should be <0> or <1>. 0 indicates that there > > + is no capacitor attached, 1 indicates that there > > + is a capacitor attached. > > What does not present mean? If you don't have a defined meaning (such > as maintain the default/bootloader initialized setting), then make this > boolean. That is the intended meaning (leave the current setting unmodified). I can add that to the documentation so it is clear.
