diff mbox series

[RESEND,net-next] net: ethernet: ti: cpts: fix build failure for powerpc

Message ID 20190611111632.9444-1-grygorii.strashko@ti.com
State Accepted
Commit a41efedfaf956698bcf5658daaadbe43b4ca1f46
Headers show
Series [RESEND,net-next] net: ethernet: ti: cpts: fix build failure for powerpc | expand

Commit Message

Grygorii Strashko June 11, 2019, 11:16 a.m. UTC
Add dependency to TI CPTS from Common CLK framework COMMON_CLK to fix
allyesconfig build for Powerpc:

drivers/net/ethernet/ti/cpts.c: In function 'cpts_of_mux_clk_setup':
drivers/net/ethernet/ti/cpts.c:567:2: error: implicit declaration of function 'of_clk_parent_fill'; did you mean 'of_clk_get_parent_name'? [-Werror=implicit-function-declaration]
  of_clk_parent_fill(refclk_np, parent_names, num_parents);
  ^~~~~~~~~~~~~~~~~~
  of_clk_get_parent_name

Fixes: a3047a81ba13 ("net: ethernet: ti: cpts: add support for ext rftclk selection")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

---
 Resending due to missed netdev@vger.kernel.org list in prev post.

 drivers/net/ethernet/ti/Kconfig | 1 +
 1 file changed, 1 insertion(+)

-- 
2.17.1

Comments

Stephen Rothwell June 12, 2019, 12:54 a.m. UTC | #1
Hi all,

On Tue, 11 Jun 2019 14:16:32 +0300 Grygorii Strashko <grygorii.strashko@ti.com> wrote:
>

> Add dependency to TI CPTS from Common CLK framework COMMON_CLK to fix

> allyesconfig build for Powerpc:

> 

> drivers/net/ethernet/ti/cpts.c: In function 'cpts_of_mux_clk_setup':

> drivers/net/ethernet/ti/cpts.c:567:2: error: implicit declaration of function 'of_clk_parent_fill'; did you mean 'of_clk_get_parent_name'? [-Werror=implicit-function-declaration]

>   of_clk_parent_fill(refclk_np, parent_names, num_parents);

>   ^~~~~~~~~~~~~~~~~~

>   of_clk_get_parent_name

> 

> Fixes: a3047a81ba13 ("net: ethernet: ti: cpts: add support for ext rftclk selection")

> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>

> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>


I have applied this to linu-next today instead of reverting a3047a81ba13.

-- 
Cheers,
Stephen Rothwell
David Miller June 12, 2019, 6:16 p.m. UTC | #2
From: Grygorii Strashko <grygorii.strashko@ti.com>

Date: Tue, 11 Jun 2019 14:16:32 +0300

> Add dependency to TI CPTS from Common CLK framework COMMON_CLK to fix

> allyesconfig build for Powerpc:

> 

> drivers/net/ethernet/ti/cpts.c: In function 'cpts_of_mux_clk_setup':

> drivers/net/ethernet/ti/cpts.c:567:2: error: implicit declaration of function 'of_clk_parent_fill'; did you mean 'of_clk_get_parent_name'? [-Werror=implicit-function-declaration]

>   of_clk_parent_fill(refclk_np, parent_names, num_parents);

>   ^~~~~~~~~~~~~~~~~~

>   of_clk_get_parent_name

> 

> Fixes: a3047a81ba13 ("net: ethernet: ti: cpts: add support for ext rftclk selection")

> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>

> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

> ---

>  Resending due to missed netdev@vger.kernel.org list in prev post.


Applied, thanks.
diff mbox series

Patch

diff --git a/drivers/net/ethernet/ti/Kconfig b/drivers/net/ethernet/ti/Kconfig
index bd05a977ee7e..a800d3417411 100644
--- a/drivers/net/ethernet/ti/Kconfig
+++ b/drivers/net/ethernet/ti/Kconfig
@@ -60,6 +60,7 @@  config TI_CPSW
 config TI_CPTS
 	bool "TI Common Platform Time Sync (CPTS) Support"
 	depends on TI_CPSW || TI_KEYSTONE_NETCP || COMPILE_TEST
+	depends on COMMON_CLK
 	depends on POSIX_TIMERS
 	---help---
 	  This driver supports the Common Platform Time Sync unit of