mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* smp-cmp.c: CDFIXMES
@ 2014-07-23  4:40 Nick Krause
  2014-07-23  6:46 ` Paul Burton
  0 siblings, 1 reply; 3+ messages in thread
From: Nick Krause @ 2014-07-23  4:40 UTC (permalink / raw)
  To: Ralf Baechle
  Cc: Paul Burton, John Crispin, markos.chandras, Steven.Hill,
	Linux-MIPS, linux-kernel

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: smp-cmp.c: CDFIXMES
  2014-07-23  4:40 smp-cmp.c: CDFIXMES Nick Krause
@ 2014-07-23  6:46 ` Paul Burton
  2014-07-23 15:19   ` Nick Krause
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Burton @ 2014-07-23  6:46 UTC (permalink / raw)
  To: Nick Krause
  Cc: Ralf Baechle, John Crispin, markos.chandras, Steven.Hill,
	Linux-MIPS, linux-kernel

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: smp-cmp.c: CDFIXMES
  2014-07-23  6:46 ` Paul Burton
@ 2014-07-23 15:19   ` Nick Krause
  0 siblings, 0 replies; 3+ messages in thread
From: Nick Krause @ 2014-07-23 15:19 UTC (permalink / raw)
  To: Paul Burton
  Cc: Ralf Baechle, John Crispin, markos.chandras, Steven.Hill,
	Linux-MIPS, linux-kernel

On Wed, Jul 23, 2014 at 2:46 AM, Paul Burton <paul.burton@imgtec.com> 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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-07-23 15:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-23  4:40 smp-cmp.c: CDFIXMES Nick Krause
2014-07-23  6:46 ` Paul Burton
2014-07-23 15:19   ` Nick Krause

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®