All of lore.kernel.org
 help / color / mirror / Atom feed
* [plbossart-sound:sdw/bios-workaround 17/17] sound/soc/codecs/rt711-sdw.c:474:28: warning: data argument not used by format string
@ 2021-01-22  8:32 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2021-01-22  8:32 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 12485 bytes --]

tree:   https://github.com/plbossart/sound sdw/bios-workaround
head:   c19a33acf26ede628da999ba4880d5900d9f0275
commit: c19a33acf26ede628da999ba4880d5900d9f0275 [17/17] [SQUASHME] rt1711-sdw: better DMI quirk with link_id
config: s390-randconfig-r025-20210121 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bd3a387ee76f58caa0d7901f3f84e9bb3d006f27)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://github.com/plbossart/sound/commit/c19a33acf26ede628da999ba4880d5900d9f0275
        git remote add plbossart-sound https://github.com/plbossart/sound
        git fetch --no-tags plbossart-sound sdw/bios-workaround
        git checkout c19a33acf26ede628da999ba4880d5900d9f0275
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=s390 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   include/uapi/linux/swab.h:19:12: note: expanded from macro '___constant_swab32'
           (((__u32)(x) & (__u32)0x000000ffUL) << 24) |            \
                     ^
   In file included from sound/soc/codecs/rt711-sdw.c:17:
   In file included from include/linux/regmap.h:20:
   In file included from include/linux/iopoll.h:14:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
           ___constant_swab32(x) :                 \
                              ^
   include/uapi/linux/swab.h:20:12: note: expanded from macro '___constant_swab32'
           (((__u32)(x) & (__u32)0x0000ff00UL) <<  8) |            \
                     ^
   In file included from sound/soc/codecs/rt711-sdw.c:17:
   In file included from include/linux/regmap.h:20:
   In file included from include/linux/iopoll.h:14:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
           ___constant_swab32(x) :                 \
                              ^
   include/uapi/linux/swab.h:21:12: note: expanded from macro '___constant_swab32'
           (((__u32)(x) & (__u32)0x00ff0000UL) >>  8) |            \
                     ^
   In file included from sound/soc/codecs/rt711-sdw.c:17:
   In file included from include/linux/regmap.h:20:
   In file included from include/linux/iopoll.h:14:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
           ___constant_swab32(x) :                 \
                              ^
   include/uapi/linux/swab.h:22:12: note: expanded from macro '___constant_swab32'
           (((__u32)(x) & (__u32)0xff000000UL) >> 24)))
                     ^
   In file included from sound/soc/codecs/rt711-sdw.c:17:
   In file included from include/linux/regmap.h:20:
   In file included from include/linux/iopoll.h:14:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:120:12: note: expanded from macro '__swab32'
           __fswab32(x))
                     ^
   In file included from sound/soc/codecs/rt711-sdw.c:17:
   In file included from include/linux/regmap.h:20:
   In file included from include/linux/iopoll.h:14:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:80:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> sound/soc/codecs/rt711-sdw.c:474:28: warning: data argument not used by format string [-Wformat-extra-args]
                                   __func__, id->part_id, slave->bus->link_id);
                                                          ^
   include/linux/dev_printk.h:131:47: note: expanded from macro 'dev_dbg'
                   dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
                                                       ~~~     ^
   21 warnings generated.


vim +474 sound/soc/codecs/rt711-sdw.c

38dc8f0688634a65 Pierre-Louis Bossart 2021-01-21  456  
320b8b0d13b81f36 Shuming Fan          2019-12-27  457  static int rt711_sdw_probe(struct sdw_slave *slave,
320b8b0d13b81f36 Shuming Fan          2019-12-27  458  				const struct sdw_device_id *id)
320b8b0d13b81f36 Shuming Fan          2019-12-27  459  {
320b8b0d13b81f36 Shuming Fan          2019-12-27  460  	struct regmap *sdw_regmap, *regmap;
38dc8f0688634a65 Pierre-Louis Bossart 2021-01-21  461  	const struct dmi_system_id *dmi_id;
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  462  	unsigned long link_id;
320b8b0d13b81f36 Shuming Fan          2019-12-27  463  
d8b1f84994d843ee Pierre-Louis Bossart 2021-01-20  464  	/* only allow BIOS work-around in specific configurations */
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  465  	if (id->part_id == 0x701) {
38dc8f0688634a65 Pierre-Louis Bossart 2021-01-21  466  		dmi_id = dmi_first_match(rt701_quirk_table);
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  467  		if (!dmi_id) {
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  468  			dev_dbg(&slave->dev, "%s: probe workaround requires DMI quirk\n", __func__);
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  469  			return -ENODEV;
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  470  		}
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  471  		link_id = (unsigned long)dmi_id->driver_data;
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  472  		if (link_id != slave->bus->link_id) {
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  473  			dev_dbg(&slave->dev, "%s: probe workaround on link %d not supported\n",
d8b1f84994d843ee Pierre-Louis Bossart 2021-01-20 @474  				__func__, id->part_id, slave->bus->link_id);
d8b1f84994d843ee Pierre-Louis Bossart 2021-01-20  475  			return -ENODEV;
d8b1f84994d843ee Pierre-Louis Bossart 2021-01-20  476  		}
c19a33acf26ede62 Pierre-Louis Bossart 2021-01-21  477  	}
d8b1f84994d843ee Pierre-Louis Bossart 2021-01-20  478  
320b8b0d13b81f36 Shuming Fan          2019-12-27  479  	/* Regmap Initialization */
320b8b0d13b81f36 Shuming Fan          2019-12-27  480  	sdw_regmap = devm_regmap_init_sdw(slave, &rt711_sdw_regmap);
be1a4b2c56db860a Vinod Koul           2020-08-26  481  	if (IS_ERR(sdw_regmap))
be1a4b2c56db860a Vinod Koul           2020-08-26  482  		return PTR_ERR(sdw_regmap);
320b8b0d13b81f36 Shuming Fan          2019-12-27  483  
320b8b0d13b81f36 Shuming Fan          2019-12-27  484  	regmap = devm_regmap_init(&slave->dev, NULL,
320b8b0d13b81f36 Shuming Fan          2019-12-27  485  		&slave->dev, &rt711_regmap);
b03c29c1b13afa18 Wei Yongjun          2020-01-15  486  	if (IS_ERR(regmap))
b03c29c1b13afa18 Wei Yongjun          2020-01-15  487  		return PTR_ERR(regmap);
320b8b0d13b81f36 Shuming Fan          2019-12-27  488  
d8b1f84994d843ee Pierre-Louis Bossart 2021-01-20  489  	slave->id.part_id = 0x711;
320b8b0d13b81f36 Shuming Fan          2019-12-27  490  	rt711_init(&slave->dev, sdw_regmap, regmap, slave);
320b8b0d13b81f36 Shuming Fan          2019-12-27  491  
320b8b0d13b81f36 Shuming Fan          2019-12-27  492  	return 0;
320b8b0d13b81f36 Shuming Fan          2019-12-27  493  }
320b8b0d13b81f36 Shuming Fan          2019-12-27  494  

:::::: The code at line 474 was first introduced by commit
:::::: d8b1f84994d843eeac2eff6618c82878798c2179 ASoC: codecs: rt711-sdw: add _ADR to work-around broken DSDT

:::::: TO: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
:::::: CC: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 31878 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-01-22  8:32 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-22  8:32 [plbossart-sound:sdw/bios-workaround 17/17] sound/soc/codecs/rt711-sdw.c:474:28: warning: data argument not used by format string kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.