archive mirror
 help / color / mirror / Atom feed
From: "tip-bot2 for 周琰杰" <>
	Rob Herring <>,
	Daniel Lezcano <>,,
Subject: [tip: timers/core] dt-bindings: timer: Add new OST support for the upcoming new driver.
Date: Thu, 03 Dec 2020 23:47:51 -0000	[thread overview]
Message-ID: <160703927157.3364.4500387551340496666.tip-bot2@tip-bot2> (raw)
In-Reply-To: <>

The following commit has been merged into the timers/core branch of tip:

Commit-ID:     0fce2e02a29ca5420472f03d3f2858eedded3fe7
Author:        周琰杰 (Zhou Yanjie) <>
AuthorDate:    Mon, 26 Oct 2020 23:58:42 +08:00
Committer:     Daniel Lezcano <>
CommitterDate: Thu, 03 Dec 2020 19:16:18 +01:00

dt-bindings: timer: Add new OST support for the upcoming new driver.

The new OST has one global timer and two or four percpu timers, so there
will be three combinations in the upcoming new OST driver: the original
GLOBAL_TIMER + PERCPU_TIMER0/1/2/3, For this, add the macro definition
about OST_CLK_PERCPU_TIMER0/1/2/3. And in order to ensure that all the
combinations work normally, the original ABI values of OST_CLK_PERCPU_TIMER
and OST_CLK_GLOBAL_TIMER need to be exchanged to ensure that in any
combinations, the clock can be registered (by calling clk_hw_register())
from index 0.

Before this patch, OST_CLK_PERCPU_TIMER and OST_CLK_GLOBAL_TIMER are only
used in two places, one is when using "assigned-clocks" to configure the
clocks in the DTS file; the other is when registering the clocks in the
sysost driver. When the values of these two ABIs are exchanged, the ABI
value used by sysost driver when registering the clock, and the ABI value
used by DTS when configuring the clock using "assigned-clocks" will also
change accordingly. Therefore, there is no situation that causes the wrong
clock to the configured. Therefore, exchanging ABI values will not cause
errors in the existing codes when registering and configuring the clocks.

Currently, in the mainline, only X1000 and X1830 are using sysost driver,
and the upcoming X2000 will also use sysost driver. This patch has been
tested on all three SoCs and all works fine.

Tested-by: 周正 (Zhou Zheng) <>
Signed-off-by: 周琰杰 (Zhou Yanjie) <>
Reviewed-by: Rob Herring <>
Signed-off-by: Daniel Lezcano <>
 include/dt-bindings/clock/ingenic,sysost.h | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/include/dt-bindings/clock/ingenic,sysost.h b/include/dt-bindings/clock/ingenic,sysost.h
index 9ac88e9..063791b 100644
--- a/include/dt-bindings/clock/ingenic,sysost.h
+++ b/include/dt-bindings/clock/ingenic,sysost.h
@@ -1,12 +1,16 @@
 /* SPDX-License-Identifier: GPL-2.0 */
- * This header provides clock numbers for the ingenic,tcu DT binding.
+ * This header provides clock numbers for the Ingenic OST DT binding.

      parent reply	other threads:[~2020-12-03 23:49 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-26 15:58 [PATCH v3 0/1] Add macro definition for the upcoming new OST driver 周琰杰 (Zhou Yanjie)
2020-10-26 15:58 ` [PATCH v3 1/1] dt-bindings: timer: Add new OST support for the upcoming new driver 周琰杰 (Zhou Yanjie)
2020-10-30 18:45   ` Rob Herring
2020-12-03 23:47   ` tip-bot2 for 周琰杰 [this message]

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:

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

  git send-email \
    --in-reply-to=160703927157.3364.4500387551340496666.tip-bot2@tip-bot2 \ \ \ \ \ \ \ \ \

* 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).