* [asoc:topic/intel 53/55] sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type
@ 2016-09-25 8:13 kbuild test robot
2016-09-26 8:39 ` Jeeja KP
0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2016-09-25 8:13 UTC (permalink / raw)
To: Sandeep Tayal; +Cc: Jeeja KP, alsa-devel, Mark Brown, kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3161 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/intel
head: 2d995e5dc283adbfbf9c1eb81bf35ca7af2d22a6
commit: fe27f4e0545d3fc1b0518fafb4fe0460d757651d [53/55] ASoC: hdac_hdmi: use audio component framework to read ELD
config: x86_64-randconfig-n0-09251512 (attached as .config)
compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4
reproduce:
git checkout fe27f4e0545d3fc1b0518fafb4fe0460d757651d
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
sound/soc/codecs/hdac_hdmi.c: In function 'hdac_hdmi_present_sense':
sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 3 of 'snd_hdac_acomp_get_eld' makes pointer from integer without a cast [enabled by default]
ELD_MAX_SIZE);
^
In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
include/sound/hda_i915.h:14:5: note: expected 'bool *' but argument is of type 'int'
int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
^
>> sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type [enabled by default]
ELD_MAX_SIZE);
^
In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
include/sound/hda_i915.h:14:5: note: expected 'char *' but argument is of type 'bool *'
int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
^
sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 5 of 'snd_hdac_acomp_get_eld' makes integer from pointer without a cast [enabled by default]
ELD_MAX_SIZE);
^
In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
include/sound/hda_i915.h:14:5: note: expected 'int' but argument is of type 'char *'
int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
^
sound/soc/codecs/hdac_hdmi.c:1028:5: error: too many arguments to function 'snd_hdac_acomp_get_eld'
ELD_MAX_SIZE);
^
In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
include/sound/hda_i915.h:14:5: note: declared here
int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
^
vim +/snd_hdac_acomp_get_eld +1028 sound/soc/codecs/hdac_hdmi.c
1012
1013 return 0;
1014 }
1015
1016 static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin)
1017 {
1018 struct hdac_ext_device *edev = pin->edev;
1019 struct hdac_hdmi_priv *hdmi = edev->private_data;
1020 struct hdac_hdmi_pcm *pcm;
1021 int size;
1022
1023 mutex_lock(&hdmi->pin_mutex);
1024 pin->eld.monitor_present = false;
1025
1026 size = snd_hdac_acomp_get_eld(&edev->hdac, pin->nid, -1,
1027 &pin->eld.monitor_present, pin->eld.eld_buffer,
> 1028 ELD_MAX_SIZE);
1029
1030 if (size > 0) {
1031 size = min(size, ELD_MAX_SIZE);
1032 if (hdac_hdmi_parse_eld(edev, pin) < 0)
1033 size = -EINVAL;
1034 }
1035
1036 if (size > 0) {
---
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: 22544 bytes --]
[-- Attachment #3: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [asoc:topic/intel 53/55] sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type
2016-09-25 8:13 [asoc:topic/intel 53/55] sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type kbuild test robot
@ 2016-09-26 8:39 ` Jeeja KP
2016-09-26 16:02 ` Mark Brown
0 siblings, 1 reply; 3+ messages in thread
From: Jeeja KP @ 2016-09-26 8:39 UTC (permalink / raw)
To: kbuild test robot; +Cc: Sandeep Tayal, Mark Brown, kbuild-all, alsa-devel
On Sun, Sep 25, 2016 at 04:13:04PM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/intel
> head: 2d995e5dc283adbfbf9c1eb81bf35ca7af2d22a6
> commit: fe27f4e0545d3fc1b0518fafb4fe0460d757651d [53/55] ASoC: hdac_hdmi: use audio component framework to read ELD
> config: x86_64-randconfig-n0-09251512 (attached as .config)
> compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4
> reproduce:
> git checkout fe27f4e0545d3fc1b0518fafb4fe0460d757651d
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
Hi Mark,
This patch has dependency [1] with the i915 API changes for MST. This is
not merged yet
Can this patch be reverted or staged until the i915 API changes are
merged?
Sorry I should have explicitly mention this in cover letter
[1]: https://patchwork.freedesktop.org/series/10571/.
--
Jeeja
> All warnings (new ones prefixed by >>):
>
> sound/soc/codecs/hdac_hdmi.c: In function 'hdac_hdmi_present_sense':
> sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 3 of 'snd_hdac_acomp_get_eld' makes pointer from integer without a cast [enabled by default]
> ELD_MAX_SIZE);
> ^
> In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
> include/sound/hda_i915.h:14:5: note: expected 'bool *' but argument is of type 'int'
> int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
> ^
> >> sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type [enabled by default]
> ELD_MAX_SIZE);
> ^
> In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
> include/sound/hda_i915.h:14:5: note: expected 'char *' but argument is of type 'bool *'
> int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
> ^
> sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 5 of 'snd_hdac_acomp_get_eld' makes integer from pointer without a cast [enabled by default]
> ELD_MAX_SIZE);
> ^
> In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
> include/sound/hda_i915.h:14:5: note: expected 'int' but argument is of type 'char *'
> int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
> ^
> sound/soc/codecs/hdac_hdmi.c:1028:5: error: too many arguments to function 'snd_hdac_acomp_get_eld'
> ELD_MAX_SIZE);
> ^
> In file included from sound/soc/codecs/hdac_hdmi.c:30:0:
> include/sound/hda_i915.h:14:5: note: declared here
> int snd_hdac_acomp_get_eld(struct hdac_device *codec, hda_nid_t nid,
> ^
>
> vim +/snd_hdac_acomp_get_eld +1028 sound/soc/codecs/hdac_hdmi.c
>
> 1012
> 1013 return 0;
> 1014 }
> 1015
> 1016 static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin)
> 1017 {
> 1018 struct hdac_ext_device *edev = pin->edev;
> 1019 struct hdac_hdmi_priv *hdmi = edev->private_data;
> 1020 struct hdac_hdmi_pcm *pcm;
> 1021 int size;
> 1022
> 1023 mutex_lock(&hdmi->pin_mutex);
> 1024 pin->eld.monitor_present = false;
> 1025
> 1026 size = snd_hdac_acomp_get_eld(&edev->hdac, pin->nid, -1,
> 1027 &pin->eld.monitor_present, pin->eld.eld_buffer,
> > 1028 ELD_MAX_SIZE);
> 1029
> 1030 if (size > 0) {
> 1031 size = min(size, ELD_MAX_SIZE);
> 1032 if (hdac_hdmi_parse_eld(edev, pin) < 0)
> 1033 size = -EINVAL;
> 1034 }
> 1035
> 1036 if (size > 0) {
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
--
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [asoc:topic/intel 53/55] sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type
2016-09-26 8:39 ` Jeeja KP
@ 2016-09-26 16:02 ` Mark Brown
0 siblings, 0 replies; 3+ messages in thread
From: Mark Brown @ 2016-09-26 16:02 UTC (permalink / raw)
To: Jeeja KP; +Cc: Sandeep Tayal, kbuild test robot, kbuild-all, alsa-devel
[-- Attachment #1.1: Type: text/plain, Size: 229 bytes --]
On Mon, Sep 26, 2016 at 02:09:51PM +0530, Jeeja KP wrote:
> Can this patch be reverted or staged until the i915 API changes are
> merged?
I'll revert.
> Sorry I should have explicitly mention this in cover letter
No problem.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-26 16:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-25 8:13 [asoc:topic/intel 53/55] sound/soc/codecs/hdac_hdmi.c:1028:5: warning: passing argument 4 of 'snd_hdac_acomp_get_eld' from incompatible pointer type kbuild test robot
2016-09-26 8:39 ` Jeeja KP
2016-09-26 16:02 ` Mark Brown
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.