linux-kernel - Re: [PATCH v15 06/19] leds: lp50xx: Add the LP50XX family of the RGB LED driver

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:   Wed, 30 Oct 2019 23:03:02 +0100
From:   Jacek Anaszewski <jacek.anaszewski@...il.com>
To:     Dan Murphy <dmurphy@...com>
Cc:     pavel@....cz, linux-leds@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v15 06/19] leds: lp50xx: Add the LP50XX family of the RGB
 LED driver



On 10/30/19 11:00 PM, Jacek Anaszewski wrote:
> Dan,
> 
> On 10/30/19 9:55 PM, Dan Murphy wrote:
>> Jacek
>>
>> On 10/30/19 3:50 PM, Jacek Anaszewski wrote:
>>> Dan,
>>>
>>> On 10/30/19 9:23 PM, Dan Murphy wrote:
>>>> Jacek
>>>>
>>>>
>>>>>> Does not appear here
>>>>>>
>>>>>> Finally not sure why the MIPS compiler is complaining about this
>>>>>> but the
>>>>>> ARM and x86 is not
>>>>> Compilation breaks also for ARM.
>>>> Thanks for the information.
>>>>
>>>> I am not seeing this issue or even a warning when using the ARM 8.3
>>>> toolchain even in a clean build.
>>>>
>>>> arm-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture
>>>> 8.3-2019.03 (arm-rel-8.36)) 8.3.0
>>>>
>>>> So is a bug in this toolchain?
>>>  From what I've just googled C specification allows for a compiler
>>> implementation to accept also other forms of constants expressions
>>> than standard ones pointed out in the spec.
>>>
>>> So this is not necessarily a bug.
>>>
>> OK.  Well I will try to repo with the above instructions.  And wait till
>> Friday for any other comments.  If I get no other comments I will make
>> the couple of changes and then post v16.
> 
> I confirm that with arm-unknown-linux-gnueabi-gcc just built with
> crosstool-NG 1.24.0 I don't observe the issue either.

Of course version 8.3.0.

> Previously I used quite old gcc 5.2 or so.
> 

-- 
Best regards,
Jacek Anaszewski