From: Junyi Zhao via B4 Relay <devnull+junyi.zhao.amlogic.com@kernel.org>
To: Alexandre Belloni <alexandre.belloni@bootlin.com>,
Frank Li <Frank.Li@nxp.com>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Neil Armstrong <neil.armstrong@linaro.org>,
Kevin Hilman <khilman@baylibre.com>,
Jerome Brunet <jbrunet@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
Kees Cook <kees@kernel.org>,
"Gustavo A. R. Silva" <gustavoars@kernel.org>
Cc: linux-i3c@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-amlogic@lists.infradead.org, imx@lists.linux.dev,
linux-hardening@vger.kernel.org, xianwei.zhao@amlogic.com,
Kelvin.Zhang@amlogic.com, Junyi Zhao <junyi.zhao@amlogic.com>
Subject: [PATCH 3/3] i3c: master: svc: Add Amlogic A9 glue layer
Date: Mon, 19 Oct 2026 13:34:28 +0800 [thread overview]
Message-ID: <20261019-a9_i3c-v1-3-329d819d621f@amlogic.com> (raw)
In-Reply-To: <20261019-a9_i3c-v1-0-329d819d621f@amlogic.com>
From: Junyi Zhao <junyi.zhao@amlogic.com>
Add support for the Amlogic A9 implementation of the Silvaco
SVC I3C controller.
The glue layer provides Amlogic-specific operations and resource
handling while reusing the common SVC I3C core driver.
Add a corresponding MAINTAINERS entry for the Amlogic I3C
controller.
Signed-off-by: Junyi Zhao <junyi.zhao@amlogic.com>
---
MAINTAINERS | 6 +
drivers/i3c/master/svc/Kconfig | 7 +
drivers/i3c/master/svc/Makefile | 1 +
drivers/i3c/master/svc/i3c-amlogic.c | 250 +++++++++++++++++++++++++++++++++++
4 files changed, 264 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index dffc63af2..478979446 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1398,6 +1398,12 @@ F: Documentation/userspace-api/media/v4l/metafmt-c3-isp.rst
F: drivers/media/platform/amlogic/c3/isp/
F: include/uapi/linux/media/amlogic/
+AMLOGIC I3C DRIVER
+M: Junyi Zhao <junyi.zhao@amlogic.com>
+S: Maintained
+F: Documentation/devicetree/bindings/i3c/amlogic,i3c-a9.yaml
+F: drivers/i3c/master/svc/i3c-amlogic.c
+
AMLOGIC MIPI ADAPTER DRIVER
M: Keke Li <keke.li@amlogic.com>
L: linux-media@vger.kernel.org
diff --git a/drivers/i3c/master/svc/Kconfig b/drivers/i3c/master/svc/Kconfig
index 14fa1e542..df3528bc1 100644
--- a/drivers/i3c/master/svc/Kconfig
+++ b/drivers/i3c/master/svc/Kconfig
@@ -18,4 +18,11 @@ config I3C_SILVACO
compatible strings such as "silvaco,i3c-master-v1" and
"nuvoton,npcm845-i3c".
+config I3C_AMLOGIC
+ tristate "Amlogic I3C support"
+ default y
+ help
+ This selects the Amlogic SoC glue layer support for
+ the Silvaco device driver. This driver is used for A9, C5 SoCs.
+
endif # SVC_I3C
diff --git a/drivers/i3c/master/svc/Makefile b/drivers/i3c/master/svc/Makefile
index cf18c5da6..695863d1f 100644
--- a/drivers/i3c/master/svc/Makefile
+++ b/drivers/i3c/master/svc/Makefile
@@ -3,3 +3,4 @@
obj-$(CONFIG_SVC_I3C) += i3c-main.o
obj-$(CONFIG_I3C_SILVACO) += i3c-silvaco.o
+obj-$(CONFIG_I3C_AMLOGIC) += i3c-amlogic.o
diff --git a/drivers/i3c/master/svc/i3c-amlogic.c b/drivers/i3c/master/svc/i3c-amlogic.c
new file mode 100644
index 000000000..82300d6ae
--- /dev/null
+++ b/drivers/i3c/master/svc/i3c-amlogic.c
@@ -0,0 +1,250 @@
+// SPDX-License-Identifier: (GPL-2.0-only OR MIT)
+/*
+ * Copyright (C) 2026 Amlogic, Inc. All rights reserved
+ */
+#include "i3c-main.h"
+#include <linux/math64.h>
+
+/* Amlogic specific registers */
+#define I3C_BG_RD_ADDR 0x200
+#define I3C_BG_WR_ADDR 0x204
+#define I3C_BG_WR_CTRL 0x208
+#define I3C_BG_R_TT_LEN 0x20c
+#define I3C_BG_X_CNT 0x210
+#define I3C_BUS_MON_CTRL 0x214
+#define I3C_BUS_MON_ADDR 0x218
+#define I3C_BUS_MON_DATA 0x21c
+#define I3C_BUS_MON_DATA_MAK 0x220
+#define I3C_CLK_SLOW_DIV_CTRL 0x224
+#define I3C_BG_R_WCNT_CLR 0x228
+#define I3C_DEBUG_MODIFY 0x22c
+
+#define I3C_AO_CLK_SLOW_DIV_CTRL 0x300
+#define I3C_AO_FCLK_DIV_CTRL 0x304
+#define I3C_AO_FCLK_DIV_MASK GENMASK(7, 0)
+#define I3C_AO_FCLK_DIV_EN BIT(8)
+#define I3C_AO_FCLK_EN BIT(9)
+#define I3C_AO_DEBUG_MODIFY 0x308
+
+#define I3C_DEFAULT_FCLK_RATE 25000000UL
+
+struct amlogic_i3c_priv {
+ int num_clks;
+ struct clk_bulk_data *clks;
+};
+
+struct amlogic_i3c_drvdata {
+ int (*fclk_rate_init)(struct svc_i3c_master *master,
+ struct clk *fclk, unsigned long rate);
+};
+
+static int default_i3c_fclk_rate_init(struct svc_i3c_master *master,
+ struct clk *fclk, unsigned long rate)
+{
+ int err;
+
+ err = clk_set_rate(fclk, rate);
+ if (err)
+ return dev_err_probe(master->dev, err,
+ "setting fast clock rate failed\n");
+
+ return 0;
+}
+
+static int ao_i3c_fclk_rate_init(struct svc_i3c_master *master,
+ struct clk *fclk, unsigned long rate)
+{
+ unsigned int div_cnt, reg_val;
+ unsigned long fin_freq = clk_get_rate(fclk);
+
+ if (!fin_freq)
+ return dev_err_probe(master->dev, -EINVAL,
+ "can't get I3C fast clock rate\n");
+
+ div_cnt = div64_u64(fin_freq, rate);
+ if (!div_cnt || div_cnt > I3C_AO_FCLK_DIV_MASK)
+ return dev_err_probe(master->dev, -EINVAL,
+ "requested fast clock frequency too low\n");
+
+ reg_val = I3C_AO_FCLK_DIV_EN | I3C_AO_FCLK_EN | (div_cnt - 1);
+ writel(reg_val, master->regs + I3C_AO_FCLK_DIV_CTRL);
+
+ return 0;
+}
+
+static int amlogic_i3c_probe(struct platform_device *pdev)
+{
+ struct device *dev = &pdev->dev;
+ const struct amlogic_i3c_drvdata *data;
+ struct svc_i3c_master *master;
+ struct amlogic_i3c_priv *priv;
+ struct clk *fclk;
+ int ret, i;
+
+ master = svc_i3c_master_alloc(dev);
+ if (!master)
+ return -ENOMEM;
+
+ priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
+ if (!priv)
+ return -ENOMEM;
+
+ master->drvdata = of_device_get_match_data(dev);
+ if (!master->drvdata)
+ return -EINVAL;
+
+ data = master->drvdata->bsp_data;
+ if (!data)
+ return -EINVAL;
+
+ master->regs = devm_platform_ioremap_resource(pdev, 0);
+ if (IS_ERR(master->regs))
+ return PTR_ERR(master->regs);
+
+ master->irq = platform_get_irq(pdev, 0);
+ if (master->irq < 0)
+ return master->irq;
+
+ priv->num_clks = devm_clk_bulk_get_all(dev, &priv->clks);
+ if (priv->num_clks < 0)
+ return dev_err_probe(dev, -EINVAL, "can't get I3C clocks\n");
+
+ for (i = 0; i < priv->num_clks; i++) {
+ if (!strcmp(priv->clks[i].id, "fast_clk"))
+ break;
+ }
+
+ if (i == priv->num_clks)
+ return dev_err_probe(dev, -EINVAL,
+ "can't get I3C peripheral clock\n");
+
+ if (IS_ERR(priv->clks[i].clk))
+ return PTR_ERR(priv->clks[i].clk);
+ fclk = priv->clks[i].clk;
+
+ ret = clk_bulk_prepare_enable(priv->num_clks, priv->clks);
+ if (ret)
+ return dev_err_probe(dev, ret, "can't enable I3C clocks\n");
+
+ master->dev = dev;
+ master->bsp_priv = priv;
+ ret = data->fclk_rate_init(master, fclk, I3C_DEFAULT_FCLK_RATE);
+ if (ret)
+ goto err_disable_clks;
+ master->fclk_rate = I3C_DEFAULT_FCLK_RATE;
+
+ pm_runtime_set_autosuspend_delay(dev, SVC_I3C_PM_TIMEOUT_MS);
+ pm_runtime_use_autosuspend(dev);
+ pm_runtime_get_noresume(dev);
+ pm_runtime_set_active(dev);
+ pm_runtime_enable(dev);
+
+ ret = svc_i3c_master_probe(pdev, master);
+ if (ret)
+ goto rpm_disable;
+
+ platform_set_drvdata(pdev, master);
+ pm_runtime_put_autosuspend(dev);
+
+ return 0;
+
+rpm_disable:
+ pm_runtime_dont_use_autosuspend(dev);
+ pm_runtime_put_noidle(dev);
+ pm_runtime_disable(dev);
+ pm_runtime_set_suspended(dev);
+
+err_disable_clks:
+ clk_bulk_disable_unprepare(priv->num_clks, priv->clks);
+
+ return ret;
+}
+
+static void amlogic_i3c_remove(struct platform_device *pdev)
+{
+ struct svc_i3c_master *master = platform_get_drvdata(pdev);
+
+ svc_i3c_master_remove(master);
+
+ pm_runtime_dont_use_autosuspend(&pdev->dev);
+ pm_runtime_disable(&pdev->dev);
+}
+
+static int __maybe_unused amlogic_i3c_runtime_suspend(struct device *dev)
+{
+ struct svc_i3c_master *master = dev_get_drvdata(dev);
+ struct amlogic_i3c_priv *priv = master->bsp_priv;
+
+ svc_i3c_save_regs(master);
+ clk_bulk_disable_unprepare(priv->num_clks, priv->clks);
+ pinctrl_pm_select_sleep_state(dev);
+
+ return 0;
+}
+
+static int __maybe_unused amlogic_i3c_runtime_resume(struct device *dev)
+{
+ struct svc_i3c_master *master = dev_get_drvdata(dev);
+ struct amlogic_i3c_priv *priv = master->bsp_priv;
+ int ret;
+
+ pinctrl_pm_select_default_state(dev);
+ ret = clk_bulk_prepare_enable(priv->num_clks, priv->clks);
+ if (ret)
+ return ret;
+
+ svc_i3c_restore_regs(master);
+
+ return 0;
+}
+
+static const struct dev_pm_ops amlogic_i3c_pm_ops = {
+ SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
+ pm_runtime_force_resume)
+ SET_RUNTIME_PM_OPS(amlogic_i3c_runtime_suspend,
+ amlogic_i3c_runtime_resume, NULL)
+};
+
+static const struct amlogic_i3c_drvdata amlogic_i3c_data = {
+ .fclk_rate_init = default_i3c_fclk_rate_init,
+};
+
+static const struct amlogic_i3c_drvdata amlogic_i3c_ao_data = {
+ .fclk_rate_init = ao_i3c_fclk_rate_init,
+};
+
+static const struct svc_i3c_drvdata amlogic_a9_drvdata = {
+ .quirks = SVC_I3C_QUIRK_FIFO_EMPTY |
+ SVC_I3C_QUIRK_FALSE_SLVSTART |
+ SVC_I3C_QUIRK_DAA_CORRUPT,
+ .bsp_data = &amlogic_i3c_data,
+};
+
+static const struct svc_i3c_drvdata amlogic_a9_ao_drvdata = {
+ .quirks = SVC_I3C_QUIRK_FIFO_EMPTY |
+ SVC_I3C_QUIRK_FALSE_SLVSTART |
+ SVC_I3C_QUIRK_DAA_CORRUPT,
+ .bsp_data = &amlogic_i3c_ao_data,
+};
+
+static const struct of_device_id amlogic_i3c_of_match_tbl[] = {
+ { .compatible = "amlogic,i3c-a9", .data = &amlogic_a9_drvdata },
+ { .compatible = "amlogic,i3c-ao-a9", .data = &amlogic_a9_ao_drvdata },
+ { /* sentinel */ },
+};
+MODULE_DEVICE_TABLE(of, amlogic_i3c_of_match_tbl);
+
+static struct platform_driver amlogic_i3c_master = {
+ .probe = amlogic_i3c_probe,
+ .remove = amlogic_i3c_remove,
+ .driver = {
+ .name = "amlogic-i3c-master",
+ .of_match_table = amlogic_i3c_of_match_tbl,
+ .pm = &amlogic_i3c_pm_ops,
+ },
+};
+module_platform_driver(amlogic_i3c_master);
+
+MODULE_AUTHOR("Junyi Zhao <junyi.zhao@amlogic.com>");
+MODULE_DESCRIPTION("Amlogic I3C master driver");
+MODULE_LICENSE("GPL");
--
2.42.0
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
next prev parent reply other threads:[~2026-10-19 5:34 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-19 5:34 [PATCH 0/3] Add vendor glue layer support and Amlogic A9 implementation Junyi Zhao via B4 Relay
2026-10-19 5:34 ` [PATCH 1/3] i3c: master: Rework SVC I3C to support vendor glue layers Junyi Zhao via B4 Relay
2026-10-19 5:34 ` [PATCH 2/3] dt-bindings: i3c: Add Amlogic A9 I3C controller Junyi Zhao via B4 Relay
2026-10-19 6:00 ` Krzysztof Kozlowski
2026-10-19 6:01 ` Krzysztof Kozlowski
2026-10-19 5:34 ` Junyi Zhao via B4 Relay [this message]
2026-10-19 5:52 ` [PATCH 3/3] i3c: master: svc: Add Amlogic A9 glue layer sashiko-bot
2026-10-19 7:40 ` Neil Armstrong
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=20261019-a9_i3c-v1-3-329d819d621f@amlogic.com \
--to=devnull+junyi.zhao.amlogic.com@kernel.org \
--cc=Frank.Li@nxp.com \
--cc=Kelvin.Zhang@amlogic.com \
--cc=alexandre.belloni@bootlin.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=gustavoars@kernel.org \
--cc=imx@lists.linux.dev \
--cc=jbrunet@baylibre.com \
--cc=junyi.zhao@amlogic.com \
--cc=kees@kernel.org \
--cc=khilman@baylibre.com \
--cc=krzk+dt@kernel.org \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-hardening@vger.kernel.org \
--cc=linux-i3c@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=miquel.raynal@bootlin.com \
--cc=neil.armstrong@linaro.org \
--cc=robh@kernel.org \
--cc=xianwei.zhao@amlogic.com \
/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
all inboxes | Powered by JetHome®