On Fri, Oct 05, 2018 at 07:38:35AM +0200, Wojciech Slenska wrote:
Commit msg?
> Signed-off-by: Wojciech Slenska <wojciech.slenska@xxxxxxxxx>
> ---
> Documentation/devicetree/bindings/hwmon/sht3x.txt | 16 +++++++++++++
Please split bindings to separate patch.
> drivers/hwmon/sht3x.c | 28 ++++++++++++++++++++---
> 2 files changed, 41 insertions(+), 3 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/hwmon/sht3x.txt
>
> diff --git a/Documentation/devicetree/bindings/hwmon/sht3x.txt b/Documentation/devicetree/bindings/hwmon/sht3x.txt
> new file mode 100644
> index 0000000..80b117e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/hwmon/sht3x.txt
> @@ -0,0 +1,16 @@
> +Sensirion SHT3x Humidity and Temperature Sensor
> +
> +Required node properties:
> +- compatible: "sensirion,sht3x" or "sensirion,sts3x"
> +- reg: I2C bus address of the device
> +
> +Optional properties:
> +- sensirion,blocking-io: enable blocking mode on i2c
This is not a h/w parameter and shouldn't be in DT.
> +- sensirion,no-high-precision: disable high accuracy
Maybe this one is okay, but couldn't the user want to set this? If so,
then it should be a sysfs attr.
> +
> +Example sht3x node:
> +
> +sensor {
sensor@4a
> + compatible = "sensirion,sht3x";
> + reg = <0x4a>;
> +}
