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

kernel test robot
2020-06-09T12:13:11Z

[-- Attachment #1: Type: text/plain, Size: 4808 bytes --]

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@intel.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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 55392 bytes --]