ALSA: cs5535audio: clean up OLPC code

PATCH In review Andres Salomon 1 participant 1 message Nov 6, 2008

Subsystems (2)

Sound CS5535 Audio ALSA driver

lore

Share & tools

lore

Andres Salomon

author

2008-11-06 21:54 UTC #0 lore raw

PATCH 13/14 ALSA: cs5535audio: clean up OLPC code

- add copyright info to _olpc.c
- minor layout fixes
- make Makefile more concise
- silence a warning

Signed-off-by Andres Salomon <[email protected]>

---
 sound/pci/cs5535audio/Makefile           |    5 +----
 sound/pci/cs5535audio/cs5535audio.c      |    4 ++--
 sound/pci/cs5535audio/cs5535audio_olpc.c |   15 ++++++++++++---
 3 files changed, 15 insertions(+), 9 deletions(-)

sound/pci/cs5535audio/Makefile +1 -4

index 3e41fd3..ccc6422 100644
@@ -4,10 +4,7 @@
 
 snd-cs5535audio-y := cs5535audio.o cs5535audio_pcm.o
 snd-cs5535audio-$(CONFIG_PM) += cs5535audio_pm.o
-
-ifdef CONFIG_OLPC
-snd-cs5535audio-objs += cs5535audio_olpc.o
-endif
+snd-cs5535audio-$(CONFIG_OLPC) += cs5535audio_olpc.o
 
 # Toplevel Module Dependency
 obj-$(CONFIG_SND_CS5535AUDIO) += snd-cs5535audio.o

sound/pci/cs5535audio/cs5535audio.c +2 -2

index 0f5ce36..826e6de 100644
@@ -174,8 +174,8 @@ static int __devinit snd_cs5535audio_mixer(struct cs5535audio *cs5535au)
 
 	snd_ac97_tune_hardware(cs5535au->ac97, ac97_quirks, ac97_quirk);
 
-	/* olpc_quirks is dummied out if not olpc */
-	if (( err = olpc_quirks(card, cs5535au->ac97)) < 0) {
+	err = olpc_quirks(card, cs5535au->ac97);
+	if (err < 0) {
 		snd_printk(KERN_ERR "olpc quirks failed\n");
 		return err;
 	}

sound/pci/cs5535audio/cs5535audio_olpc.c +12 -3

index 45b24f7..164f6bd 100644
@@ -1,4 +1,14 @@
-#include <sound/driver.h>
+/*
+ * OLPC XO-1 additional sound features
+ *
+ * Copyright © 2006  Jaya Kumar <[email protected]>
+ * Copyright © 2007-2008  Andres Salomon <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
 #include <sound/core.h>
 #include <sound/info.h>
 #include <sound/control.h>

@@ -106,7 +116,7 @@ static struct snd_kcontrol_new olpc_cs5535audio_ctls[] __devinitdata = {
 	.info = olpc_dc_info,
 	.get = olpc_dc_get,
 	.put = olpc_dc_put,
-	.private_value = 0
+	.private_value = 0,
 },
 {
 	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,

@@ -161,4 +171,3 @@ int __devinit olpc_quirks(struct snd_card *card, struct snd_ac97 *ac97)
 	olpc_mic_bias(ac97, 0);
 	return 0;
 }
-

Signature · 1 line

1.5.6.5