[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ed2d1106-f388-6a36-cedf-33ebb5987d28@gmail.com>
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
Powered by blists - more mailing lists