Re: [PATCH v2 1/4] ASoc: tas2783A: Add soundwire based codec driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Niranjan,

kernel test robot noticed the following build errors:

[auto build test ERROR on broonie-sound/for-next]
[also build test ERROR on linus/master v6.17-rc5 next-20250909]
[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/Niranjan-H-Y/ASoc-tas2783A-machine-driver-amp-utility-for-TI-devices/20250909-201553
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
patch link:    https://lore.kernel.org/r/20250901051144.1112-1-niranjan.hy%40ti.com
patch subject: [PATCH v2 1/4] ASoc: tas2783A: Add soundwire based codec driver
config: hexagon-randconfig-002-20250910 (https://download.01.org/0day-ci/archive/20250910/202509101012.OvT5ytrT-lkp@xxxxxxxxx/config)
compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project 7fb1dc08d2f025aad5777bb779dfac1197e9ef87)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250910/202509101012.OvT5ytrT-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202509101012.OvT5ytrT-lkp@xxxxxxxxx/

All errors (new ones prefixed by >>):

>> sound/soc/codecs/tas2783-sdw.c:791:8: error: incompatible pointer types passing 'size_t *' (aka 'unsigned int *') to parameter of type 'unsigned long *' [-Werror,-Wincompatible-pointer-types]
     791 |                                           &size, NULL);
         |                                           ^~~~~
   1 error generated.


vim +791 sound/soc/codecs/tas2783-sdw.c

   769	
   770	static s32 tas2783_update_calibdata(struct tas2783_prv *tas_dev)
   771	{
   772		efi_guid_t efi_guid = TAS2783_CALI_GUID;
   773		u32 attr, i, *tmp_val;
   774		size_t size;
   775		s32 ret;
   776		efi_status_t status;
   777		static efi_char16_t efi_names[][32] = {
   778			L"SmartAmpCalibrationData", L"CALI_DATA"};
   779	
   780		tmp_val = (u32 *)tas_dev->cali_data.data;
   781		attr = 0;
   782		i = 0;
   783	
   784		/*
   785		 * In some cases, the calibration is performed in Windows,
   786		 * and data was saved in UEFI. Linux can access it.
   787		 */
   788		for (i = 0; i < ARRAY_SIZE(efi_names); i++) {
   789			size = 0;
   790			status = efi.get_variable(efi_names[i], &efi_guid, &attr,
 > 791						  &size, NULL);
   792			if (size > TAS2783_CALIB_DATA_SZ) {
   793				dev_err(tas_dev->dev, "cali data too large\n");
   794				break;
   795			}
   796	
   797			tas_dev->cali_data.read_sz = size;
   798			if (status == EFI_BUFFER_TOO_SMALL) {
   799				status = efi.get_variable(efi_names[i], &efi_guid, &attr,
   800								&tas_dev->cali_data.read_sz,
   801								tas_dev->cali_data.data);
   802				dev_dbg(tas_dev->dev, "cali get %lu bytes result:%ld\n",
   803					tas_dev->cali_data.read_sz, status);
   804			}
   805			if (status == EFI_SUCCESS)
   806				break;
   807		}
   808	
   809		if (status != EFI_SUCCESS) {
   810			/* Failed got calibration data from EFI. */
   811			dev_dbg(tas_dev->dev, "No calibration data in UEFI.");
   812			return 0;
   813		}
   814	
   815		mutex_lock(&tas_dev->calib_lock);
   816		ret = tas2783_validate_calibdata(tas_dev, tas_dev->cali_data.data,
   817						 tas_dev->cali_data.read_sz);
   818		if (!ret)
   819			tas2783_set_calib_params_to_device(tas_dev, tmp_val);
   820		mutex_unlock(&tas_dev->calib_lock);
   821	
   822		return ret;
   823	}
   824	

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



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux