Hi Wolfram, On 27 July 2017 at 17:29, Baolin Wang <baolin.wang@xxxxxxxxxx> wrote: > Hi Wolfram, > > On 24 July 2017 at 14:51, Baolin Wang <baolin.wang@xxxxxxxxxxxxxx> wrote: >> Hi Wolfram, >> >> On 五, 7月 14, 2017 at 05:01:10下午 +0800, Baolin Wang wrote: >>> This patch adds the binding documentation for Spreadtrum I2C >>> controller device. >>> >>> Signed-off-by: Baolin Wang <baolin.wang@xxxxxxxxxxxxxx> >>> Acked-by: Rob Herring <robh@xxxxxxxxxx> >>> --- >>> Changes since v3: >>> - Add Ack from RobH. >>> >>> Changes since v2: >>> - Change compatible strings to be SoC specific. >>> >>> Changes since v1: >>> - No updates. >>> --- >> >> Could you apply this patchset if there are no other comments? >> It is a long time since the v4 was send out. Thanks. > > Ping? Since this patchset has waited for a long time, could you pick up this patchset into your branch? Thanks. > >> >>> Documentation/devicetree/bindings/i2c/i2c-sprd.txt | 31 ++++++++++++++++++++ >>> 1 file changed, 31 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/i2c/i2c-sprd.txt >>> >>> diff --git a/Documentation/devicetree/bindings/i2c/i2c-sprd.txt b/Documentation/devicetree/bindings/i2c/i2c-sprd.txt >>> new file mode 100644 >>> index 0000000..60b7cda >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/i2c/i2c-sprd.txt >>> @@ -0,0 +1,31 @@ >>> +I2C for Spreadtrum platforms >>> + >>> +Required properties: >>> +- compatible: Should be "sprd,sc9860-i2c". >>> +- reg: Specify the physical base address of the controller and length >>> + of memory mapped region. >>> +- interrupts: Should contain I2C interrupt. >>> +- clock-names: Should contain following entries: >>> + "i2c" for I2C clock, >>> + "source" for I2C source (parent) clock, >>> + "enable" for I2C module enable clock. >>> +- clocks: Should contain a clock specifier for each entry in clock-names. >>> +- clock-frequency: Constains desired I2C bus clock frequency in Hz. >>> +- #address-cells: Should be 1 to describe address cells for I2C device address. >>> +- #size-cells: Should be 0 means no size cell for I2C device address. >>> + >>> +Optional properties: >>> +- Child nodes conforming to I2C bus binding >>> + >>> +Examples: >>> +i2c0: i2c@70500000 { >>> + compatible = "sprd,sc9860-i2c"; >>> + reg = <0 0x70500000 0 0x1000>; >>> + interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>; >>> + clock-names = "i2c", "source", "enable"; >>> + clocks = <&clk_i2c3>, <&ext_26m>, <&clk_ap_apb_gates 11>; >>> + clock-frequency = <400000>; >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> +}; >>> + >>> -- >>> 1.7.9.5 >>> > > > > -- > Baolin.wang > Best Regards -- Baolin.wang Best Regards