Re: [PATCH net-next v8 2/4] net: netcp: Add Keystone NetCP core ethernet driver

From: Murali Karicheri
Date: Tue Jan 20 2015 - 14:24:05 EST


On 01/20/2015 12:26 PM, Paul Bolle wrote:
On Tue, 2015-01-20 at 12:10 -0500, Murali Karicheri wrote:
On 01/20/2015 03:18 AM, Paul Bolle wrote:
On Thu, 2015-01-15 at 19:12 -0500, Murali Karicheri wrote:
diff --git a/drivers/net/ethernet/ti/Kconfig b/drivers/net/ethernet/ti/Kconfig
index 605dd90..e11bcfa 100644
--- a/drivers/net/ethernet/ti/Kconfig
+++ b/drivers/net/ethernet/ti/Kconfig
@@ -73,12 +73,23 @@ config TI_CPSW
config TI_CPTS
boolean "TI Common Platform Time Sync (CPTS) Support"
depends on TI_CPSW
+ depends on TI_CPSW || TI_KEYSTONE_NET

You probably meant to add
|| TI_KEYSTONE_NETCP

here. Ie, add CP. But as this slipped through testing it _might_ not be
needed at all.

Currently CPTS driver is not used for NetCP. So I want to remove the
above Kconfig dependency from Kconfig and cpts.o from the Makefile. Do
you expect me to send an incremental patch for this to the netdev list?

That's Dave's call. I think that Dave works with incremental patches
exclusively once things have hit (one of the trees that feed into)
linux-next.

or can pick the attached patch that addresses this issue. Let me know.

I'm just the reporter. Please send it through the regular channels. That
would certainly include netdev.

Sure! I just posted the patch to the netdev list.

Thanks for letting me know.

Murali

Paul Bolle



--
Murali Karicheri
Linux Kernel, Texas Instruments
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/