| Message ID | 20170420154216.32709-1-marek.vasut+renesas@gmail.com |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
On Thu, Apr 20, 2017 at 5:42 PM, Marek Vasut <marek.vasut@gmail.com> wrote: > The "if" interface clock speed is actually derived from the "fck" > block clock, as in the hardware they are the same clock. Drop the > incorrect second "if" clock and retain only the "fck" clock. > > Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com> > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > Cc: Jonathan Cameron <jic23@kernel.org> > Cc: Rob Herring <robh@kernel.org> > Cc: linux-renesas-soc@vger.kernel.org > To: devicetree@vger.kernel.org Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 20/04/17 18:30, Geert Uytterhoeven wrote: > On Thu, Apr 20, 2017 at 5:42 PM, Marek Vasut <marek.vasut@gmail.com> wrote: >> The "if" interface clock speed is actually derived from the "fck" >> block clock, as in the hardware they are the same clock. Drop the >> incorrect second "if" clock and retain only the "fck" clock. >> >> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com> >> Cc: Geert Uytterhoeven <geert+renesas@glider.be> >> Cc: Jonathan Cameron <jic23@kernel.org> >> Cc: Rob Herring <robh@kernel.org> >> Cc: linux-renesas-soc@vger.kernel.org >> To: devicetree@vger.kernel.org > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Applied to the togreg branch of iio.git and pushed out as testing. Device tree ack welcome, but I 'think' this is safe enough to take without as it won't break any existing device trees. Jonathan > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Apr 20, 2017 at 05:42:16PM +0200, Marek Vasut wrote: > The "if" interface clock speed is actually derived from the "fck" > block clock, as in the hardware they are the same clock. Drop the > incorrect second "if" clock and retain only the "fck" clock. > > Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com> > Cc: Geert Uytterhoeven <geert+renesas@glider.be> > Cc: Jonathan Cameron <jic23@kernel.org> > Cc: Rob Herring <robh@kernel.org> > Cc: linux-renesas-soc@vger.kernel.org > To: devicetree@vger.kernel.org > --- > Documentation/devicetree/bindings/iio/adc/renesas,gyroadc.txt | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) Acked-by: Rob Herring <robh@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/iio/adc/renesas,gyroadc.txt b/Documentation/devicetree/bindings/iio/adc/renesas,gyroadc.txt index f5b0adae6010..2a62908a774a 100644 --- a/Documentation/devicetree/bindings/iio/adc/renesas,gyroadc.txt +++ b/Documentation/devicetree/bindings/iio/adc/renesas,gyroadc.txt @@ -16,8 +16,7 @@ Required properties: - clocks: References to all the clocks specified in the clock-names property as specified in Documentation/devicetree/bindings/clock/clock-bindings.txt. -- clock-names: Shall contain "fck" and "if". The "fck" is the GyroADC block - clock, the "if" is the interface clock. +- clock-names: Shall contain "fck". The "fck" is the GyroADC block clock. - power-domains: Must contain a reference to the PM domain, if available. - #address-cells: Should be <1> (setting for the subnodes) for all ADCs except for "fujitsu,mb88101a". Should be <0> (setting for @@ -75,8 +74,8 @@ Example: adc@e6e54000 { compatible = "renesas,r8a7791-gyroadc", "renesas,rcar-gyroadc"; reg = <0 0xe6e54000 0 64>; - clocks = <&mstp9_clks R8A7791_CLK_GYROADC>, <&clk_65m>; - clock-names = "fck", "if"; + clocks = <&mstp9_clks R8A7791_CLK_GYROADC>; + clock-names = "fck"; power-domains = <&sysc R8A7791_PD_ALWAYS_ON>; pinctrl-0 = <&adc_pins>;
The "if" interface clock speed is actually derived from the "fck" block clock, as in the hardware they are the same clock. Drop the incorrect second "if" clock and retain only the "fck" clock. Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com> Cc: Geert Uytterhoeven <geert+renesas@glider.be> Cc: Jonathan Cameron <jic23@kernel.org> Cc: Rob Herring <robh@kernel.org> Cc: linux-renesas-soc@vger.kernel.org To: devicetree@vger.kernel.org --- Documentation/devicetree/bindings/iio/adc/renesas,gyroadc.txt | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-)