2014-07-23 04:41:02

by Nicholas Krause

[permalink] [raw]
Subject: smp-cmp.c: CDFIXMES

Are the lines with CDFIXME still needed? If not please tell me as I
will send in a patch removing these
two from this file in order to help you guys out :).
Cheers Nick


2014-07-23 06:47:06

by Paul Burton

[permalink] [raw]
Subject: Re: smp-cmp.c: CDFIXMES

On Wed, Jul 23, 2014 at 12:40:59AM -0400, Nick Krause wrote:
> Are the lines with CDFIXME still needed? If not please tell me as I
> will send in a patch removing these
> two from this file in order to help you guys out :).
> Cheers Nick

Hi Nick,

I imagine the only answer any of us can give you is "we don't know". If
we did then we'd have removed the code or the comments already.

Please do note that the smp-cmp.c file lives behind a Kconfig option
that is now marked as deprecated, and that there is other work going on
in areas related to clocksource & clock events on the applicable
systems. So whilst someone could spend the time figuring out whether
those lines are useful, I expect that cleaning up these old FIXMEs is
not a particularly high priority for anyone right now.

If you'll leave them alone for a while I expect they'll disappear one
way or another in a few cycles time, along with the rest of the file.

Thanks,
Paul

2014-07-23 15:19:26

by Nicholas Krause

[permalink] [raw]
Subject: Re: smp-cmp.c: CDFIXMES

On Wed, Jul 23, 2014 at 2:46 AM, Paul Burton <[email protected]> wrote:
> On Wed, Jul 23, 2014 at 12:40:59AM -0400, Nick Krause wrote:
>> Are the lines with CDFIXME still needed? If not please tell me as I
>> will send in a patch removing these
>> two from this file in order to help you guys out :).
>> Cheers Nick
>
> Hi Nick,
>
> I imagine the only answer any of us can give you is "we don't know". If
> we did then we'd have removed the code or the comments already.
>
> Please do note that the smp-cmp.c file lives behind a Kconfig option
> that is now marked as deprecated, and that there is other work going on
> in areas related to clocksource & clock events on the applicable
> systems. So whilst someone could spend the time figuring out whether
> those lines are useful, I expect that cleaning up these old FIXMEs is
> not a particularly high priority for anyone right now.
>
> If you'll leave them alone for a while I expect they'll disappear one
> way or another in a few cycles time, along with the rest of the file.
>
> Thanks,
> Paul


That's Ok Paul, just wanted to known when the file was going away as it seemed
old if you want help with this please let me known :).
Cheers Nick