All of lore.kernel.org
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 2/2] Documentation: devicetree: Add PL310 PM bindings
Date: Tue, 19 Apr 2016 23:41:22 +0100	[thread overview]
Message-ID: <20160419224122.GE19428@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <CAL_JsqL-LCt06ZYQL4FEu7vEdHaFCvBe5xnHbkZTf55yAf5Svw@mail.gmail.com>

On Tue, Apr 19, 2016 at 04:38:14PM -0500, Rob Herring wrote:
> On Mon, Apr 18, 2016 at 4:36 PM, Brad Mouring <brad.mouring@ni.com> wrote:
> > Document the DT bindings for controlling ARM PL310 Power Control
> > settings.
> >
> > Signed-off-by: Brad Mouring <brad.mouring@ni.com>
> 
> What happened to Josh's ack?
> 
> > ---
> >  Documentation/devicetree/bindings/arm/l2c2x0.txt | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/l2c2x0.txt b/Documentation/devicetree/bindings/arm/l2c2x0.txt
> > index fe0398c..c1c756e 100644
> > --- a/Documentation/devicetree/bindings/arm/l2c2x0.txt
> > +++ b/Documentation/devicetree/bindings/arm/l2c2x0.txt
> > @@ -84,6 +84,10 @@ Optional properties:
> >  - prefetch-instr : Instruction prefetch. Value: <0> (forcibly disable),
> >    <1> (forcibly enable), property absent (retain settings set by
> >    firmware)
> > +- arm,dynamic-clock-gating : L2 dynamic clock gating. Value: <0> (forcibly
> > +  disable), <1> or property absent (forcibly enable)
> > +- arm,standby-mode: L2 standby mode enable. Value <0> (forcibly disable),
> > +  <1>  or property absent (forcibly enable)
> 
> What happened to "retain settings set by firmware"?

I said we shouldn't.  Current behaviour is that we enable these features,
and moving to missing-means-retain means that everything today ends up
with these features disabled.  IOW, it's yet another change of actual
behaviour that every platform would see.

-- 
RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

  reply	other threads:[~2016-04-19 22:41 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-23 22:03 [PATCH 0/2] Control PL310 pwr_ctrl register through DT Brad Mouring
2016-02-23 22:03 ` Brad Mouring
     [not found] ` <1456265019-23900-1-git-send-email-brad.mouring-acOepvfBmUk@public.gmane.org>
2016-02-23 22:03   ` [PATCH 1/2] pl2x0: Add OF control of cache power management Brad Mouring
2016-02-23 22:03     ` Brad Mouring
2016-02-23 22:03   ` [PATCH 2/2] Documentation: devicetree: Add PL310 PM bindings Brad Mouring
2016-02-23 22:03     ` Brad Mouring
     [not found]     ` <1456265019-23900-3-git-send-email-brad.mouring-acOepvfBmUk@public.gmane.org>
2016-02-24  1:33       ` Rob Herring
2016-02-24  1:33         ` Rob Herring
2016-02-29 16:01   ` [PATCH 1/2] pl2x0: Add OF control of cache power management Brad Mouring
2016-02-29 16:01     ` Brad Mouring
     [not found]     ` <1456761716-10174-1-git-send-email-brad.mouring-acOepvfBmUk@public.gmane.org>
2016-02-29 16:01       ` [PATCH 2/2] Documentation: devicetree: Add PL310 PM bindings Brad Mouring
2016-02-29 16:01         ` Brad Mouring
2016-04-18 21:36 ` [PATCH v2 0/2] Control PL310 pwr_ctrl register through DT Brad Mouring
2016-04-18 21:36   ` [PATCH v2 1/2] pl2x0: Add OF control of cache power management Brad Mouring
2016-04-18 21:36   ` [PATCH v2 2/2] Documentation: devicetree: Add PL310 PM bindings Brad Mouring
2016-04-19 21:38     ` Rob Herring
2016-04-19 22:41       ` Russell King - ARM Linux [this message]
2016-04-20 14:04         ` Rob Herring
2016-04-21 20:48           ` Brad Mouring
2016-04-27  8:59           ` Russell King - ARM Linux
2016-04-27 12:48             ` Rob Herring
2016-04-27 14:08               ` Brad Mouring
2016-04-27 14:34                 ` Rob Herring
2016-04-28 13:57               ` Russell King - ARM Linux
2016-04-20 14:23       ` Brad Mouring
2016-04-27 15:35   ` [PATCH v3 0/2] Control PL310 pwr_ctrl register through DT Brad Mouring
2016-04-27 15:35     ` [PATCH v3 1/2] arm: pl2x0: Add OF control of cache power management Brad Mouring
2016-04-27 15:35     ` [PATCH 2/2] Documentation: devicetree: Add PL310 PM bindings Brad Mouring
2016-04-28 19:56       ` Rob Herring

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=20160419224122.GE19428@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.