All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Jyri Sarha <jyri.sarha@linux.intel.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: [jsarha:topology2_sched_domain_support 1/1] sound/soc/sof/ipc4-topology.c:510:3: warning: pointer/integer type mismatch in conditional expression ('unsigned int' and 'void *')
Date: Tue, 8 Apr 2025 01:23:02 +0800	[thread overview]
Message-ID: <202504080147.AExWIzWG-lkp@intel.com> (raw)

tree:   https://github.com/jsarha/linux topology2_sched_domain_support
head:   e7f2745320e2fae66eb1e6d7620062d5377b25b9
commit: e7f2745320e2fae66eb1e6d7620062d5377b25b9 [1/1] ASoC: sof: ipc4-topology: Add support to sched_domain attribute
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20250408/202504080147.AExWIzWG-lkp@intel.com/config)
compiler: clang version 20.1.2 (https://github.com/llvm/llvm-project 58df0ef89dd64126512e4ee27b4ac3fd8ddf6247)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250408/202504080147.AExWIzWG-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/202504080147.AExWIzWG-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> sound/soc/sof/ipc4-topology.c:510:3: warning: pointer/integer type mismatch in conditional expression ('unsigned int' and 'void *') [-Wconditional-type-mismatch]
     510 |                 dev_err("Bad domain val %d", swidget->comp_domain);
         |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:154:2: note: expanded from macro 'dev_err'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:109:3: note: expanded from macro 'dev_printk_index_wrap'
     109 |                 dev_printk_index_emit(level, fmt);                      \
         |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:105:2: note: expanded from macro 'dev_printk_index_emit'
     105 |         printk_index_subsys_emit("%s %s: ", level, fmt)
         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/printk.h:468:2: note: expanded from macro 'printk_index_subsys_emit'
     468 |         __printk_index_emit(fmt, level, subsys_fmt_prefix)
         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/printk.h:434:39: note: expanded from macro '__printk_index_emit'
     434 |                                 .fmt = __builtin_constant_p(_fmt) ? (_fmt) : NULL, \
         |                                                                   ^ ~~~~~~   ~~~~
   sound/soc/sof/ipc4-topology.c:510:11: error: incompatible pointer types passing 'char[18]' to parameter of type 'const struct device *' [-Werror,-Wincompatible-pointer-types]
     510 |                 dev_err("Bad domain val %d", swidget->comp_domain);
         |                         ^~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:154:44: note: expanded from macro 'dev_err'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                   ^~~
   include/linux/dev_printk.h:110:11: note: expanded from macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                         ^~~
   include/linux/dev_printk.h:50:36: note: passing argument to parameter 'dev' here
      50 | void _dev_err(const struct device *dev, const char *fmt, ...);
         |                                    ^
   sound/soc/sof/ipc4-topology.c:510:32: error: incompatible integer to pointer conversion passing 'enum sof_comp_domain' to parameter of type 'const char *' [-Wint-conversion]
     510 |                 dev_err("Bad domain val %d", swidget->comp_domain);
         |                                              ^~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:154:57: note: expanded from macro 'dev_err'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                                ^~~
   include/linux/dev_printk.h:19:22: note: expanded from macro 'dev_fmt'
      19 | #define dev_fmt(fmt) fmt
         |                      ^~~
   include/linux/dev_printk.h:110:16: note: expanded from macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:50:53: note: passing argument to parameter 'fmt' here
      50 | void _dev_err(const struct device *dev, const char *fmt, ...);
         |                                                     ^
   1 warning and 2 errors generated.


vim +510 sound/soc/sof/ipc4-topology.c

   482	
   483	static int sof_ipc4_widget_setup_msg(struct snd_sof_widget *swidget, struct sof_ipc4_msg *msg)
   484	{
   485		struct sof_ipc4_fw_module *fw_module;
   486		uint32_t type;
   487		int ret;
   488	
   489		ret = sof_ipc4_widget_set_module_info(swidget);
   490		if (ret)
   491			return ret;
   492	
   493		fw_module = swidget->module_info;
   494	
   495		msg->primary = fw_module->man4_module_entry.id;
   496		msg->primary |= SOF_IPC4_MSG_TYPE_SET(SOF_IPC4_MOD_INIT_INSTANCE);
   497		msg->primary |= SOF_IPC4_MSG_DIR(SOF_IPC4_MSG_REQUEST);
   498		msg->primary |= SOF_IPC4_MSG_TARGET(SOF_IPC4_MODULE_MSG);
   499	
   500		msg->extension = SOF_IPC4_MOD_EXT_CORE_ID(swidget->core);
   501	
   502		switch (swidget->comp_domain) {
   503		case SOF_COMP_DOMAIN_LL:
   504			type = 0;
   505			break;
   506		case SOF_COMP_DOMAIN_DP:
   507			type = 1;
   508			break;
   509		default:
 > 510			dev_err("Bad domain val %d", swidget->comp_domain);
   511			/* fall through */
   512		case SOF_COMP_DOMAIN_UNSET:
   513			type = (fw_module->man4_module_entry.type & SOF_IPC4_MODULE_DP) ? 1 : 0;
   514			break;
   515		}
   516		msg->extension |= SOF_IPC4_MOD_EXT_DOMAIN(type);
   517	
   518		return 0;
   519	}
   520	

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

                 reply	other threads:[~2025-04-07 17:24 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=202504080147.AExWIzWG-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=jyri.sarha@linux.intel.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /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.