All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Arnd Bergmann <arnd@arndb.de>, Sujeev Dias <sdias@codeaurora.org>,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	Tony Truong <truong@codeaurora.org>
Subject: Re: [PATCH v1 1/4] mhi_bus: core: Add support for MHI host interface
Date: Sat, 28 Apr 2018 08:28:58 +0800	[thread overview]
Message-ID: <201804280814.62cyZ2X2%fengguang.wu@intel.com> (raw)
In-Reply-To: <1524795811-21399-2-git-send-email-sdias@codeaurora.org>

[-- Attachment #1: Type: text/plain, Size: 21610 bytes --]

Hi Sujeev,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.17-rc2 next-20180426]
[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/Sujeev-Dias/mhi_bus-core-Add-support-for-MHI-host-interface/20180428-065959
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
>> drivers/bus/mhi/core/mhi_init.c:608:5: error: redefinition of 'mhi_device_configure'
    int mhi_device_configure(struct mhi_device *mhi_dev,
        ^~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:545:19: note: previous definition of 'mhi_device_configure' was here
    static inline int mhi_device_configure(struct mhi_device *mhi_div,
                      ^~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:914:5: error: redefinition of 'of_register_mhi_controller'
    int of_register_mhi_controller(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:599:19: note: previous definition of 'of_register_mhi_controller' was here
    static inline int of_register_mhi_controller(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:995:6: error: redefinition of 'mhi_unregister_mhi_controller'
    void mhi_unregister_mhi_controller(struct mhi_controller *mhi_cntrl)
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:604:20: note: previous definition of 'mhi_unregister_mhi_controller' was here
    static inline void mhi_unregister_mhi_controller(
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1015:24: error: redefinition of 'mhi_alloc_controller'
    struct mhi_controller *mhi_alloc_controller(size_t size)
                           ^~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:594:38: note: previous definition of 'mhi_alloc_controller' was here
    static inline struct mhi_controller *mhi_alloc_controller(size_t size)
                                         ^~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1028:5: error: redefinition of 'mhi_prepare_for_power_up'
    int mhi_prepare_for_power_up(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:617:19: note: previous definition of 'mhi_prepare_for_power_up' was here
    static inline int mhi_prepare_for_power_up(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1070:6: error: redefinition of 'mhi_unprepare_after_power_down'
    void mhi_unprepare_after_power_down(struct mhi_controller *mhi_cntrl)
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:637:20: note: previous definition of 'mhi_unprepare_after_power_down' was here
    static inline void mhi_unprepare_after_power_down(
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1225:5: error: redefinition of 'mhi_driver_register'
    int mhi_driver_register(struct mhi_driver *mhi_drv)
        ^~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:536:19: note: previous definition of 'mhi_driver_register' was here
    static inline int mhi_driver_register(struct mhi_driver *mhi_drv)
                      ^~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1239:6: error: redefinition of 'mhi_driver_unregister'
    void mhi_driver_unregister(struct mhi_driver *mhi_drv)
         ^~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:541:20: note: previous definition of 'mhi_driver_unregister' was here
    static inline void mhi_driver_unregister(struct mhi_driver *mhi_drv)
                       ^~~~~~~~~~~~~~~~~~~~~
--
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
   drivers/bus/mhi/core/mhi_main.c: In function 'mhi_debugfs_mhi_event_show':
>> drivers/bus/mhi/core/mhi_main.c:1335:44: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 5 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " rp:0x%llx wp:0x%llx local_rp:0x%llx db:0x%llx\n",
                                            ~~~^
                                            %x
   drivers/bus/mhi/core/mhi_main.c:1337:8:
           mhi_to_physical(ring, ring->rp),
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~      
   drivers/bus/mhi/core/mhi_main.c:1335:54: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " rp:0x%llx wp:0x%llx local_rp:0x%llx db:0x%llx\n",
                                                      ~~~^
                                                      %x
   drivers/bus/mhi/core/mhi_main.c:1338:8:
           mhi_event->db_cfg.db_val);
           ~~~~~~~~~~~~~~~~~~~~~~~~                       
   drivers/bus/mhi/core/mhi_main.c: In function 'mhi_debugfs_mhi_chan_show':
   drivers/bus/mhi/core/mhi_main.c:1368:57: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " base:0x%llx len:0x%llx wp:0x%llx local_rp:0x%llx local_wp:0x%llx db:0x%llx\n",
                                                         ~~~^
                                                         %x
   drivers/bus/mhi/core/mhi_main.c:1371:8:
           mhi_to_physical(ring, ring->rp),
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                   
   drivers/bus/mhi/core/mhi_main.c:1368:73: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " base:0x%llx len:0x%llx wp:0x%llx local_rp:0x%llx local_wp:0x%llx db:0x%llx\n",
                                                                         ~~~^
                                                                         %x
   drivers/bus/mhi/core/mhi_main.c:1372:8:
           mhi_to_physical(ring, ring->wp),
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                   
   drivers/bus/mhi/core/mhi_main.c:1368:83: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 8 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " base:0x%llx len:0x%llx wp:0x%llx local_rp:0x%llx local_wp:0x%llx db:0x%llx\n",
                                                                                   ~~~^
                                                                                   %x
   drivers/bus/mhi/core/mhi_main.c:1373:8:
           mhi_chan->db_cfg.db_val);
           ~~~~~~~~~~~~~~~~~~~~~~~                                                     
   drivers/bus/mhi/core/mhi_main.c: At top level:
>> drivers/bus/mhi/core/mhi_main.c:1381:5: error: redefinition of 'mhi_prepare_for_transfer'
    int mhi_prepare_for_transfer(struct mhi_device *mhi_dev)
        ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:566:19: note: previous definition of 'mhi_prepare_for_transfer' was here
    static inline int mhi_prepare_for_transfer(struct mhi_device *mhi_dev)
                      ^~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1417:6: error: redefinition of 'mhi_unprepare_from_transfer'
    void mhi_unprepare_from_transfer(struct mhi_device *mhi_dev)
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:571:20: note: previous definition of 'mhi_unprepare_from_transfer' was here
    static inline void mhi_unprepare_from_transfer(struct mhi_device *mhi_dev)
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1434:5: error: redefinition of 'mhi_get_no_free_descriptors'
    int mhi_get_no_free_descriptors(struct mhi_device *mhi_dev,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:576:19: note: previous definition of 'mhi_get_no_free_descriptors' was here
    static inline int mhi_get_no_free_descriptors(struct mhi_device *mhi_dev,
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1446:24: error: redefinition of 'mhi_bdf_to_controller'
    struct mhi_controller *mhi_bdf_to_controller(u32 domain,
                           ^~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:609:38: note: previous definition of 'mhi_bdf_to_controller' was here
    static inline struct mhi_controller *mhi_bdf_to_controller(u32 domain,
                                         ^~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1462:5: error: redefinition of 'mhi_poll'
    int mhi_poll(struct mhi_device *mhi_dev,
        ^~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:582:19: note: previous definition of 'mhi_poll' was here
    static inline int mhi_poll(struct mhi_device *mhi_dev, u32 budget)
                      ^~~~~~~~
--
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
>> drivers/bus/mhi/core/mhi_pm.c:782:5: error: redefinition of 'mhi_async_power_up'
    int mhi_async_power_up(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:622:19: note: previous definition of 'mhi_async_power_up' was here
    static inline int mhi_async_power_up(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:873:6: error: redefinition of 'mhi_power_down'
    void mhi_power_down(struct mhi_controller *mhi_cntrl, bool graceful)
         ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:632:20: note: previous definition of 'mhi_power_down' was here
    static inline void mhi_power_down(struct mhi_controller *mhi_cntrl,
                       ^~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:907:5: error: redefinition of 'mhi_sync_power_up'
    int mhi_sync_power_up(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:627:19: note: previous definition of 'mhi_sync_power_up' was here
    static inline int mhi_sync_power_up(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:923:5: error: redefinition of 'mhi_pm_suspend'
    int mhi_pm_suspend(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:642:19: note: previous definition of 'mhi_pm_suspend' was here
    static inline int mhi_pm_suspend(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1012:5: error: redefinition of 'mhi_pm_resume'
    int mhi_pm_resume(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:647:19: note: previous definition of 'mhi_pm_resume' was here
    static inline int mhi_pm_resume(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1097:6: error: redefinition of 'mhi_device_get'
    void mhi_device_get(struct mhi_device *mhi_dev)
         ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:553:20: note: previous definition of 'mhi_device_get' was here
    static inline void mhi_device_get(struct mhi_device *mhi_dev)
                       ^~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1108:5: error: redefinition of 'mhi_device_get_sync'
    int mhi_device_get_sync(struct mhi_device *mhi_dev)
        ^~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:557:19: note: previous definition of 'mhi_device_get_sync' was here
    static inline int mhi_device_get_sync(struct mhi_device *mhi_dev)
                      ^~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1121:6: error: redefinition of 'mhi_device_put'
    void mhi_device_put(struct mhi_device *mhi_dev)
         ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:562:20: note: previous definition of 'mhi_device_put' was here
    static inline void mhi_device_put(struct mhi_device *mhi_dev)
                       ^~~~~~~~~~~~~~
--
   In file included from drivers/bus/mhi/core/mhi_boot.c:26:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
>> drivers/bus/mhi/core/mhi_boot.c:138:5: error: redefinition of 'mhi_download_rddm_img'
    int mhi_download_rddm_img(struct mhi_controller *mhi_cntrl, bool in_panic)
        ^~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_boot.c:26:0:
   include/linux/mhi.h:652:19: note: previous definition of 'mhi_download_rddm_img' was here
    static inline int mhi_download_rddm_img(struct mhi_controller *mhi_cntrl,
                      ^~~~~~~~~~~~~~~~~~~~~
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   drivers/bus/mhi/core/mhi_boot.c: In function 'mhi_download_rddm_img':
   include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
>> drivers/bus/mhi/core/mhi_boot.c:196:2: note: in expansion of macro 'MHI_LOG'
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
     ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:196:41: note: format string is defined here
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
                                          ~~^
                                          %x
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   drivers/bus/mhi/core/mhi_boot.c: In function 'mhi_fw_load_amss':
>> include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka const unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:248:2: note: in expansion of macro 'MHI_LOG'
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
     ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:248:41: note: format string is defined here
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
                                          ~~^
                                          %x
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   drivers/bus/mhi/core/mhi_boot.c: In function 'mhi_alloc_bhie_table':
   include/linux/kern_levels.h:5:18: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:3: note: in expansion of macro 'MHI_LOG'
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
      ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:34: note: format string is defined here
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
                                  ~~~^
                                  %x
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:3: note: in expansion of macro 'MHI_LOG'
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
      ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:43: note: format string is defined here
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
                                            ~~^
                                            %u
..

vim +658 include/linux/mhi.h

   657	
 > 658	static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
   659	{
   660		return -EINVAL;
   661	}
   662	
   663	#endif
   664	
   665	#ifdef CONFIG_MHI_DEBUG
   666	
   667	#define MHI_VERB(fmt, ...) do { \
   668			if (mhi_cntrl->klog_lvl <= MHI_MSG_LVL_VERBOSE) \
   669				pr_debug("[D][%s] " fmt, __func__, ##__VA_ARGS__);\
   670	} while (0)
   671	
   672	#else
   673	
   674	#define MHI_VERB(fmt, ...)
   675	
   676	#endif
   677	
   678	#define MHI_LOG(fmt, ...) do {	\
   679			if (mhi_cntrl->klog_lvl <= MHI_MSG_LVL_INFO) \
 > 680				pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
   681	} while (0)
   682	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 62932 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
To: Sujeev Dias <sdias@codeaurora.org>
Cc: kbuild-all@01.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Arnd Bergmann <arnd@arndb.de>, Sujeev Dias <sdias@codeaurora.org>,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	Tony Truong <truong@codeaurora.org>
Subject: Re: [PATCH v1 1/4] mhi_bus: core: Add support for MHI host interface
Date: Sat, 28 Apr 2018 08:28:58 +0800	[thread overview]
Message-ID: <201804280814.62cyZ2X2%fengguang.wu@intel.com> (raw)
In-Reply-To: <1524795811-21399-2-git-send-email-sdias@codeaurora.org>

[-- Attachment #1: Type: text/plain, Size: 21610 bytes --]

Hi Sujeev,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.17-rc2 next-20180426]
[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/Sujeev-Dias/mhi_bus-core-Add-support-for-MHI-host-interface/20180428-065959
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
>> drivers/bus/mhi/core/mhi_init.c:608:5: error: redefinition of 'mhi_device_configure'
    int mhi_device_configure(struct mhi_device *mhi_dev,
        ^~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:545:19: note: previous definition of 'mhi_device_configure' was here
    static inline int mhi_device_configure(struct mhi_device *mhi_div,
                      ^~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:914:5: error: redefinition of 'of_register_mhi_controller'
    int of_register_mhi_controller(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:599:19: note: previous definition of 'of_register_mhi_controller' was here
    static inline int of_register_mhi_controller(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:995:6: error: redefinition of 'mhi_unregister_mhi_controller'
    void mhi_unregister_mhi_controller(struct mhi_controller *mhi_cntrl)
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:604:20: note: previous definition of 'mhi_unregister_mhi_controller' was here
    static inline void mhi_unregister_mhi_controller(
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1015:24: error: redefinition of 'mhi_alloc_controller'
    struct mhi_controller *mhi_alloc_controller(size_t size)
                           ^~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:594:38: note: previous definition of 'mhi_alloc_controller' was here
    static inline struct mhi_controller *mhi_alloc_controller(size_t size)
                                         ^~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1028:5: error: redefinition of 'mhi_prepare_for_power_up'
    int mhi_prepare_for_power_up(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:617:19: note: previous definition of 'mhi_prepare_for_power_up' was here
    static inline int mhi_prepare_for_power_up(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1070:6: error: redefinition of 'mhi_unprepare_after_power_down'
    void mhi_unprepare_after_power_down(struct mhi_controller *mhi_cntrl)
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:637:20: note: previous definition of 'mhi_unprepare_after_power_down' was here
    static inline void mhi_unprepare_after_power_down(
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1225:5: error: redefinition of 'mhi_driver_register'
    int mhi_driver_register(struct mhi_driver *mhi_drv)
        ^~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:536:19: note: previous definition of 'mhi_driver_register' was here
    static inline int mhi_driver_register(struct mhi_driver *mhi_drv)
                      ^~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_init.c:1239:6: error: redefinition of 'mhi_driver_unregister'
    void mhi_driver_unregister(struct mhi_driver *mhi_drv)
         ^~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_init.c:23:0:
   include/linux/mhi.h:541:20: note: previous definition of 'mhi_driver_unregister' was here
    static inline void mhi_driver_unregister(struct mhi_driver *mhi_drv)
                       ^~~~~~~~~~~~~~~~~~~~~
--
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
   drivers/bus/mhi/core/mhi_main.c: In function 'mhi_debugfs_mhi_event_show':
>> drivers/bus/mhi/core/mhi_main.c:1335:44: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 5 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " rp:0x%llx wp:0x%llx local_rp:0x%llx db:0x%llx\n",
                                            ~~~^
                                            %x
   drivers/bus/mhi/core/mhi_main.c:1337:8:
           mhi_to_physical(ring, ring->rp),
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~      
   drivers/bus/mhi/core/mhi_main.c:1335:54: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " rp:0x%llx wp:0x%llx local_rp:0x%llx db:0x%llx\n",
                                                      ~~~^
                                                      %x
   drivers/bus/mhi/core/mhi_main.c:1338:8:
           mhi_event->db_cfg.db_val);
           ~~~~~~~~~~~~~~~~~~~~~~~~                       
   drivers/bus/mhi/core/mhi_main.c: In function 'mhi_debugfs_mhi_chan_show':
   drivers/bus/mhi/core/mhi_main.c:1368:57: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " base:0x%llx len:0x%llx wp:0x%llx local_rp:0x%llx local_wp:0x%llx db:0x%llx\n",
                                                         ~~~^
                                                         %x
   drivers/bus/mhi/core/mhi_main.c:1371:8:
           mhi_to_physical(ring, ring->rp),
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                   
   drivers/bus/mhi/core/mhi_main.c:1368:73: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " base:0x%llx len:0x%llx wp:0x%llx local_rp:0x%llx local_wp:0x%llx db:0x%llx\n",
                                                                         ~~~^
                                                                         %x
   drivers/bus/mhi/core/mhi_main.c:1372:8:
           mhi_to_physical(ring, ring->wp),
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                   
   drivers/bus/mhi/core/mhi_main.c:1368:83: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 8 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
           " base:0x%llx len:0x%llx wp:0x%llx local_rp:0x%llx local_wp:0x%llx db:0x%llx\n",
                                                                                   ~~~^
                                                                                   %x
   drivers/bus/mhi/core/mhi_main.c:1373:8:
           mhi_chan->db_cfg.db_val);
           ~~~~~~~~~~~~~~~~~~~~~~~                                                     
   drivers/bus/mhi/core/mhi_main.c: At top level:
>> drivers/bus/mhi/core/mhi_main.c:1381:5: error: redefinition of 'mhi_prepare_for_transfer'
    int mhi_prepare_for_transfer(struct mhi_device *mhi_dev)
        ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:566:19: note: previous definition of 'mhi_prepare_for_transfer' was here
    static inline int mhi_prepare_for_transfer(struct mhi_device *mhi_dev)
                      ^~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1417:6: error: redefinition of 'mhi_unprepare_from_transfer'
    void mhi_unprepare_from_transfer(struct mhi_device *mhi_dev)
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:571:20: note: previous definition of 'mhi_unprepare_from_transfer' was here
    static inline void mhi_unprepare_from_transfer(struct mhi_device *mhi_dev)
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1434:5: error: redefinition of 'mhi_get_no_free_descriptors'
    int mhi_get_no_free_descriptors(struct mhi_device *mhi_dev,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:576:19: note: previous definition of 'mhi_get_no_free_descriptors' was here
    static inline int mhi_get_no_free_descriptors(struct mhi_device *mhi_dev,
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1446:24: error: redefinition of 'mhi_bdf_to_controller'
    struct mhi_controller *mhi_bdf_to_controller(u32 domain,
                           ^~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:609:38: note: previous definition of 'mhi_bdf_to_controller' was here
    static inline struct mhi_controller *mhi_bdf_to_controller(u32 domain,
                                         ^~~~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_main.c:1462:5: error: redefinition of 'mhi_poll'
    int mhi_poll(struct mhi_device *mhi_dev,
        ^~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_main.c:23:0:
   include/linux/mhi.h:582:19: note: previous definition of 'mhi_poll' was here
    static inline int mhi_poll(struct mhi_device *mhi_dev, u32 budget)
                      ^~~~~~~~
--
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
>> drivers/bus/mhi/core/mhi_pm.c:782:5: error: redefinition of 'mhi_async_power_up'
    int mhi_async_power_up(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:622:19: note: previous definition of 'mhi_async_power_up' was here
    static inline int mhi_async_power_up(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:873:6: error: redefinition of 'mhi_power_down'
    void mhi_power_down(struct mhi_controller *mhi_cntrl, bool graceful)
         ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:632:20: note: previous definition of 'mhi_power_down' was here
    static inline void mhi_power_down(struct mhi_controller *mhi_cntrl,
                       ^~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:907:5: error: redefinition of 'mhi_sync_power_up'
    int mhi_sync_power_up(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:627:19: note: previous definition of 'mhi_sync_power_up' was here
    static inline int mhi_sync_power_up(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:923:5: error: redefinition of 'mhi_pm_suspend'
    int mhi_pm_suspend(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:642:19: note: previous definition of 'mhi_pm_suspend' was here
    static inline int mhi_pm_suspend(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1012:5: error: redefinition of 'mhi_pm_resume'
    int mhi_pm_resume(struct mhi_controller *mhi_cntrl)
        ^~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:647:19: note: previous definition of 'mhi_pm_resume' was here
    static inline int mhi_pm_resume(struct mhi_controller *mhi_cntrl)
                      ^~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1097:6: error: redefinition of 'mhi_device_get'
    void mhi_device_get(struct mhi_device *mhi_dev)
         ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:553:20: note: previous definition of 'mhi_device_get' was here
    static inline void mhi_device_get(struct mhi_device *mhi_dev)
                       ^~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1108:5: error: redefinition of 'mhi_device_get_sync'
    int mhi_device_get_sync(struct mhi_device *mhi_dev)
        ^~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:557:19: note: previous definition of 'mhi_device_get_sync' was here
    static inline int mhi_device_get_sync(struct mhi_device *mhi_dev)
                      ^~~~~~~~~~~~~~~~~~~
>> drivers/bus/mhi/core/mhi_pm.c:1121:6: error: redefinition of 'mhi_device_put'
    void mhi_device_put(struct mhi_device *mhi_dev)
         ^~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_pm.c:24:0:
   include/linux/mhi.h:562:20: note: previous definition of 'mhi_device_put' was here
    static inline void mhi_device_put(struct mhi_device *mhi_dev)
                       ^~~~~~~~~~~~~~
--
   In file included from drivers/bus/mhi/core/mhi_boot.c:26:0:
>> include/linux/mhi.h:658:15: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int'
    static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
                  ^~~
>> drivers/bus/mhi/core/mhi_boot.c:138:5: error: redefinition of 'mhi_download_rddm_img'
    int mhi_download_rddm_img(struct mhi_controller *mhi_cntrl, bool in_panic)
        ^~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/bus/mhi/core/mhi_boot.c:26:0:
   include/linux/mhi.h:652:19: note: previous definition of 'mhi_download_rddm_img' was here
    static inline int mhi_download_rddm_img(struct mhi_controller *mhi_cntrl,
                      ^~~~~~~~~~~~~~~~~~~~~
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   drivers/bus/mhi/core/mhi_boot.c: In function 'mhi_download_rddm_img':
   include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
>> drivers/bus/mhi/core/mhi_boot.c:196:2: note: in expansion of macro 'MHI_LOG'
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
     ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:196:41: note: format string is defined here
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
                                          ~~^
                                          %x
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   drivers/bus/mhi/core/mhi_boot.c: In function 'mhi_fw_load_amss':
>> include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka const unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:248:2: note: in expansion of macro 'MHI_LOG'
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
     ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:248:41: note: format string is defined here
     MHI_LOG("Upper:0x%x Lower:0x%x len:0x%lx sequence:%u\n",
                                          ~~^
                                          %x
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   drivers/bus/mhi/core/mhi_boot.c: In function 'mhi_alloc_bhie_table':
   include/linux/kern_levels.h:5:18: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:3: note: in expansion of macro 'MHI_LOG'
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
      ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:34: note: format string is defined here
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
                                  ~~~^
                                  %x
   In file included from include/linux/printk.h:7:0,
                    from include/linux/kernel.h:14,
                    from include/linux/list.h:9,
                    from include/linux/wait.h:7,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:6,
                    from include/linux/debugfs.h:15,
                    from drivers/bus/mhi/core/mhi_boot.c:13:
   include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka unsigned int}' [-Wformat=]
    #define KERN_SOH "\001"  /* ASCII Start Of Header */
                     ^
   include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
    #define KERN_INFO KERN_SOH "6" /* informational */
                      ^~~~~~~~
   include/linux/printk.h:311:9: note: in expansion of macro 'KERN_INFO'
     printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
            ^~~~~~~~~
>> include/linux/mhi.h:680:4: note: in expansion of macro 'pr_info'
       pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
       ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:3: note: in expansion of macro 'MHI_LOG'
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
      ^~~~~~~
   drivers/bus/mhi/core/mhi_boot.c:403:43: note: format string is defined here
      MHI_LOG("Entry:%d Address:0x%llx size:%lu\n", i,
                                            ~~^
                                            %u
..

vim +658 include/linux/mhi.h

   657	
 > 658	static inlint int mhi_force_rddm_mode(struct mhi_controller *mhi_cntrl)
   659	{
   660		return -EINVAL;
   661	}
   662	
   663	#endif
   664	
   665	#ifdef CONFIG_MHI_DEBUG
   666	
   667	#define MHI_VERB(fmt, ...) do { \
   668			if (mhi_cntrl->klog_lvl <= MHI_MSG_LVL_VERBOSE) \
   669				pr_debug("[D][%s] " fmt, __func__, ##__VA_ARGS__);\
   670	} while (0)
   671	
   672	#else
   673	
   674	#define MHI_VERB(fmt, ...)
   675	
   676	#endif
   677	
   678	#define MHI_LOG(fmt, ...) do {	\
   679			if (mhi_cntrl->klog_lvl <= MHI_MSG_LVL_INFO) \
 > 680				pr_info("[I][%s] " fmt, __func__, ##__VA_ARGS__);\
   681	} while (0)
   682	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 62932 bytes --]

  parent reply	other threads:[~2018-04-28  0:28 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-27  2:23 MHI initial design review Sujeev Dias
2018-04-27  2:23 ` [PATCH v1 1/4] mhi_bus: core: Add support for MHI host interface Sujeev Dias
2018-04-27  7:22   ` Greg Kroah-Hartman
2018-04-28 14:28     ` Sujeev Dias
2018-04-28 15:50       ` Greg Kroah-Hartman
2018-04-27  7:23   ` Greg Kroah-Hartman
2018-04-27 12:18   ` Arnd Bergmann
2018-04-28 16:08     ` Sujeev Dias
2018-04-28  0:28   ` kbuild test robot [this message]
2018-04-28  0:28     ` kbuild test robot
2018-04-28  2:52   ` kbuild test robot
2018-04-28  2:52     ` kbuild test robot
2018-05-03 19:21   ` Pavel Machek
2018-05-04  3:05     ` Sujeev Dias
2018-06-22 23:03   ` Randy Dunlap
2018-04-27  2:23 ` [PATCH v1 2/4] mhi_bus: controller: MHI support for QCOM modems Sujeev Dias
2018-04-27 11:32   ` Arnd Bergmann
2018-04-28 15:40     ` Sujeev Dias
2018-04-28  3:05   ` kbuild test robot
2018-04-28  3:05     ` kbuild test robot
2018-04-28  3:12   ` kbuild test robot
2018-04-28  3:12     ` kbuild test robot
2018-04-27  2:23 ` [PATCH v1 3/4] mhi_bus: dev: netdev: add network interface driver Sujeev Dias
2018-04-27 11:19   ` Arnd Bergmann
2018-04-28 15:25     ` Sujeev Dias
2018-04-27  2:23 ` [PATCH v1 4/4] mhi_bus: dev: uci: add user space " Sujeev Dias
2018-04-27 11:36   ` Arnd Bergmann
2018-04-28  1:03   ` kbuild test robot
2018-04-28  1:03     ` kbuild test robot
2018-04-28  5:16   ` [PATCH] mhi_bus: dev: uci: fix semicolon.cocci warnings kbuild test robot
2018-04-28  5:16     ` kbuild test robot
2018-04-28  5:16   ` [PATCH v1 4/4] mhi_bus: dev: uci: add user space interface driver kbuild test robot
2018-04-28  5:16     ` kbuild test robot
2018-07-09 20:08 ` MHI code review Sujeev Dias
2018-07-09 20:08   ` [PATCH v2 1/7] mhi_bus: core: initial checkin for modem host interface bus driver Sujeev Dias
2018-07-09 20:50     ` Greg Kroah-Hartman
2018-07-09 20:52     ` Greg Kroah-Hartman
2018-07-10  6:36     ` Greg Kroah-Hartman
2018-07-11 19:30     ` Rob Herring
2018-08-09 18:39     ` Randy Dunlap
2018-07-09 20:08   ` [PATCH v2 2/7] mhi_bus: core: add power management support Sujeev Dias
2018-07-09 20:08   ` [PATCH v2 3/7] mhi_bus: core: add support for data transfer Sujeev Dias
2018-07-10  6:29     ` Greg Kroah-Hartman
2018-07-09 20:08   ` [PATCH v2 4/7] mhi_bus: core: add support for handling ioctl cmds Sujeev Dias
2018-07-09 20:08   ` [PATCH v2 5/7] mhi_bus: core: add support to get external modem time Sujeev Dias
2018-07-11 19:32     ` Rob Herring
2018-08-09 20:17     ` Randy Dunlap
2018-07-09 20:08   ` [PATCH v2 6/7] mhi_bus: controller: MHI support for QCOM modems Sujeev Dias
2018-07-11 19:36     ` Rob Herring
2018-07-09 20:08   ` [PATCH v2 7/7] mhi_bus: dev: uci: add user space interface driver Sujeev Dias
2019-04-30 15:10   ` MHI code review Daniele Palmas
2019-06-12 17:54     ` Sujeev Dias
2019-06-12 20:58       ` Daniele Palmas
2019-06-12 18:00     ` Sujeev Dias

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201804280814.62cyZ2X2%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=arnd@arndb.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=kbuild-all@01.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sdias@codeaurora.org \
    --cc=truong@codeaurora.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.