Hi Can,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on scsi/for-next]
[cannot apply to v5.4-rc4 next-20191023]
[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/Can-Guo/scsi-ufs-Add-command-logging-infrastructure/20191023-151035
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: x86_64-rhel (attached as .config)
compiler: gcc-7 (Debian 7.4.0-14) 7.4.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from drivers/scsi/ufs/ufshcd.c:45:0:
>> drivers/scsi/ufs/ufshcd.h:426:10: error: 'UFS_ERR_REG_HIST_LENGTH' undeclared here (not in a function); did you mean 'UIC_ERR_REG_HIST_LENGTH'?
     u32 reg[UFS_ERR_REG_HIST_LENGTH];
             ^~~~~~~~~~~~~~~~~~~~~~~
             UIC_ERR_REG_HIST_LENGTH

vim +426 drivers/scsi/ufs/ufshcd.h

3a4bf06d1f6d7d Yaniv Gardi 2014-09-25  416  
7425e502f13960 Can Guo     2019-10-22  417  #define UIC_ERR_REG_HIST_LENGTH 20
ff8e20c6624921 Dolev Raviv 2016-12-22  418  /**
d3c615bf586f27 Stanley Chu 2019-07-10  419   * struct ufs_err_reg_hist - keeps history of errors
ff8e20c6624921 Dolev Raviv 2016-12-22  420   * @pos: index to indicate cyclic buffer position
ff8e20c6624921 Dolev Raviv 2016-12-22  421   * @reg: cyclic buffer for registers value
ff8e20c6624921 Dolev Raviv 2016-12-22  422   * @tstamp: cyclic buffer for time stamp
ff8e20c6624921 Dolev Raviv 2016-12-22  423   */
48d5b9732580bc Stanley Chu 2019-07-10  424  struct ufs_err_reg_hist {
ff8e20c6624921 Dolev Raviv 2016-12-22  425  	int pos;
48d5b9732580bc Stanley Chu 2019-07-10 @426  	u32 reg[UFS_ERR_REG_HIST_LENGTH];
48d5b9732580bc Stanley Chu 2019-07-10  427  	ktime_t tstamp[UFS_ERR_REG_HIST_LENGTH];
ff8e20c6624921 Dolev Raviv 2016-12-22  428  };
ff8e20c6624921 Dolev Raviv 2016-12-22  429  

:::::: The code at line 426 was first introduced by commit
:::::: 48d5b9732580bca18fed9ef20da8aa210c646755 scsi: ufs: Change names related to error history

:::::: TO: Stanley Chu <stanley.chu@mediatek.com>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation