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

[kbuild test robot]

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@xxxxxxxxx>
:::::: CC: Takashi Iwai <tiwai@xxxxxxx>

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

Attachment: .config.gz
Description: application/gzip