All of lore.kernel.org
 help / color / mirror / Atom feed
* [cris:scmi_telemetry_ng_V15 13/35] drivers/firmware/arm_scmi/telemetry.c:1037:26: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'}
@ 2026-11-09 14:24 kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2026-11-09 14:24 UTC (permalink / raw)
  To: Cristian Marussi; +Cc: oe-kbuild-all

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/cris/linux.git scmi_telemetry_ng_V15
head:   d51f22cab86f4ad2f08b4e88803f85fdffd81a62
commit: be715d5b6657e41ce80f4b150a138ee4c5924d97 [13/35] firmware: arm_scmi: Add Telemetry SHMTI areas enumeration
config: csky-randconfig-2200-20261109 (https://download.01.org/0day-ci/archive/20261109/202611091504.ef4O1O8S-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 16.2.0
smatch: v0.5.0-9187-g5189e3fb
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261109/202611091504.ef4O1O8S-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202611091504.ef4O1O8S-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/linux/device.h:15,
                    from drivers/firmware/arm_scmi/telemetry.c:13:
   drivers/firmware/arm_scmi/telemetry.c: In function 'scmi_telemetry_area_check':
   include/linux/printk.h:109:25: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
     109 | #define FW_BUG          "[Firmware Bug]: "
         |                         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:452:17: note: in expansion of macro 'dev_err'
     452 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) pointer too big.\n",
         |                 ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:452:30: note: in expansion of macro 'FW_BUG'
     452 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) pointer too big.\n",
         |                              ^~~~~~
   include/linux/printk.h:109:25: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
     109 | #define FW_BUG          "[Firmware Bug]: "
         |                         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:458:17: note: in expansion of macro 'dev_err'
     458 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) misaligned.\n",
         |                 ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:458:30: note: in expansion of macro 'FW_BUG'
     458 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) misaligned.\n",
         |                              ^~~~~~
   include/linux/printk.h:109:25: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
     109 | #define FW_BUG          "[Firmware Bug]: "
         |                         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:470:17: note: in expansion of macro 'dev_err'
     470 |                 dev_err(dev, FW_BUG "%s area (0x%llX/%u) wrap-around.\n",
         |                 ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:470:30: note: in expansion of macro 'FW_BUG'
     470 |                 dev_err(dev, FW_BUG "%s area (0x%llX/%u) wrap-around.\n",
         |                              ^~~~~~
   drivers/firmware/arm_scmi/telemetry.c: In function 'iter_shmti_process_response':
>> drivers/firmware/arm_scmi/telemetry.c:1037:26: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
    1037 |                          "SHMTI ID:%u is NOT 64bit aligned (paddr:0x%llX)\n",
         |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:156:61: note: in expansion of macro 'dev_fmt'
     156 |         dev_printk_index_wrap(_dev_warn, KERN_WARNING, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                             ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:1036:17: note: in expansion of macro 'dev_warn'
    1036 |                 dev_warn(ph->dev,
         |                 ^~~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:1037:72: note: format string is defined here
    1037 |                          "SHMTI ID:%u is NOT 64bit aligned (paddr:0x%llX)\n",
         |                                                                     ~~~^
         |                                                                        |
         |                                                                        long long unsigned int
         |                                                                     %X


vim +1037 drivers/firmware/arm_scmi/telemetry.c

   995	
   996	static int iter_shmti_process_response(const struct scmi_protocol_handle *ph,
   997					       const void *response,
   998					       struct scmi_iterator_state *st,
   999					       void *priv)
  1000	{
  1001		const struct scmi_msg_resp_telemetry_shmti_list *r = response;
  1002		struct telemetry_info *ti = priv;
  1003		struct telemetry_shmti *shmti;
  1004		const struct scmi_shmti_desc *desc;
  1005		void __iomem *addr;
  1006		phys_addr_t phys_addr;
  1007		size_t len;
  1008		int ret;
  1009	
  1010		desc = &r->desc[st->loop_idx];
  1011		shmti = &ti->shmti[st->desc_index + st->loop_idx];
  1012	
  1013		shmti->info.sid = le32_to_cpu(desc->id);
  1014		shmti->flags = le32_to_cpu(desc->flags);
  1015	
  1016		ret = scmi_telemetry_area_check(ph->dev, "SHMTI",
  1017						le32_to_cpu(desc->addr_low),
  1018						le32_to_cpu(desc->addr_high),
  1019						le32_to_cpu(desc->length),
  1020						&phys_addr, &len);
  1021		if (ret) {
  1022			dev_err(ph->dev, "Malformed SHMTI ID:%u. Abort.\n",
  1023				shmti->info.sid);
  1024			return ret;
  1025		}
  1026	
  1027		if (len < SHMTI_MIN_SIZE || len > SHMTI_MAX_SIZE) {
  1028			dev_err(ph->dev,
  1029				FW_BUG "Invalid length for SHMTI ID:%u len:%zu. Abort.\n",
  1030				shmti->info.sid, len);
  1031			return -EINVAL;
  1032		}
  1033	
  1034		/* SHMTI base phys_addr SHOULD be 64bit aligned by the SCMI spec */
  1035		if (!IS_ALIGNED(phys_addr, 8))
  1036			dev_warn(ph->dev,
> 1037				 "SHMTI ID:%u is NOT 64bit aligned (paddr:0x%llX)\n",
  1038				 shmti->info.sid, phys_addr);
  1039	
  1040		addr = devm_ioremap(ph->dev, phys_addr, len);
  1041		if (!addr)
  1042			return -EADDRNOTAVAIL;
  1043	
  1044		shmti->base = addr;
  1045		shmti->info.len = len;
  1046		shmti->info.phys = phys_addr;
  1047		shmti->info.offset = shmti->info.phys & ~PAGE_MASK;
  1048	
  1049		ret = scmi_telemetry_shmti_validate(ph->dev, shmti);
  1050		if (ret)
  1051			return ret;
  1052	
  1053		ti->num_shmti++;
  1054	
  1055		return 0;
  1056	}
  1057	

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 2+ messages in thread
* [cris:scmi_telemetry_ng_V15 13/35] drivers/firmware/arm_scmi/telemetry.c:1037:26: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'}
@ 2026-11-09 15:02 kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2026-11-09 15:02 UTC (permalink / raw)
  To: Cristian Marussi; +Cc: oe-kbuild-all

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/cris/linux.git scmi_telemetry_ng_V15
head:   d51f22cab86f4ad2f08b4e88803f85fdffd81a62
commit: be715d5b6657e41ce80f4b150a138ee4c5924d97 [13/35] firmware: arm_scmi: Add Telemetry SHMTI areas enumeration
config: sh-randconfig-1000-20261109 (https://download.01.org/0day-ci/archive/20261109/202611092306.36wJrIAE-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 16.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261109/202611092306.36wJrIAE-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202611092306.36wJrIAE-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/linux/device.h:15,
                    from drivers/firmware/arm_scmi/telemetry.c:13:
   drivers/firmware/arm_scmi/telemetry.c: In function 'scmi_telemetry_area_check':
   include/linux/printk.h:109:25: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
     109 | #define FW_BUG          "[Firmware Bug]: "
         |                         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:452:17: note: in expansion of macro 'dev_err'
     452 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) pointer too big.\n",
         |                 ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:452:30: note: in expansion of macro 'FW_BUG'
     452 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) pointer too big.\n",
         |                              ^~~~~~
   include/linux/printk.h:109:25: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
     109 | #define FW_BUG          "[Firmware Bug]: "
         |                         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:458:17: note: in expansion of macro 'dev_err'
     458 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) misaligned.\n",
         |                 ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:458:30: note: in expansion of macro 'FW_BUG'
     458 |                 dev_err(dev, FW_BUG "%s phys_addr (0x%llX) misaligned.\n",
         |                              ^~~~~~
   include/linux/printk.h:109:25: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
     109 | #define FW_BUG          "[Firmware Bug]: "
         |                         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:470:17: note: in expansion of macro 'dev_err'
     470 |                 dev_err(dev, FW_BUG "%s area (0x%llX/%u) wrap-around.\n",
         |                 ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:470:30: note: in expansion of macro 'FW_BUG'
     470 |                 dev_err(dev, FW_BUG "%s area (0x%llX/%u) wrap-around.\n",
         |                              ^~~~~~
   drivers/firmware/arm_scmi/telemetry.c: In function 'iter_shmti_process_response':
>> drivers/firmware/arm_scmi/telemetry.c:1037:26: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
    1037 |                          "SHMTI ID:%u is NOT 64bit aligned (paddr:0x%llX)\n",
         |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:156:61: note: in expansion of macro 'dev_fmt'
     156 |         dev_printk_index_wrap(_dev_warn, KERN_WARNING, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                             ^~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:1036:17: note: in expansion of macro 'dev_warn'
    1036 |                 dev_warn(ph->dev,
         |                 ^~~~~~~~
   drivers/firmware/arm_scmi/telemetry.c:1037:72: note: format string is defined here
    1037 |                          "SHMTI ID:%u is NOT 64bit aligned (paddr:0x%llX)\n",
         |                                                                     ~~~^
         |                                                                        |
         |                                                                        long long unsigned int
         |                                                                     %X


vim +1037 drivers/firmware/arm_scmi/telemetry.c

   995	
   996	static int iter_shmti_process_response(const struct scmi_protocol_handle *ph,
   997					       const void *response,
   998					       struct scmi_iterator_state *st,
   999					       void *priv)
  1000	{
  1001		const struct scmi_msg_resp_telemetry_shmti_list *r = response;
  1002		struct telemetry_info *ti = priv;
  1003		struct telemetry_shmti *shmti;
  1004		const struct scmi_shmti_desc *desc;
  1005		void __iomem *addr;
  1006		phys_addr_t phys_addr;
  1007		size_t len;
  1008		int ret;
  1009	
  1010		desc = &r->desc[st->loop_idx];
  1011		shmti = &ti->shmti[st->desc_index + st->loop_idx];
  1012	
  1013		shmti->info.sid = le32_to_cpu(desc->id);
  1014		shmti->flags = le32_to_cpu(desc->flags);
  1015	
  1016		ret = scmi_telemetry_area_check(ph->dev, "SHMTI",
  1017						le32_to_cpu(desc->addr_low),
  1018						le32_to_cpu(desc->addr_high),
  1019						le32_to_cpu(desc->length),
  1020						&phys_addr, &len);
  1021		if (ret) {
  1022			dev_err(ph->dev, "Malformed SHMTI ID:%u. Abort.\n",
  1023				shmti->info.sid);
  1024			return ret;
  1025		}
  1026	
  1027		if (len < SHMTI_MIN_SIZE || len > SHMTI_MAX_SIZE) {
  1028			dev_err(ph->dev,
  1029				FW_BUG "Invalid length for SHMTI ID:%u len:%zu. Abort.\n",
  1030				shmti->info.sid, len);
  1031			return -EINVAL;
  1032		}
  1033	
  1034		/* SHMTI base phys_addr SHOULD be 64bit aligned by the SCMI spec */
  1035		if (!IS_ALIGNED(phys_addr, 8))
  1036			dev_warn(ph->dev,
> 1037				 "SHMTI ID:%u is NOT 64bit aligned (paddr:0x%llX)\n",
  1038				 shmti->info.sid, phys_addr);
  1039	
  1040		addr = devm_ioremap(ph->dev, phys_addr, len);
  1041		if (!addr)
  1042			return -EADDRNOTAVAIL;
  1043	
  1044		shmti->base = addr;
  1045		shmti->info.len = len;
  1046		shmti->info.phys = phys_addr;
  1047		shmti->info.offset = shmti->info.phys & ~PAGE_MASK;
  1048	
  1049		ret = scmi_telemetry_shmti_validate(ph->dev, shmti);
  1050		if (ret)
  1051			return ret;
  1052	
  1053		ti->num_shmti++;
  1054	
  1055		return 0;
  1056	}
  1057	

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-11-09 15:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-09 14:24 [cris:scmi_telemetry_ng_V15 13/35] drivers/firmware/arm_scmi/telemetry.c:1037:26: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' {aka 'unsigned int'} kernel test robot
  -- strict thread matches above, loose matches on Subject: below --
2026-11-09 15:02 kernel test robot

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.