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]

Message-ID: <f32e2063-3c56-607c-0e74-f75f68e62e16@ti.com>
Date:   Wed, 30 Oct 2019 15:23:09 -0500
From:   Dan Murphy <dmurphy@...com>
To:     Jacek Anaszewski <jacek.anaszewski@...il.com>,
        kbuild test robot <lkp@...el.com>
CC:     <kbuild-all@...ts.01.org>, <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

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?

> This is not architecture specific but language specific thing.
>
> If you enter this error message got Google you will get an answer
> to why this happens in the first result.

I did Google the issue and got the answer I guess I just question to why 
I am not seeing the error because for me the patchset builds fine no errors.

And why is this only reported on the MIPS arch.  Not saying that it is 
not an issue in the code just wondering how it gets there.

>
> You need to patch your driver with the attached one.
>
I will try to reproduce the error and then apply the changes.

Dan