linux-next: build failure after merge of the final tree (sound treerelated)

[Stephen Rothwell]

Hi all,

After merging the final tree, today's linux-next build (i386 defconfig)
failed like this:

sound/pci/hda/patch_via.c:185:28: error: field 'loopback' has incomplete type
sound/pci/hda/patch_via.c:187:22: error: array type has incomplete element type
sound/pci/hda/patch_via.c: In function 'is_aa_path_mute':
sound/pci/hda/patch_via.c:891:39: error: dereferencing pointer to incomplete type
sound/pci/hda/patch_via.c:891:51: error: dereferencing pointer to incomplete type
sound/pci/hda/patch_via.c:892:11: error: dereferencing pointer to incomplete type
sound/pci/hda/patch_via.c: In function 'add_loopback_list':
sound/pci/hda/patch_via.c:2004:29: warning: type defaults to 'int' in type name
sound/pci/hda/patch_via.c:2004:29: warning: type defaults to 'int' in type name
sound/pci/hda/patch_via.c:2004:29: error: negative width in bit-field '<anonymous>'
sound/pci/hda/patch_via.c:2007:6: error: dereferencing pointer to incomplete type
sound/pci/hda/patch_via.c:2008:6: error: dereferencing pointer to incomplete type
sound/pci/hda/patch_via.c:2009:6: error: dereferencing pointer to incomplete type

Caused by commit ada509ec00e4 ("ALSA: hda - Simplify analog-low-current
mode check for VIA codecs") which removed the CONFIG_SND_HDA_POWER_SAVE
protection of the usage, but not the definition in
sound/pci/hda/hda_local.h.

I have added the below patch for today:

From: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
Date: Tue, 28 Jun 2011 16:46:09 +1000
Subject: [PATCH] ALSA: hda - remove SND_HDA_POWER_SAVE protection of struct
 hda_loopback_check

to fix build problems when it is disabled.

Signed-off-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
---
 sound/pci/hda/hda_local.h |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/sound/pci/hda/hda_local.h b/sound/pci/hda/hda_local.h
index b333bf4..88b277e 100644
--- a/sound/pci/hda/hda_local.h
+++ b/sound/pci/hda/hda_local.h
@@ -565,7 +565,6 @@ int snd_hda_get_bool_hint(struct hda_codec *codec, const char *key)
  * power-management
  */
 
-#ifdef CONFIG_SND_HDA_POWER_SAVE
 void snd_hda_schedule_power_save(struct hda_codec *codec);
 
 struct hda_amp_list {
@@ -582,7 +581,6 @@ struct hda_loopback_check {
 int snd_hda_check_amp_list_power(struct hda_codec *codec,
 				 struct hda_loopback_check *check,
 				 hda_nid_t nid);
-#endif /* CONFIG_SND_HDA_POWER_SAVE */
 
 /*
  * AMP control callbacks
-- 
1.7.5.4

-- 
Cheers,
Stephen Rothwell                    sfr@xxxxxxxxxxxxxxxx
http://www.canb.auug.org.au/~sfr/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/