All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Bard Liao <yung-chuan.liao@linux.intel.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: [thesofproject:topic/sof-dev 9451/9453] sound/soc/sof/amd/acp7x.c:180:45: error: assignment to 'int (*)(struct snd_sof_dev *, const char *)' from incompatible pointer type 'int (*)(struct snd_sof_dev *)'
Date: Fri, 23 Oct 2026 00:10:14 +0800	[thread overview]
Message-ID: <202610230026.Sb41drsw-lkp@intel.com> (raw)

tree:   https://github.com/thesofproject/linux topic/sof-dev
head:   fc5eeea8dbdc973a5994c5ab91fc9daade2673e3
commit: ec69dd1e9c24d7b84073055d0fd87c436919a6b0 [9451/9453] Merge branch 'sound/upstream-20261015' into merge/sound-upstream-20261015
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20261023/202610230026.Sb41drsw-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261023/202610230026.Sb41drsw-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/202610230026.Sb41drsw-lkp@intel.com/

Note: the thesofproject/topic/sof-dev HEAD fc5eeea8dbdc973a5994c5ab91fc9daade2673e3 builds fine.
      It only hurts bisectability.

All errors (new ones prefixed by >>):

   sound/soc/sof/amd/acp7x.c: In function 'sof_acp7x_ops_init':
>> sound/soc/sof/amd/acp7x.c:180:45: error: assignment to 'int (*)(struct snd_sof_dev *, const char *)' from incompatible pointer type 'int (*)(struct snd_sof_dev *)' [-Wincompatible-pointer-types]
     180 |                 sof_acp7x_ops.load_firmware = acp7x_load_firmware_carveout;
         |                                             ^


vim +180 sound/soc/sof/amd/acp7x.c

0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  147  
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  148  int sof_acp7x_ops_init(struct snd_sof_dev *sdev)
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  149  {
7a81f54a0483edc Vijendar Mukunda 2026-07-01  150  	struct acpi_device *adev = ACPI_COMPANION(&to_pci_dev(sdev->dev)->dev);
7a81f54a0483edc Vijendar Mukunda 2026-07-01  151  	const union acpi_object *obj;
7af6ac41cd470b0 Vijendar Mukunda 2026-07-01  152  	int acp_sof_signed_firmware_image = 0;
7a81f54a0483edc Vijendar Mukunda 2026-07-01  153  	int acp_sof_post_fw_run_delay = 0;
7a81f54a0483edc Vijendar Mukunda 2026-07-01  154  
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  155  	/* common defaults */
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  156  	memcpy(&sof_acp7x_ops, &sof_acp_common_ops, sizeof(struct snd_sof_dsp_ops));
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  157  
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  158  	sof_acp7x_ops.drv = acp7x_sof_dai;
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  159  	sof_acp7x_ops.num_drv = ARRAY_SIZE(acp7x_sof_dai);
78d99c2c5eee0f6 Vijendar Mukunda 2026-07-01  160  	sof_acp7x_ops.probe = amd_sof_acp7x_probe;
78d99c2c5eee0f6 Vijendar Mukunda 2026-07-01  161  	sof_acp7x_ops.remove = amd_sof_acp7x_remove;
0ec5afad9d31c4f Vijendar Mukunda 2026-07-01  162  
7a81f54a0483edc Vijendar Mukunda 2026-07-01  163  	if (adev) {
7af6ac41cd470b0 Vijendar Mukunda 2026-07-01  164  		if (!acpi_dev_get_property(adev, "acp-sof-signed-firmware-image",
7af6ac41cd470b0 Vijendar Mukunda 2026-07-01  165  					   ACPI_TYPE_INTEGER, &obj))
7af6ac41cd470b0 Vijendar Mukunda 2026-07-01  166  			acp_sof_signed_firmware_image = obj->integer.value;
7af6ac41cd470b0 Vijendar Mukunda 2026-07-01  167  
7a81f54a0483edc Vijendar Mukunda 2026-07-01  168  		if (!acpi_dev_get_property(adev, "acp-sof-post_fw_run_delay",
7a81f54a0483edc Vijendar Mukunda 2026-07-01  169  					   ACPI_TYPE_INTEGER, &obj))
7a81f54a0483edc Vijendar Mukunda 2026-07-01  170  			acp_sof_post_fw_run_delay = obj->integer.value;
7a81f54a0483edc Vijendar Mukunda 2026-07-01  171  	}
7a81f54a0483edc Vijendar Mukunda 2026-07-01  172  
cec3598305a6df3 Vijendar Mukunda 2026-10-13  173  	if (acp_sof_signed_firmware_image) {
cec3598305a6df3 Vijendar Mukunda 2026-10-13  174  		/*
cec3598305a6df3 Vijendar Mukunda 2026-10-13  175  		 * This driver handles only ACP7.B/7.F which both use the ASP
cec3598305a6df3 Vijendar Mukunda 2026-10-13  176  		 * carveout firmware load path. Assign unconditionally rather
cec3598305a6df3 Vijendar Mukunda 2026-10-13  177  		 * than rechecking the revision so the correct loader is always
cec3598305a6df3 Vijendar Mukunda 2026-10-13  178  		 * selected when signed firmware is enabled.
cec3598305a6df3 Vijendar Mukunda 2026-10-13  179  		 */
cec3598305a6df3 Vijendar Mukunda 2026-10-13 @180  		sof_acp7x_ops.load_firmware = acp7x_load_firmware_carveout;

:::::: The code at line 180 was first introduced by commit
:::::: cec3598305a6df306fc4eb0c49a72e0f9949c06f ASoC: SOF: amd: add ASP carveout firmware loading for ACP7.B/7.F

:::::: TO: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
:::::: CC: Mark Brown <broonie@kernel.org>

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

                 reply	other threads:[~2026-10-22 16:10 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=202610230026.Sb41drsw-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=yung-chuan.liao@linux.intel.com \
    /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.