From nobody Mon Feb 9 12:29:35 2026 Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2762018A93F; Fri, 26 Dec 2025 06:56:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=140.211.166.183 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766732168; cv=none; b=QcLKUXmgfLK4qwcgr9eqmQWIBRPsvqdCQS3PpEeg04spaa50NXcOHTRxEbiJzEJ2h0N2qegmEssOoH1u02v7OxFRvB+a6SzBucf0i9QdVJpNt0ZqiNOByCtBBlW+bWf06aDtuMJ2JaJk7omfFORNyLfC6432K6FyFaFZrAMBa6w= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766732168; c=relaxed/simple; bh=iXQGtbst7HSGFd1x1UD6Mbf7sKGICkcsk8p9F0XKLng=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=na/di4nfu7pAIIFtmQvIJtZgjNGeLr33g0elifiqZDAn/VZ5PqJyccUl2jyhawRqDdj32aHvfwNjsNvlKZK7k1l8VZ1rzAuAaxMOXyJOxUUfalScDKyt0a1f2S6PRDNcxsCTSxa7zA6I/3hP5qUhKtGVqFFrQEzbjdOwWPBAYMU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gentoo.org; spf=pass smtp.mailfrom=gentoo.org; arc=none smtp.client-ip=140.211.166.183 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gentoo.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gentoo.org Received: from [127.0.0.1] (unknown [116.232.18.222]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: dlan) by smtp.gentoo.org (Postfix) with ESMTPSA id D8572340FF7; Fri, 26 Dec 2025 06:56:02 +0000 (UTC) From: Yixun Lan Date: Fri, 26 Dec 2025 14:55:25 +0800 Subject: [PATCH v2 1/3] clk: spacemit: prepare common ccu header Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251226-06-k1-clk-common-v2-1-28b59418b4df@gentoo.org> References: <20251226-06-k1-clk-common-v2-0-28b59418b4df@gentoo.org> In-Reply-To: <20251226-06-k1-clk-common-v2-0-28b59418b4df@gentoo.org> To: Stephen Boyd , Michael Turquette , Philipp Zabel Cc: Alex Elder , Guodong Xu , Inochi Amaoto , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-riscv@lists.infradead.org, spacemit@lists.linux.dev, Yixun Lan X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=2046; i=dlan@gentoo.org; h=from:subject:message-id; bh=iXQGtbst7HSGFd1x1UD6Mbf7sKGICkcsk8p9F0XKLng=; b=owEB6QIW/ZANAwAKATGq6kdZTbvtAcsmYgBpTjFziZV4+t/TQxLXrsx21nn+sdYgMIlQeSkao gvLVlBgV2yJAq8EAAEKAJkWIQS1urjJwxtxFWcCI9wxqupHWU277QUCaU4xcxsUgAAAAAAEAA5t YW51MiwyLjUrMS4xMSwyLDJfFIAAAAAALgAoaXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5 maWZ0aGhvcnNlbWFuLm5ldEI1QkFCOEM5QzMxQjcxMTU2NzAyMjNEQzMxQUFFQTQ3NTk0REJCRU QACgkQMarqR1lNu+1W1w/+Ix0WncaPZ9YxSaKtIAT6dj3lQhiufqEHDPrn81n7dyZLsxpTMuvk0 mc6ejQBhET4qcEoiGDkKJgcpg7PZjj5coggaHZS+38hXBbUi7GC45Hlx4PToVDMyYdXS8W5JMJ4 Z5fZ4D6I60al0g1yjIgbUOZ0oZGszTzcz65kRa+/y+dGWl0+2Wq5h4juZZv58mM51Q9s0V9TRr8 sFqICxGOVilIHukp+mprDgq0Zb4VEnJqSOX/fDBFp+rA/cs5ZmPTiDs/IiHP9qHcQX4NXhuydZi z8eHkbbAE4U8zxB6zxchE1pjg5UpugMWlBZOLy7dYNA/jezZ1allwy+ecdfYYS7zr1PKDX1+5LT BW1znyE1Gs3qOMZQx07FWqI9U9gPVMZMSmkzS+mPEm25Mx2M/oVsPwaY4eqJxGPhSQ2Idbo4IfM hlUzY/0HMHHMSH4VqqkwGNuuzOSrQQab465ka8PjuH7uO60OM+aFymmApllLHT0qe41A1QqRlyc DG1OZgtJPIiMlfxsMBrO1Jv5Wk0vOsn+0jv8ZilPUNAzMqXmFaTsI7lfskYRl+oK6GYk7fGqJ13 0Vd9grmpic5svjY1+b98DVjvdL7IqcPwKhdvqbkojb2PmCoIw9Y2dqokMqpjIb7W9e3q8aFnjr2 L+GmXHkgADCjekzSdvhxYmCY/PRBU4= X-Developer-Key: i=dlan@gentoo.org; a=openpgp; fpr=50B03A1A5CBCD33576EF8CD7920C0DBCAABEFD55 In order to prepare adding clock driver for new SoC, extract common ccu header file, so it can be shared by all drivers. Also introduce a reset name macro, so it can be both used in clock and reset subsystem, explicitly to make them match each other. Signed-off-by: Yixun Lan --- include/soc/spacemit/ccu.h | 21 +++++++++++++++++++++ include/soc/spacemit/k1-syscon.h | 13 +++---------- 2 files changed, 24 insertions(+), 10 deletions(-) diff --git a/include/soc/spacemit/ccu.h b/include/soc/spacemit/ccu.h new file mode 100644 index 000000000000..84dcdecccc05 --- /dev/null +++ b/include/soc/spacemit/ccu.h @@ -0,0 +1,21 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#ifndef __SOC_SPACEMIT_CCU_H__ +#define __SOC_SPACEMIT_CCU_H__ + +#include +#include + +/* Auxiliary device used to represent a CCU reset controller */ +struct spacemit_ccu_adev { + struct auxiliary_device adev; + struct regmap *regmap; +}; + +static inline struct spacemit_ccu_adev * +to_spacemit_ccu_adev(struct auxiliary_device *adev) +{ + return container_of(adev, struct spacemit_ccu_adev, adev); +} + +#endif /* __SOC_SPACEMIT_CCU_H__ */ diff --git a/include/soc/spacemit/k1-syscon.h b/include/soc/spacemit/k1-sys= con.h index 354751562c55..13efa7a30853 100644 --- a/include/soc/spacemit/k1-syscon.h +++ b/include/soc/spacemit/k1-syscon.h @@ -5,17 +5,10 @@ #ifndef __SOC_K1_SYSCON_H__ #define __SOC_K1_SYSCON_H__ =20 -/* Auxiliary device used to represent a CCU reset controller */ -struct spacemit_ccu_adev { - struct auxiliary_device adev; - struct regmap *regmap; -}; +#include "ccu.h" =20 -static inline struct spacemit_ccu_adev * -to_spacemit_ccu_adev(struct auxiliary_device *adev) -{ - return container_of(adev, struct spacemit_ccu_adev, adev); -} +/* Reset name macro, should match in clock and reset */ +#define _K_RST(_unit) "k1-" #_unit "-reset" =20 /* APBS register offset */ #define APBS_PLL1_SWCR1 0x100 --=20 2.52.0 From nobody Mon Feb 9 12:29:35 2026 Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D05DC18A93F; Fri, 26 Dec 2025 06:56:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=140.211.166.183 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766732175; cv=none; b=kLijbpXGy45l2x79YTu7EAZ/NzVPAUwT+hl4QL73qnhIVA56u2Cn4lKb5np4uLcYMKLqHURYSlk/4ItSCS/Sajt4q3bGVt0wop802YuWP/e5SzGFv59AlDpJjZa0aKdVpFohjX90qb4UhqLoY+Kodv6lBt3UYyicua6tSUlnRUY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766732175; c=relaxed/simple; bh=bqqRKD27Y7PX902QrpjeI87r8IoEUsvgCMncbegXi5A=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=eoS+QSCrXHSPBTMDKCo2uPRPmJtmVcOrW0/SU2nqbwALjW0xVAroqVr97bfUXxw8r8K/k9t8A2AjM18qoHMmLCZFi73YH/ZQDW+3WPuaICFpSNvjVs8/BDXuRW2sbqi3G7wvTOP0K7xEWXHi6fNxHOVi5l2OsfCiK5FmRu/Pccw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gentoo.org; spf=pass smtp.mailfrom=gentoo.org; arc=none smtp.client-ip=140.211.166.183 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gentoo.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gentoo.org Received: from [127.0.0.1] (unknown [116.232.18.222]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: dlan) by smtp.gentoo.org (Postfix) with ESMTPSA id 85292341042; Fri, 26 Dec 2025 06:56:07 +0000 (UTC) From: Yixun Lan Date: Fri, 26 Dec 2025 14:55:26 +0800 Subject: [PATCH v2 2/3] clk: spacemit: extract common ccu functions Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251226-06-k1-clk-common-v2-2-28b59418b4df@gentoo.org> References: <20251226-06-k1-clk-common-v2-0-28b59418b4df@gentoo.org> In-Reply-To: <20251226-06-k1-clk-common-v2-0-28b59418b4df@gentoo.org> To: Stephen Boyd , Michael Turquette , Philipp Zabel Cc: Alex Elder , Guodong Xu , Inochi Amaoto , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-riscv@lists.infradead.org, spacemit@lists.linux.dev, Yixun Lan , Yao Zi X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=13189; i=dlan@gentoo.org; h=from:subject:message-id; bh=bqqRKD27Y7PX902QrpjeI87r8IoEUsvgCMncbegXi5A=; b=owEB6QIW/ZANAwAKATGq6kdZTbvtAcsmYgBpTjF2eig99H2qv0ZinpPeFJkZCWHGo3gHaDfzN dHrjUqIBI2JAq8EAAEKAJkWIQS1urjJwxtxFWcCI9wxqupHWU277QUCaU4xdhsUgAAAAAAEAA5t YW51MiwyLjUrMS4xMSwyLDJfFIAAAAAALgAoaXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5 maWZ0aGhvcnNlbWFuLm5ldEI1QkFCOEM5QzMxQjcxMTU2NzAyMjNEQzMxQUFFQTQ3NTk0REJCRU QACgkQMarqR1lNu+3gWw//RAN2yfgPU5IUxvM5tA43WlMqwoIQyXHm5R7d8F8QZtFq4gmrd3Dg4 zqW5pEiAZ1RdHyx+QbucaeFKZdPckY4C5m2jg1G79r6/5Ph0Do4P4iDc3tt2cdXIYm2rqG4xvhH yLEPqQs4/6q/JNvSVkS4uHvShvmLRI32nTRiUy8h1AO13zBBgmRFfERy1s3caY65jLnmFvN77UL gIsoURqQqbnqFpRFYF8NoJcLdrBe92PRHzEKvYON261VA0mt449w5sDkJwZr+Xq1hXgP2j8S6cU H44uV4XGaky1dt13euY4wHm8Qzk7R0YkSaV8FG31MV/Udko92lfXreNgV3ekeO07+vq53HrR8Fq q8ZvbG18ZD/WcbEZ1vyVsR7TzT4q/QMv5rk8fkgNnbdVZYspUZw5KkL5+KmkwLzGQOEK2OqGhlU H7ARTFdQQnZAvU6681dTdt0ZZPI4Qjh8HgWzmXdwmDdkh2NXiKEt+YJvNkOpg+iHLPPNe14FXli Mr0KrzKLMyEYpJTSwragiCku5r/ap/y2LbY6pBxNV+24UtaMiG4iSsqUcFcCL2SYp56fasSf91n sYbvlws3T7I0O1ahOh3pJFRzdWf2CijaU8PkDo6RM+Pke6p8OI4BFBMumvdxuv8ykpVUaGwSuej DljJABQIs3T3PgbCfu11xIIdZJ6hB8= X-Developer-Key: i=dlan@gentoo.org; a=openpgp; fpr=50B03A1A5CBCD33576EF8CD7920C0DBCAABEFD55 Refactor the probe function of SpacemiT's clock, and extract a common ccu file, so new clock driver added in the future can share the same code, which would lower the burden of maintenance. Since this commit changes the module name where the auxiliary device registered, the auxiliary device id need to be adjusted. Idea of the patch is come from review of K3 clock driver, please refer this disucssion [1]. Link: https://lore.kernel.org/all/aTo8sCPpVM1o9PKX@pie/ [1] Suggested-by: Yao Zi Signed-off-by: Yixun Lan --- drivers/clk/spacemit/ccu-k1.c | 188 ++--------------------------------= ---- drivers/clk/spacemit/ccu_common.c | 171 ++++++++++++++++++++++++++++++++++ drivers/clk/spacemit/ccu_common.h | 10 ++ 3 files changed, 188 insertions(+), 181 deletions(-) diff --git a/drivers/clk/spacemit/ccu-k1.c b/drivers/clk/spacemit/ccu-k1.c index 01d9485b615d..f97d6f0f0c39 100644 --- a/drivers/clk/spacemit/ccu-k1.c +++ b/drivers/clk/spacemit/ccu-k1.c @@ -5,15 +5,10 @@ */ =20 #include -#include #include -#include -#include -#include #include #include #include -#include #include =20 #include "ccu_common.h" @@ -23,14 +18,6 @@ =20 #include =20 -struct spacemit_ccu_data { - const char *reset_name; - struct clk_hw **hws; - size_t num; -}; - -static DEFINE_IDA(auxiliary_ids); - /* APBS clocks start, APBS region contains and only contains all PLL clock= s */ =20 /* @@ -802,7 +789,7 @@ static struct clk_hw *k1_ccu_mpmu_hws[] =3D { }; =20 static const struct spacemit_ccu_data k1_ccu_mpmu_data =3D { - .reset_name =3D "mpmu-reset", + .reset_name =3D _K_RST(mpmu), .hws =3D k1_ccu_mpmu_hws, .num =3D ARRAY_SIZE(k1_ccu_mpmu_hws), }; @@ -913,7 +900,7 @@ static struct clk_hw *k1_ccu_apbc_hws[] =3D { }; =20 static const struct spacemit_ccu_data k1_ccu_apbc_data =3D { - .reset_name =3D "apbc-reset", + .reset_name =3D _K_RST(apbc), .hws =3D k1_ccu_apbc_hws, .num =3D ARRAY_SIZE(k1_ccu_apbc_hws), }; @@ -984,184 +971,23 @@ static struct clk_hw *k1_ccu_apmu_hws[] =3D { }; =20 static const struct spacemit_ccu_data k1_ccu_apmu_data =3D { - .reset_name =3D "apmu-reset", + .reset_name =3D _K_RST(apmu), .hws =3D k1_ccu_apmu_hws, .num =3D ARRAY_SIZE(k1_ccu_apmu_hws), }; =20 static const struct spacemit_ccu_data k1_ccu_rcpu_data =3D { - .reset_name =3D "rcpu-reset", + .reset_name =3D _K_RST(rcpu), }; =20 static const struct spacemit_ccu_data k1_ccu_rcpu2_data =3D { - .reset_name =3D "rcpu2-reset", + .reset_name =3D _K_RST(rcpu2), }; =20 static const struct spacemit_ccu_data k1_ccu_apbc2_data =3D { - .reset_name =3D "apbc2-reset", + .reset_name =3D _K_RST(apbc2), }; =20 -static int spacemit_ccu_register(struct device *dev, - struct regmap *regmap, - struct regmap *lock_regmap, - const struct spacemit_ccu_data *data) -{ - struct clk_hw_onecell_data *clk_data; - int i, ret; - - /* Nothing to do if the CCU does not implement any clocks */ - if (!data->hws) - return 0; - - clk_data =3D devm_kzalloc(dev, struct_size(clk_data, hws, data->num), - GFP_KERNEL); - if (!clk_data) - return -ENOMEM; - - clk_data->num =3D data->num; - - for (i =3D 0; i < data->num; i++) { - struct clk_hw *hw =3D data->hws[i]; - struct ccu_common *common; - const char *name; - - if (!hw) { - clk_data->hws[i] =3D ERR_PTR(-ENOENT); - continue; - } - - name =3D hw->init->name; - - common =3D hw_to_ccu_common(hw); - common->regmap =3D regmap; - common->lock_regmap =3D lock_regmap; - - ret =3D devm_clk_hw_register(dev, hw); - if (ret) { - dev_err(dev, "Cannot register clock %d - %s\n", - i, name); - return ret; - } - - clk_data->hws[i] =3D hw; - } - - ret =3D devm_of_clk_add_hw_provider(dev, of_clk_hw_onecell_get, clk_data); - if (ret) - dev_err(dev, "failed to add clock hardware provider (%d)\n", ret); - - return ret; -} - -static void spacemit_cadev_release(struct device *dev) -{ - struct auxiliary_device *adev =3D to_auxiliary_dev(dev); - - ida_free(&auxiliary_ids, adev->id); - kfree(to_spacemit_ccu_adev(adev)); -} - -static void spacemit_adev_unregister(void *data) -{ - struct auxiliary_device *adev =3D data; - - auxiliary_device_delete(adev); - auxiliary_device_uninit(adev); -} - -static int spacemit_ccu_reset_register(struct device *dev, - struct regmap *regmap, - const char *reset_name) -{ - struct spacemit_ccu_adev *cadev; - struct auxiliary_device *adev; - int ret; - - /* Nothing to do if the CCU does not implement a reset controller */ - if (!reset_name) - return 0; - - cadev =3D kzalloc(sizeof(*cadev), GFP_KERNEL); - if (!cadev) - return -ENOMEM; - - cadev->regmap =3D regmap; - - adev =3D &cadev->adev; - adev->name =3D reset_name; - adev->dev.parent =3D dev; - adev->dev.release =3D spacemit_cadev_release; - adev->dev.of_node =3D dev->of_node; - ret =3D ida_alloc(&auxiliary_ids, GFP_KERNEL); - if (ret < 0) - goto err_free_cadev; - adev->id =3D ret; - - ret =3D auxiliary_device_init(adev); - if (ret) - goto err_free_aux_id; - - ret =3D auxiliary_device_add(adev); - if (ret) { - auxiliary_device_uninit(adev); - return ret; - } - - return devm_add_action_or_reset(dev, spacemit_adev_unregister, adev); - -err_free_aux_id: - ida_free(&auxiliary_ids, adev->id); -err_free_cadev: - kfree(cadev); - - return ret; -} - -static int k1_ccu_probe(struct platform_device *pdev) -{ - struct regmap *base_regmap, *lock_regmap =3D NULL; - const struct spacemit_ccu_data *data; - struct device *dev =3D &pdev->dev; - int ret; - - base_regmap =3D device_node_to_regmap(dev->of_node); - if (IS_ERR(base_regmap)) - return dev_err_probe(dev, PTR_ERR(base_regmap), - "failed to get regmap\n"); - - /* - * The lock status of PLLs locate in MPMU region, while PLLs themselves - * are in APBS region. Reference to MPMU syscon is required to check PLL - * status. - */ - if (of_device_is_compatible(dev->of_node, "spacemit,k1-pll")) { - struct device_node *mpmu =3D of_parse_phandle(dev->of_node, - "spacemit,mpmu", 0); - if (!mpmu) - return dev_err_probe(dev, -ENODEV, - "Cannot parse MPMU region\n"); - - lock_regmap =3D device_node_to_regmap(mpmu); - of_node_put(mpmu); - - if (IS_ERR(lock_regmap)) - return dev_err_probe(dev, PTR_ERR(lock_regmap), - "failed to get lock regmap\n"); - } - - data =3D of_device_get_match_data(dev); - - ret =3D spacemit_ccu_register(dev, base_regmap, lock_regmap, data); - if (ret) - return dev_err_probe(dev, ret, "failed to register clocks\n"); - - ret =3D spacemit_ccu_reset_register(dev, base_regmap, data->reset_name); - if (ret) - return dev_err_probe(dev, ret, "failed to register resets\n"); - - return 0; -} - static const struct of_device_id of_k1_ccu_match[] =3D { { .compatible =3D "spacemit,k1-pll", @@ -1200,7 +1026,7 @@ static struct platform_driver k1_ccu_driver =3D { .name =3D "spacemit,k1-ccu", .of_match_table =3D of_k1_ccu_match, }, - .probe =3D k1_ccu_probe, + .probe =3D spacemit_ccu_probe, }; module_platform_driver(k1_ccu_driver); =20 diff --git a/drivers/clk/spacemit/ccu_common.c b/drivers/clk/spacemit/ccu_c= ommon.c index 4412c4104dab..f1a837aafb46 100644 --- a/drivers/clk/spacemit/ccu_common.c +++ b/drivers/clk/spacemit/ccu_common.c @@ -1,6 +1,177 @@ // SPDX-License-Identifier: GPL-2.0-only =20 +#include +#include +#include #include +#include +#include +#include + +#include "ccu_common.h" + +static DEFINE_IDA(auxiliary_ids); +static int spacemit_ccu_register(struct device *dev, + struct regmap *regmap, + struct regmap *lock_regmap, + const struct spacemit_ccu_data *data) +{ + struct clk_hw_onecell_data *clk_data; + int i, ret; + + /* Nothing to do if the CCU does not implement any clocks */ + if (!data->hws) + return 0; + + clk_data =3D devm_kzalloc(dev, struct_size(clk_data, hws, data->num), + GFP_KERNEL); + if (!clk_data) + return -ENOMEM; + + clk_data->num =3D data->num; + + for (i =3D 0; i < data->num; i++) { + struct clk_hw *hw =3D data->hws[i]; + struct ccu_common *common; + const char *name; + + if (!hw) { + clk_data->hws[i] =3D ERR_PTR(-ENOENT); + continue; + } + + name =3D hw->init->name; + + common =3D hw_to_ccu_common(hw); + common->regmap =3D regmap; + common->lock_regmap =3D lock_regmap; + + ret =3D devm_clk_hw_register(dev, hw); + if (ret) { + dev_err(dev, "Cannot register clock %d - %s\n", + i, name); + return ret; + } + + clk_data->hws[i] =3D hw; + } + + ret =3D devm_of_clk_add_hw_provider(dev, of_clk_hw_onecell_get, clk_data); + if (ret) + dev_err(dev, "failed to add clock hardware provider (%d)\n", ret); + + return ret; +} + +static void spacemit_cadev_release(struct device *dev) +{ + struct auxiliary_device *adev =3D to_auxiliary_dev(dev); + + ida_free(&auxiliary_ids, adev->id); + kfree(to_spacemit_ccu_adev(adev)); +} + +static void spacemit_adev_unregister(void *data) +{ + struct auxiliary_device *adev =3D data; + + auxiliary_device_delete(adev); + auxiliary_device_uninit(adev); +} + +static int spacemit_ccu_reset_register(struct device *dev, + struct regmap *regmap, + const char *reset_name) +{ + struct spacemit_ccu_adev *cadev; + struct auxiliary_device *adev; + int ret; + + /* Nothing to do if the CCU does not implement a reset controller */ + if (!reset_name) + return 0; + + cadev =3D kzalloc(sizeof(*cadev), GFP_KERNEL); + if (!cadev) + return -ENOMEM; + + cadev->regmap =3D regmap; + + adev =3D &cadev->adev; + adev->name =3D reset_name; + adev->dev.parent =3D dev; + adev->dev.release =3D spacemit_cadev_release; + adev->dev.of_node =3D dev->of_node; + ret =3D ida_alloc(&auxiliary_ids, GFP_KERNEL); + if (ret < 0) + goto err_free_cadev; + adev->id =3D ret; + + ret =3D auxiliary_device_init(adev); + if (ret) + goto err_free_aux_id; + + ret =3D auxiliary_device_add(adev); + if (ret) { + auxiliary_device_uninit(adev); + return ret; + } + + return devm_add_action_or_reset(dev, spacemit_adev_unregister, adev); + +err_free_aux_id: + ida_free(&auxiliary_ids, adev->id); +err_free_cadev: + kfree(cadev); + + return ret; +} + +int spacemit_ccu_probe(struct platform_device *pdev) +{ + struct regmap *base_regmap, *lock_regmap =3D NULL; + const struct spacemit_ccu_data *data; + struct device *dev =3D &pdev->dev; + int ret; + + base_regmap =3D device_node_to_regmap(dev->of_node); + if (IS_ERR(base_regmap)) + return dev_err_probe(dev, PTR_ERR(base_regmap), + "failed to get regmap\n"); + + /* + * The lock status of PLLs locate in MPMU region, while PLLs themselves + * are in APBS region. Reference to MPMU syscon is required to check PLL + * status. + */ + if (of_device_is_compatible(dev->of_node, "spacemit,k1-pll")) { + struct device_node *mpmu =3D of_parse_phandle(dev->of_node, + "spacemit,mpmu", 0); + if (!mpmu) + return dev_err_probe(dev, -ENODEV, + "Cannot parse MPMU region\n"); + + lock_regmap =3D device_node_to_regmap(mpmu); + of_node_put(mpmu); + + if (IS_ERR(lock_regmap)) + return dev_err_probe(dev, PTR_ERR(lock_regmap), + "failed to get lock regmap\n"); + } + + data =3D of_device_get_match_data(dev); + + ret =3D spacemit_ccu_register(dev, base_regmap, lock_regmap, data); + if (ret) + return dev_err_probe(dev, ret, "failed to register clocks\n"); + + ret =3D spacemit_ccu_reset_register(dev, base_regmap, data->reset_name); + if (ret) + return dev_err_probe(dev, ret, "failed to register resets\n"); + + return 0; +} +EXPORT_SYMBOL_NS_GPL(spacemit_ccu_probe, "CLK_SPACEMIT"); =20 MODULE_DESCRIPTION("SpacemiT CCU common clock driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/spacemit/ccu_common.h b/drivers/clk/spacemit/ccu_c= ommon.h index da72f3836e0b..9b4ef24875e5 100644 --- a/drivers/clk/spacemit/ccu_common.h +++ b/drivers/clk/spacemit/ccu_common.h @@ -7,6 +7,8 @@ #ifndef _CCU_COMMON_H_ #define _CCU_COMMON_H_ =20 +#include +#include #include =20 struct ccu_common { @@ -36,6 +38,12 @@ static inline struct ccu_common *hw_to_ccu_common(struct= clk_hw *hw) return container_of(hw, struct ccu_common, hw); } =20 +struct spacemit_ccu_data { + const char *reset_name; + struct clk_hw **hws; + size_t num; +}; + #define ccu_read(c, reg) \ ({ \ u32 tmp; \ @@ -45,4 +53,6 @@ static inline struct ccu_common *hw_to_ccu_common(struct = clk_hw *hw) #define ccu_update(c, reg, mask, val) \ regmap_update_bits((c)->regmap, (c)->reg_##reg, mask, val) =20 +int spacemit_ccu_probe(struct platform_device *pdev); + #endif /* _CCU_COMMON_H_ */ --=20 2.52.0 From nobody Mon Feb 9 12:29:35 2026 Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0791E2797B5; Fri, 26 Dec 2025 06:56:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=140.211.166.183 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766732181; cv=none; b=eEAeysVQMzDAi2RKt3eKV2UR1X2RX0Rpvj3L8VxKuiDzQvTUHKkFW/OOqyI1XHp3DQJM+UdhbvDw6cmQEoLWlYYoLuY+PYXKeBM0tNd721JokF/IMZOUnemwBS/BglQd0goyK1dYlPP/ZV7mAvzlEqfgqLVW/yk35F5L9kXu8sk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766732181; c=relaxed/simple; bh=nZt8LczdeEYtuXJI/rXMtv2yMXU27iWRkv3bAqS2KIg=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=jBPFzyXcmDRxDbc6GslwCls8cfw0wVDnIrvwo78qAYdpU6k8HpUKNcsHCllHr8E+hs0kwmhgPBSrMo0oTj9PHrRkdOMkwgxov/sUq77ymim9KXRb6X5DLcucYTbX1Tpfkx+4RgSH1ECpefJH9T1SxNt2/4EnSlnyGsAhjfJSZCs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gentoo.org; spf=pass smtp.mailfrom=gentoo.org; arc=none smtp.client-ip=140.211.166.183 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gentoo.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gentoo.org Received: from [127.0.0.1] (unknown [116.232.18.222]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: dlan) by smtp.gentoo.org (Postfix) with ESMTPSA id 92449340FF7; Fri, 26 Dec 2025 06:56:13 +0000 (UTC) From: Yixun Lan Date: Fri, 26 Dec 2025 14:55:27 +0800 Subject: [PATCH v2 3/3] reset: spacemit: fix auxiliary device id Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251226-06-k1-clk-common-v2-3-28b59418b4df@gentoo.org> References: <20251226-06-k1-clk-common-v2-0-28b59418b4df@gentoo.org> In-Reply-To: <20251226-06-k1-clk-common-v2-0-28b59418b4df@gentoo.org> To: Stephen Boyd , Michael Turquette , Philipp Zabel Cc: Alex Elder , Guodong Xu , Inochi Amaoto , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-riscv@lists.infradead.org, spacemit@lists.linux.dev, Yixun Lan X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=807; i=dlan@gentoo.org; h=from:subject:message-id; bh=nZt8LczdeEYtuXJI/rXMtv2yMXU27iWRkv3bAqS2KIg=; b=owEB6QIW/ZANAwAKATGq6kdZTbvtAcsmYgBpTjF5VDd6l/FLTRfQrAj3nlV36Lr8SOL8oeon8 fkW0K12Y0CJAq8EAAEKAJkWIQS1urjJwxtxFWcCI9wxqupHWU277QUCaU4xeRsUgAAAAAAEAA5t YW51MiwyLjUrMS4xMSwyLDJfFIAAAAAALgAoaXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5 maWZ0aGhvcnNlbWFuLm5ldEI1QkFCOEM5QzMxQjcxMTU2NzAyMjNEQzMxQUFFQTQ3NTk0REJCRU QACgkQMarqR1lNu+1iLxAAjNip2N4yTDRtJTqrhY8r8mz8gTkNfoCQEMKFNdEINtSAILbGasR8C 1pWHgkxG+N4hU9EbxDYSJDngPpw/pExvST7FtKhNabyZps9xIgoPcXyeCaV/dQj9twprwdeswkn QFREIkGEpDAFiNPVnIfxWTNRzQFXjVHcIBbpebmqX31FoP1IGEky+SqsP5xpL4pnLmparPvsMcK iC+sP0kMuh23dGuz33HGXA1OPycxz0Y713L5YHj+JOjKOiz3km6dDv6cAc798h5WDR4YUCIEk76 8gaxk70zU6IJ4Y9+9/3JU/Un5YtfUGTgfr0yDkiaxJJgPvbhp+d+e6aedOoN++lcy+nxNTtwxos 2nf0kK/Lhv2B44e9LCt8AghD848V70sq4JU+4Wd+MloCWcm/ObojqBy+pBkoWlweHqQfQMw2o/7 FtiDrLj1gS0KjnI88ccjfZZu9bAsEUVaPQfWnoLMuoFCPZxhxE2Ce7jHkraDo2ghL+TqLj8cN8U ml2wOD5TimEt7/jbOwEKSxxe6AqIAHbtm5zy6PeQNCLtQVvzUS2aO4nb+uubHpgeDMM9wEIFJrj hA7HzjHu6Yb0pU/R9HxGJXnQcA03VyAFKuYPsp/QHiC4fNibcYNCU+JotYQNlFTyu4WVCkKo+s/ SIhP09p0t8mJdB8oQ/QX0oQBRJY0oE= X-Developer-Key: i=dlan@gentoo.org; a=openpgp; fpr=50B03A1A5CBCD33576EF8CD7920C0DBCAABEFD55 Due to the auxiliary register procedure moved to ccu common module, the auxiliary device id need to be adjusted, otherwise reset driver will fail to probe. Signed-off-by: Yixun Lan --- drivers/reset/reset-spacemit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/reset/reset-spacemit.c b/drivers/reset/reset-spacemit.c index e1272aff28f7..8922e14fa836 100644 --- a/drivers/reset/reset-spacemit.c +++ b/drivers/reset/reset-spacemit.c @@ -278,7 +278,7 @@ static int spacemit_reset_probe(struct auxiliary_device= *adev, =20 #define K1_AUX_DEV_ID(_unit) \ { \ - .name =3D "spacemit_ccu_k1." #_unit "-reset", \ + .name =3D "spacemit_ccu." _K_RST(_unit), \ .driver_data =3D (kernel_ulong_t)&k1_ ## _unit ## _reset_data, \ } =20 --=20 2.52.0