linux-kernel - Re: [PATCH v4 03/10] clk: actions: Add MMC clock-register reset bits

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:   Tue, 9 Jun 2020 19:29:09 +0800
From:   kernel test robot <lkp@...el.com>
To:     Amit Singh Tomar <amittomer25@...il.com>, andre.przywara@....com,
        afaerber@...e.de, manivannan.sadhasivam@...aro.org,
        sboyd@...nel.org
Cc:     kbuild-all@...ts.01.org, cristian.ciocaltea@...il.com,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-actions@...ts.infradead.org
Subject: Re: [PATCH v4 03/10] clk: actions: Add MMC clock-register reset bits

Hi Amit,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on clk/clk-next pza/reset/next linus/master v5.7 next-20200608]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Amit-Singh-Tomar/Add-MMC-and-DMA-support-for-Actions-S700/20200609-182123
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: nios2-allyesconfig (attached as .config)
compiler: nios2-linux-gcc (GCC) 9.3.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
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=nios2 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

Note: the linux-review/Amit-Singh-Tomar/Add-MMC-and-DMA-support-for-Actions-S700/20200609-182123 HEAD 1aa888462e44552a129b6ea35f603cf6ef65a83c builds fine.
      It only hurts bisectibility.

All error/warnings (new ones prefixed by >>, old ones prefixed by <<):

>> drivers/clk/actions/owl-s700.c:580:3: error: 'RESET_SD0' undeclared here (not in a function); did you mean 'RESET_SI'?
580 |  [RESET_SD0]     = { CMU_DEVRST0, BIT(22) },
|   ^~~~~~~~~
|   RESET_SI
>> drivers/clk/actions/owl-s700.c:580:3: error: array index in initializer not of integer type
drivers/clk/actions/owl-s700.c:580:3: note: (near initialization for 's700_resets')
>> drivers/clk/actions/owl-s700.c:581:3: error: 'RESET_SD1' undeclared here (not in a function); did you mean 'RESET_SI'?
581 |  [RESET_SD1]     = { CMU_DEVRST0, BIT(23) },
|   ^~~~~~~~~
|   RESET_SI
drivers/clk/actions/owl-s700.c:581:3: error: array index in initializer not of integer type
drivers/clk/actions/owl-s700.c:581:3: note: (near initialization for 's700_resets')
>> drivers/clk/actions/owl-s700.c:582:3: error: 'RESET_SD2' undeclared here (not in a function); did you mean 'RESET_SI'?
582 |  [RESET_SD2]     = { CMU_DEVRST0, BIT(24) },
|   ^~~~~~~~~
|   RESET_SI
drivers/clk/actions/owl-s700.c:582:3: error: array index in initializer not of integer type
drivers/clk/actions/owl-s700.c:582:3: note: (near initialization for 's700_resets')
>> drivers/clk/actions/owl-s700.c:587:17: warning: initialized field overwritten [-Woverride-init]
587 |  [RESET_SPI0] = { CMU_DEVRST1, BIT(4) },
|                 ^
drivers/clk/actions/owl-s700.c:587:17: note: (near initialization for 's700_resets[12]')
drivers/clk/actions/owl-s700.c:588:17: warning: initialized field overwritten [-Woverride-init]
588 |  [RESET_SPI1] = { CMU_DEVRST1, BIT(5) },
|                 ^
drivers/clk/actions/owl-s700.c:588:17: note: (near initialization for 's700_resets[13]')
drivers/clk/actions/owl-s700.c:589:17: warning: initialized field overwritten [-Woverride-init]
589 |  [RESET_SPI2] = { CMU_DEVRST1, BIT(6) },
|                 ^
drivers/clk/actions/owl-s700.c:589:17: note: (near initialization for 's700_resets[14]')

vim +580 drivers/clk/actions/owl-s700.c

   573	
   574	static const struct owl_reset_map s700_resets[] = {
   575		[RESET_DE]	= { CMU_DEVRST0, BIT(0) },
   576		[RESET_LCD0]	= { CMU_DEVRST0, BIT(1) },
   577		[RESET_DSI]	= { CMU_DEVRST0, BIT(2) },
   578		[RESET_CSI]	= { CMU_DEVRST0, BIT(13) },
   579		[RESET_SI]	= { CMU_DEVRST0, BIT(14) },
 > 580		[RESET_SD0]     = { CMU_DEVRST0, BIT(22) },
 > 581		[RESET_SD1]     = { CMU_DEVRST0, BIT(23) },
 > 582		[RESET_SD2]     = { CMU_DEVRST0, BIT(24) },
   583		[RESET_I2C0]	= { CMU_DEVRST1, BIT(0) },
   584		[RESET_I2C1]	= { CMU_DEVRST1, BIT(1) },
   585		[RESET_I2C2]	= { CMU_DEVRST1, BIT(2) },
   586		[RESET_I2C3]	= { CMU_DEVRST1, BIT(3) },
 > 587		[RESET_SPI0]	= { CMU_DEVRST1, BIT(4) },
   588		[RESET_SPI1]	= { CMU_DEVRST1, BIT(5) },
   589		[RESET_SPI2]	= { CMU_DEVRST1, BIT(6) },
   590		[RESET_SPI3]	= { CMU_DEVRST1, BIT(7) },
   591		[RESET_UART0]	= { CMU_DEVRST1, BIT(8) },
   592		[RESET_UART1]	= { CMU_DEVRST1, BIT(9) },
   593		[RESET_UART2]	= { CMU_DEVRST1, BIT(10) },
   594		[RESET_UART3]	= { CMU_DEVRST1, BIT(11) },
   595		[RESET_UART4]	= { CMU_DEVRST1, BIT(12) },
   596		[RESET_UART5]	= { CMU_DEVRST1, BIT(13) },
   597		[RESET_UART6]	= { CMU_DEVRST1, BIT(14) },
   598		[RESET_KEY]	= { CMU_DEVRST1, BIT(24) },
   599		[RESET_GPIO]	= { CMU_DEVRST1, BIT(25) },
   600		[RESET_AUDIO]	= { CMU_DEVRST1, BIT(29) },
   601	};
   602	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

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