All of lore.kernel.org
 help / color / mirror / Atom feed
From: Auto Configured <romain.perier@gmail.com>
To: heiko@sntech.de
Cc: grant.likely@linaro.org, robh@kernel.org,
	devicetree@vger.kernel.org, lgirdwood@gmail.com,
	broonie@kernel.org, johan@kernel.org, mark.rutland@arm.com,
	linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, balbi@ti.com
Subject: [PATCH v4 0/2] poweroff-source DT property renaming
Date: Thu, 13 Nov 2014 13:34:57 +0000	[thread overview]
Message-ID: <1415885699-1465-1-git-send-email-user@radxa> (raw)

The goal of this serie is to rename the property "poweroff-source" to
"system-power-controller". After discussions, it makes more sense to rename back
to the old established property name. This patches serie reverts
commit a4b4e0461ec5 ("of: Add standard property for poweroff capability").

Changes since v3:
  - Move backward compatibility from of_is_system_power_controller to
    tps65910, as it was only required for this driver.

Changes since v2:
  - of_is_system_power_controller is inline in of.h and only supports
    backward compatibility with device drivers which used this helper
    (which is tps65910)
  - Quashed all changes into a single patch (except documentation)
  - Add a separated commit to fix typos and mispellings in documentation

Changes since v1:

  - The first patch defines "of_is_system_power_controller" which is compatible
    with both "vendor,system-power-controller" and "system-power-controller"
    properties. Also, It keeps the old helper function of_system_has_poweroff_source
    for source compatibility until everything is renamed (in this way, bisections
    are not broken and change is made "atomically" between each commit)

    Note: the property "poweroff-source" itself is not used in dts files yet.
    Before this patch tps65910 was broken due to missing backward compatibility
    with "vendor,system-power-controller". As the old helper uses the new one,
    it works again.

  - act8865 and tps65910 are ported to the new helper function
  - The last commit removes the olf helper which was only used for source compatibility

Romain Perier (2):
  of: Rename "poweroff-source" property to "system-power-controller"
  dt-bindings: Update documentation for "system-power-controller" and
    fix misspellings

 .../devicetree/bindings/power/power-controller.txt     | 18 ++++++++++++++++++
 Documentation/devicetree/bindings/power/poweroff.txt   | 18 ------------------
 .../bindings/regulator/act8865-regulator.txt           |  4 ++--
 drivers/mfd/tps65910.c                                 |  9 ++++++++-
 drivers/regulator/act8865-regulator.c                  |  2 +-
 include/linux/of.h                                     |  6 +++---
 6 files changed, 32 insertions(+), 25 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/power/power-controller.txt
 delete mode 100644 Documentation/devicetree/bindings/power/poweroff.txt

-- 
1.9.1


             reply	other threads:[~2014-11-13 13:35 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-13 13:34 Auto Configured [this message]
2014-11-13 13:34 ` [PATCH v4 1/2] of: Rename "poweroff-source" property to "system-power-controller" Auto Configured
2014-11-13 20:55   ` Grant Likely
2014-11-13 20:55     ` Grant Likely
2014-11-13 20:55     ` Grant Likely
2014-11-14  7:22     ` Romain Perier
2014-11-14  7:22       ` Romain Perier
2014-11-17 10:51       ` Romain Perier
2014-11-17 10:51         ` Romain Perier
2014-11-20 10:44         ` Romain Perier
2014-11-20 10:44           ` Romain Perier
2014-11-21 10:59   ` Johan Hovold
2014-11-21 10:59     ` Johan Hovold
2014-11-21 10:59     ` Johan Hovold
2014-11-21 11:00     ` [PATCH] Revert "mfd: tps65910: Convert ti,system-power-controller DT property to poweroff-source" Johan Hovold
2014-11-21 11:00       ` [PATCH] Revert "mfd: tps65910: Convert ti, system-power-controller " Johan Hovold
2014-11-21 13:18     ` [PATCH v4 1/2] of: Rename "poweroff-source" property to "system-power-controller" Romain Perier
2014-11-21 13:18       ` Romain Perier
2014-11-21 13:22       ` Romain Perier
2014-11-21 13:22         ` Romain Perier
2014-11-24 10:24     ` Lee Jones
2014-11-24 10:24       ` Lee Jones
2014-11-24 11:35       ` Johan Hovold
2014-11-24 11:35         ` Johan Hovold
2014-11-24 12:58         ` Romain Perier
2014-11-24 12:58           ` Romain Perier
2014-11-24 13:01           ` Romain Perier
2014-11-24 13:01             ` Romain Perier
2014-11-24 13:12             ` Heiko Stübner
2014-11-24 13:12               ` Heiko Stübner
2014-11-24 13:21           ` Johan Hovold
2014-11-24 13:21             ` Johan Hovold
2014-11-13 13:34 ` [PATCH v4 2/2] dt-bindings: Update documentation for "system-power-controller" and fix misspellings Auto Configured

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=1415885699-1465-1-git-send-email-user@radxa \
    --to=romain.perier@gmail.com \
    --cc=balbi@ti.com \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=grant.likely@linaro.org \
    --cc=heiko@sntech.de \
    --cc=johan@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh@kernel.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.