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]
Date:   Sat, 20 May 2023 00:21:00 +0800
From:   kernel test robot <lkp@...el.com>
To:     Shenghao Ding <13916275206@....com>, broonie@...nel.org,
        devicetree@...r.kernel.org, krzysztof.kozlowski+dt@...aro.org,
        robh+dt@...nel.org, lgirdwood@...il.com, perex@...ex.cz,
        pierre-louis.bossart@...ux.intel.com
Cc:     oe-kbuild-all@...ts.linux.dev, kevin-lu@...com,
        shenghao-ding@...com, alsa-devel@...a-project.org,
        linux-kernel@...r.kernel.org, x1077012@...com, peeyush@...com,
        navada@...com, gentuser@...il.com, Ryan_Chu@...tron.com,
        Sam_Wu@...tron.com, Shenghao Ding <13916275206@....com>
Subject: Re: [PATCH v3 3/5] ASoC: tas2781: Add tas2781 driver

Hi Shenghao,

kernel test robot noticed the following build warnings:

[auto build test WARNING on broonie-sound/for-next]
[also build test WARNING on tiwai-sound/for-next tiwai-sound/for-linus linus/master v6.4-rc2 next-20230519]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Shenghao-Ding/ASoC-tas2781-firmware-lib/20230519-160451
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
patch link:    https://lore.kernel.org/r/20230519080211.20205-1-13916275206%40139.com
patch subject: [PATCH v3 3/5] ASoC: tas2781: Add tas2781 driver
config: arm64-randconfig-s033-20230518
compiler: aarch64-linux-gcc (GCC) 12.1.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # apt-get install sparse
        # sparse version: v0.6.4-39-gce1a6720-dirty
        # https://github.com/intel-lab-lkp/linux/commit/1138e1878d682a13d5ba424d90bb89dd7a05b637
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Shenghao-Ding/ASoC-tas2781-firmware-lib/20230519-160451
        git checkout 1138e1878d682a13d5ba424d90bb89dd7a05b637
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=arm64 olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=arm64 SHELL=/bin/bash sound/soc/codecs/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202305192322.gBZ4JIyr-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> sound/soc/codecs/tas2781-i2c.c:585:66: sparse: sparse: incorrect type in argument 1 (different base types) @@     expected restricted __be32 const [usertype] *cell @@     got unsigned int const *[assigned] reg @@
   sound/soc/codecs/tas2781-i2c.c:585:66: sparse:     expected restricted __be32 const [usertype] *cell
   sound/soc/codecs/tas2781-i2c.c:585:66: sparse:     got unsigned int const *[assigned] reg

vim +585 sound/soc/codecs/tas2781-i2c.c

   549	
   550	static void tasdevice_parse_dt(struct tasdevice_priv *tas_priv)
   551	{
   552		struct i2c_client *client = (struct i2c_client *)tas_priv->client;
   553		unsigned int dev_addrs[max_chn];
   554		int rc, i, ndev = 0;
   555	
   556		if (tas_priv->isacpi) {
   557			ndev = device_property_read_u32_array(&client->dev,
   558				"ti,audio-slots", NULL, 0);
   559			if (ndev <= 0) {
   560				ndev = 1;
   561				dev_addrs[0] = client->addr;
   562			} else {
   563				ndev = (ndev < ARRAY_SIZE(dev_addrs))
   564					? ndev : ARRAY_SIZE(dev_addrs);
   565				ndev = device_property_read_u32_array(&client->dev,
   566					"ti,audio-slots", dev_addrs, ndev);
   567			}
   568	
   569			tas_priv->irq_info.irq_gpio =
   570				acpi_dev_gpio_irq_get(ACPI_COMPANION(&client->dev), 0);
   571		} else {
   572			struct device_node *np = tas_priv->dev->of_node;
   573	#ifdef CONFIG_OF
   574			const unsigned int *reg, *reg_end;
   575			int len, sw, aw;
   576	
   577			aw = of_n_addr_cells(np);
   578			sw = of_n_size_cells(np);
   579			if (sw == 0) {
   580				reg = (const unsigned int *)of_get_property(np,
   581					"reg", &len);
   582				reg_end = reg + len/sizeof(*reg);
   583				ndev = 0;
   584				do {
 > 585					dev_addrs[ndev] = of_read_number(reg, aw);
   586					reg += aw;
   587					ndev++;
   588				} while (reg < reg_end);
   589			} else {
   590				ndev = 1;
   591				dev_addrs[0] = client->addr;
   592			}
   593	#else
   594			ndev = 1;
   595			dev_addrs[0] = client->addr;
   596	#endif
   597			tas_priv->irq_info.irq_gpio = of_irq_get(np, 0);
   598		}
   599		tas_priv->ndev = ndev;
   600		for (i = 0; i < ndev; i++)
   601			tas_priv->tasdevice[i].dev_addr = dev_addrs[i];
   602	
   603		if (ndev > 1) {
   604			rc = device_property_read_u32(&client->dev,
   605				"ti,broadcast-addr",
   606				&(tas_priv->glb_addr.dev_addr));
   607			if (rc) {
   608				dev_err(tas_priv->dev,
   609					"Looking up broadcast-addr failed %d\n", rc);
   610				tas_priv->glb_addr.dev_addr = 0;
   611			}
   612		}
   613	
   614		tas_priv->reset = devm_gpiod_get_optional(&client->dev,
   615				"reset-gpios", GPIOD_OUT_HIGH);
   616		if (IS_ERR(tas_priv->reset))
   617			dev_err(tas_priv->dev, "%s Can't get reset GPIO\n",
   618				__func__);
   619	
   620		strcpy(tas_priv->dev_name, tasdevice_id[tas_priv->chip_id].name);
   621	
   622		if (gpio_is_valid(tas_priv->irq_info.irq_gpio)) {
   623			rc = gpio_request(tas_priv->irq_info.irq_gpio,
   624					"AUDEV-IRQ");
   625			if (!rc) {
   626				gpio_direction_input(
   627					tas_priv->irq_info.irq_gpio);
   628	
   629				tas_priv->irq_info.irq =
   630					gpio_to_irq(tas_priv->irq_info.irq_gpio);
   631			} else
   632				dev_err(tas_priv->dev, "%s: GPIO %d request error\n",
   633					__func__, tas_priv->irq_info.irq_gpio);
   634		} else
   635			dev_err(tas_priv->dev,
   636				"Looking up irq-gpio property failed %d\n",
   637				tas_priv->irq_info.irq_gpio);
   638	}
   639	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

View attachment "config" of type "text/plain" (171934 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ