Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1398668 > unrolled thread
| Started by | David Wu <david.wu@rock-chips.com> |
|---|---|
| First post | 2016-05-11 06:10 +0200 |
| Last post | 2016-05-11 20:30 +0200 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH v8 6/8] dt-bindings: i2c: rk3x: add support for rk3399 David Wu <david.wu@rock-chips.com> - 2016-05-11 06:10 +0200
Re: [PATCH v8 6/8] dt-bindings: i2c: rk3x: add support for rk3399 Doug Anderson <dianders@chromium.org> - 2016-05-11 18:40 +0200
Re: [PATCH v8 6/8] dt-bindings: i2c: rk3x: add support for rk3399 "David.Wu" <david.wu@rock-chips.com> - 2016-05-12 03:20 +0200
Re: [PATCH v8 6/8] dt-bindings: i2c: rk3x: add support for rk3399 Heiko Stuebner <heiko@sntech.de> - 2016-05-11 20:30 +0200
| From | David Wu <david.wu@rock-chips.com> |
|---|---|
| Date | 2016-05-11 06:10 +0200 |
| Subject | [PATCH v8 6/8] dt-bindings: i2c: rk3x: add support for rk3399 |
| Message-ID | <rxtDc-wb-17@gated-at.bofh.it> |
The bus clock and function clock are separated at rk3399, and others use one clock as the bus clock and function clock. Signed-off-by: David Wu <david.wu@rock-chips.com> Reviewed-by: Douglas Anderson <dianders@chromium.org> --- Change in v8: - remove error description. Documentation/devicetree/bindings/i2c/i2c-rk3x.txt | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt index 0b4a85f..5429301 100644 --- a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt +++ b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt @@ -6,10 +6,20 @@ RK3xxx SoCs. Required properties : - reg : Offset and length of the register set for the device - - compatible : should be "rockchip,rk3066-i2c", "rockchip,rk3188-i2c", - "rockchip,rk3228-i2c" or "rockchip,rk3288-i2c". + - compatible: should be one of the following: + - "rockchip,rk3066-i2c": for rk3066 + - "rockchip,rk3188-i2c": for rk3188 + - "rockchip,rk3228-i2c": for rk3228 + - "rockchip,rk3288-i2c": for rk3288 + - "rockchip,rk3399-i2c": for rk3399 - interrupts : interrupt number - - clocks : parent clock + - clocks: See ../clock/clock-bindings.txt + - For older hardware (rk3066, rk3188, rk3228, rk3288): + - There is one clock that's used both to derive the functional clock + for the device and as the bus clock. + - For newer hardware (rk3399): specified by name + - "i2c": REQUIRED. This is used to derive the functional clock. + - "pclk": REQUIRED. This is the bus clock. Required on RK3066, RK3188 : -- 1.9.1
[toc] | [next] | [standalone]
| From | Doug Anderson <dianders@chromium.org> |
|---|---|
| Date | 2016-05-11 18:40 +0200 |
| Message-ID | <rxFl1-3u1-45@gated-at.bofh.it> |
| In reply to | #1398668 |
Hi, On Tue, May 10, 2016 at 12:30 PM, David Wu <david.wu@rock-chips.com> wrote: > The bus clock and function clock are separated at rk3399, > and others use one clock as the bus clock and function clock. > > Signed-off-by: David Wu <david.wu@rock-chips.com> > Reviewed-by: Douglas Anderson <dianders@chromium.org> > --- > Change in v8: > - remove error description. > > Documentation/devicetree/bindings/i2c/i2c-rk3x.txt | 16 +++++++++++++--- > 1 file changed, 13 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt > index 0b4a85f..5429301 100644 > --- a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt > +++ b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt > @@ -6,10 +6,20 @@ RK3xxx SoCs. > Required properties : > > - reg : Offset and length of the register set for the device > - - compatible : should be "rockchip,rk3066-i2c", "rockchip,rk3188-i2c", > - "rockchip,rk3228-i2c" or "rockchip,rk3288-i2c". > + - compatible: should be one of the following: > + - "rockchip,rk3066-i2c": for rk3066 > + - "rockchip,rk3188-i2c": for rk3188 > + - "rockchip,rk3228-i2c": for rk3228 > + - "rockchip,rk3288-i2c": for rk3288 > + - "rockchip,rk3399-i2c": for rk3399 > - interrupts : interrupt number > - - clocks : parent clock > + - clocks: See ../clock/clock-bindings.txt > + - For older hardware (rk3066, rk3188, rk3228, rk3288): > + - There is one clock that's used both to derive the functional clock > + for the device and as the bus clock. > + - For newer hardware (rk3399): specified by name > + - "i2c": REQUIRED. This is used to derive the functional clock. > + - "pclk": REQUIRED. This is the bus clock. Depending on what Rob thinks, it might make sense to remove the above two "REQUIRED" bits. That would match his earlier feedback since we're still in the "Required" section and thus it is redundant. -Doug
[toc] | [prev] | [next] | [standalone]
| From | "David.Wu" <david.wu@rock-chips.com> |
|---|---|
| Date | 2016-05-12 03:20 +0200 |
| Message-ID | <rxNsd-3dK-9@gated-at.bofh.it> |
| In reply to | #1399342 |
Hi Doug, 在 2016/5/12 0:35, Doug Anderson 写道: > Hi, > > On Tue, May 10, 2016 at 12:30 PM, David Wu <david.wu@rock-chips.com> wrote: >> The bus clock and function clock are separated at rk3399, >> and others use one clock as the bus clock and function clock. >> >> Signed-off-by: David Wu <david.wu@rock-chips.com> >> Reviewed-by: Douglas Anderson <dianders@chromium.org> >> --- >> Change in v8: >> - remove error description. >> >> Documentation/devicetree/bindings/i2c/i2c-rk3x.txt | 16 +++++++++++++--- >> 1 file changed, 13 insertions(+), 3 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt >> index 0b4a85f..5429301 100644 >> --- a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt >> +++ b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt >> @@ -6,10 +6,20 @@ RK3xxx SoCs. >> Required properties : >> >> - reg : Offset and length of the register set for the device >> - - compatible : should be "rockchip,rk3066-i2c", "rockchip,rk3188-i2c", >> - "rockchip,rk3228-i2c" or "rockchip,rk3288-i2c". >> + - compatible: should be one of the following: >> + - "rockchip,rk3066-i2c": for rk3066 >> + - "rockchip,rk3188-i2c": for rk3188 >> + - "rockchip,rk3228-i2c": for rk3228 >> + - "rockchip,rk3288-i2c": for rk3288 >> + - "rockchip,rk3399-i2c": for rk3399 >> - interrupts : interrupt number >> - - clocks : parent clock >> + - clocks: See ../clock/clock-bindings.txt >> + - For older hardware (rk3066, rk3188, rk3228, rk3288): >> + - There is one clock that's used both to derive the functional clock >> + for the device and as the bus clock. >> + - For newer hardware (rk3399): specified by name >> + - "i2c": REQUIRED. This is used to derive the functional clock. >> + - "pclk": REQUIRED. This is the bus clock. > > Depending on what Rob thinks, it might make sense to remove the above > two "REQUIRED" bits. That would match his earlier feedback since > we're still in the "Required" section and thus it is redundant. > Okay, i make a little misunderstand for that, so i will fix it in next version. > -Doug > > >
[toc] | [prev] | [next] | [standalone]
| From | Heiko Stuebner <heiko@sntech.de> |
|---|---|
| Date | 2016-05-11 20:30 +0200 |
| Message-ID | <rxH3s-5o7-21@gated-at.bofh.it> |
| In reply to | #1398668 |
Am Mittwoch, 11. Mai 2016, 03:30:47 schrieb David Wu: > The bus clock and function clock are separated at rk3399, > and others use one clock as the bus clock and function clock. > > Signed-off-by: David Wu <david.wu@rock-chips.com> > Reviewed-by: Douglas Anderson <dianders@chromium.org> I'd second what Doug said wrt. the removal of the unnecessary "REQUIRED" statements of the clock names, especially as the surrounding text makes that requirement pretty clear. Otherwiese looks nice, so Reviewed-by: Heiko Stuebner <heiko@sntech.de> > --- > Change in v8: > - remove error description. > > Documentation/devicetree/bindings/i2c/i2c-rk3x.txt | 16 +++++++++++++--- > 1 file changed, 13 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt > b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt index > 0b4a85f..5429301 100644 > --- a/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt > +++ b/Documentation/devicetree/bindings/i2c/i2c-rk3x.txt > @@ -6,10 +6,20 @@ RK3xxx SoCs. > Required properties : > > - reg : Offset and length of the register set for the device > - - compatible : should be "rockchip,rk3066-i2c", "rockchip,rk3188-i2c", > - "rockchip,rk3228-i2c" or "rockchip,rk3288-i2c". > + - compatible: should be one of the following: > + - "rockchip,rk3066-i2c": for rk3066 > + - "rockchip,rk3188-i2c": for rk3188 > + - "rockchip,rk3228-i2c": for rk3228 > + - "rockchip,rk3288-i2c": for rk3288 > + - "rockchip,rk3399-i2c": for rk3399 > - interrupts : interrupt number > - - clocks : parent clock > + - clocks: See ../clock/clock-bindings.txt > + - For older hardware (rk3066, rk3188, rk3228, rk3288): > + - There is one clock that's used both to derive the functional clock > + for the device and as the bus clock. > + - For newer hardware (rk3399): specified by name > + - "i2c": REQUIRED. This is used to derive the functional clock. > + - "pclk": REQUIRED. This is the bus clock. > > Required on RK3066, RK3188 :
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web