Re: [PATCH net 1/3] cpsw/netcp: cpts depends on posix_timers
From: Arnd Bergmann
Date: Mon Mar 13 2017 - 12:56:11 EST
On Fri, Dec 16, 2016 at 7:18 PM, Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> wrote:
> On Fri, 16 Dec 2016, Arnd Bergmann wrote:
>> With posix timers having become optional, we get a build error with
>> the cpts time sync option of the CPSW driver:
>> drivers/net/ethernet/ti/cpts.c: In function 'cpts_find_ts':
>> drivers/net/ethernet/ti/cpts.c:291:23: error: implicit declaration of function 'ptp_classify_raw';did you mean 'ptp_classifier_init'? [-Werror=implicit-function-declaration]
>> It really makes no sense to build this driver if we can't use PTP,
>> so it's better to go back to 'select PTP_1588_CLOCK' but instead
>> add a dependency on POSIX_TIMERS.
> Why not depend on PTP_1588_CLOCK directly instead?
Sorry for dropping the ball on this. I just noticed my two patches in my patch
are still required, so I'm resending them now.
I think using 'select' here is better for consistency as we use 'imply'
elsewhere, and mixing the two might cause circular dependencies.
If you still have concerns about this, please follow up on the resend.