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

Groups > linux.kernel > #1589399

Re: [PATCH 0/4] fujitsu_init() cleanup

Show all headers | View raw

On Tue, Feb 28, 2017 at 09:33:28AM +0100, Micha?? K??pie?? wrote:
> GregKH wrote:
> > On Mon, Feb 27, 2017 at 10:17:55PM -0800, Darren Hart wrote:
> > > GregKH - Can you please confirm the above? Moving an attribute is different than
> > > the format and contents, which is what I explicitly documented in
> > > Documentation/admin-guide/sysfs-rules.rst (last section).
> > 
> > Moving an attribute to a different device structure is usually a bad
> > idea, if the userspace tool counting on it to be present in a specific
> > place breaks.
> 
> Yes, I am familiar with that premise.  Here is the thing though: I am
> unaware of any userspace tool which uses these attributes.  Though,
> obviously, that does not mean such tools do not exist.

For what it's worth I too am unaware of any utilities which use the
/sys/devices/platform/fujitsu-laptop/ attributes associated with the
backlight - this after using the S7020 since 2005.  I would be surprised if
any existed since they would have to be specifically for the Fujitsu
hardware.  If writing any utility to control the backlight the logical thing
to do would be to use the standard backlight attributes in
/sys/devices/virtual/backlight/fujitsu-laptop/.

> > But, as you can't be consistent here, don't break userspace please, I'd
> > recommend just leaving it alone for now.
> 
> Darren, in the light of the above I will be awaiting your final call on
> this before posting any further patches touching this area.  My number
> one priority was to drop the broken backlight-related attributes,
> because leaving the other attributes where they currently are does not
> prevent achieving a clean split between the two drivers registered by
> fujitsu-laptop, which is the ultimate objective of all these cleanups.

As I explained in my response to GregKH earlier and having investigated this
in more detail, I have no objection to the removal of the non-standard
backlight-related sysfs attributes (brightness_changed, lcd_level and
max_brightness).  They are almost certainly unused and their removal will
allow a significant cleanup of fujitsu-laptop.  Obviously however there are
competing viewpoints which take a bigger picture view so I will defer to
Darren's judgement.

Regards
  jonathan

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

Thread

Re: [PATCH 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-02-27 08:30 +0100
  Re: [PATCH 0/4] fujitsu_init() cleanup Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-02-28 09:10 +0100
    Re: [PATCH 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-02-28 09:40 +0100
      Re: [PATCH 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-02-28 13:20 +0100
        Re: [PATCH 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-02-28 14:40 +0100
          Re: [PATCH 0/4] fujitsu_init() cleanup Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-02-28 15:10 +0100
            Re: [PATCH 0/4] fujitsu_init() cleanup Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-02-28 16:00 +0100
            Re: [PATCH 0/4] fujitsu_init() cleanup Darren Hart <dvhart@infradead.org> - 2017-03-01 04:30 +0100
    Re: [PATCH 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-02-28 12:30 +0100
      Re: [PATCH 0/4] fujitsu_init() cleanup Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-02-28 13:40 +0100
  Re: [PATCH 0/4] fujitsu_init() cleanup Darren Hart <dvhart@infradead.org> - 2017-02-28 09:20 +0100

csiph-web