All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: Rob Herring <rob.herring@linaro.org>
Cc: andrew@lunn.ch, Jason Cooper <jason@lakedaemon.net>,
	linux-pm@vger.kernel.org, antoine.tenart@bootlin.com,
	Viresh Kumar <viresh.kumar@linaro.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	maxime.chevallier@bootlin.com, nadavh@marvell.com,
	make@marvell.com, xigu@marvell.com, neta@marvell.com,
	thomas.petazzoni@bootlin.com, miquel.raynal@bootlin.com,
	jinghua@marvell.com, mw@semihalf.com, dingwei@marvell.com,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	sebastian.hesselbarth@gmail.com
Subject: Re: [PATCH 1/3] dt-bindings: marvell: Add documentation for the Armada 3700 AVS binding
Date: Wed, 04 Jul 2018 15:26:08 +0200	[thread overview]
Message-ID: <87tvpf2kun.fsf@bootlin.com> (raw)
In-Reply-To: <CABGGisyEO+XVQm87sVn-D0SbkRrqHfRo0R9iVc=PjbY5uyUemA@mail.gmail.com> (Rob Herring's message of "Tue, 3 Jul 2018 10:13:29 -0600")

Hi Rob,
 
 On mar., juil. 03 2018, Rob Herring <rob.herring@linaro.org> wrote:

> On Tue, Jul 3, 2018 at 8:46 AM Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
>>
>> On Tuesday, June 19, 2018 2:44:00 PM CEST Gregory CLEMENT wrote:
>> > Extend the documentation of the Armada 37xx SoC with the Adaptive Voltage
>> > Scaling (AVS) registers.
>> >
>> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>
>> > ---
>> >  .../bindings/arm/marvell/armada-37xx.txt         | 16 ++++++++++++++++
>> >  1 file changed, 16 insertions(+)
>> >
>> > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> > index 35c3c3460d17..22438f659d1e 100644
>> > --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> > @@ -33,3 +33,19 @@ nb_pm: syscon@14000 {
>> >       compatible = "marvell,armada-3700-nb-pm", "syscon";
>> >       reg = <0x14000 0x60>;
>> >  }
>> > +
>> > +AVS
>> > +---
>> > +
>> > +For AVS an other component is needed:
>> > +
>> > +Required properties:
>> > +- compatible     : should contain "marvell,armada-3700-avs", "syscon";
>> > +- reg            : the register start and length for the AVS
>> > +
>> > +Example:
>> > +avs: avs@11500 {
>> > +     compatible = "marvell,armada-3700-avs", "syscon";
>> > +     reg = <0x11500 0x40>;
>> > +}
>> > +
>> >
>>
>> I can't apply this one without a Rob's ACK.
>
> Which won't happen if not sent to the DT list.

Now it's done.

Gregory

>
> Rob

-- 
Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

WARNING: multiple messages have this Message-ID (diff)
From: gregory.clement@bootlin.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] dt-bindings: marvell: Add documentation for the Armada 3700 AVS binding
Date: Wed, 04 Jul 2018 15:26:08 +0200	[thread overview]
Message-ID: <87tvpf2kun.fsf@bootlin.com> (raw)
In-Reply-To: <CABGGisyEO+XVQm87sVn-D0SbkRrqHfRo0R9iVc=PjbY5uyUemA@mail.gmail.com> (Rob Herring's message of "Tue, 3 Jul 2018 10:13:29 -0600")

Hi Rob,
 
 On mar., juil. 03 2018, Rob Herring <rob.herring@linaro.org> wrote:

> On Tue, Jul 3, 2018 at 8:46 AM Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
>>
>> On Tuesday, June 19, 2018 2:44:00 PM CEST Gregory CLEMENT wrote:
>> > Extend the documentation of the Armada 37xx SoC with the Adaptive Voltage
>> > Scaling (AVS) registers.
>> >
>> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>
>> > ---
>> >  .../bindings/arm/marvell/armada-37xx.txt         | 16 ++++++++++++++++
>> >  1 file changed, 16 insertions(+)
>> >
>> > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> > index 35c3c3460d17..22438f659d1e 100644
>> > --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> > @@ -33,3 +33,19 @@ nb_pm: syscon at 14000 {
>> >       compatible = "marvell,armada-3700-nb-pm", "syscon";
>> >       reg = <0x14000 0x60>;
>> >  }
>> > +
>> > +AVS
>> > +---
>> > +
>> > +For AVS an other component is needed:
>> > +
>> > +Required properties:
>> > +- compatible     : should contain "marvell,armada-3700-avs", "syscon";
>> > +- reg            : the register start and length for the AVS
>> > +
>> > +Example:
>> > +avs: avs at 11500 {
>> > +     compatible = "marvell,armada-3700-avs", "syscon";
>> > +     reg = <0x11500 0x40>;
>> > +}
>> > +
>> >
>>
>> I can't apply this one without a Rob's ACK.
>
> Which won't happen if not sent to the DT list.

Now it's done.

Gregory

>
> Rob

-- 
Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

  reply	other threads:[~2018-07-04 13:26 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-19 12:43 [PATCH 0/3] cpufreq: add AVS support for Armada 3700 Gregory CLEMENT
2018-06-19 12:43 ` Gregory CLEMENT
2018-06-19 12:44 ` [PATCH 1/3] dt-bindings: marvell: Add documentation for the Armada 3700 AVS binding Gregory CLEMENT
2018-06-19 12:44   ` Gregory CLEMENT
2018-06-29  9:54   ` Viresh Kumar
2018-06-29  9:54     ` Viresh Kumar
2018-07-03 14:44   ` Rafael J. Wysocki
2018-07-03 14:44     ` Rafael J. Wysocki
2018-07-03 16:13     ` Rob Herring
2018-07-03 16:13       ` Rob Herring
2018-07-04 13:26       ` Gregory CLEMENT [this message]
2018-07-04 13:26         ` Gregory CLEMENT
2018-06-19 12:44 ` [PATCH 2/3] cpufreq: armada-37xx: Add AVS support Gregory CLEMENT
2018-06-19 12:44   ` Gregory CLEMENT
2018-06-29  9:54   ` Viresh Kumar
2018-06-29  9:54     ` Viresh Kumar
2018-06-19 12:44 ` [PATCH 3/3] arm64: dts: marvell: armada-37xx: add the node allowing " Gregory CLEMENT
2018-06-19 12:44   ` Gregory CLEMENT
2018-06-29  9:54   ` Viresh Kumar
2018-06-29  9:54     ` Viresh Kumar
2018-06-29 14:36     ` Gregory CLEMENT
2018-06-29 14:36       ` Gregory CLEMENT

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tvpf2kun.fsf@bootlin.com \
    --to=gregory.clement@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=antoine.tenart@bootlin.com \
    --cc=dingwei@marvell.com \
    --cc=jason@lakedaemon.net \
    --cc=jinghua@marvell.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=make@marvell.com \
    --cc=maxime.chevallier@bootlin.com \
    --cc=miquel.raynal@bootlin.com \
    --cc=mw@semihalf.com \
    --cc=nadavh@marvell.com \
    --cc=neta@marvell.com \
    --cc=rjw@rjwysocki.net \
    --cc=rob.herring@linaro.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=viresh.kumar@linaro.org \
    --cc=xigu@marvell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.