On 27/10/15 10:46, Stephen Boyd wrote: > On 10/26, Florian Fainelli wrote: >> diff --git a/drivers/clk/bcm/Kconfig b/drivers/clk/bcm/Kconfig >> index 85260fb96b36..a0c18528b70d 100644 >> --- a/drivers/clk/bcm/Kconfig >> +++ b/drivers/clk/bcm/Kconfig >> @@ -1,3 +1,13 @@ >> +config CLK_BCM_63XX >> + bool "Broadcom BCM63xx clock support" >> + depends on ARCH_BCM_63XX || COMPILE_TEST >> + depends on COMMON_CLK >> + select COMMON_CLK_IPROC >> + default y > > perhaps default CLK_BCM_63XX? Did you mean default ARCH_BCM_63XX instead? > >> + help >> + Enable common clock framework support for Broadcom BCM63xx DSL SoCs >> + based on the ARM architecture >> + >> config CLK_BCM_KONA >> bool "Broadcom Kona CCU clock support" >> depends on ARCH_BCM_MOBILE || COMPILE_TEST >> diff --git a/drivers/clk/bcm/clk-bcm63xx.c b/drivers/clk/bcm/clk-bcm63xx.c >> new file mode 100644 >> index 000000000000..b7e0469d0522 >> --- /dev/null >> +++ b/drivers/clk/bcm/clk-bcm63xx.c >> @@ -0,0 +1,22 @@ >> +/* >> + * Copyright (C) 2015 Broadcom Corporation >> + * >> + * This program is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU General Public License as >> + * published by the Free Software Foundation version 2. >> + * >> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any >> + * kind, whether express or implied; without even the implied warranty >> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + */ >> +#include <linux/kernel.h> >> +#include <linux/clk-provider.h> >> +#include <linux/of.h> > > #include <linux/init.h> ? > > I'm not sure what the kernel.h include is for. Nothing, init.h is what should be included instead. -- Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html