Hi Andrew On 29/06/2016 01:44, Andrew Duggan wrote: > On 06/28/2016 01:56 PM, Rob Herring wrote: >> On Fri, Jun 24, 2016 at 05:18:37PM -0700, Andrew Duggan wrote: >>> Signed-off-by: Andrew Duggan <aduggan@xxxxxxxxxxxxx> >>> --- >>> .../devicetree/bindings/input/rmi4/rmi_2d_sensor.txt | 4 ++++ >>> drivers/input/rmi4/rmi_2d_sensor.c | 11 >>> +++++++++++ >>> 2 files changed, 15 insertions(+) >>> >>> diff --git >>> a/Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt >>> b/Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt >>> index f2c30c8..86cb8e3 100644 >>> --- a/Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt >>> +++ b/Documentation/devicetree/bindings/input/rmi4/rmi_2d_sensor.txt >>> @@ -37,6 +37,10 @@ Optional Properties: >>> disable reporing absolute position data. >>> - syna,rezero-wait-ms: Time in miliseconds to wait after issuing a rezero >>> command. >>> +- syna,dribble-enable: Enable reporting of dribble packets in the >>> firmware. >>> +- syna,dribble-disable: Disable reporting of dribble packets in the >>> firmware. >>> +- syna,palm-detect-enable: Enable reporting the palm detect gesture in >>> the firmware. >>> +- syna,palm-detect-disable: Disable reporting the palm detect gesture >>> in the firmware. >> No need for enable and disable. Make the common case be no property and >> the property is the opposite. > > There are actually three states here. No property means to use the default > value set in the firmware. The enable parameter means that the default > should be overwritten and the feature should be explicitly enabled. Disable > means the feature should be explicitly disabled. An earlier version of this > patch had one parameter per feature which needed to be set to 0, 1, or 2 > which correspond the the values of an enum in the driver. It was suggested > that the current approach would be clearer then setting 0, 1, or 2. I wonder if it might be worth mentioning explicitly in the devicetree documentation that the firmware is being overridden - perhaps "syna,dribble-force-disable"? cheers Nick -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html