lkml.org 
[lkml]   [2019]   [Oct]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v15 06/19] leds: lp50xx: Add the LP50XX family of the RGB LED driver
From
Date
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.

Previously I used quite old gcc 5.2 or so.

--
Best regards,
Jacek Anaszewski

\
 
 \ /
  Last update: 2019-10-30 23:01    [W:0.073 / U:0.844 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site