Re: [net-next 11/13] igb: Update PTP function names/variables and locations.

On Fri, 2012-08-24 at 11:12 -0400, David Miller wrote:

From: Joe Perches <joe@perches.com>
Date: Fri, 24 Aug 2012 02:22:34 -0700

quoted

On Fri, 2012-08-24 at 08:32 +0200, Richard Cochran wrote:

quoted

On Thu, Aug 23, 2012 at 08:02:25PM -0700, Joe Perches wrote:

quoted

Improving code clarity and consistency isn't churn.

This patch series moves code around for no good reason. That is, by
definition, churn.

For your definition of good.

I think the people doing all of the actual development and
maintainence of the code get to decide how to define good.  And in
this case that's basically the Intel NIC development team, not you.

Just for clarity, you do mean Richard's view and not mine.
I agree with these proposed changes.

Moving functions around is a very valuable and useful cleanup quite
often.

So you can count me in on their definition of "good" as well.