linux-kernel - Re: [PATCH v1 1/2] ALSA: atmel: Remove AVR32 bits from the driver

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] [day] [month] [year] [list]

Date:   Thu, 4 May 2017 04:23:41 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc:     kbuild-all@...org,
        Hans-Christian Noren Egtvedt <egtvedt@...fundet.no>,
        linux-kernel@...r.kernel.org,
        Haavard Skinnemoen <hskinnemoen@...il.com>,
        Nicolas Ferre <nicolas.ferre@...el.com>,
        Boris Brezillon <boris.brezillon@...e-electrons.com>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Subject: Re: [PATCH v1 1/2] ALSA: atmel: Remove AVR32 bits from the driver

Hi Andy,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11 next-20170503]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andy-Shevchenko/ALSA-atmel-Remove-AVR32-bits-from-the-driver/20170502-214413
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=arm 

All errors (new ones prefixed by >>):

   sound//atmel/ac97c.c: In function 'atmel_ac97c_playback_hw_free':
   sound//atmel/ac97c.c:205:22: warning: unused variable 'chip' [-Wunused-variable]
     struct atmel_ac97c *chip = snd_pcm_substream_chip(substream);
                         ^~~~
   sound//atmel/ac97c.c: In function 'atmel_ac97c_capture_hw_free':
   sound//atmel/ac97c.c:211:22: warning: unused variable 'chip' [-Wunused-variable]
     struct atmel_ac97c *chip = snd_pcm_substream_chip(substream);
                         ^~~~
   sound//atmel/ac97c.c: In function 'atmel_ac97c_capture_trigger':
   sound//atmel/ac97c.c:444:1: warning: label 'out' defined but not used [-Wunused-label]
    out:
    ^~~
   sound//atmel/ac97c.c: In function 'atmel_ac97c_interrupt':
>> sound//atmel/ac97c.c:540:3: error: expected ';' before '}' token
      }
      ^

vim +540 sound//atmel/ac97c.c

7177395f Sedji Gaouaou   2010-03-01  524  			runtime = chip->playback_substream->runtime;
e9d5ef6b Andy Shevchenko 2017-05-02  525  			block_size = frames_to_bytes(runtime, runtime->period_size);
7177395f Sedji Gaouaou   2010-03-01  526  			chip->playback_period++;
7177395f Sedji Gaouaou   2010-03-01  527  
7177395f Sedji Gaouaou   2010-03-01  528  			if (chip->playback_period == runtime->periods)
7177395f Sedji Gaouaou   2010-03-01  529  				chip->playback_period = 0;
7177395f Sedji Gaouaou   2010-03-01  530  			next_period = chip->playback_period + 1;
7177395f Sedji Gaouaou   2010-03-01  531  			if (next_period == runtime->periods)
7177395f Sedji Gaouaou   2010-03-01  532  				next_period = 0;
7177395f Sedji Gaouaou   2010-03-01  533  
7177395f Sedji Gaouaou   2010-03-01  534  			offset = block_size * next_period;
7177395f Sedji Gaouaou   2010-03-01  535  
e9d5ef6b Andy Shevchenko 2017-05-02  536  			writel(runtime->dma_addr + offset, chip->regs + ATMEL_PDC_TNPR);
e9d5ef6b Andy Shevchenko 2017-05-02  537  			writel(block_size / 2, chip->regs + ATMEL_PDC_TNCR);
7177395f Sedji Gaouaou   2010-03-01  538  
e9d5ef6b Andy Shevchenko 2017-05-02  539  			snd_pcm_period_elapsed(chip->playback_substream)
7177395f Sedji Gaouaou   2010-03-01 @540  		}
7177395f Sedji Gaouaou   2010-03-01  541  		if ((casr & camr) & AC97C_CSR_ENDRX) {
7177395f Sedji Gaouaou   2010-03-01  542  			runtime = chip->capture_substream->runtime;
e9d5ef6b Andy Shevchenko 2017-05-02  543  			block_size = frames_to_bytes(runtime, runtime->period_size);
7177395f Sedji Gaouaou   2010-03-01  544  			chip->capture_period++;
7177395f Sedji Gaouaou   2010-03-01  545  
7177395f Sedji Gaouaou   2010-03-01  546  			if (chip->capture_period == runtime->periods)
7177395f Sedji Gaouaou   2010-03-01  547  				chip->capture_period = 0;
7177395f Sedji Gaouaou   2010-03-01  548  			next_period = chip->capture_period + 1;

:::::: The code at line 540 was first introduced by commit
:::::: 7177395fdd919e561544a7d1c0ac196098a2ae2d ALSA: AC97: add AC97 support for AT91.

:::::: TO: Sedji Gaouaou <sedji.gaouaou@...el.com>
:::::: CC: Takashi Iwai <tiwai@...e.de>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (61359 bytes)