Re: [PATCH 3/7] staging: ccree: add support for older HW revisions
Hi Gilad, [auto build test WARNING on staging/staging-testing] [also build test WARNING on next-20170623] [cannot apply to v4.12-rc6] [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-bug-fixes-and-TODO-items-for-4-13/20170623-134445 config: x86_64-randconfig-b0-06241039 (attached as .config) compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All warnings (new ones prefixed by >>): drivers/staging/ccree/ssi_sram_mgr.c: In function 'ssi_sram_mgr_init': >> drivers/staging/ccree/ssi_sram_mgr.c:76: warning: format '%x' expects type >> 'unsigned int', but argument 3 has type 'dma_addr_t' vim +76 drivers/staging/ccree/ssi_sram_mgr.c 60 /* Allocate "this" context */ 61 drvdata->sram_mgr_handle = kzalloc( 62 sizeof(struct ssi_sram_mgr_ctx), GFP_KERNEL); 63 if (!drvdata->sram_mgr_handle) { 64 SSI_LOG_ERR("Not enough memory to allocate SRAM_MGR ctx (%zu)\n", 65 sizeof(struct ssi_sram_mgr_ctx)); 66 rc = -ENOMEM; 67 goto out; 68 } 69 smgr_ctx = drvdata->sram_mgr_handle; 70 71 if (drvdata->hw_rev < CC_HW_REV_712) { 72 /* Pool starts after ROM bytes */ 73 start = (dma_addr_t)CC_HAL_READ_REGISTER(CC_REG_OFFSET(HOST_RGF, 74 HOST_SEP_SRAM_THRESHOLD)); 75 if ((start & 0x3) != 0) { > 76 SSI_LOG_ERR("Invalid SRAM offset 0x%x\n", start); 77 rc = -ENODEV; 78 goto out; 79 } 80 } 81 82 smgr_ctx->sram_free_offset = start; 83 return 0; 84 --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation .config.gz Description: application/gzip
Re: [PATCH 3/7] staging: ccree: add support for older HW revisions
Hi Gilad, [auto build test WARNING on staging/staging-testing] [also build test WARNING on next-20170623] [cannot apply to v4.12-rc6] [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-bug-fixes-and-TODO-items-for-4-13/20170623-134445 config: x86_64-randconfig-b0-06241039 (attached as .config) compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All warnings (new ones prefixed by >>): drivers/staging/ccree/ssi_sram_mgr.c: In function 'ssi_sram_mgr_init': >> drivers/staging/ccree/ssi_sram_mgr.c:76: warning: format '%x' expects type >> 'unsigned int', but argument 3 has type 'dma_addr_t' vim +76 drivers/staging/ccree/ssi_sram_mgr.c 60 /* Allocate "this" context */ 61 drvdata->sram_mgr_handle = kzalloc( 62 sizeof(struct ssi_sram_mgr_ctx), GFP_KERNEL); 63 if (!drvdata->sram_mgr_handle) { 64 SSI_LOG_ERR("Not enough memory to allocate SRAM_MGR ctx (%zu)\n", 65 sizeof(struct ssi_sram_mgr_ctx)); 66 rc = -ENOMEM; 67 goto out; 68 } 69 smgr_ctx = drvdata->sram_mgr_handle; 70 71 if (drvdata->hw_rev < CC_HW_REV_712) { 72 /* Pool starts after ROM bytes */ 73 start = (dma_addr_t)CC_HAL_READ_REGISTER(CC_REG_OFFSET(HOST_RGF, 74 HOST_SEP_SRAM_THRESHOLD)); 75 if ((start & 0x3) != 0) { > 76 SSI_LOG_ERR("Invalid SRAM offset 0x%x\n", start); 77 rc = -ENODEV; 78 goto out; 79 } 80 } 81 82 smgr_ctx->sram_free_offset = start; 83 return 0; 84 --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation .config.gz Description: application/gzip
Re: [PATCH 3/7] staging: ccree: add support for older HW revisions
Hi Gilad, [auto build test WARNING on staging/staging-testing] [also build test WARNING on next-20170622] [cannot apply to v4.12-rc6] [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-bug-fixes-and-TODO-items-for-4-13/20170623-134445 config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-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=sparc64 All warnings (new ones prefixed by >>): In file included from drivers/staging/ccree/ssi_sram_mgr.c:17:0: drivers/staging/ccree/ssi_sram_mgr.c: In function 'ssi_sram_mgr_init': include/linux/kern_levels.h:4:18: warning: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'dma_addr_t {aka long long unsigned int}' [-Wformat=] #define KERN_SOH "\001" /* ASCII Start Of Header */ ^ drivers/staging/ccree/ssi_driver.h:97:9: note: in definition of macro 'SSI_LOG' printk(level "ccree::%s: " format, __func__, ##__VA_ARGS__) ^ include/linux/kern_levels.h:10:18: note: in expansion of macro 'KERN_SOH' #define KERN_ERR KERN_SOH "3" /* error conditions */ ^~~~ >> drivers/staging/ccree/ssi_driver.h:98:42: note: in expansion of macro >> 'KERN_ERR' #define SSI_LOG_ERR(format, ...) SSI_LOG(KERN_ERR, format, ##__VA_ARGS__) ^~~~ >> drivers/staging/ccree/ssi_sram_mgr.c:76:4: note: in expansion of macro >> 'SSI_LOG_ERR' SSI_LOG_ERR("Invalid SRAM offset 0x%x\n", start); ^~~ vim +/KERN_ERR +98 drivers/staging/ccree/ssi_driver.h abefd674 Gilad Ben-Yossef 2017-04-23 91 /* AXI_ID is not actually the AXI ID of the transaction but the value of AXI_ID 250a00a7 Derek Robson 2017-05-30 92 * field in the HW descriptor. The DMA engine +8 that value. 250a00a7 Derek Robson 2017-05-30 93 */ abefd674 Gilad Ben-Yossef 2017-04-23 94 abefd674 Gilad Ben-Yossef 2017-04-23 95 /* Logging macros */ abefd674 Gilad Ben-Yossef 2017-04-23 96 #define SSI_LOG(level, format, ...) \ 891144d7 Gilad Ben-Yossef 2017-06-22 @97 printk(level "ccree::%s: " format, __func__, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 @98 #define SSI_LOG_ERR(format, ...) SSI_LOG(KERN_ERR, format, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 99 #define SSI_LOG_WARNING(format, ...) SSI_LOG(KERN_WARNING, format, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 100 #define SSI_LOG_NOTICE(format, ...) SSI_LOG(KERN_NOTICE, format, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 101 #define SSI_LOG_INFO(format, ...) SSI_LOG(KERN_INFO, format, ##__VA_ARGS__) :: The code at line 98 was first introduced by commit :: abefd6741d540fc624e73a2a3bdef2397bcbd064 staging: ccree: introduce CryptoCell HW driver :: TO: Gilad Ben-Yossef:: CC: Greg Kroah-Hartman --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation .config.gz Description: application/gzip
Re: [PATCH 3/7] staging: ccree: add support for older HW revisions
Hi Gilad, [auto build test WARNING on staging/staging-testing] [also build test WARNING on next-20170622] [cannot apply to v4.12-rc6] [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-bug-fixes-and-TODO-items-for-4-13/20170623-134445 config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-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=sparc64 All warnings (new ones prefixed by >>): In file included from drivers/staging/ccree/ssi_sram_mgr.c:17:0: drivers/staging/ccree/ssi_sram_mgr.c: In function 'ssi_sram_mgr_init': include/linux/kern_levels.h:4:18: warning: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'dma_addr_t {aka long long unsigned int}' [-Wformat=] #define KERN_SOH "\001" /* ASCII Start Of Header */ ^ drivers/staging/ccree/ssi_driver.h:97:9: note: in definition of macro 'SSI_LOG' printk(level "ccree::%s: " format, __func__, ##__VA_ARGS__) ^ include/linux/kern_levels.h:10:18: note: in expansion of macro 'KERN_SOH' #define KERN_ERR KERN_SOH "3" /* error conditions */ ^~~~ >> drivers/staging/ccree/ssi_driver.h:98:42: note: in expansion of macro >> 'KERN_ERR' #define SSI_LOG_ERR(format, ...) SSI_LOG(KERN_ERR, format, ##__VA_ARGS__) ^~~~ >> drivers/staging/ccree/ssi_sram_mgr.c:76:4: note: in expansion of macro >> 'SSI_LOG_ERR' SSI_LOG_ERR("Invalid SRAM offset 0x%x\n", start); ^~~ vim +/KERN_ERR +98 drivers/staging/ccree/ssi_driver.h abefd674 Gilad Ben-Yossef 2017-04-23 91 /* AXI_ID is not actually the AXI ID of the transaction but the value of AXI_ID 250a00a7 Derek Robson 2017-05-30 92 * field in the HW descriptor. The DMA engine +8 that value. 250a00a7 Derek Robson 2017-05-30 93 */ abefd674 Gilad Ben-Yossef 2017-04-23 94 abefd674 Gilad Ben-Yossef 2017-04-23 95 /* Logging macros */ abefd674 Gilad Ben-Yossef 2017-04-23 96 #define SSI_LOG(level, format, ...) \ 891144d7 Gilad Ben-Yossef 2017-06-22 @97 printk(level "ccree::%s: " format, __func__, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 @98 #define SSI_LOG_ERR(format, ...) SSI_LOG(KERN_ERR, format, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 99 #define SSI_LOG_WARNING(format, ...) SSI_LOG(KERN_WARNING, format, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 100 #define SSI_LOG_NOTICE(format, ...) SSI_LOG(KERN_NOTICE, format, ##__VA_ARGS__) abefd674 Gilad Ben-Yossef 2017-04-23 101 #define SSI_LOG_INFO(format, ...) SSI_LOG(KERN_INFO, format, ##__VA_ARGS__) :: The code at line 98 was first introduced by commit :: abefd6741d540fc624e73a2a3bdef2397bcbd064 staging: ccree: introduce CryptoCell HW driver :: TO: Gilad Ben-Yossef :: CC: Greg Kroah-Hartman --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation .config.gz Description: application/gzip