Re: [PATCH v2 0/2] make POSIX timers optional

From: Nicolas Pitre
Date: Tue Sep 20 2016 - 16:59:21 EST


On Tue, 20 Sep 2016, Richard Cochran wrote:

> On Tue, Sep 20, 2016 at 10:25:56PM +0200, Richard Cochran wrote:
> > After this series, if I don't pay enough attention to dmesg, then I
> > have lost functionality that I had in step #1. That sucks, and it has
> > nothing to do with the tinification option at all. It will bite even
> > if I have no knowledge of it. That isn't acceptable to me.
>
> Can't you leave all the "select PTP_1588_CLOCK" alone and simply add
>
> #ifdef CONFIG_POSIX_TIMERS
> // global declarations
> #else
> // static inlines
> #endif
>
> to ptp_clock_kernel.h, and then sandwich ptp_clock.c in
> #ifdef CONFIG_POSIX_TIMERS ... #endif ?

Sure I could... but I'm sure I'll be flamed by others for making things
even more obscure and hackish than they are right now.

Oh well... Let's go fix the Kconfig parser then.


Nicolas