All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Tzung-Bi Shih <tzungbi@google.com>
Cc: gwendal@google.com, devicetree@vger.kernel.org,
	alsa-devel@alsa-project.org, kbuild-all@lists.01.org,
	cychiang@google.com, drinkcat@google.com, robh+dt@kernel.org,
	tzungbi@google.com, broonie@kernel.org,
	enric.balletbo@collabora.com, bleung@google.com,
	dgreid@google.com
Subject: Re: [alsa-devel] [PATCH v3 07/10] ASoC: cros_ec_codec: support WoV
Date: Tue, 15 Oct 2019 09:34:30 +0800	[thread overview]
Message-ID: <201910150924.II2vi71d%lkp@intel.com> (raw)
In-Reply-To: <20191014180059.07.I5388b69a7a9c551078fed216a77440cee6dedf49@changeid>

[-- Attachment #1: Type: text/plain, Size: 3949 bytes --]

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@intel.com>

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 #2: .config.gz --]
[-- Type: application/gzip, Size: 29599 bytes --]

[-- Attachment #3: Type: text/plain, Size: 161 bytes --]

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

WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
Cc: gwendal@google.com, devicetree@vger.kernel.org,
	alsa-devel@alsa-project.org, kbuild-all@lists.01.org,
	cychiang@google.com, drinkcat@google.com, robh+dt@kernel.org,
	tzungbi@google.com, broonie@kernel.org,
	enric.balletbo@collabora.com, bleung@google.com,
	dgreid@google.com
Subject: Re: [PATCH v3 07/10] ASoC: cros_ec_codec: support WoV
Date: Tue, 15 Oct 2019 09:34:30 +0800	[thread overview]
Message-ID: <201910150924.II2vi71d%lkp@intel.com> (raw)
In-Reply-To: <20191014180059.07.I5388b69a7a9c551078fed216a77440cee6dedf49@changeid>

[-- Attachment #1: Type: text/plain, Size: 3949 bytes --]

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@intel.com>

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 #2: .config.gz --]
[-- Type: application/gzip, Size: 29599 bytes --]

[-- Attachment #3: Type: text/plain, Size: 0 bytes --]



WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [alsa-devel] [PATCH v3 07/10] ASoC: cros_ec_codec: support WoV
Date: Tue, 15 Oct 2019 09:34:30 +0800	[thread overview]
Message-ID: <201910150924.II2vi71d%lkp@intel.com> (raw)
In-Reply-To: <20191014180059.07.I5388b69a7a9c551078fed216a77440cee6dedf49@changeid>

[-- Attachment #1: Type: text/plain, Size: 4054 bytes --]

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@intel.com>

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 #2: config.gz --]
[-- Type: application/gzip, Size: 29599 bytes --]

  reply	other threads:[~2019-10-15  1:35 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-14 10:20 [alsa-devel] [PATCH v3 00/10] ASoC: mediatek: mt8183-mt6358-ts3a227-max98357: support WoV Tzung-Bi Shih
2019-10-14 10:20 ` Tzung-Bi Shih
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 01/10] platform/chrome: cros_ec: remove unused EC feature Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-15 11:48   ` [alsa-devel] " Mark Brown
2019-10-15 15:37     ` Tzung-Bi Shih
2019-10-17  2:25   ` Benson Leung
2019-10-18 18:06   ` [alsa-devel] Applied "platform/chrome: cros_ec: remove unused EC feature" to the asoc tree Mark Brown
2019-10-18 18:06     ` Mark Brown
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 02/10] ASoC: cros_ec_codec: refactor I2S RX Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-17  2:30   ` [alsa-devel] " Benson Leung
2019-10-18 18:06   ` [alsa-devel] Applied "ASoC: cros_ec_codec: refactor I2S RX" to the asoc tree Mark Brown
2019-10-18 18:06     ` Mark Brown
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 03/10] ASoC: cros_ec_codec: extract DMIC EC command from I2S RX Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-17  2:48   ` [alsa-devel] " Benson Leung
2019-10-18 18:06   ` [alsa-devel] Applied "ASoC: cros_ec_codec: extract DMIC EC command from I2S RX" to the asoc tree Mark Brown
2019-10-18 18:06     ` Mark Brown
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 04/10] platform/chrome: cros_ec: add common commands for EC codec Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-17  2:57   ` [alsa-devel] " Benson Leung
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 05/10] ASoC: cros_ec_codec: read max DMIC gain from " Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-17  3:07   ` [alsa-devel] " Benson Leung
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 06/10] ASoC: dt-bindings: cros_ec_codec: add SHM bindings Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-14 17:23   ` [alsa-devel] " Rob Herring
2019-10-14 17:23     ` Rob Herring
2019-10-17  3:12   ` [alsa-devel] " Benson Leung
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 07/10] ASoC: cros_ec_codec: support WoV Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-15  1:34   ` kbuild test robot [this message]
2019-10-15  1:34     ` [alsa-devel] " kbuild test robot
2019-10-15  1:34     ` kbuild test robot
2019-10-15  6:49     ` [alsa-devel] " Tzung-Bi Shih
2019-10-15  6:49       ` Tzung-Bi Shih
2019-10-15  6:49       ` Tzung-Bi Shih
2019-10-17 11:43       ` [alsa-devel] " Mark Brown
2019-10-17 11:43         ` Mark Brown
2019-10-15  3:20   ` kbuild test robot
2019-10-15  3:20     ` kbuild test robot
2019-10-15  3:20     ` kbuild test robot
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 08/10] ASoC: mediatek: mt6358: " Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 09/10] ASoC: dt-bindings: mt8183: add ec-codec Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-14 17:23   ` [alsa-devel] " Rob Herring
2019-10-14 17:23     ` Rob Herring
2019-10-14 10:20 ` [alsa-devel] [PATCH v3 10/10] ASoC: mediatek: mt8183: support WoV Tzung-Bi Shih
2019-10-14 10:20   ` Tzung-Bi Shih
2019-10-14 12:34 ` [alsa-devel] [PATCH v3 00/10] ASoC: mediatek: mt8183-mt6358-ts3a227-max98357: " Tzung-Bi Shih
2019-10-14 12:34   ` Tzung-Bi Shih

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=201910150924.II2vi71d%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=bleung@google.com \
    --cc=broonie@kernel.org \
    --cc=cychiang@google.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dgreid@google.com \
    --cc=drinkcat@google.com \
    --cc=enric.balletbo@collabora.com \
    --cc=gwendal@google.com \
    --cc=kbuild-all@lists.01.org \
    --cc=robh+dt@kernel.org \
    --cc=tzungbi@google.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.