Re: [PATCH v3 07/10] ASoC: cros_ec_codec: support WoV

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tzung-Bi,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on asoc/for-next]
[cannot apply to v5.4-rc3 next-20191014]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Tzung-Bi-Shih/ASoC-mediatek-mt8183-mt6358-ts3a227-max98357-support-WoV/20191014-183227
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
config: x86_64-randconfig-g002-201941 (attached as .config)
compiler: gcc-7 (Debian 7.4.0-13) 7.4.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

   sound/soc/codecs/cros_ec_codec.c: In function 'cros_ec_codec_platform_probe':
>> sound/soc/codecs/cros_ec_codec.c:993:23: error: implicit declaration of function 'of_read_number'; did you mean 'wov_read_audio'? [-Werror=implicit-function-declaration]
      priv->ec_shm_addr = of_read_number(regaddr_p, 2);
                          ^~~~~~~~~~~~~~
                          wov_read_audio
   cc1: some warnings being treated as errors

vim +993 sound/soc/codecs/cros_ec_codec.c

   973	
   974	static int cros_ec_codec_platform_probe(struct platform_device *pdev)
   975	{
   976		struct device *dev = &pdev->dev;
   977		struct cros_ec_device *ec_device = dev_get_drvdata(pdev->dev.parent);
   978		struct cros_ec_codec_priv *priv;
   979		struct device_node *node;
   980		struct resource res;
   981		struct ec_param_ec_codec p;
   982		struct ec_response_ec_codec_get_capabilities r;
   983		int ret;
   984		u64 ec_shm_size;
   985		const __be32 *regaddr_p;
   986	
   987		priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
   988		if (!priv)
   989			return -ENOMEM;
   990	
   991		regaddr_p = of_get_address(dev->of_node, 0, &ec_shm_size, NULL);
   992		if (regaddr_p) {
 > 993			priv->ec_shm_addr = of_read_number(regaddr_p, 2);
   994			priv->ec_shm_len = ec_shm_size;
   995	
   996			dev_dbg(dev, "ec_shm_addr=%#llx len=%#x\n",
   997				priv->ec_shm_addr, priv->ec_shm_len);
   998		}
   999	
  1000		node = of_parse_phandle(dev->of_node, "memory-region", 0);
  1001		if (node) {
  1002			ret = of_address_to_resource(node, 0, &res);
  1003			if (!ret) {
  1004				priv->ap_shm_phys_addr = res.start;
  1005				priv->ap_shm_len = resource_size(&res);
  1006				priv->ap_shm_addr =
  1007					(uint64_t)(uintptr_t)devm_ioremap_wc(
  1008						dev, priv->ap_shm_phys_addr,
  1009						priv->ap_shm_len);
  1010				priv->ap_shm_last_alloc = priv->ap_shm_phys_addr;
  1011	
  1012				dev_dbg(dev, "ap_shm_phys_addr=%#llx len=%#x\n",
  1013					priv->ap_shm_phys_addr, priv->ap_shm_len);
  1014			}
  1015		}
  1016	
  1017		priv->dev = dev;
  1018		priv->ec_device = ec_device;
  1019		atomic_set(&priv->dmic_probed, 0);
  1020	
  1021		p.cmd = EC_CODEC_GET_CAPABILITIES;
  1022		ret = send_ec_host_command(priv->ec_device, EC_CMD_EC_CODEC,
  1023					   (uint8_t *)&p, sizeof(p),
  1024					   (uint8_t *)&r, sizeof(r));
  1025		if (ret) {
  1026			dev_err(dev, "failed to EC_CODEC_GET_CAPABILITIES\n");
  1027			return ret;
  1028		}
  1029		priv->ec_capabilities = r.capabilities;
  1030	
  1031		platform_set_drvdata(pdev, priv);
  1032	
  1033		ret = devm_snd_soc_register_component(dev, &i2s_rx_component_driver,
  1034						      &i2s_rx_dai_driver, 1);
  1035		if (ret)
  1036			return ret;
  1037	
  1038		return devm_snd_soc_register_component(dev, &wov_component_driver,
  1039						       &wov_dai_driver, 1);
  1040	}
  1041	

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

Attachment: .config.gz
Description: application/gzip

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux