Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1194973

Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform data

Show all headers | View raw

Hi Lee,

On Tue, 28 Jul 2015 18:32:16 +0100, Lee Jones wrote:
> On Tue, 28 Jul 2015, Guenter Roeck wrote:
> > On 07/28/2015 08:28 AM, Lee Jones wrote:
> > >On Tue, 28 Jul 2015, Guenter Roeck wrote:
> > >>Sure, we could have changed it to lowercase, but so far no one bothered.
> > >>Plus, of course, there is always the element that some maintainers hate
> > >>that kind of cleanup,
> > >
> > >Really?  Surely any kind of clean-up is good clean-up.  Especially as
> > >Greg KH et. al, have been doing public presentations telling everyone
> > >that there is always kernel work for anyone who has the time; spelling
> > >corrections and all.
> > 
> > Yes, really. Just try to submit cleanup patches to maintainers other than
> > Greg and myself, and you'll see. It is a minefield.
> 
> Admittedly some of us have our quirks, but I'm happy to challenge
> anyone that won't accept clean-up patches that make things better.

I may be one of these, to some degree, under certain circumstances.

The problem is that what you call "better" may not actually sound
better to me. Or it might be better in some respect but worse in others.

Specifically, your proposal to rename a kernel driver to remove
capitals, obviously has upsides, but it may also have downsides. Are
modprobe and friends case-insensitive? If not then renaming the module
that way may break existing "options" or "blacklist" statements
in /etc/modprobe.conf, or initialization scripts.

Such a change may also require some changes on the distribution side,
from a packaging perspective.

Likewise, renaming variables in the code makes it look better, but at
the cost of making future backports to that driver more difficult.

And if nothing else, the time you (or others) spend on this, is time
you won't spend somewhere else where it may be more useful. Or fun.

So in the end there's always a balance between the costs and the
benefits. Which may explain why sometimes some maintainers aren't so
interested in certain clean-up patches.

-- 
Jean Delvare
SUSE L3 Support
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/5] iTCO_wdt: Add support for Intel Sunrisepoint Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-27 15:40 +0200
  [PATCH 2/5] i2c: i801: Create iTCO device on newer Intel PCHs Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-27 15:40 +0200
    Re: [PATCH 2/5] i2c: i801: Create iTCO device on newer Intel PCHs Guenter Roeck <linux@roeck-us.net> - 2015-07-27 16:10 +0200
      Re: [PATCH 2/5] i2c: i801: Create iTCO device on newer Intel PCHs Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-28 11:40 +0200
  [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform data Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-27 15:40 +0200
    Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Guenter Roeck <linux@roeck-us.net> - 2015-07-27 15:50 +0200
      Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-27 16:20 +0200
        Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Guenter Roeck <linux@roeck-us.net> - 2015-07-27 16:30 +0200
          Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-28 12:00 +0200
    Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Lee Jones <lee.jones@linaro.org> - 2015-07-27 23:40 +0200
      Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-28 11:20 +0200
    Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Lee Jones <lee.jones@linaro.org> - 2015-07-28 11:50 +0200
      Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Matt Fleming <matt@codeblueprint.co.uk> - 2015-07-28 13:10 +0200
        Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Jean Delvare <jdelvare@suse.de> - 2015-07-29 11:10 +0200
          Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Lee Jones <lee.jones@linaro.org> - 2015-07-29 12:10 +0200
        Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Aaron Sierra <asierra@xes-inc.com> - 2015-07-29 17:00 +0200
          Re: [PATCH 1/5] iTCO_wdt: Expose watchdog properties using platform  data Lee Jones <lee.jones@linaro.org> - 2015-07-30 11:00 +0200

csiph-web