On Thu, Sep 21, 2023, at 08:34, Takashi Iwai wrote:
> On Thu, 21 Sep 2023 11:23:14 +0200, Richard Fitzgerald wrote:
>> On 21/9/23 00:35, kernel test robot wrote:
>> > In file included from arch/hexagon/include/asm/io.h:337:
>> > include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
>> > val = __raw_readb(PCI_IOBASE + addr);
>> > ~~~~~~~~~~ ^
>> > include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
>> > val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
>> > ~~~~~~~~~~ ^
>> > include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
>> > #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
>> > ^
>>
>> This looks like it has nothing to do with cirrus_scodec_test, it's just
>> unfortunate that including gpio/driver.h results in also including io.h.
>
> Yeah, I believe this part is irrelevant can be ignored. It's included
> as a part of (single) log.
Correct, it's a known bug that we've been working on for (literally) years,
it should be fixed soon by enforcing CONFIG_HAS_IOPORT soon to blank out
these functions on architectures that cannot support them. I hope we can
land the last patch for this in 6.7.
Arnd
