Re: [PATCH 1/4] i2c: iProc: define Broadcom iProc I2C binding
flat view
From: Varka Bhadram <hidden>
Date: 2014-12-10 01:27:27
Also in:
linux-arm-kernel, linux-i2c, lkml
Hi, On Wednesday 10 December 2014 06:24 AM, Ray Jui wrote:
quoted hunk ↗ jump to hunk
Document the I2C device tree binding for Broadcom iProc family of SoCs Signed-off-by: Ray Jui <rjui@broadcom.com> Reviewed-by: Scott Branden <sbranden@broadcom.com> --- .../devicetree/bindings/i2c/brcm,iproc-i2c.txt | 37 ++++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100644 Documentation/devicetree/bindings/i2c/brcm,iproc-i2c.txtdiff --git a/Documentation/devicetree/bindings/i2c/brcm,iproc-i2c.txt b/Documentation/devicetree/bindings/i2c/brcm,iproc-i2c.txt new file mode 100644 index 0000000..81f982c --- /dev/null +++ b/Documentation/devicetree/bindings/i2c/brcm,iproc-i2c.txt@@ -0,0 +1,37 @@ +Broadcom iProc I2C controller + +Required properties: + +- compatible: + Must be "brcm,iproc-i2c" + +- reg: + Define the base and range of the I/O address space that contain the iProc + I2C controller registers + +- interrupts: + Should contain the I2C interrupt + +- clock-frequency: + This is the I2C bus clock. Need to be either 100000 or 400000 + +- #address-cells: + Always 1 (for I2C addresses) + +- #size-cells: + Always 0 +
All the properties defined with two lines of statements. Why cant they be with single line statement, like: compatible: Must be "brcm,iproc-i2c" reg: Define the base and range of the I/O address space that contain the iProc I2C controller registers .... -- Thanks and Regards, Varka Bhadram.