RE: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK dependencies

-----Original Message-----
From: Richard Cochran <richardcochran@gmail.com>
Sent: Tuesday, August 03, 2021 1:55 PM
To: Arnd Bergmann <arnd@kernel.org>
Cc: Nicolas Pitre <nico@fluxnic.net>; Keller, Jacob E <jacob.e.keller@intel.com>;
Brandeburg, Jesse [off-list ref]; Nguyen, Anthony L
[off-list ref]; David S. Miller [off-list ref]; Jakub
Kicinski [off-list ref]; Arnd Bergmann [off-list ref]; Kurt
Kanzenbach [off-list ref]; Saleem, Shiraz [off-list ref];
Ertman, David M [off-list ref]; intel-wired-lan@lists.osuosl.org;
netdev@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK
dependencies


On Tue, Aug 03, 2021 at 07:00:49PM +0200, Arnd Bergmann wrote:

quoted

If you turn all those 'select' lines into 'depends on', this will work, but it's
not actually much different from what I'm suggesting.

"depends" instead of "select" works for me.  I just want it simple and clear.

quoted

Maybe we can do it
in two steps: first fix the build failure by replacing all the 'imply'
statements
with the correct dependencies, and then you send a patch on top that
turns PPS and PTP_1588_CLOCK into bool options.

Sounds good.

Thanks,
Richard

Works for me too.

Thanks,
Jake