linux-clk.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] clk: sunxi-ng: Add hardware dependency
@ 2016-09-08 15:36 Jean Delvare
  2016-09-08 19:32 ` Stephen Boyd
  0 siblings, 1 reply; 3+ messages in thread
From: Jean Delvare @ 2016-09-08 15:36 UTC (permalink / raw)
  To: linux-clk; +Cc: Michael Turquette, Stephen Boyd, Maxime Ripard, Chen-Yu Tsai

The sunxi-ng clock driver is useless for other architectures.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: Michael Turquette <mturquette@baylibre.com>
---
 drivers/clk/sunxi-ng/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- linux-4.8-rc5.orig/drivers/clk/sunxi-ng/Kconfig	2016-09-04 23:31:46.000000000 +0200
+++ linux-4.8-rc5/drivers/clk/sunxi-ng/Kconfig	2016-09-08 17:29:46.179508283 +0200
@@ -1,6 +1,6 @@
 config SUNXI_CCU
 	bool "Clock support for Allwinner SoCs"
-	default ARCH_SUNXI
+	depends on ARCH_SUNXI || COMPILE_TEST
 
 if SUNXI_CCU
 


-- 
Jean Delvare
SUSE L3 Support

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

* Re: [PATCH] clk: sunxi-ng: Add hardware dependency
  2016-09-08 15:36 [PATCH] clk: sunxi-ng: Add hardware dependency Jean Delvare
@ 2016-09-08 19:32 ` Stephen Boyd
  2016-09-08 19:52   ` Jean Delvare
  0 siblings, 1 reply; 3+ messages in thread
From: Stephen Boyd @ 2016-09-08 19:32 UTC (permalink / raw)
  To: Jean Delvare, linux-clk; +Cc: Michael Turquette, Maxime Ripard, Chen-Yu Tsai

On 09/08/2016 08:36 AM, Jean Delvare wrote:
> The sunxi-ng clock driver is useless for other architectures.
>
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
> Cc: Michael Turquette <mturquette@baylibre.com>
> ---
>  drivers/clk/sunxi-ng/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-4.8-rc5.orig/drivers/clk/sunxi-ng/Kconfig	2016-09-04 23:31:46.000000000 +0200
> +++ linux-4.8-rc5/drivers/clk/sunxi-ng/Kconfig	2016-09-08 17:29:46.179508283 +0200
> @@ -1,6 +1,6 @@
>  config SUNXI_CCU
>  	bool "Clock support for Allwinner SoCs"
> -	default ARCH_SUNXI
> +	depends on ARCH_SUNXI || COMPILE_TEST

Please leave the default there and just add the depends and compile test
part.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

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

* Re: [PATCH] clk: sunxi-ng: Add hardware dependency
  2016-09-08 19:32 ` Stephen Boyd
@ 2016-09-08 19:52   ` Jean Delvare
  0 siblings, 0 replies; 3+ messages in thread
From: Jean Delvare @ 2016-09-08 19:52 UTC (permalink / raw)
  To: Stephen Boyd; +Cc: linux-clk, Michael Turquette, Maxime Ripard, Chen-Yu Tsai

On Thu, 8 Sep 2016 12:32:07 -0700, Stephen Boyd wrote:
> On 09/08/2016 08:36 AM, Jean Delvare wrote:
> > The sunxi-ng clock driver is useless for other architectures.
> >
> > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
> > Cc: Michael Turquette <mturquette@baylibre.com>
> > ---
> >  drivers/clk/sunxi-ng/Kconfig |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > --- linux-4.8-rc5.orig/drivers/clk/sunxi-ng/Kconfig	2016-09-04 23:31:46.000000000 +0200
> > +++ linux-4.8-rc5/drivers/clk/sunxi-ng/Kconfig	2016-09-08 17:29:46.179508283 +0200
> > @@ -1,6 +1,6 @@
> >  config SUNXI_CCU
> >  	bool "Clock support for Allwinner SoCs"
> > -	default ARCH_SUNXI
> > +	depends on ARCH_SUNXI || COMPILE_TEST
> 
> Please leave the default there and just add the depends and compile test
> part.

Yes, makes sense sorry. Will send v2 shortly.

Thanks for the review,
-- 
Jean Delvare
SUSE L3 Support

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

end of thread, other threads:[~2016-09-08 19:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-08 15:36 [PATCH] clk: sunxi-ng: Add hardware dependency Jean Delvare
2016-09-08 19:32 ` Stephen Boyd
2016-09-08 19:52   ` Jean Delvare

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