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

Groups > linux.kernel > #1646182

Re: [PATCH v3] staging: iio: light: Replace symbolic permissions as per coding style

Show all headers | View raw

On Sat, May 20, 2017 at 06:55:02PM +0100, Jonathan Cameron wrote:
> On 19/05/17 10:37, surenderpolsani@gmail.com wrote:
> >From: Surender Polsani <surenderpolsani@gmail.com>
> >
> >Fixed the following checkpatch.pl warnings:
> >octal permissions are more preferable than symbolic permissions
> >
> >Replaced DEVICE_ATTR family macros with DEVICE_ATTR_RW family
> >as suggested by Greg K-H. Changed attributes and function
> >names where ever required to satisfy internal macro definitions
> >like __ATTR__RW().
> >
> >Signed-off-by: Surender Polsani <surenderpolsani@gmail.com>
> Nicely presented patch, but it runs into the fact that some of these
> shouldn't exist as hand rolled attrs in the first place.
> 
> Some of theses should be handled through the various info_mask and
> event_info_mask bitmaps + read_raw etc.
> 
> This would be a much less mechanical change however...
> 
> See inline and I'll try and pick out which ones.
> 
> Brian is working on this driver as well at the moment so there may
> well be some clashes.
> 
> Perhaps you two could confer on who will target what?  Saves
> everyone time to work together!

You can apply this if you'd like. I just got 7 different hardware
samples from Jon this week that are supported by this driver. I haven't
gotten far yet with coding since I'm currently working on getting
them all setup so that it will be easy for me to test my upcoming
driver changes.

For my first patch series, I'm planning to migrate the driver to use
IIO channels, cleaning up the I2C calls, and runtime power management
support. Hopefully I'll have this to you for next weekend.

Brian

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

Thread

[PATCH] staging: iio: light: Replace symbolic permissions as per coding style surenderpolsani@gmail.com - 2017-05-18 10:40 +0200
  Re: [PATCH] staging: iio: light: Replace symbolic permissions as per  coding style Greg KH <gregkh@linuxfoundation.org> - 2017-05-18 11:10 +0200
    [PATCH v2] staging: iio: light: Replace symbolic permissions as per coding style surenderpolsani@gmail.com - 2017-05-19 09:10 +0200
      Re: [PATCH v2] staging: iio: light: Replace symbolic permissions as  per coding style Greg KH <gregkh@linuxfoundation.org> - 2017-05-19 09:20 +0200
        Re: [PATCH v2] staging: iio: light: Replace symbolic permissions as  per coding style suri <surenderpolsani@gmail.com> - 2017-05-19 09:40 +0200
    [PATCH v2] staging: iio: light: Replace symbolic permissions as per coding style surenderpolsani@gmail.com - 2017-05-19 10:20 +0200
      Re: [PATCH v2] staging: iio: light: Replace symbolic permissions as  per coding style Greg KH <gregkh@linuxfoundation.org> - 2017-05-19 10:30 +0200
        [PATCH v3] staging: iio: light: Replace symbolic permissions as per coding style surenderpolsani@gmail.com - 2017-05-19 11:40 +0200
          Re: [PATCH v3] staging: iio: light: Replace symbolic permissions as  per coding style Jonathan Cameron <jic23@kernel.org> - 2017-05-20 20:00 +0200
            Re: [PATCH v3] staging: iio: light: Replace symbolic permissions as  per coding style Brian Masney <masneyb@onstation.org> - 2017-05-20 20:40 +0200
              Re: [PATCH v3] staging: iio: light: Replace symbolic permissions as  per coding style Jonathan Cameron <jic23@kernel.org> - 2017-05-21 13:00 +0200

csiph-web