linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: <Sergey.Semin@baikalelectronics.ru>
To: unlisted-recipients:; (no To-header on input)
Cc: Serge Semin <Sergey.Semin@baikalelectronics.ru>,
	Serge Semin <fancer.lancer@gmail.com>,
	Alexey Malahov <Alexey.Malahov@baikalelectronics.ru>,
	Maxim Kaurkin <Maxim.Kaurkin@baikalelectronics.ru>,
	Pavel Parkhomenko <Pavel.Parkhomenko@baikalelectronics.ru>,
	Ramil Zaripov <Ramil.Zaripov@baikalelectronics.ru>,
	Ekaterina Skachko <Ekaterina.Skachko@baikalelectronics.ru>,
	Vadim Vlasov <V.Vlasov@baikalelectronics.ru>,
	Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	Paul Burton <paulburton@kernel.org>,
	Ralf Baechle <ralf@linux-mips.org>,
	Lee Jones <lee.jones@linaro.org>,
	Sebastian Reichel <sre@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>, <devicetree@vger.kernel.org>,
	<linux-pm@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: [PATCH 0/4] syscon: Alter syscon and reboot-mode drivers
Date: Fri, 6 Mar 2020 16:03:37 +0300	[thread overview]
Message-ID: <20200306130356.6ABDD8030703@mail.baikalelectronics.ru> (raw)

From: Serge Semin <fancer.lancer@gmail.com>

A lot of custom functionality can be tuned by means of a syscon-registers
placed in Baikal-T1 CCU memory space (though in documentation they are
called system devices registers). While in most of the case the syscon
registers usage will be distributed between corresponding subsystem drivers
(thanks to pretty much coherent design of the registers fields), there are
some peculiarities we should reflect in the generic syscon code.

First of all seeing there is an map endiannes config acceptable by the syscon
it would be justified to add the corresponding properties declared in the
mfd/syscon.yaml bindings file. Don't really know why this hasn't been done
before.

Then as it's going to be general in the framework of the Baikal-T1 SoC support
integration into the kernel, we suggest to replace the legacy text-based
syscon-reboot-mode dts-bindings file with yaml-based one.

Finally seeing the syscon-reboot-mode devices are mostly used in conjuction
with just syscon-reboot device (in particular Baikal-T1 CCU WDT RCR register is
preserved during any type of reboots) and for several other reasons (see the
commit message for details), we suggest to add the 'regmap' property support
to the syscon-reboot-mode driver. This would eliminate the requirement of
placing the syscon-reboot-mode dts-node in the syscon sub-nodes area.

This patchset is rebased and tested on the mainline Linux kernel 5.6-rc4:
commit 98d54f81e36b ("Linux 5.6-rc4").

Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru>
Signed-off-by: Alexey Malahov <Alexey.Malahov@baikalelectronics.ru>
Cc: Maxim Kaurkin <Maxim.Kaurkin@baikalelectronics.ru>
Cc: Pavel Parkhomenko <Pavel.Parkhomenko@baikalelectronics.ru>
Cc: Ramil Zaripov <Ramil.Zaripov@baikalelectronics.ru>
Cc: Ekaterina Skachko <Ekaterina.Skachko@baikalelectronics.ru>
Cc: Vadim Vlasov <V.Vlasov@baikalelectronics.ru>
Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Cc: Paul Burton <paulburton@kernel.org>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Sebastian Reichel <sre@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: devicetree@vger.kernel.org
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org

Serge Semin (4):
  dt-bindings: syscon: Add syscon endian properties support
  dt-bindings: power: reset: Replace SYSCON reboot-mode legacy bindings
    with YAML-based one
  dt-bindings: power: reset: Add regmap support to the SYSCON
    reboot-mode bindings
  power: reset: syscon-reboot-mode: Add regmap dts-property support

 .../devicetree/bindings/mfd/syscon.yaml       | 14 +++++
 .../power/reset/syscon-reboot-mode.txt        | 35 -----------
 .../power/reset/syscon-reboot-mode.yaml       | 58 +++++++++++++++++++
 drivers/power/reset/syscon-reboot-mode.c      | 11 +++-
 4 files changed, 80 insertions(+), 38 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt
 create mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml

-- 
2.25.1


             reply	other threads:[~2020-03-06 13:04 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-06 13:03 Sergey.Semin [this message]
2020-03-09  7:13 ` [PATCH 0/4] syscon: Alter syscon and reboot-mode drivers Lee Jones
     [not found] ` <20200309071233.12D6980307C8@mail.baikalelectronics.ru>
2020-03-10  0:35   ` Sergey Semin
2020-05-07 23:38 ` [PATCH v2 0/3] syscon: Alter syscon and reboot drivers Serge Semin
2020-05-07 23:38   ` [PATCH v2 1/3] dt-bindings: power: reset: Convert syscon-reboot-mode to DT schema Serge Semin
2020-05-09  0:30     ` Sebastian Reichel
2020-05-10  8:47       ` Serge Semin
2020-05-07 23:38   ` [PATCH v2 2/3] dt-bindings: power: reset: Unrequire regmap property in syscon-reboot node Serge Semin
2020-05-15  3:14     ` Rob Herring
2020-05-16 13:26       ` Serge Semin
2020-05-07 23:38   ` [PATCH v2 3/3] power: reset: syscon-reboot: Add parental syscon support Serge Semin

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=20200306130356.6ABDD8030703@mail.baikalelectronics.ru \
    --to=sergey.semin@baikalelectronics.ru \
    --cc=Alexey.Malahov@baikalelectronics.ru \
    --cc=Ekaterina.Skachko@baikalelectronics.ru \
    --cc=Maxim.Kaurkin@baikalelectronics.ru \
    --cc=Pavel.Parkhomenko@baikalelectronics.ru \
    --cc=Ramil.Zaripov@baikalelectronics.ru \
    --cc=V.Vlasov@baikalelectronics.ru \
    --cc=devicetree@vger.kernel.org \
    --cc=fancer.lancer@gmail.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=paulburton@kernel.org \
    --cc=ralf@linux-mips.org \
    --cc=robh+dt@kernel.org \
    --cc=sre@kernel.org \
    --cc=tsbogend@alpha.franken.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).