lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Tue, 16 May 2017 19:24:50 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     harinath Nampally <harinath922@...il.com>
Cc:     lars@...afoo.de, Michael.Hennerich@...log.com, knaack.h@....de,
        Peter Meerwald-Stadler <pmeerw@...erw.net>,
        Greg KH <gregkh@...uxfoundation.org>,
        linux-iio@...r.kernel.org, devel@...verdev.osuosl.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: iio: meter: Replace symbolic permissions with
 octal permissions

On 15/05/17 05:38, harinath Nampally wrote:
> Jonathan,
> 
> Thank you for letting me know about correct iio git tree.
> 
> I checked out 'togreg' branch, and noticed that
> latest ade7758_core.c still has checkpatch.pl warnings.
> 
> Actually Quentin Swain fixed the similar warnings in the ade7854.c,
> and my patch is fix for similar warnings but it is for different file
> ade7758_core.c
I guess he did both.
http://marc.info/?l=linux-iio&m=149359424405696&w=2

Chances are it was still sat in my testing branch as I hadn't
yet pushed it out as togreg post the autobuilders running
build tests on it.  I pushed out a few minutes ago so
should be up to date on the front end servers by now.

Jonathan
> 
> Please correct me if I am wrong.
> 
> Thanks,
> Harinath
> 
> On Mon, May 15, 2017 at 12:27 AM, harinath Nampally
> <harinath922@...il.com> wrote:
>> Jonathan,
>>
>> Thank you for letting me know about correct iio git tree.
>>
>> I checked out 'togreg' branch, and noticed that
>> latest ade7758_core.c still has checkpatch.pl warnings.
>>
>> Actually Quentin Swain fixed the similar warnings in the ade7854.c,
>> and my patch is fix for similar warnings but it is for different file
>> ade7758_core.c
>>
>> Please refer to below link for Quentin Swain's patch.
>> https://www.spinics.net/lists/linux-iio/msg33010.html
>>
>> Please let me know if I am wrong.
>>
>> Thanks,
>> Harinath
>>
>> On Mon, May 15, 2017 at 12:22 AM, harinath Nampally
>> <harinath922@...il.com> wrote:
>>> Jonathan,
>>>
>>> Thank you for letting me know about correct iio git tree.
>>>
>>> I checked out 'togreg' branch, and noticed that
>>> latest ade7758_core.c still has checkpatch.pl warnings.
>>>
>>> Actually Quentin Swain fixed the similar warnings in the ade7854.c,
>>> and my patch is fix for similar warnings but it is for different file
>>> ade7758_core.c
>>>
>>> Please refer to below link for Quentin Swain's patch.
>>> https://www.spinics.net/lists/linux-iio/msg33010.html
>>>
>>> Please let me know if I am wrong.
>>>
>>> Thanks,
>>> Harinath
>>>
>>> On Sun, May 14, 2017 at 10:41 AM, Jonathan Cameron <jic23@...nel.org> wrote:
>>>>
>>>> On 12/05/17 22:19, Harinath Nampally wrote:
>>>>>
>>>>> This patch fixes below kind of warnings:
>>>>> WARNING: Symbolic permissions 'S_IXXX | S_IXXX' are not preferred.
>>>>>
>>>>> Issue found and fixed by checkpatch.pl
>>>>>
>>>>> Signed-off-by: Harinath Nampally <harinath922@...il.com>
>>>>
>>>> Already done by Quentin Swain <dudebrobro179@...il.com> last month...
>>>>
>>>> For IIO related patches, stuff gets queued up in iio.git on
>>>> kernel.org before it gets sent on to Greg.  This is relatively
>>>> unusual for staging, but does lead to lots of repeats of
>>>> patches like this unfortunately.
>>>>
>>>> Jonathan
>>>>
>>>>> ---
>>>>>    drivers/staging/iio/meter/ade7758_core.c | 50
>>>>> ++++++++++++++++----------------
>>>>>    1 file changed, 25 insertions(+), 25 deletions(-)
>>>>>
>>>>> diff --git a/drivers/staging/iio/meter/ade7758_core.c
>>>>> b/drivers/staging/iio/meter/ade7758_core.c
>>>>> index 99c89e6..40498af 100644
>>>>> --- a/drivers/staging/iio/meter/ade7758_core.c
>>>>> +++ b/drivers/staging/iio/meter/ade7758_core.c
>>>>> @@ -301,103 +301,103 @@ static int ade7758_reset(struct device *dev)
>>>>>          return ret;
>>>>>    }
>>>>>    -static IIO_DEV_ATTR_VPEAK(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_VPEAK(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_VPEAK);
>>>>> -static IIO_DEV_ATTR_IPEAK(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_IPEAK(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_VPEAK);
>>>>> -static IIO_DEV_ATTR_APHCAL(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_APHCAL(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_APHCAL);
>>>>> -static IIO_DEV_ATTR_BPHCAL(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BPHCAL(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_BPHCAL);
>>>>> -static IIO_DEV_ATTR_CPHCAL(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CPHCAL(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_CPHCAL);
>>>>> -static IIO_DEV_ATTR_WDIV(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_WDIV(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_WDIV);
>>>>> -static IIO_DEV_ATTR_VADIV(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_VADIV(0644,
>>>>>                  ade7758_read_8bit,
>>>>>                  ade7758_write_8bit,
>>>>>                  ADE7758_VADIV);
>>>>> -static IIO_DEV_ATTR_AIRMS(S_IRUGO,
>>>>> +static IIO_DEV_ATTR_AIRMS(0444,
>>>>>                  ade7758_read_24bit,
>>>>>                  NULL,
>>>>>                  ADE7758_AIRMS);
>>>>> -static IIO_DEV_ATTR_BIRMS(S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BIRMS(0444,
>>>>>                  ade7758_read_24bit,
>>>>>                  NULL,
>>>>>                  ADE7758_BIRMS);
>>>>> -static IIO_DEV_ATTR_CIRMS(S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CIRMS(0444,
>>>>>                  ade7758_read_24bit,
>>>>>                  NULL,
>>>>>                  ADE7758_CIRMS);
>>>>> -static IIO_DEV_ATTR_AVRMS(S_IRUGO,
>>>>> +static IIO_DEV_ATTR_AVRMS(0444,
>>>>>                  ade7758_read_24bit,
>>>>>                  NULL,
>>>>>                  ADE7758_AVRMS);
>>>>> -static IIO_DEV_ATTR_BVRMS(S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BVRMS(0444,
>>>>>                  ade7758_read_24bit,
>>>>>                  NULL,
>>>>>                  ADE7758_BVRMS);
>>>>> -static IIO_DEV_ATTR_CVRMS(S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CVRMS(0444,
>>>>>                  ade7758_read_24bit,
>>>>>                  NULL,
>>>>>                  ADE7758_CVRMS);
>>>>> -static IIO_DEV_ATTR_AIRMSOS(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_AIRMSOS(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_AIRMSOS);
>>>>> -static IIO_DEV_ATTR_BIRMSOS(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BIRMSOS(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_BIRMSOS);
>>>>> -static IIO_DEV_ATTR_CIRMSOS(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CIRMSOS(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_CIRMSOS);
>>>>> -static IIO_DEV_ATTR_AVRMSOS(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_AVRMSOS(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_AVRMSOS);
>>>>> -static IIO_DEV_ATTR_BVRMSOS(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BVRMSOS(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_BVRMSOS);
>>>>> -static IIO_DEV_ATTR_CVRMSOS(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CVRMSOS(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_CVRMSOS);
>>>>> -static IIO_DEV_ATTR_AIGAIN(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_AIGAIN(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_AIGAIN);
>>>>> -static IIO_DEV_ATTR_BIGAIN(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BIGAIN(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_BIGAIN);
>>>>> -static IIO_DEV_ATTR_CIGAIN(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CIGAIN(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_CIGAIN);
>>>>> -static IIO_DEV_ATTR_AVRMSGAIN(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_AVRMSGAIN(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_AVRMSGAIN);
>>>>> -static IIO_DEV_ATTR_BVRMSGAIN(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_BVRMSGAIN(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_BVRMSGAIN);
>>>>> -static IIO_DEV_ATTR_CVRMSGAIN(S_IWUSR | S_IRUGO,
>>>>> +static IIO_DEV_ATTR_CVRMSGAIN(0644,
>>>>>                  ade7758_read_16bit,
>>>>>                  ade7758_write_16bit,
>>>>>                  ADE7758_CVRMSGAIN);
>>>>>
>>>>
>>>
>>>
>>>
>>> --
>>> Thanks,
>>> Harinath
>>> +1 312 560 8565
>>>
>>>
>>
>>
>>
>> --
>> Thanks,
>> Harinath
>> +1 312 560 8565
> 
> 
> 

Powered by blists - more mailing lists