Add support for AST2700 clk driver

PATCH Applied Ryan Chen 4 participants 11 messages Oct 23, 2024 – Feb 3, 2025

Subsystems (6)

Device tree bindings Common clk Multifunction devices (MFD) Arm/Aspeed machine support +2 more subsystems

lore

Share & tools

lore

Ryan Chen

author maintainer

2024-10-23 09:02 UTC #0 lore raw

v6 cover Add support for AST2700 clk driver

This patch series is add clk driver for AST2700.

AST2700 is the 8th generation of Integrated Remote Management Processor introduced by ASPEED Technology Inc. Which is Board Management controller (BMC) SoC family. AST2700 have two SoC connected, one is SoC0, another is SoC1, it has it's own scu, this driver inlcude SCU0 and SCU1 driver.

v6:
-patch-2: add reset-aspeed.h
-reset-aspeed: add include cleanup.h for guard() -reset-aspeed: change ids name clk_aspeed to reset_aspeed -reset-aspeed: move aspeed_reset_controller_register, aspeed_reset_adev_release, aspeed_reset_unregister_adev from clk-ast2700.c -reset-aspeed: drop base check, since it check in clk-ast2700.c -clk-ast2700: sync each gate name from *clk to *clk-gate name. -clk-ast2700: add CLK_GATE_ASPEED to diff clk_hw_register_gate and ast2700_clk_hw_register_gate.

v5:
-patch-2 Kconfig: add select AUXILIARY_BUS
-reset-aspeed: #define to_aspeed_reset(p) turn into static inline function. -reset-aspeed: modify spin_lock_irqsave to guard(spinlock_irqsave) -reset-aspeed: remove unnecessary parentheses.
-clk-ast2700: use <linux/units.h> and refrain from define clk

v4:
-yaml: keep size-cells=<1>.
-merge clk,reset dt binding header with yaml the same patch. -rename clk,reset dt binding header to aspeed,ast2700-scu.h -reset-aspeed: update tables tabs sapces to consistent spaces. -reset-aspeed: remove no use dev_set_drvdata.
-clk-ast2700: modify reset_name to const int scu in struct clk_data. -clk-ast2700: use scu number in clk_data generate reset_name for reset
driver register.
-clk-ast2700: fix pll number mix up scu0,scu1. -clk-ast2700: update dt-binding clock include file.

v3:
-yaml: v2 missing send yaml patch, v3 add. -yaml: drop 64bits address example.
-yaml: add discription about soc0 and soc1 -dt-bindings: remove (), *_NUMS, reserved. -dt-bindings: remove dulipated define number. -dt-bindings: merge clk and reset to be one patch. -reset-aspeed: add auxiliary device for reset driver. -clk-ast2700: modify reset to be auxiliary add. -clk-ast2700: modify to be platform driver. -clk-ast2700: modify each clk to const clk array.

v2:
-yaml: drop 64bits address example.
-yaml: add discription about soc0 and soc1
-dt-bindings: remove (), *_NUMS, reserved.
-dt-bindings: remove dulipated define number
-clk-ast2700: drop WARN_ON, weird comment.

Ryan Chen (3):
dt-bindings: mfd: aspeed: support for AST2700
reset: aspeed: register AST2700 reset auxiliary bus device
clk: aspeed: add AST2700 clock driver.

 .../bindings/mfd/aspeed,ast2x00-scu.yaml      |    8 +-
 drivers/clk/Kconfig                           |    8 +
 drivers/clk/Makefile                          |    1 +
 drivers/clk/clk-ast2700.c                     | 1513 +++++++++++++++++
 drivers/reset/Kconfig                         |    7 +
 drivers/reset/Makefile                        |    1 +
 drivers/reset/reset-aspeed.c                  |  302 ++++
 .../dt-bindings/clock/aspeed,ast2700-scu.h    |  163 ++
 .../dt-bindings/reset/aspeed,ast2700-scu.h    |  124 ++
 include/soc/aspeed/reset-aspeed.h             |   21 +
 10 files changed, 2147 insertions(+), 1 deletion(-)
 create mode 100644 drivers/clk/clk-ast2700.c
 create mode 100644 drivers/reset/reset-aspeed.c
 create mode 100644 include/dt-bindings/clock/aspeed,ast2700-scu.h
 create mode 100644 include/dt-bindings/reset/aspeed,ast2700-scu.h
 create mode 100644 include/soc/aspeed/reset-aspeed.h

Signature · 1 line

2.34.1

Ryan Chen

author maintainer

2024-10-23 09:02 UTC #1 lore raw

PATCH v6 1/3 dt-bindings: mfd: aspeed: support for AST2700

Add reset, clk dt bindings headers, and update compatible support for AST2700 clk, silicon-id in yaml.

Signed-off-by Ryan Chen <[email protected]>

Reviewed-by Krzysztof Kozlowski <[email protected]>

---
 .../bindings/mfd/aspeed,ast2x00-scu.yaml      |   8 +-
 .../dt-bindings/clock/aspeed,ast2700-scu.h    | 163 ++++++++++++++++++
 .../dt-bindings/reset/aspeed,ast2700-scu.h    | 124 +++++++++++++
 3 files changed, 294 insertions(+), 1 deletion(-)
 create mode 100644 include/dt-bindings/clock/aspeed,ast2700-scu.h
 create mode 100644 include/dt-bindings/reset/aspeed,ast2700-scu.h

Documentation/devicetree/bindings/mfd/aspeed,ast2x00-scu.yaml +7 -1

index 86ee69c0f45b..c800d5e53b65 100644
@@ -9,6 +9,8 @@ title: Aspeed System Control Unit
 description:
   The Aspeed System Control Unit manages the global behaviour of the SoC,
   configuring elements such as clocks, pinmux, and reset.
+  In AST2700 SOC which has two soc connection, each soc have its own scu
+  register control, ast2700-scu0 for soc0, ast2700-scu1 for soc1.
 
 maintainers:
   - Joel Stanley <[email protected]>

@@ -21,6 +23,8 @@ properties:
           - aspeed,ast2400-scu
           - aspeed,ast2500-scu
           - aspeed,ast2600-scu
+          - aspeed,ast2700-scu0
+          - aspeed,ast2700-scu1
       - const: syscon
       - const: simple-mfd

@@ -30,7 +34,8 @@ properties:
   ranges: true
 
   '#address-cells':
-    const: 1
+    minimum: 1
+    maximum: 2
 
   '#size-cells':
     const: 1

@@ -76,6 +81,7 @@ patternProperties:
               - aspeed,ast2400-silicon-id
               - aspeed,ast2500-silicon-id
               - aspeed,ast2600-silicon-id
+              - aspeed,ast2700-silicon-id
           - const: aspeed,silicon-id
 
       reg:

include/dt-bindings/clock/aspeed,ast2700-scu.h new file +163 -0

163 lines not on this page · raw message

include/dt-bindings/reset/aspeed,ast2700-scu.h new file +124 -0

124 lines not on this page · raw message

Signature · 1 line

2.34.1

Lee Jones

maintainer Applied

2024-11-01 16:12 UTC #2 lore raw

Re: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed: support for AST2700

Quoted 4 lines · Ryan Chen

Add reset, clk dt bindings headers, and update compatible support for AST2700 clk, silicon-id in yaml.

Applied, thanks!

[1/3] dt-bindings: mfd: aspeed: support for AST2700

      commit: 76c6217c31266e800b67a476bba59dfeb9858a90

--
Lee Jones [李琼斯]

Ryan Chen

author maintainer

2025-01-31 03:51 UTC #3 lore raw

RE: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed: support for AST2700

Quoted 20 lines

-----Original Message-----
From: Lee Jones <[email protected]>
Sent: Saturday, November 2, 2024 12:12 AM
To: [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; Ryan Chen <[email protected]>
Cc: Krzysztof Kozlowski <[email protected]>
Subject: Re: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed: support for AST2700
Quoted 4 lines · Ryan Chen
Add reset, clk dt bindings headers, and update compatible support for AST2700 clk, silicon-id in yaml.

Quoted 7 lines

Applied, thanks!
[1/3] dt-bindings: mfd: aspeed: support for AST2700
      commit: 76c6217c31266e800b67a476bba59dfeb9858a90
--
Lee Jones [李琼斯]

Hello,
Since this patch have been applied, if I still want modify I will do the rebase and still patch base on this. Am I right?

For example I want to modify include/dt-bindings/clock/aspeed,ast2700-scu.h I send the patchv8 [1/3] patch for aspeed,ast2700-scu.h
Others [2/3], [3/3] still progress, am I right?

Or I should go for new patch series?

Ryan

Andrew Jeffery

maintainer

2025-02-03 03:40 UTC #4 lore raw

Re: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed: support for AST2700

Hi Ryan,

Quoted 35 lines · Ryan Chen

Quoted 22 lines
-----Original Message-----
From: Lee Jones <[email protected]>
Sent: Saturday, November 2, 2024 12:12 AM
To: [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; Ryan Chen <[email protected]>
Cc: Krzysztof Kozlowski <[email protected]>
Subject: Re: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed:
support for
AST2700
Quoted 5 lines · Ryan Chen
Add reset, clk dt bindings headers, and update compatible support for
AST2700 clk, silicon-id in yaml.
Quoted 7 lines
Applied, thanks!
[1/3] dt-bindings: mfd: aspeed: support for AST2700
commit: 76c6217c31266e800b67a476bba59dfeb9858a90
--
Lee Jones [李琼斯]
Hello,
Since this patch have been applied, if I still want modify I will do the rebase and still patch base on this.
Am I right?

Quoted 4 lines

For example I want to modify include/dt- bindings/clock/aspeed,ast2700-scu.h
I send the patchv8 [1/3] patch for aspeed,ast2700-scu.h Others [2/3], [3/3] still progress, am I right?

I'd rebase the series on top of v6.14-rc1 and continue incrementing the series version. You've replied to v6 here, but I see v7 at [1], so I guess v8 as you suggest?

[1]: https://lore.kernel.org/all/[email protected]/

Andrew

Ryan Chen

author maintainer

2025-02-03 06:15 UTC #5 lore raw

RE: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed: support for AST2700

Quoted 41 lines

Subject: Re: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed: support for AST2700
Hi Ryan,
Quoted 35 lines · Ryan Chen
Quoted 22 lines
-----Original Message-----
From: Lee Jones <[email protected]>
Sent: Saturday, November 2, 2024 12:12 AM
To: [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; Ryan Chen <[email protected]>
Cc: Krzysztof Kozlowski <[email protected]>
Subject: Re: (subset) [PATCH v6 1/3] dt-bindings: mfd: aspeed:
support for
AST2700
Quoted 5 lines · Ryan Chen
Add reset, clk dt bindings headers, and update compatible support for
AST2700 clk, silicon-id in yaml.
Quoted 7 lines
Applied, thanks!
[1/3] dt-bindings: mfd: aspeed: support for AST2700
commit: 76c6217c31266e800b67a476bba59dfeb9858a90
--
Lee Jones [李琼斯]
Hello,
Since this patch have been applied, if I still want modify I will do the rebase and still patch base on this.
Am I right?

Quoted 8 lines

Quoted 4 lines
For example I want to modify include/dt- bindings/clock/aspeed,ast2700-scu.h
I send the patchv8 [1/3] patch for aspeed,ast2700-scu.h Others [2/3], [3/3] still progress, am I right?
I'd rebase the series on top of v6.14-rc1 and continue incrementing the series version. You've replied to v6 here, but I see v7 at [1], so I guess v8 as you suggest?

Yes, I am going to progress v8. Thank your advice.

Quoted 6 lines

[1]:
https://lore.kernel.org/all/20241028053018.2579200-1-ryan_chen@aspeedtec
h.com/
Andrew

Ryan Chen

author maintainer

2024-10-23 09:02 UTC #6 lore raw

PATCH v6 2/3 reset: aspeed: register AST2700 reset auxiliary bus device

In reply to Ryan Chen

The AST2700 reset driver is registered as an auxiliary device due to reset and clock controller share the same register region.

Signed-off-by Ryan Chen <[email protected]>

Reviewed-by Philipp Zabel <[email protected]>

---
 drivers/reset/Kconfig             |   7 +
 drivers/reset/Makefile            |   1 +
 drivers/reset/reset-aspeed.c      | 302 ++++++++++++++++++++++++++++++
 include/soc/aspeed/reset-aspeed.h |  21 +++
 4 files changed, 331 insertions(+)
 create mode 100644 drivers/reset/reset-aspeed.c
 create mode 100644 include/soc/aspeed/reset-aspeed.h

drivers/reset/Kconfig +7 -0

index 67bce340a87e..71a3accea7cb 100644
@@ -22,6 +22,13 @@ config RESET_A10SR
 	  This option enables support for the external reset functions for
 	  peripheral PHYs on the Altera Arria10 System Resource Chip.
 
+config RESET_ASPEED
+	tristate "ASPEED Reset Driver"
+	depends on ARCH_ASPEED || COMPILE_TEST
+	select AUXILIARY_BUS
+	help
+	  This enables the reset controller driver for AST2700.
+
 config RESET_ATH79
 	bool "AR71xx Reset Driver" if COMPILE_TEST
 	default ATH79

drivers/reset/Makefile +1 -0

index 27b0bbdfcc04..97482bb56416 100644
@@ -5,6 +5,7 @@ obj-y += starfive/
 obj-y += sti/
 obj-y += tegra/
 obj-$(CONFIG_RESET_A10SR) += reset-a10sr.o
+obj-$(CONFIG_RESET_ASPEED) += reset-aspeed.o
 obj-$(CONFIG_RESET_ATH79) += reset-ath79.o
 obj-$(CONFIG_RESET_AXS10X) += reset-axs10x.o
 obj-$(CONFIG_RESET_BCM6345) += reset-bcm6345.o

drivers/reset/reset-aspeed.c new file +302 -0

302 lines not on this page · raw message

include/soc/aspeed/reset-aspeed.h new file +21 -0

new file mode 100644
index 000000000000..54ac18551b61
@@ -0,0 +1,21 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright (c) 2024 ASPEED Technology Inc.
+ * Author: Ryan Chen <[email protected]>
+ */
+
+#ifndef __RESET_ASPEED_H__
+#define __RESET_ASPEED_H__
+
+#if IS_ENABLED(CONFIG_RESET_ASPEED)
+int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
+				     const char *adev_name);
+#else
+int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
+				     const char *adev_name)
+{
+	return -ENODEV;
+}
+#endif /* if IS_ENABLED(CONFIG_RESET_ASPEED) */
+
+#endif /* __RESET_ASPEED_H__ */

Signature · 1 line

2.34.1

Ryan Chen

author maintainer

2024-10-23 09:02 UTC #7 lore raw

PATCH v6 3/3 clk: aspeed: add AST2700 clock driver.

In reply to Ryan Chen

Add AST2700 clock controller driver and also use axiliary device framework register the reset controller driver. Due to clock and reset using the same register region.

Signed-off-by Ryan Chen <[email protected]>

---
 drivers/clk/Kconfig       |    8 +
 drivers/clk/Makefile      |    1 +
 drivers/clk/clk-ast2700.c | 1513 +++++++++++++++++++++++++++++++++++++
 3 files changed, 1522 insertions(+)
 create mode 100644 drivers/clk/clk-ast2700.c

drivers/clk/Kconfig +8 -0

index 983ef4f36d8c..4cc35ecba1c0 100644
@@ -269,6 +269,14 @@ config COMMON_CLK_ASPEED
 	  The G4 and G5 series, including the ast2400 and ast2500, are supported
 	  by this driver.
 
+config COMMON_CLK_AST2700
+	bool "Clock driver for AST2700 SoC"
+	depends on ARCH_ASPEED || COMPILE_TEST
+	help
+	  This driver provides support for clock on AST2700 SoC.
+	  The driver is responsible for managing the various clocks required
+	  by the peripherals and cores within the AST2700.
+
 config COMMON_CLK_S2MPS11
 	tristate "Clock driver for S2MPS1X/S5M8767 MFD"
 	depends on MFD_SEC_CORE || COMPILE_TEST

drivers/clk/Makefile +1 -0

index f793a16cad40..fe95203c3138 100644
@@ -38,6 +38,7 @@ obj-$(CONFIG_COMMON_CLK_FSL_SAI)	+= clk-fsl-sai.o
 obj-$(CONFIG_COMMON_CLK_GEMINI)		+= clk-gemini.o
 obj-$(CONFIG_COMMON_CLK_ASPEED)		+= clk-aspeed.o
 obj-$(CONFIG_MACH_ASPEED_G6)		+= clk-ast2600.o
+obj-$(CONFIG_COMMON_CLK_AST2700)	+= clk-ast2700.o
 obj-$(CONFIG_ARCH_HIGHBANK)		+= clk-highbank.o
 obj-$(CONFIG_CLK_HSDK)			+= clk-hsdk-pll.o
 obj-$(CONFIG_COMMON_CLK_K210)		+= clk-k210.o

drivers/clk/clk-ast2700.c new file +1513 -0

1,513 lines not on this page · raw message

Signature · 1 line

2.34.1

kernel test robot

build bot

2024-10-24 12:53 UTC #8 lore raw

Re: [PATCH v6 3/3] clk: aspeed: add AST2700 clock driver.

kernel test robot noticed the following build warnings:

Hi Ryan,

kernel test robot noticed the following build warnings:

[auto build test WARNING on clk/clk-next]
[also build test WARNING on pza/reset/next lee-mfd/for-mfd-next lee-leds/for-leds-next linus/master lee-mfd/for-mfd-fixes v6.12-rc4 next-20241024] [cannot apply to pza/imx-drm/next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Ryan-Chen/dt-bindings-mfd-aspeed-support-for-AST2700/20241023-170434
base:   https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
patch link:    https://lore.kernel.org/r/20241023090153.1395220-4-ryan_chen%40aspeedtech.com
patch subject: [PATCH v6 3/3] clk: aspeed: add AST2700 clock driver.
config: arm64-randconfig-r133-20241024 (https://download.01.org/0day-ci/archive/20241024/[email protected]/config)
compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 5886454669c3c9026f7f27eab13509dd0241f2d6)
reproduce: (https://download.01.org/0day-ci/archive/20241024/[email protected]/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All warnings (new ones prefixed by >>):

In file included from drivers/clk/clk-ast2700.c:17:

>> include/soc/aspeed/reset-aspeed.h:14:5: warning: no previous prototype for function 'aspeed_reset_controller_register' [-Wmissing-prototypes]
      14 | int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
         |     ^
   include/soc/aspeed/reset-aspeed.h:14:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
      14 | int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
         | ^
         | static

1 warning generated.

vim +/aspeed_reset_controller_register +14 include/soc/aspeed/reset-aspeed.h

1476d29e4461f6 Ryan Chen 2024-10-23 9

1476d29e4461f6 Ryan Chen 2024-10-23  10  #if IS_ENABLED(CONFIG_RESET_ASPEED)
1476d29e4461f6 Ryan Chen 2024-10-23  11  int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
1476d29e4461f6 Ryan Chen 2024-10-23  12  				     const char *adev_name);
1476d29e4461f6 Ryan Chen 2024-10-23  13  #else
1476d29e4461f6 Ryan Chen 2024-10-23 @14  int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
1476d29e4461f6 Ryan Chen 2024-10-23  15  				     const char *adev_name)
1476d29e4461f6 Ryan Chen 2024-10-23  16  {
1476d29e4461f6 Ryan Chen 2024-10-23  17  	return -ENODEV;
1476d29e4461f6 Ryan Chen 2024-10-23  18  }
1476d29e4461f6 Ryan Chen 2024-10-23  19  #endif /* if IS_ENABLED(CONFIG_RESET_ASPEED) */

1476d29e4461f6 Ryan Chen 2024-10-23 20

Signature · 2 lines

0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

kernel test robot

build bot

2024-10-24 20:45 UTC #9 lore raw

Re: [PATCH v6 3/3] clk: aspeed: add AST2700 clock driver.

In reply to Ryan Chen

kernel test robot noticed the following build warnings:

Hi Ryan,

kernel test robot noticed the following build warnings:

url:    https://github.com/intel-lab-lkp/linux/commits/Ryan-Chen/dt-bindings-mfd-aspeed-support-for-AST2700/20241023-170434
base:   https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
patch link:    https://lore.kernel.org/r/20241023090153.1395220-4-ryan_chen%40aspeedtech.com
patch subject: [PATCH v6 3/3] clk: aspeed: add AST2700 clock driver.
config: loongarch-randconfig-r054-20241024 (https://download.01.org/0day-ci/archive/20241025/[email protected]/config)
compiler: loongarch64-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241025/[email protected]/reproduce)

All warnings (new ones prefixed by >>):

In file included from drivers/clk/clk-ast2700.c:17:

>> include/soc/aspeed/reset-aspeed.h:14:5: warning: no previous prototype for 'aspeed_reset_controller_register' [-Wmissing-prototypes]

14 | int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

vim +/aspeed_reset_controller_register +14 include/soc/aspeed/reset-aspeed.h

1476d29e4461f6 Ryan Chen 2024-10-23 9

1476d29e4461f6 Ryan Chen 2024-10-23  10  #if IS_ENABLED(CONFIG_RESET_ASPEED)
1476d29e4461f6 Ryan Chen 2024-10-23  11  int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
1476d29e4461f6 Ryan Chen 2024-10-23  12  				     const char *adev_name);
1476d29e4461f6 Ryan Chen 2024-10-23  13  #else
1476d29e4461f6 Ryan Chen 2024-10-23 @14  int aspeed_reset_controller_register(struct device *clk_dev, void __iomem *base,
1476d29e4461f6 Ryan Chen 2024-10-23  15  				     const char *adev_name)
1476d29e4461f6 Ryan Chen 2024-10-23  16  {
1476d29e4461f6 Ryan Chen 2024-10-23  17  	return -ENODEV;
1476d29e4461f6 Ryan Chen 2024-10-23  18  }
1476d29e4461f6 Ryan Chen 2024-10-23  19  #endif /* if IS_ENABLED(CONFIG_RESET_ASPEED) */

1476d29e4461f6 Ryan Chen 2024-10-23 20

Signature · 2 lines

0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

kernel test robot

build bot

2024-10-25 14:57 UTC #10 lore raw

Re: [PATCH v6 3/3] clk: aspeed: add AST2700 clock driver.

In reply to Ryan Chen

kernel test robot noticed the following build errors:

Hi Ryan,

kernel test robot noticed the following build errors:

[auto build test ERROR on clk/clk-next]
[also build test ERROR on pza/reset/next lee-mfd/for-mfd-next lee-leds/for-leds-next linus/master lee-mfd/for-mfd-fixes v6.12-rc4 next-20241025] [cannot apply to pza/imx-drm/next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Ryan-Chen/dt-bindings-mfd-aspeed-support-for-AST2700/20241023-170434
base:   https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
patch link:    https://lore.kernel.org/r/20241023090153.1395220-4-ryan_chen%40aspeedtech.com
patch subject: [PATCH v6 3/3] clk: aspeed: add AST2700 clock driver.
config: hexagon-allmodconfig (https://download.01.org/0day-ci/archive/20241025/[email protected]/config)
compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 5886454669c3c9026f7f27eab13509dd0241f2d6)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241025/[email protected]/reproduce)

All errors (new ones prefixed by >>):

>> ld.lld: error: undefined symbol: aspeed_reset_controller_register
   >>> referenced by clk-ast2700.c
   >>>               drivers/clk/clk-ast2700.o:(ast2700_soc_clk_probe) in archive vmlinux.a
   >>> referenced by clk-ast2700.c
   >>>               drivers/clk/clk-ast2700.o:(ast2700_soc_clk_probe) in archive vmlinux.a

Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for MODVERSIONS
Depends on [n]: MODULES [=y] && !COMPILE_TEST [=y]
Selected by [y]:
- RANDSTRUCT_FULL [=y] && (CC_HAS_RANDSTRUCT [=y] || GCC_PLUGINS [=n]) && MODULES [=y]
WARNING: unmet direct dependencies detected for GET_FREE_REGION
Depends on [n]: SPARSEMEM [=n]
Selected by [m]:
- RESOURCE_KUNIT_TEST [=m] && RUNTIME_TESTING_MENU [=y] && KUNIT [=m]

Signature · 2 lines

0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki