All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Amit Kumar Mahapatra <amit.kumar-mahapatra@amd.com>
Cc: oe-kbuild-all@lists.linux.dev, Mark Brown <broonie@kernel.org>
Subject: [broonie-ci:v3_20230202_amit_kumar_mahapatra_spi_add_support_for_stacked_parallel_memories 35/35] sound/pci/hda/cs35l41_hda_spi.c:28:61: warning: passing argument 3 of 'cs35l41_hda_probe' makes integer from pointer without a cast
Date: Fri, 10 Feb 2023 11:56:05 +0800	[thread overview]
Message-ID: <202302101127.AxwfAPSu-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/ci.git v3_20230202_amit_kumar_mahapatra_spi_add_support_for_stacked_parallel_memories
head:   b1bff13bcb2439c0ba5142fb0851ad602f3a4a02
commit: b1bff13bcb2439c0ba5142fb0851ad602f3a4a02 [35/35] spi: Add stacked and parallel memories support in SPI core
config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20230210/202302101127.AxwfAPSu-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce (this is a W=1 build):
        # https://git.kernel.org/pub/scm/linux/kernel/git/broonie/ci.git/commit/?id=b1bff13bcb2439c0ba5142fb0851ad602f3a4a02
        git remote add broonie-ci https://git.kernel.org/pub/scm/linux/kernel/git/broonie/ci.git
        git fetch --no-tags broonie-ci v3_20230202_amit_kumar_mahapatra_spi_add_support_for_stacked_parallel_memories
        git checkout b1bff13bcb2439c0ba5142fb0851ad602f3a4a02
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 O=build_dir ARCH=i386 olddefconfig
        make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash sound/pci/hda/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202302101127.AxwfAPSu-lkp@intel.com/

All warnings (new ones prefixed by >>):

   sound/pci/hda/cs35l41_hda_spi.c: In function 'cs35l41_hda_spi_probe':
>> sound/pci/hda/cs35l41_hda_spi.c:28:61: warning: passing argument 3 of 'cs35l41_hda_probe' makes integer from pointer without a cast [-Wint-conversion]
      28 |         return cs35l41_hda_probe(&spi->dev, device_name, spi->chip_select, spi->irq,
         |                                                          ~~~^~~~~~~~~~~~~
         |                                                             |
         |                                                             u8 * {aka unsigned char *}
   In file included from sound/pci/hda/cs35l41_hda_spi.c:13:
   sound/pci/hda/cs35l41_hda.h:83:72: note: expected 'int' but argument is of type 'u8 *' {aka 'unsigned char *'}
      83 | int cs35l41_hda_probe(struct device *dev, const char *device_name, int id, int irq,
         |                                                                    ~~~~^~


vim +/cs35l41_hda_probe +28 sound/pci/hda/cs35l41_hda_spi.c

7b2f3eb492dac7 Lucas Tanure    2021-12-17  14  
7b2f3eb492dac7 Lucas Tanure    2021-12-17  15  static int cs35l41_hda_spi_probe(struct spi_device *spi)
7b2f3eb492dac7 Lucas Tanure    2021-12-17  16  {
7b2f3eb492dac7 Lucas Tanure    2021-12-17  17  	const char *device_name;
7b2f3eb492dac7 Lucas Tanure    2021-12-17  18  
642999365da3b7 Andy Shevchenko 2022-07-11  19  	/*
642999365da3b7 Andy Shevchenko 2022-07-11  20  	 * Compare against the device name so it works for SPI, normal ACPI
642999365da3b7 Andy Shevchenko 2022-07-11  21  	 * and for ACPI by serial-multi-instantiate matching cases.
7b2f3eb492dac7 Lucas Tanure    2021-12-17  22  	 */
7b2f3eb492dac7 Lucas Tanure    2021-12-17  23  	if (strstr(dev_name(&spi->dev), "CSC3551"))
7b2f3eb492dac7 Lucas Tanure    2021-12-17  24  		device_name = "CSC3551";
7b2f3eb492dac7 Lucas Tanure    2021-12-17  25  	else
7b2f3eb492dac7 Lucas Tanure    2021-12-17  26  		return -ENODEV;
7b2f3eb492dac7 Lucas Tanure    2021-12-17  27  
7b2f3eb492dac7 Lucas Tanure    2021-12-17 @28  	return cs35l41_hda_probe(&spi->dev, device_name, spi->chip_select, spi->irq,
7b2f3eb492dac7 Lucas Tanure    2021-12-17  29  				 devm_regmap_init_spi(spi, &cs35l41_regmap_spi));
7b2f3eb492dac7 Lucas Tanure    2021-12-17  30  }
7b2f3eb492dac7 Lucas Tanure    2021-12-17  31  

:::::: The code at line 28 was first introduced by commit
:::::: 7b2f3eb492dac7665c75df067e4d8e4869589f4a ALSA: hda: cs35l41: Add support for CS35L41 in HDA systems

:::::: TO: Lucas Tanure <tanureal@opensource.cirrus.com>
:::::: CC: Takashi Iwai <tiwai@suse.de>

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

                 reply	other threads:[~2023-02-10  3:56 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=202302101127.AxwfAPSu-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=amit.kumar-mahapatra@amd.com \
    --cc=broonie@kernel.org \
    --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.