Hi Gilad, [auto build test ERROR on linus/master] [also build test ERROR on v4.11-rc7] [cannot apply to staging/staging-testing next-20170420] [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/Gilad-Ben-Yossef/staging-ccree-add-Arm-TrustZone-CryptoCell-REE-driver/20170420-222023 config: i386-allmodconfig (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=i386 All error/warnings (new ones prefixed by >>): In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_driver.c:60: >> drivers/staging/ccree/cc_hal.h:29:2: error: #error Unsupported platform #error Unsupported platform ^~~~~ drivers/staging/ccree/ssi_driver.c: In function 'cc_isr': >> drivers/staging/ccree/cc_hal.h:33:38: error: implicit declaration of function 'READ_REGISTER' [-Werror=implicit-function-declaration] #define CC_HAL_READ_REGISTER(offset) READ_REGISTER(cc_base + offset) ^ >> drivers/staging/ccree/ssi_driver.c:120:8: note: in expansion of macro 'CC_HAL_READ_REGISTER' irr = CC_HAL_READ_REGISTER(CC_REG_OFFSET(HOST_RGF, HOST_IRR)); ^~~~~~~~~~~~~~~~~~~~ >> drivers/staging/ccree/cc_hal.h:32:44: error: implicit declaration of function 'WRITE_REGISTER' [-Werror=implicit-function-declaration] #define CC_HAL_WRITE_REGISTER(offset, val) WRITE_REGISTER(cc_base + offset, val) ^ >> drivers/staging/ccree/ssi_driver.c:129:2: note: in expansion of macro 'CC_HAL_WRITE_REGISTER' CC_HAL_WRITE_REGISTER(CC_REG_OFFSET(HOST_RGF, HOST_ICR), irr); ^~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_sysfs.c:19: >> drivers/staging/ccree/cc_hal.h:29:2: error: #error Unsupported platform #error Unsupported platform ^~~~~ drivers/staging/ccree/ssi_sysfs.c: In function 'ssi_sys_regdump_show': >> drivers/staging/ccree/cc_hal.h:33:38: error: implicit declaration of function 'READ_REGISTER' [-Werror=implicit-function-declaration] #define CC_HAL_READ_REGISTER(offset) READ_REGISTER(cc_base + offset) ^ >> drivers/staging/ccree/ssi_sysfs.c:291:19: note: in expansion of macro 'CC_HAL_READ_REGISTER' register_value = CC_HAL_READ_REGISTER(CC_REG_OFFSET(HOST_RGF, HOST_SIGNATURE)); ^~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_buffer_mgr.h:27, from drivers/staging/ccree/ssi_buffer_mgr.c:28: >> drivers/staging/ccree/cc_hal.h:29:2: error: #error Unsupported platform #error Unsupported platform ^~~~~ -- In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_request_mgr.c:27: >> drivers/staging/ccree/cc_hal.h:29:2: error: #error Unsupported platform #error Unsupported platform ^~~~~ drivers/staging/ccree/ssi_request_mgr.c: In function 'request_mgr_init': >> drivers/staging/ccree/ssi_request_mgr.c:198:29: error: implicit declaration of function 'READ_REGISTER' [-Werror=implicit-function-declaration] req_mgr_h->hw_queue_size = READ_REGISTER(drvdata->cc_base + ^~~~~~~~~~~~~ In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_request_mgr.c:27: drivers/staging/ccree/ssi_request_mgr.c: In function 'comp_handler': >> drivers/staging/ccree/cc_hal.h:32:44: error: implicit declaration of function 'WRITE_REGISTER' [-Werror=implicit-function-declaration] #define CC_HAL_WRITE_REGISTER(offset, val) WRITE_REGISTER(cc_base + offset, val) ^ >> drivers/staging/ccree/ssi_request_mgr.c:595:3: note: in expansion of macro 'CC_HAL_WRITE_REGISTER' CC_HAL_WRITE_REGISTER(CC_REG_OFFSET(HOST_RGF, HOST_ICR), SSI_COMP_IRQ_MASK); ^~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_pm.c:24: >> drivers/staging/ccree/cc_hal.h:29:2: error: #error Unsupported platform #error Unsupported platform ^~~~~ drivers/staging/ccree/ssi_pm.c: In function 'ssi_power_mgr_runtime_suspend': >> drivers/staging/ccree/ssi_pm.c:46:2: error: implicit declaration of function 'WRITE_REGISTER' [-Werror=implicit-function-declaration] WRITE_REGISTER(drvdata->cc_base + CC_REG_OFFSET(HOST_RGF, HOST_POWER_DOWN_EN), POWER_DOWN_ENABLE); ^~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- In file included from drivers/staging/ccree/ssi_driver.h:48:0, from drivers/staging/ccree/ssi_pm_ext.c:24: >> drivers/staging/ccree/cc_hal.h:29:2: error: #error Unsupported platform #error Unsupported platform ^~~~~ drivers/staging/ccree/ssi_pm_ext.c: In function 'ssi_pm_ext_hw_suspend': >> drivers/staging/ccree/cc_hal.h:32:44: error: implicit declaration of function 'WRITE_REGISTER' [-Werror=implicit-function-declaration] #define CC_HAL_WRITE_REGISTER(offset, val) WRITE_REGISTER(cc_base + offset, val) ^ >> drivers/staging/ccree/ssi_pm_ext.c:41:2: note: in expansion of macro 'CC_HAL_WRITE_REGISTER' CC_HAL_WRITE_REGISTER(CC_REG_OFFSET(HOST_RGF, SRAM_ADDR), sram_addr); ^~~~~~~~~~~~~~~~~~~~~ >> drivers/staging/ccree/cc_hal.h:33:38: error: implicit declaration of function 'READ_REGISTER' [-Werror=implicit-function-declaration] #define CC_HAL_READ_REGISTER(offset) READ_REGISTER(cc_base + offset) ^ >> drivers/staging/ccree/ssi_pm_ext.c:47:10: note: in expansion of macro 'CC_HAL_READ_REGISTER' val = CC_HAL_READ_REGISTER(CC_REG_OFFSET(HOST_RGF, SRAM_DATA_READY)); ^~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors coccinelle warnings: (new ones prefixed by >>) >> drivers/staging/ccree/ssi_sysfs.c:319:34-35: WARNING: Use ARRAY_SIZE drivers/staging/ccree/ssi_sysfs.c:429:34-35: WARNING: Use ARRAY_SIZE -- >> drivers/staging/ccree/ssi_buffer_mgr.c:530:3-19: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values. -- >> drivers/staging/ccree/ssi_driver.c:484:6-11: No need to set .owner here. The core will do it. -- >> drivers/staging/ccree/ssi_request_mgr.c:623:3-4: Unneeded semicolon Please review and possibly fold the followup patch. vim +29 drivers/staging/ccree/cc_hal.h 23 24 #if defined(CONFIG_ARM) || defined(CONFIG_ARM64) 25 /* CC registers are always 32 bit wide (even on 64 bit platforms) */ 26 #define READ_REGISTER(_addr) ioread32((_addr)) 27 #define WRITE_REGISTER(_addr, _data) iowrite32((_data), (_addr)) 28 #else > 29 #error Unsupported platform 30 #endif 31 > 32 #define CC_HAL_WRITE_REGISTER(offset, val) WRITE_REGISTER(cc_base + offset, val) > 33 #define CC_HAL_READ_REGISTER(offset) READ_REGISTER(cc_base + offset) 34 35 #endif --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
Attachment:
.config.gz
Description: application/gzip
