[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <547F9958.1020807@gmx.de>
Date: Thu, 04 Dec 2014 00:14:32 +0100
From: Hartmut Knaack <knaack.h@....de>
To: Karol Wrona <k.wrona@...sung.com>,
Jonathan Cameron <jic23@...nel.org>, linux-iio@...r.kernel.org,
Arnd Bergmann <arnd@...db.de>, linux-kernel@...r.kernel.org
CC: Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
Kyungmin Park <kyungmin.park@...sung.com>
Subject: Re: [PATCH v2 2/5] sensorhub: Add sensorhub bindings
Karol Wrona schrieb am 21.11.2014 um 19:19:
> Add sensorhub bindings for sensorhub on Galaxy Gear 2.
Some comments inline.
>
> Signed-off-by: Karol Wrona <k.wrona@...sung.com>
> Acked-by: Kyungmin Park <kyungmin.park@...sung.com>
> ---
> .../devicetree/bindings/misc/sensorhub.txt | 46 ++++++++++++++++++++
> 1 file changed, 46 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/misc/sensorhub.txt
>
> diff --git a/Documentation/devicetree/bindings/misc/sensorhub.txt b/Documentation/devicetree/bindings/misc/sensorhub.txt
> new file mode 100644
> index 0000000..088bf32
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/misc/sensorhub.txt
> @@ -0,0 +1,46 @@
> +Samsung Sensorhub driver
> +
> +Sensorhub is MCU which manages several sensors and also play the role
... is a MCU ... also plays the ...
> +of virtual sensor device.
of a virtual ...
> +
> +Required properties:
> +- compatible: "samsung,sensorhub-rinato" or "samsung,sensorhub-thermostat"
> +- spi-max-frequency: max SPI clock frequency
> +- interrupt-parent
Missing description.
> +- interrupts: communication interrupt
> +- ap-mcu-gpio: [out] ap to sensorhub line - used during communication
> +- mcu-ap-gpio: [in] sensorhub to ap - used during communication
> +- mcu-reset: [out] sensorhub reset
> +
> +Optional properties:
> +- sensor's nodes:
> + compatible = "samsung,mpu6500-accel"
> + compatible = "samsung,mpu6500-gyro"
> + compatible = "samsung,adpd142"
> +
> +Sensor compatibles are used to match proper sensor driver to real sensor on
> +the board. The firmware does not give such information, so it helps to specify
> +some sensors properties. Sensors have "samsung" prefixes because frequently
> +they will not have much in common with sensors used without sensorhub because
> +it can do some data processing.
> +
> +Example:
> +
> + shub_spi: shub {
> + compatible = "samsung,sensorhub-rinato";
> + spi-max-frequency = <5000000>;
> + interrupt-parent = <&gpx0>;
> + interrupts = <2 0>;
> + ap-mcu-gpio = <&gpx0 0 0>;
> + mcu-ap-gpio = <&gpx0 4 0>;
> + mcu-reset = <&gpx0 5 0>;
> + sensor@0 {
> + compatible = "samsung,mpu6500-accel";
> + };
> + sensor@1 {
> + compatible = "samsung,mpu6500-gyro";
> + };
> + sensor@2 {
> + compatible = "samsung,adpd142";
> + };
> + };
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists