Messages in this thread | | | Subject | Re: [PATCH v2 2/2] devicetree: i2c-hid: Add reset property | From | hl <> | Date | Mon, 6 Nov 2017 09:00:52 +0800 |
| |
On Saturday, November 04, 2017 12:35 PM, Brian Norris wrote: > On Mon, Oct 30, 2017 at 8:03 PM, Lin Huang <hl@rock-chips.com> wrote: >> Document a "reset" and "assert-reset-us", it can be used for >> driver control reset property. And reuse post-power-on-delay-ms >> for deassert reset delay. >> >> Signed-off-by: Lin Huang <hl@rock-chips.com> >> --- >> Documentation/devicetree/bindings/input/hid-over-i2c.txt | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/input/hid-over-i2c.txt b/Documentation/devicetree/bindings/input/hid-over-i2c.txt >> index 28e8bd8..6ab0eed 100644 >> --- a/Documentation/devicetree/bindings/input/hid-over-i2c.txt >> +++ b/Documentation/devicetree/bindings/input/hid-over-i2c.txt >> @@ -31,7 +31,9 @@ device-specific compatible properties, which should be used in addition to the >> >> - vdd-supply: phandle of the regulator that provides the supply voltage. >> - post-power-on-delay-ms: time required by the device after enabling its regulators >> - before it is ready for communication. Must be used with 'vdd-supply'. >> + or deassert reset pin before it is ready for communication. >> +- reset: phandle of the gpio that provides for hid reset pin. >> +- assert-reset-us: the device require reset assert time. > If there was any point in adding the device-specific description > around "wacom,w9013"...then you should probably mention these > properties there too. The idea was to document possible properties > here (where you're adding them already), and to note the property > names under the devices (or so far, just 1 device) that support them. > Or IOW, you need an addition like this: > > - compatible: > * "wacom,w9013" (Wacom W9013 digitizer). Supports: > - vdd-supply > - post-power-on-delay-ms > + - reset-gpios > + - assert-reset-us Okay, got it, will fix it next version. > > Brian > > >
| |