mbox series

[0/4] soc: amlogic: clk-measure: add axg and g12a support

Message ID 20190117164838.8008-1-jbrunet@baylibre.com
Headers show
Series soc: amlogic: clk-measure: add axg and g12a support | expand

Message

Jerome Brunet Jan. 17, 2019, 4:48 p.m. UTC 
This patchset adds support for the axg and g12a SoC family in amlogic's
clock measure driver and enable it on the related devices

Jerome Brunet (4):
  dt-bindings: amlogic: add new compatible devices to clk_measure
  soc: amlogic: clk-measure: add axg and g12a support
  arm64: dts: meson: axg: add clk measure support
  arm64: dts: meson: g12a: add clk measure support

 .../bindings/soc/amlogic/clk-measure.txt      |   2 +
 arch/arm64/boot/dts/amlogic/meson-axg.dtsi    |   5 +
 arch/arm64/boot/dts/amlogic/meson-g12a.dtsi   |   5 +
 drivers/soc/amlogic/meson-clk-measure.c       | 194 ++++++++++++++++++
 4 files changed, 206 insertions(+)

-- 
2.20.1

Comments

Martin Blumenstingl Jan. 17, 2019, 7:57 p.m. UTC | #1 
Hi Jerome,

On Thu, Jan 17, 2019 at 5:48 PM Jerome Brunet <jbrunet@baylibre.com> wrote:
>

> Add the clock measure device to the axg SoC family

>

> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>

> ---

>  arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 5 +++++

>  1 file changed, 5 insertions(+)

>

> diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> index 0e0abb1a03e2..94ea36a2417a 100644

> --- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> +++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> @@ -1611,6 +1611,11 @@

>                                 status = "disabled";

>                         };

>

> +                       clk_msr: clock-sensor@18000 {

in the dt-bindings specify "clock-measure@..."
this also applies to patch 4/4 from this series


[0] https://elixir.bootlin.com/linux/v5.0-rc1/source/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt
Jerome Brunet Jan. 18, 2019, 7:52 a.m. UTC | #2 
On Thu, 2019-01-17 at 20:57 +0100, Martin Blumenstingl wrote:
> Hi Jerome,

> 

> On Thu, Jan 17, 2019 at 5:48 PM Jerome Brunet <jbrunet@baylibre.com> wrote:

> > Add the clock measure device to the axg SoC family

> > 

> > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>

> > ---

> >  arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 5 +++++

> >  1 file changed, 5 insertions(+)

> > 

> > diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> > b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> > index 0e0abb1a03e2..94ea36a2417a 100644

> > --- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> > +++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi

> > @@ -1611,6 +1611,11 @@

> >                                 status = "disabled";

> >                         };

> > 

> > +                       clk_msr: clock-sensor@18000 {

> in the dt-bindings specify "clock-measure@..."


That's the exemple, not a specification, AFAIK.
I'm not sure I see the problem ?

> this also applies to patch 4/4 from this series



Anyway, The node was not placed correctly in g12, I might as well rename it. I
don't really care

> 

> 

> [0] 

> https://elixir.bootlin.com/linux/v5.0-rc1/source/Documentation/devicetree/bindings/soc/amlogic/clk-measure.txt