All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ulrich Hecht <uli+renesas@fpond.eu>
To: linux-renesas-soc@vger.kernel.org
Cc: linux-clk@vger.kernel.org, wsa@the-dreams.de,
	geert@linux-m68k.org, magnus.damm@gmail.com,
	Ulrich Hecht <uli+renesas@fpond.eu>
Subject: [PATCH v4 0/3] clk: renesas: cpg-mssr: add RWDT to critical clocks
Date: Tue, 16 Jun 2020 18:26:23 +0200	[thread overview]
Message-ID: <20200616162626.27944-1-uli+renesas@fpond.eu> (raw)

(was: "clk: renesas: cpg-mssr: add never-disable option")

Hi!

The purpose of this series is to allow a WDT that has been enabled by the
bootloader to survive these events:

- deferred probing of the WDT device, which can lead the clock driver
  to disable the WDT clock until the WDT is re-probed, giving it a
  blind spot
- probe failure in the WDT driver

Following a suggestion by Geert, this revision, instead of adding another
list of clocks with special handling, changes the semantics of the
crit_mod_clks[] array slightly by only marking clocks critical that are
enabled at boot time. That way it can be used for RWDT without forcing the
clock on unnecessarily.

The other existing user of crit_mod_clks[] (INTC-AP) is always on at boot time
and should thus not be affected by this change.

There are a number of Gen2 and RZ/G1 SoCs that have the RWDT clock declared
as critical already in order to allow SMP bringup code to work. That should
still work with this series applied, assuming that the WDT clock is on at
boot time.

CU
Uli


Changes since v3:
- drop separate array, add RWDT to crit_mod_clks[]
- only mark clocks as critical if they are enabled on boot

Changes since v2:
- use the term "never-disable" instead of "ignore-unused"
- do the handling internally instead of relying on the behavior of
  CLK_IGNORE_UNUSED

Changes since v1:
- rename data structures for clarity
- squash SoC-specific patches into one per family


Ulrich Hecht (3):
  clk: renesas: cpg-mssr: mark clocks as critical only if on at boot
  clk: renesas: rcar-gen3: mark RWDT clocks as critical
  clk: renesas: rzg2: mark RWDT clock as critical

 drivers/clk/renesas/r8a774a1-cpg-mssr.c |  1 +
 drivers/clk/renesas/r8a774b1-cpg-mssr.c |  1 +
 drivers/clk/renesas/r8a774c0-cpg-mssr.c |  1 +
 drivers/clk/renesas/r8a7795-cpg-mssr.c  |  2 +-
 drivers/clk/renesas/r8a7796-cpg-mssr.c  |  2 +-
 drivers/clk/renesas/r8a77965-cpg-mssr.c |  1 +
 drivers/clk/renesas/r8a77970-cpg-mssr.c |  2 +-
 drivers/clk/renesas/r8a77980-cpg-mssr.c |  2 +-
 drivers/clk/renesas/r8a77990-cpg-mssr.c |  1 +
 drivers/clk/renesas/r8a77995-cpg-mssr.c |  2 +-
 drivers/clk/renesas/renesas-cpg-mssr.c  | 17 +++++++++--------
 11 files changed, 19 insertions(+), 13 deletions(-)

-- 
2.20.1


             reply	other threads:[~2020-06-16 16:26 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-16 16:26 Ulrich Hecht [this message]
2020-06-16 16:26 ` [PATCH v4 1/3] clk: renesas: cpg-mssr: mark clocks as critical only if on at boot Ulrich Hecht
2020-06-18 11:46   ` Geert Uytterhoeven
2020-06-16 16:26 ` [PATCH v4 2/3] clk: renesas: rcar-gen3: mark RWDT clocks as critical Ulrich Hecht
2020-06-18 11:49   ` Geert Uytterhoeven
2020-06-16 16:26 ` [PATCH v4 3/3] clk: renesas: rzg2: mark RWDT clock " Ulrich Hecht
2020-06-18 11:50   ` Geert Uytterhoeven
2020-06-18 11:45 ` [PATCH v4 0/3] clk: renesas: cpg-mssr: add RWDT to critical clocks Geert Uytterhoeven

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=20200616162626.27944-1-uli+renesas@fpond.eu \
    --to=uli+renesas@fpond.eu \
    --cc=geert@linux-m68k.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=wsa@the-dreams.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 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.