| Message ID | 1530624110-4687-3-git-send-email-narmstrong@baylibre.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | None | expand |
On Tue, Jul 03, 2018 at 03:21:49PM +0200, Neil Armstrong wrote: > The Amlogic Meson GX SoCs embeds a clock measurer IP to measure the internal > clock paths frequencies. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > .../devicetree/bindings/soc/amlogic/clk-measure.txt | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt > > diff --git a/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt b/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt > new file mode 100644 > index 0000000..ba9183a > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt > @@ -0,0 +1,15 @@ > +Amlogic Internal Clock Measurer > +=============================== > + > +The Amlogic SoCs contains an IP to measure the internal clocks. > +The precision is multiple of MHz, useful to debug the clock states. What a simple yet useful debug tool. > + > +Required properties: > +- compatible: Shall contain "amlogic,meson-gx-clk-measure" > +- reg: base address and size of the Clock Measurer register space. > + > +Example: > + clock-measure@8758 { > + compatible = "amlogic,meson-gx-clk-measure"; > + reg = <0x0 0x8758 0x0 0x10>; This doesn't really look like its own block though. Can't the parent device instantiate the driver for this? > + }; > -- > 2.7.4 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- 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
Hi Rob, On 16/07/2018 23:45, Rob Herring wrote: > On Tue, Jul 03, 2018 at 03:21:49PM +0200, Neil Armstrong wrote: >> The Amlogic Meson GX SoCs embeds a clock measurer IP to measure the internal >> clock paths frequencies. >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> .../devicetree/bindings/soc/amlogic/clk-measure.txt | 15 +++++++++++++++ >> 1 file changed, 15 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt >> >> diff --git a/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt b/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt >> new file mode 100644 >> index 0000000..ba9183a >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt >> @@ -0,0 +1,15 @@ >> +Amlogic Internal Clock Measurer >> +=============================== >> + >> +The Amlogic SoCs contains an IP to measure the internal clocks. >> +The precision is multiple of MHz, useful to debug the clock states. > > What a simple yet useful debug tool. > >> + >> +Required properties: >> +- compatible: Shall contain "amlogic,meson-gx-clk-measure" >> +- reg: base address and size of the Clock Measurer register space. >> + >> +Example: >> + clock-measure@8758 { >> + compatible = "amlogic,meson-gx-clk-measure"; >> + reg = <0x0 0x8758 0x0 0x10>; > > This doesn't really look like its own block though. Can't the parent > device instantiate the driver for this? As weird as it seems, on the GX family the registers are not on a aligned boundary, but yes it's its own block. On newer SoCs, they have a more aligned register address, with the exact same register set. Neil > > >> + }; >> -- >> 2.7.4 >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- 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/soc/amlogic/clk-measure.txt b/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt new file mode 100644 index 0000000..ba9183a --- /dev/null +++ b/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt @@ -0,0 +1,15 @@ +Amlogic Internal Clock Measurer +=============================== + +The Amlogic SoCs contains an IP to measure the internal clocks. +The precision is multiple of MHz, useful to debug the clock states. + +Required properties: +- compatible: Shall contain "amlogic,meson-gx-clk-measure" +- reg: base address and size of the Clock Measurer register space. + +Example: + clock-measure@8758 { + compatible = "amlogic,meson-gx-clk-measure"; + reg = <0x0 0x8758 0x0 0x10>; + };
The Amlogic Meson GX SoCs embeds a clock measurer IP to measure the internal clock paths frequencies. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- .../devicetree/bindings/soc/amlogic/clk-measure.txt | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt