On Thu, 2017-12-14 at 13:13 +0800, kbuild test robot wrote:
> Hi Ching,
>
> Thank you for the patch! Perhaps something to improve:
>
> [auto build test WARNING on scsi/for-next]
> [also build test WARNING on next-20171213]
> [cannot apply to v4.15-rc3]
> [if your patch is applied to the wrong git tree, please drop us a note to
> help improve the system]
As what I have said in [PATCH 0/4], this serial patches are apply to
Martin's 4.16/scsi-queue only. Now scsi/for-next and next-20171213 are
synchronized to Martin's 4.16/scsi-queue, so these patches can apply to
them. But v4.15-rc3 has to do some patches like 4.16/scsi-queue done
before, or this patch can not apply.
>
> url:
> https://github.com/0day-ci/linux/commits/Ching-Huang/scsi-arcmsr-simplify-hba_get_config-routine/20171213-224803
> base: https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
> reproduce:
> # apt-get install sparse
> make ARCH=x86_64 allmodconfig
> make C=1 CF=-D__CHECK_ENDIAN__
>
>
> sparse warnings: (new ones prefixed by >>)
>
>
> vim +2971 drivers/scsi/arcmsr/arcmsr_hba.c
>
> 2958
> 2959 static void arcmsr_get_adapter_config(struct
> AdapterControlBlock *pACB, uint32_t *rwbuffer)
> 2960 {
> 2961 int count;
> 2962 uint32_t *acb_firm_model = (uint32_t *)pACB->firm_model;
> 2963 uint32_t *acb_firm_version = (uint32_t
> *)pACB->firm_version;
> 2964 uint32_t *acb_device_map = (uint32_t *)pACB->device_map;
> 2965 uint32_t *firm_model = &rwbuffer[15];
> 2966 uint32_t *firm_version = &rwbuffer[17];
> 2967 uint32_t *device_map = &rwbuffer[21];
> 2968
> 2969 count = 2;
> 2970 while (count) {
> > 2971 *acb_firm_model = readl(firm_model);
> 2972 acb_firm_model++;
> 2973 firm_model++;
> 2974 count--;
> 2975 }
> 2976 count = 4;
> 2977 while (count) {
> 2978 *acb_firm_version = readl(firm_version);
> 2979 acb_firm_version++;
> 2980 firm_version++;
> 2981 count--;
> 2982 }
> 2983 count = 4;
> 2984 while (count) {
> 2985 *acb_device_map = readl(device_map);
> 2986 acb_device_map++;
> 2987 device_map++;
> 2988 count--;
> 2989 }
> 2990 pACB->signature = readl(&rwbuffer[0]);
> 2991 pACB->firm_request_len = readl(&rwbuffer[1]);
> 2992 pACB->firm_numbers_queue = readl(&rwbuffer[2]);
> > 2993 pACB->firm_sdram_size = readl(&rwbuffer[3]);
> > 2994 pACB->firm_hd_channels = readl(&rwbuffer[4]);
> 2995 pACB->firm_cfg_version = readl(&rwbuffer[25]);
> 2996 pr_notice("Areca RAID Controller%d: Model %s, F/W %s\n",
> 2997 pACB->host->host_no,
> 2998 pACB->firm_model,
> 2999 pACB->firm_version);
> 3000 }
> 3001
> 3002 static bool arcmsr_hbaA_get_config(struct AdapterControlBlock
> *acb)
> 3003 {
> 3004 struct MessageUnit_A __iomem *reg = acb->pmuA;
> 3005
> 3006 arcmsr_wait_firmware_ready(acb);
> 3007 writel(ARCMSR_INBOUND_MESG0_GET_CONFIG,
> ®->inbound_msgaddr0);
> 3008 if (!arcmsr_hbaA_wait_msgint_ready(acb)) {
> 3009 printk(KERN_NOTICE "arcmsr%d: wait 'get adapter
> firmware \
> 3010 miscellaneous data' timeout \n",
> acb->host->host_no);
> 3011 return false;
> 3012 }
> > 3013 arcmsr_get_adapter_config(acb, reg->message_rwbuffer);
> 3014 return true;
> 3015 }
> 3016 static bool arcmsr_hbaB_get_config(struct AdapterControlBlock
> *acb)
> 3017 {
> 3018 struct MessageUnit_B *reg = acb->pmuB;
> 3019
> 3020 arcmsr_wait_firmware_ready(acb);
> 3021 writel(ARCMSR_MESSAGE_START_DRIVER_MODE,
> reg->drv2iop_doorbell);
> 3022 if (!arcmsr_hbaB_wait_msgint_ready(acb)) {
> 3023 printk(KERN_ERR "arcmsr%d: can't set driver
> mode.\n", acb->host->host_no);
> 3024 return false;
> 3025 }
> 3026 writel(ARCMSR_MESSAGE_GET_CONFIG,
> reg->drv2iop_doorbell);
> 3027 if (!arcmsr_hbaB_wait_msgint_ready(acb)) {
> 3028 printk(KERN_NOTICE "arcmsr%d: wait 'get adapter
> firmware \
> 3029 miscellaneous data' timeout \n",
> acb->host->host_no);
> 3030 return false;
> 3031 }
> > 3032 arcmsr_get_adapter_config(acb, reg->message_rwbuffer);
> 3033 return true;
> 3034 }
> 3035
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
