lkml.org 
[lkml]   [2018]   [Jul]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 2/3] dt-bindings: amlogic: Add Internal Clock Measurer bindings
    From
    Date
    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

    \
     
     \ /
      Last update: 2018-07-17 13:31    [W:3.182 / U:0.700 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site