Linux clock framework development
 help / color / mirror / Atom feed
* [PATCHv2] clk: thead: allow COMPILE_TEST builds
@ 2026-07-07  0:56 Rosen Penev
  2026-07-07 14:57 ` Brian Masney
  0 siblings, 1 reply; 4+ messages in thread
From: Rosen Penev @ 2026-07-07  0:56 UTC (permalink / raw)
  To: linux-clk
  Cc: Michael Turquette, Stephen Boyd, Brian Masney, Nathan Chancellor,
	Nick Desaulniers, Bill Wendling, Justin Stitt, open list,
	open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b

Follow the pattern of many other directories in this Makefile (sunxi-ng,
tenstorrent, ti, etc.) by using obj-y. This allows COMPILE_TEST to
select the driver without having CONFIG_ARCH_THEAD enabled

Tested with:
make LLVM=1 ARCH=loongarch drivers/clk/thead/

Assisted-by: Codex:GPT-5.5
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 v2: use obj-y
 drivers/clk/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile
index f6a71fd6d872..bbcedf5d60ea 100644
--- a/drivers/clk/Makefile
+++ b/drivers/clk/Makefile
@@ -157,7 +157,7 @@ obj-$(CONFIG_ARCH_SUNXI)		+= sunxi/
 obj-y					+= sunxi-ng/
 obj-$(CONFIG_CLK_TEGRA)			+= tegra/
 obj-y					+= tenstorrent/
-obj-$(CONFIG_ARCH_THEAD)		+= thead/
+obj-y					+= thead/
 obj-y					+= ti/
 obj-$(CONFIG_CLK_UNIPHIER)		+= uniphier/
 obj-$(CONFIG_ARCH_U8500)		+= ux500/
-- 
2.55.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCHv2] clk: thead: allow COMPILE_TEST builds
  2026-07-07  0:56 [PATCHv2] clk: thead: allow COMPILE_TEST builds Rosen Penev
@ 2026-07-07 14:57 ` Brian Masney
  2026-07-09  6:30   ` Drew Fustini
  0 siblings, 1 reply; 4+ messages in thread
From: Brian Masney @ 2026-07-07 14:57 UTC (permalink / raw)
  To: Rosen Penev
  Cc: linux-clk, Michael Turquette, Stephen Boyd, Nathan Chancellor,
	Nick Desaulniers, Bill Wendling, Justin Stitt, open list,
	open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b

On Mon, Jul 06, 2026 at 05:56:19PM -0700, Rosen Penev wrote:
> Follow the pattern of many other directories in this Makefile (sunxi-ng,
> tenstorrent, ti, etc.) by using obj-y. This allows COMPILE_TEST to
> select the driver without having CONFIG_ARCH_THEAD enabled
> 
> Tested with:
> make LLVM=1 ARCH=loongarch drivers/clk/thead/
> 
> Assisted-by: Codex:GPT-5.5
> Signed-off-by: Rosen Penev <rosenp@gmail.com>

Reviewed-by: Brian Masney <bmasney@redhat.com>


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCHv2] clk: thead: allow COMPILE_TEST builds
  2026-07-07 14:57 ` Brian Masney
@ 2026-07-09  6:30   ` Drew Fustini
  2026-07-09 12:05     ` Brian Masney
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Fustini @ 2026-07-09  6:30 UTC (permalink / raw)
  To: Brian Masney
  Cc: Rosen Penev, linux-clk, Michael Turquette, Stephen Boyd,
	Nathan Chancellor, Nick Desaulniers, Bill Wendling, Justin Stitt,
	open list,
	open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b

On Tue, Jul 07, 2026 at 10:57:51AM -0400, Brian Masney wrote:
> On Mon, Jul 06, 2026 at 05:56:19PM -0700, Rosen Penev wrote:
> > Follow the pattern of many other directories in this Makefile (sunxi-ng,
> > tenstorrent, ti, etc.) by using obj-y. This allows COMPILE_TEST to
> > select the driver without having CONFIG_ARCH_THEAD enabled
> > 
> > Tested with:
> > make LLVM=1 ARCH=loongarch drivers/clk/thead/
> > 
> > Assisted-by: Codex:GPT-5.5
> > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> 
> Reviewed-by: Brian Masney <bmasney@redhat.com>

This doesn't apply to 7.2-rc or next for me due to the surrounding
context, but I agree with the change itself.

Acked-by: Drew Fustini <fustini@kernel.org>

Brian - do you think this should be routed through my thead-clk tree or
go through the main clk tree?

Thanks,
Drew

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCHv2] clk: thead: allow COMPILE_TEST builds
  2026-07-09  6:30   ` Drew Fustini
@ 2026-07-09 12:05     ` Brian Masney
  0 siblings, 0 replies; 4+ messages in thread
From: Brian Masney @ 2026-07-09 12:05 UTC (permalink / raw)
  To: Drew Fustini
  Cc: Rosen Penev, linux-clk, Michael Turquette, Stephen Boyd,
	Nathan Chancellor, Nick Desaulniers, Bill Wendling, Justin Stitt,
	open list,
	open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b

Hey Drew,

On Thu, Jul 9, 2026 at 2:30 AM Drew Fustini <fustini@kernel.org> wrote:
> On Tue, Jul 07, 2026 at 10:57:51AM -0400, Brian Masney wrote:
> > On Mon, Jul 06, 2026 at 05:56:19PM -0700, Rosen Penev wrote:
> > > Follow the pattern of many other directories in this Makefile (sunxi-ng,
> > > tenstorrent, ti, etc.) by using obj-y. This allows COMPILE_TEST to
> > > select the driver without having CONFIG_ARCH_THEAD enabled
> > >
> > > Tested with:
> > > make LLVM=1 ARCH=loongarch drivers/clk/thead/
> > >
> > > Assisted-by: Codex:GPT-5.5
> > > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> >
> > Reviewed-by: Brian Masney <bmasney@redhat.com>
>
> This doesn't apply to 7.2-rc or next for me due to the surrounding
> context, but I agree with the change itself.
>
> Acked-by: Drew Fustini <fustini@kernel.org>
>
> Brian - do you think this should be routed through my thead-clk tree or
> go through the main clk tree?

It'd be great if you could pick this up through the thead-clk tree.

Brian


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-07-09 12:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-07  0:56 [PATCHv2] clk: thead: allow COMPILE_TEST builds Rosen Penev
2026-07-07 14:57 ` Brian Masney
2026-07-09  6:30   ` Drew Fustini
2026-07-09 12:05     ` Brian Masney

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox