Re: [tiwai-sound:for-next 36/38] sound/pci/hda/cirrus_scodec_test.c:151:60: error: initializer element is not a compile-time constant

[Takashi Iwai <tiwai@suse.de>]

On Thu, 21 Sep 2023 11:23:14 +0200,
Richard Fitzgerald wrote:
> 
> On 21/9/23 00:35, kernel test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
> > head:   72f6a13022f3bf16df305b75c32f95ece263a5ce
> > commit: 2144833e7b41459fa2d52bb0676f0ab4920cf32c [36/38] ALSA: hda: cirrus_scodec: Add KUnit test
> > config: hexagon-randconfig-r041-20211219 (https://download.01.org/0day-ci/archive/20230921/202309210751.TXlnuEMF-lkp@intel.com/config)
> > compiler: clang version 16.0.4 (https://github.com/llvm/llvm-project.git ae42196bc493ffe877a7e3dff8be32035dea4d07)
> > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230921/202309210751.TXlnuEMF-lkp@intel.com/reproduce)
> > 
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@intel.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202309210751.TXlnuEMF-lkp@intel.com/
> > 
> > All errors (new ones prefixed by >>):
> > 
> >     In file included from sound/pci/hda/cirrus_scodec_test.c:9:
> >     In file included from include/linux/gpio/driver.h:6:
> >     In file included from include/linux/irqchip/chained_irq.h:10:
> >     In file included from include/linux/irq.h:20:
> >     In file included from include/linux/io.h:13:
> >     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.

Only the line indicated with the marker ">>" is interesting, and
that's what Andy already fixed in properties.h.


thanks,

Takashi