Hi Daniel, Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on wsa/i2c/for-next] [also build test WARNING on v5.12-rc5 next-20210329] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Daniel-Gomez/i2c-designware-Add-base-addr-info/20210325-232218 base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next config: powerpc-randconfig-r012-20210329 (attached as .config) compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 2a28d1d3b7bf2062288b46af34e33ccc543a99fa) 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 # install powerpc cross compiling tool for clang build # apt-get install binutils-powerpc-linux-gnu # https://github.com/0day-ci/linux/commit/5233e225fc69cfeef637d28c49f12967cbc36430 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Daniel-Gomez/i2c-designware-Add-base-addr-info/20210325-232218 git checkout 5233e225fc69cfeef637d28c49f12967cbc36430 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <l...@intel.com> All warnings (new ones prefixed by >>): >> drivers/i2c/busses/i2c-designware-master.c:770:49: warning: format specifies >> type 'unsigned long long' but the argument has type 'phys_addr_t' (aka >> 'unsigned int') [-Wformat] "Synopsys DesignWare I2C adapter at 0x%llx", dev->base_addr); ~~~~ ^~~~~~~~~~~~~~ %x 1 warning generated. vim +770 drivers/i2c/busses/i2c-designware-master.c 740 741 int i2c_dw_probe_master(struct dw_i2c_dev *dev) 742 { 743 struct i2c_adapter *adap = &dev->adapter; 744 unsigned long irq_flags; 745 int ret; 746 747 init_completion(&dev->cmd_complete); 748 749 dev->init = i2c_dw_init_master; 750 dev->disable = i2c_dw_disable; 751 dev->disable_int = i2c_dw_disable_int; 752 753 ret = i2c_dw_init_regmap(dev); 754 if (ret) 755 return ret; 756 757 ret = i2c_dw_set_timings_master(dev); 758 if (ret) 759 return ret; 760 761 ret = i2c_dw_set_fifo_size(dev); 762 if (ret) 763 return ret; 764 765 ret = dev->init(dev); 766 if (ret) 767 return ret; 768 769 snprintf(adap->name, sizeof(adap->name), > 770 "Synopsys DesignWare I2C adapter at 0x%llx", > dev->base_addr); 771 adap->retries = 3; 772 adap->algo = &i2c_dw_algo; 773 adap->quirks = &i2c_dw_quirks; 774 adap->dev.parent = dev->dev; 775 i2c_set_adapdata(adap, dev); 776 777 if (dev->flags & ACCESS_NO_IRQ_SUSPEND) { 778 irq_flags = IRQF_NO_SUSPEND; 779 } else { 780 irq_flags = IRQF_SHARED | IRQF_COND_SUSPEND; 781 } 782 783 i2c_dw_disable_int(dev); 784 ret = devm_request_irq(dev->dev, dev->irq, i2c_dw_isr, irq_flags, 785 dev_name(dev->dev), dev); 786 if (ret) { 787 dev_err(dev->dev, "failure requesting irq %i: %d\n", 788 dev->irq, ret); 789 return ret; 790 } 791 792 ret = i2c_dw_init_recovery_info(dev); 793 if (ret) 794 return ret; 795 796 /* 797 * Increment PM usage count during adapter registration in order to 798 * avoid possible spurious runtime suspend when adapter device is 799 * registered to the device core and immediate resume in case bus has 800 * registered I2C slaves that do I2C transfers in their probe. 801 */ 802 pm_runtime_get_noresume(dev->dev); 803 ret = i2c_add_numbered_adapter(adap); 804 if (ret) 805 dev_err(dev->dev, "failure adding adapter: %d\n", ret); 806 pm_runtime_put_noidle(dev->dev); 807 808 return ret; 809 } 810 EXPORT_SYMBOL_GPL(i2c_dw_probe_master); 811 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
