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]
Message-ID: <202212211125.4QXhmoUr-lkp@intel.com>
Date:   Wed, 21 Dec 2022 12:12:06 +0800
From:   kernel test robot <lkp@...el.com>
To:     Kevin Lu <luminlong@....com>, lgirdwood@...il.com,
        broonie@...nel.org, perex@...ex.cz, tiwai@...e.com
Cc:     oe-kbuild-all@...ts.linux.dev, alsa-devel@...a-project.org,
        linux-kernel@...r.kernel.org, shenghao-ding@...com,
        kevin-lu@...com, navada@...com, peeyush@...com,
        Kevin Lu <luminlong@....com>
Subject: Re: [PATCH v1] ALSA SoC: Texas Instruments TAS2781 Audio Smart Amp

Hi Kevin,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on broonie-sound/for-next]
[also build test WARNING on linus/master v6.1 next-20221220]
[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/Kevin-Lu/ALSA-SoC-Texas-Instruments-TAS2781-Audio-Smart-Amp/20221220-231421
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
patch link:    https://lore.kernel.org/r/20221220151157.2247-1-luminlong%40139.com
patch subject: [PATCH v1] ALSA SoC: Texas Instruments TAS2781 Audio Smart Amp
config: s390-allyesconfig
compiler: s390-linux-gcc (GCC) 12.1.0
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
        # https://github.com/intel-lab-lkp/linux/commit/de2ce420cebecc02e8b8f1eda156c623f9e8bc37
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Kevin-Lu/ALSA-SoC-Texas-Instruments-TAS2781-Audio-Smart-Amp/20221220-231421
        git checkout de2ce420cebecc02e8b8f1eda156c623f9e8bc37
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=s390 olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=s390 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>

All warnings (new ones prefixed by >>):

   sound/soc/codecs/tas2781-dsp.c:226:5: warning: no previous prototype for 'fw_parse_program_data_kernel' [-Wmissing-prototypes]
     226 | int fw_parse_program_data_kernel(struct tasdevice_priv *tas_dev,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   sound/soc/codecs/tas2781-dsp.c:298:5: warning: no previous prototype for 'fw_parse_configuration_data_kernel' [-Wmissing-prototypes]
     298 | int fw_parse_configuration_data_kernel(
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   sound/soc/codecs/tas2781-dsp.c:398:5: warning: no previous prototype for 'fw_parse_variable_header_kernel' [-Wmissing-prototypes]
     398 | int fw_parse_variable_header_kernel(struct tasdevice_priv *tas_dev,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   sound/soc/codecs/tas2781-dsp.c:536:5: warning: no previous prototype for 'tasdevice_load_block_kernel' [-Wmissing-prototypes]
     536 | int tasdevice_load_block_kernel(struct tasdevice_priv *pTAS2781,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   sound/soc/codecs/tas2781-dsp.c:650:5: warning: no previous prototype for 'fw_parse_variable_header_git' [-Wmissing-prototypes]
     650 | int fw_parse_variable_header_git(struct tasdevice_priv *tas_dev,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   sound/soc/codecs/tas2781-dsp.c:712:5: warning: no previous prototype for 'fw_parse_variable_header_cal' [-Wmissing-prototypes]
     712 | int fw_parse_variable_header_cal(struct tasdevice_priv *tas_dev,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from include/linux/device.h:15,
                    from include/linux/acpi.h:15,
                    from include/linux/i2c.h:13,
                    from sound/soc/codecs/tas2781-dsp.c:17:
   sound/soc/codecs/tas2781-dsp.c: In function 'fw_parse_block_data':
>> sound/soc/codecs/tas2781-dsp.c:875:25: warning: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
     875 |                         "%s: File Size(%u) error offset = %d n = %d\n",
         |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:144:56: note: in expansion of macro 'dev_fmt'
     144 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   sound/soc/codecs/tas2781-dsp.c:874:17: note: in expansion of macro 'dev_err'
     874 |                 dev_err(pFirmware->dev,
         |                 ^~~~~~~
   sound/soc/codecs/tas2781-dsp.c:875:41: note: format string is defined here
     875 |                         "%s: File Size(%u) error offset = %d n = %d\n",
         |                                        ~^
         |                                         |
         |                                         unsigned int
         |                                        %lu
   sound/soc/codecs/tas2781-dsp.c: In function 'fw_parse_header':
>> sound/soc/codecs/tas2781-dsp.c:1312:39: warning: format '%d' expects argument of type 'int', but argument 3 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
    1312 |                 dev_err(tas_dev->dev, "File size not match, %d %d", pFW->size,
         |                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:144:56: note: in expansion of macro 'dev_fmt'
     144 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   sound/soc/codecs/tas2781-dsp.c:1312:17: note: in expansion of macro 'dev_err'
    1312 |                 dev_err(tas_dev->dev, "File size not match, %d %d", pFW->size,
         |                 ^~~~~~~
   sound/soc/codecs/tas2781-dsp.c:1312:62: note: format string is defined here
    1312 |                 dev_err(tas_dev->dev, "File size not match, %d %d", pFW->size,
         |                                                             ~^
         |                                                              |
         |                                                              int
         |                                                             %ld
   sound/soc/codecs/tas2781-dsp.c: In function 'tas2781_load_calibration':
   sound/soc/codecs/tas2781-dsp.c:1986:33: warning: format '%d' expects argument of type 'int', but argument 4 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
    1986 |                                 "%s: file read error: size = %d\n",
         |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:144:56: note: in expansion of macro 'dev_fmt'
     144 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   sound/soc/codecs/tas2781-dsp.c:1985:25: note: in expansion of macro 'dev_err'
    1985 |                         dev_err(tas_dev->dev,
         |                         ^~~~~~~
   sound/soc/codecs/tas2781-dsp.c:1986:63: note: format string is defined here
    1986 |                                 "%s: file read error: size = %d\n",
         |                                                              ~^
         |                                                               |
         |                                                               int
         |                                                              %ld


vim +875 sound/soc/codecs/tas2781-dsp.c

   806	
   807	static int fw_parse_block_data(struct TFirmware *pFirmware,
   808		struct TBlock *pBlock, const struct firmware *pFW, int offset)
   809	{
   810		unsigned char *pData = (unsigned char *)pFW->data;
   811		int n;
   812	
   813		if (offset + 4 > pFW->size) {
   814			dev_err(pFirmware->dev, "%s: mnType error\n", __func__);
   815			offset = -1;
   816			goto out;
   817		}
   818		pBlock->mnType = SMS_HTONL(pData[offset],
   819			pData[offset + 1], pData[offset + 2], pData[offset + 3]);
   820		offset  += 4;
   821	
   822		if (pFirmware->fw_hdr.mnFixedHdr.mnDriverVersion >=
   823			PPC_DRIVER_CRCCHK) {
   824			if (offset + 1 > pFW->size) {
   825				dev_err(pFirmware->dev, "%s: mbPChkSumPresent error\n",
   826					__func__);
   827				offset = -1;
   828				goto out;
   829			}
   830			pBlock->mbPChkSumPresent = pData[offset];
   831			offset++;
   832	
   833			if (offset + 1 > pFW->size) {
   834				dev_err(pFirmware->dev, "%s: mnPChkSum error\n",
   835					__func__);
   836				offset = -1;
   837				goto out;
   838			}
   839			pBlock->mnPChkSum = pData[offset];
   840			offset++;
   841	
   842			if (offset + 1 > pFW->size) {
   843				dev_err(pFirmware->dev, "%s: mbYChkSumPresent error\n",
   844					__func__);
   845				offset = -1;
   846				goto out;
   847			}
   848			pBlock->mbYChkSumPresent = pData[offset];
   849			offset++;
   850	
   851			if (offset + 1 > pFW->size) {
   852				dev_err(pFirmware->dev, "%s: mnYChkSum error\n",
   853					__func__);
   854				offset = -1;
   855				goto out;
   856			}
   857			pBlock->mnYChkSum = pData[offset];
   858			offset++;
   859		} else {
   860			pBlock->mbPChkSumPresent = 0;
   861			pBlock->mbYChkSumPresent = 0;
   862		}
   863		if (offset + 4 > pFW->size) {
   864			dev_err(pFirmware->dev, "%s: mnCommands error\n", __func__);
   865			offset = -1;
   866			goto out;
   867		}
   868		pBlock->mnCommands = SMS_HTONL(pData[offset],
   869			pData[offset + 1], pData[offset + 2], pData[offset + 3]);
   870		offset  += 4;
   871	
   872		n = pBlock->mnCommands * 4;
   873		if (offset + n > pFW->size) {
   874			dev_err(pFirmware->dev,
 > 875				"%s: File Size(%u) error offset = %d n = %d\n",
   876				__func__, pFW->size, offset, n);
   877			offset = -1;
   878			goto out;
   879		}
   880		pBlock->mpData = kmemdup(&pData[offset], n, GFP_KERNEL);
   881		if (pBlock->mpData == NULL) {
   882			offset = -1;
   883			goto out;
   884		}
   885		offset  += n;
   886	out:
   887		return offset;
   888	}
   889	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ