Siemens SX1: sound cleanups

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

 



Hi!

These are cleanups for codingstyle in sound parts of siemens sx1. They
should not change any code. Please apply,

								Pavel

diff --git a/sound/arm/omap/omap-alsa-sx1-mixer.c b/sound/arm/omap/omap-alsa-sx1-mixer.c
index 8ca4c95..b036b3b 100644
--- a/sound/arm/omap/omap-alsa-sx1-mixer.c
+++ b/sound/arm/omap/omap-alsa-sx1-mixer.c
@@ -29,41 +29,34 @@ #define M_DPRINTK(ARGS...)  		/* nop */
 
 static int current_playback_target	= PLAYBACK_TARGET_LOUDSPEAKER;
 static int current_rec_src 		= REC_SRC_SINGLE_ENDED_MICIN_HED;
-static int current_volume 		= 0;	//current volume, we cant read it
-static int current_fm_volume 		= 0;	//current FM radio volume, we cant read it
+static int current_volume 		= 0;	/* current volume, we cant read it */
+static int current_fm_volume 		= 0;	/* current FM radio volume, we cant read it */
 
-/* TODO
- * For selecting SX1 recourd source.
+/* 
+ * Select SX1 recording source.
  */
 static void set_record_source(int val)
 {
-	// TODO	Recording is done on McBSP2	and Mic only
+	/* TODO Recording is done on McBSP2 and Mic only */
 	current_rec_src	= val;
 }
 
-/*
- * Converts the Alsa mixer volume (0 - 100) to SX1
- * (0 - 9) volume.
- */
-int set_mixer_volume(int mixerVol)
+int set_mixer_volume(int mixer_vol)
 {
-	int retVal;
+	/* FIXME: Alsa has mixer_vol in 0-100 range, while SX1 needs 0-9 range */
 
-	if ((mixerVol < 0) ||
-	    (mixerVol > 9) ){
-		printk(KERN_ERR "Trying a bad mixer volume (%d)!\n", mixerVol);
+	if ((mixer_vol < 0) || (mixer_vol > 9)) {
+		printk(KERN_ERR "Trying a bad mixer volume (%d)!\n", mixer_vol);
 		return -EPERM;
 	}
-	current_volume = mixerVol;	// set current volume, we cant read it
-	M_DPRINTK("mixer volume = %d\n", mixerVol);
+	current_volume = mixer_vol;	/* set current volume, we cant read it */
 
-	retVal	= cn_sx1snd_send(DAC_VOLUME_UPDATE, mixerVol, 0 );
-	return retVal;
+	return cn_sx1snd_send(DAC_VOLUME_UPDATE, mixer_vol, 0);
 }
 
 void set_loudspeaker_to_playback_target(void)
 {
-	// TODO
+	/* TODO */
 	cn_sx1snd_send(DAC_SETAUDIODEVICE, SX1_DEVICE_SPEAKER, 0);
 
 	current_playback_target	= PLAYBACK_TARGET_LOUDSPEAKER;
@@ -71,7 +64,7 @@ void set_loudspeaker_to_playback_target(
 
 void set_headphone_to_playback_target(void)
 {
-	// TODO
+	/* TODO */
 	cn_sx1snd_send(DAC_SETAUDIODEVICE, SX1_DEVICE_HEADPHONE, 0);
 
 	current_playback_target	= PLAYBACK_TARGET_HEADPHONE;
@@ -79,7 +72,7 @@ void set_headphone_to_playback_target(vo
 
 void set_telephone_to_playback_target(void)
 {
-	// TODO
+	/* TODO */
 	cn_sx1snd_send(DAC_SETAUDIODEVICE, SX1_DEVICE_PHONE, 0);
 
 	current_playback_target	= PLAYBACK_TARGET_CELLPHONE;
@@ -93,26 +86,21 @@ static void set_telephone_to_record_sour
 void init_playback_targets(void)
 {
 	set_loudspeaker_to_playback_target();
-
 	set_mixer_volume(DEFAULT_OUTPUT_VOLUME);
 }
 
 /*
- * Initializes SX1 recourd source (to mic) and playback target (to loudspeaker)
+ * Initializes SX1 record source (to mic) and playback target (to loudspeaker)
  */
 void snd_omap_init_mixer(void)
 {
-	FN_IN;
-
-	/* Select headset to record source (MIC_INHED)*/
+	/* Select headset to record source */
 	set_record_source(REC_SRC_SINGLE_ENDED_MICIN_HED);
 	/* Init loudspeaker as a default playback target*/
 	init_playback_targets();
-
-	FN_OUT(0);
 }
 
-/*--------------------------------------------------------------------------------------------*/
+/* ---------------------------------------------------------------------------------------- */
 static int __pcm_playback_target_info(snd_kcontrol_t *kcontrol, snd_ctl_elem_info_t *uinfo)
 {
 	static char *texts[PLAYBACK_TARGET_COUNT] = {
@@ -125,8 +113,8 @@ static int __pcm_playback_target_info(sn
 	if (uinfo->value.enumerated.item > PLAYBACK_TARGET_COUNT - 1) {
         	uinfo->value.enumerated.item = PLAYBACK_TARGET_COUNT - 1;
 	}
-	strcpy(uinfo->value.enumerated.name,
-       	texts[uinfo->value.enumerated.item]);
+	strcpy(uinfo->value.enumerated.name, 
+	       texts[uinfo->value.enumerated.item]);
 	return 0;
 }
 
@@ -138,29 +126,27 @@ static int __pcm_playback_target_get(snd
 
 static int __pcm_playback_target_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	int	retVal;
-	int	curVal;
+	int ret_val = 0;
+	int cur_val = ucontrol->value.integer.value[0];
 
-	retVal	= 0;
-	curVal	= ucontrol->value.integer.value[0];
-	if ((curVal >= 0) &&
-	    (curVal < PLAYBACK_TARGET_COUNT) &&
-	    (curVal != current_playback_target)) {
-		if (curVal == PLAYBACK_TARGET_LOUDSPEAKER) {
+	if ((cur_val >= 0) &&
+	    (cur_val < PLAYBACK_TARGET_COUNT) &&
+	    (cur_val != current_playback_target)) {
+		if (cur_val == PLAYBACK_TARGET_LOUDSPEAKER) {
 			set_record_source(REC_SRC_SINGLE_ENDED_MICIN_HED);
 			set_loudspeaker_to_playback_target();
  		}
-		else if (curVal == PLAYBACK_TARGET_HEADPHONE) {
+		else if (cur_val == PLAYBACK_TARGET_HEADPHONE) {
 			set_record_source(REC_SRC_SINGLE_ENDED_MICIN_HND);
  			set_headphone_to_playback_target();
  		}
-		else if (curVal == PLAYBACK_TARGET_CELLPHONE) {
+		else if (cur_val == PLAYBACK_TARGET_CELLPHONE) {
 			set_telephone_to_record_source();
 			set_telephone_to_playback_target();
 		}
-		retVal	= 1;
+		ret_val	= 1;
 	}
-	return retVal;
+	return ret_val;
 }
 
 /*--------------------------------------------------------------------------------------------*/
@@ -175,14 +161,11 @@ static int __pcm_playback_volume_info(sn
 
 /*
  * Alsa mixer interface function for getting the volume read from the SX1 in a
- * 0 -100 alsa mixer format.
+ * 0-100 alsa mixer format.
  */
 static int __pcm_playback_volume_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-
-	ucontrol->value.integer.value[0]	= current_volume;
-
-	M_DPRINTK("mixer volume = %ld\n", current_volume);
+	ucontrol->value.integer.value[0] = current_volume;
 	return 0;
 }
 
@@ -202,17 +185,17 @@ static int __pcm_playback_switch_info(sn
 
 static int __pcm_playback_switch_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	ucontrol->value.integer.value[0]	= 1;
+	ucontrol->value.integer.value[0] = 1;
 	return 0;
 }
 
 static int __pcm_playback_switch_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-//	return dac_gain_control_unmute(ucontrol->value.integer.value[0],
-//					ucontrol->value.integer.value[1]);
 	return 0;
 }
-/*--------------------------------------------------------------------------------------------*/
+
+/* -------------------------------------------------------------------------------------------- */
+
 static int __headset_playback_volume_info(snd_kcontrol_t *kcontrol, snd_ctl_elem_info_t *uinfo)
 {
 	uinfo->type			= SNDRV_CTL_ELEM_TYPE_INTEGER;
@@ -225,8 +208,6 @@ static int __headset_playback_volume_inf
 static int __headset_playback_volume_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
 	ucontrol->value.integer.value[0]	= current_volume;
-
-	M_DPRINTK("mixer volume returned = %ld\n", ucontrol->value.integer.value[0]);
 	return 0;
 }
 
@@ -246,19 +227,21 @@ static int __headset_playback_switch_inf
 
 static int __headset_playback_switch_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	ucontrol->value.integer.value[0]	= 1;
+	ucontrol->value.integer.value[0] = 1;
 	return 0;
 }
 
 static int __headset_playback_switch_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	// mute/unmute headset
-//	return adc_pga_unmute_control(ucontrol->value.integer.value[0],
-//				TSC2101_HEADSET_GAIN_CTRL,
-//				15);
+	/* mute/unmute headset */
+#if 0
+	return adc_pga_unmute_control(ucontrol->value.integer.value[0],
+				TSC2101_HEADSET_GAIN_CTRL,
+				15);
+#endif
 	return 0;
 }
-/*--------------------------------------------------------------------------------------------*/
+/* -------------------------------------------------------------------------------------------- */
 static int __fmradio_playback_volume_info(snd_kcontrol_t *kcontrol, snd_ctl_elem_info_t *uinfo)
 {
 	uinfo->type			= SNDRV_CTL_ELEM_TYPE_INTEGER;
@@ -270,9 +253,7 @@ static int __fmradio_playback_volume_inf
 
 static int __fmradio_playback_volume_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	ucontrol->value.integer.value[0]	= current_fm_volume;
-
-	M_DPRINTK("mixer volume returned = %ld\n", ucontrol->value.integer.value[0]);
+	ucontrol->value.integer.value[0] = current_fm_volume;
 	return 0;
 }
 
@@ -294,21 +275,21 @@ static int __fmradio_playback_switch_inf
 
 static int __fmradio_playback_switch_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	ucontrol->value.integer.value[0]	= 1;
+	ucontrol->value.integer.value[0] = 1;
 	return 0;
 }
 
 static int __fmradio_playback_switch_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	// mute/unmute FM radio
-	if(ucontrol->value.integer.value[0])
+	/* mute/unmute FM radio */
+	if (ucontrol->value.integer.value[0])
 		cn_sx1snd_send(DAC_FMRADIO_OPEN, current_fm_volume, 0);
 	else
 		cn_sx1snd_send(DAC_FMRADIO_CLOSE, 0, 0);
 
 	return 0;
 }
-/*--------------------------------------------------------------------------------------------*/
+/* -------------------------------------------------------------------------------------------- */
 static int __cellphone_input_switch_info(snd_kcontrol_t *kcontrol, snd_ctl_elem_info_t *uinfo)
 {
 	uinfo->type 			= SNDRV_CTL_ELEM_TYPE_BOOLEAN;
@@ -320,18 +301,19 @@ static int __cellphone_input_switch_info
 
 static int __cellphone_input_switch_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	ucontrol->value.integer.value[0]	= 1;
+	ucontrol->value.integer.value[0] = 1;
 	return 0;
 }
 
 static int __cellphone_input_switch_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-//	return adc_pga_unmute_control(ucontrol->value.integer.value[0],
-//				TSC2101_BUZZER_GAIN_CTRL,
-//				15);
+#if 0
+	return adc_pga_unmute_control(ucontrol->value.integer.value[0],
+				TSC2101_BUZZER_GAIN_CTRL, 15);
+#endif
 	return 0;
 }
-/*--------------------------------------------------------------------------------------------*/
+/* -------------------------------------------------------------------------------------------- */
 
 static int __buzzer_input_switch_info(snd_kcontrol_t *kcontrol, snd_ctl_elem_info_t *uinfo)
 {
@@ -344,15 +326,16 @@ static int __buzzer_input_switch_info(sn
 
 static int __buzzer_input_switch_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-	ucontrol->value.integer.value[0]	= 1;
+	ucontrol->value.integer.value[0] = 1;
 	return 0;
 }
 
 static int __buzzer_input_switch_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
 {
-//	return adc_pga_unmute_control(ucontrol->value.integer.value[0],
-//				TSC2101_BUZZER_GAIN_CTRL,
-//				6);
+#if 0
+	return adc_pga_unmute_control(ucontrol->value.integer.value[0],
+				TSC2101_BUZZER_GAIN_CTRL, 6);
+#endif
 	return 0;
 }
 /*--------------------------------------------------------------------------------------------*/
@@ -434,7 +417,6 @@ static snd_kcontrol_new_t egold_control[
 };
 
 #ifdef CONFIG_PM
-
 void snd_omap_suspend_mixer(void)
 {
 }
@@ -447,18 +429,16 @@ #endif
 
 int snd_omap_mixer(struct snd_card_omap_codec *egold)
 {
-	int i=0;
-	int err=0;
+	int i = 0;
+	int err = 0;
 
-	if (!egold) {
+	if (!egold)
 		return -EINVAL;
-	}
+
 	for (i=0; i < ARRAY_SIZE(egold_control); i++) {
-		if ((err = snd_ctl_add(egold->card,
-				snd_ctl_new1(&egold_control[i],
-				egold->card))) < 0) {
+		err = snd_ctl_add(egold->card, snd_ctl_new1(&egold_control[i], egold->card));
+		if (err < 0)
 			return err;
-		}
 	}
 	return 0;
 }
diff --git a/sound/arm/omap/omap-alsa-sx1-mixer.h b/sound/arm/omap/omap-alsa-sx1-mixer.h
index e67f48a..02b8b6a 100644
--- a/sound/arm/omap/omap-alsa-sx1-mixer.h
+++ b/sound/arm/omap/omap-alsa-sx1-mixer.h
@@ -30,8 +30,8 @@ #define PLAYBACK_TARGET_CELLPHONE	0x02
 
 /* following are used for register 03h Mixer PGA control bits D7-D5 for selecting record source */
 #define REC_SRC_TARGET_COUNT		0x08
-#define REC_SRC_SINGLE_ENDED_MICIN_HED	0x00	// oss code referred to MIXER_LINE
-#define REC_SRC_SINGLE_ENDED_MICIN_HND	0x01	// oss code referred to MIXER_MIC
+#define REC_SRC_SINGLE_ENDED_MICIN_HED	0x00	/* oss code referred to MIXER_LINE */
+#define REC_SRC_SINGLE_ENDED_MICIN_HND	0x01	/* oss code referred to MIXER_MIC */
 #define REC_SRC_SINGLE_ENDED_AUX1	0x02
 #define REC_SRC_SINGLE_ENDED_AUX2	0x03
 #define REC_SRC_MICIN_HED_AND_AUX1	0x04
@@ -39,7 +39,7 @@ #define REC_SRC_MICIN_HED_AND_AUX2	0x05
 #define REC_SRC_MICIN_HND_AND_AUX1	0x06
 #define REC_SRC_MICIN_HND_AND_AUX2	0x07
 
-#define DEFAULT_OUTPUT_VOLUME		5	// default output volume to dac dgc
-#define DEFAULT_INPUT_VOLUME		2	// default record volume
+#define DEFAULT_OUTPUT_VOLUME		5	/* default output volume to dac dgc */
+#define DEFAULT_INPUT_VOLUME		2	/* default record volume */
 
-#endif /*OMAPALSATSC2101MIXER_H_*/
+#endif
diff --git a/sound/arm/omap/omap-alsa-sx1.c b/sound/arm/omap/omap-alsa-sx1.c
index 0edaf95..64c09dc 100644
--- a/sound/arm/omap/omap-alsa-sx1.c
+++ b/sound/arm/omap/omap-alsa-sx1.c
@@ -20,9 +20,7 @@ #include <asm/io.h>
 #include <asm/arch/mcbsp.h>
 
 #include <linux/slab.h>
-#ifdef CONFIG_PM
 #include <linux/pm.h>
-#endif
 #include <asm/mach-types.h>
 #include <asm/arch/dma.h>
 #include <asm/arch/clock.h>
@@ -31,18 +29,11 @@ #include <asm/arch/gpio.h>
 #include <asm/arch/omap-alsa.h>
 #include "omap-alsa-sx1.h"
 
-//#include <linux/skbuff.h>
 #include <linux/connector.h>
 
-//#define M_DPRINTK(ARGS...)  printk(KERN_INFO "<%s>: ",__FUNCTION__);printk(ARGS)
-#define M_DPRINTK(ARGS...)  		/* nop */
-
-//static struct clk *egold_mclk = 0;
-
 /* Connector implementation */
 static struct cb_id cn_sx1snd_id = { CN_IDX_SX1SND, CN_VAL_SX1SND };
 static char cn_sx1snd_name[] = "cn_sx1snd";
-//static struct sock *nls;
 
 void cn_sx1snd_callback(void *data)
 {
@@ -52,39 +43,39 @@ void cn_sx1snd_callback(void *data)
            __func__, jiffies, msg->id.idx, msg->id.val,
            msg->seq, msg->ack, msg->len, (char *)msg->data);
 }
+
 /* Send IPC message to sound server */
 extern int cn_sx1snd_send(unsigned int cmd, unsigned int arg1, unsigned int arg2)
 {
 	struct cn_msg *m;
 	unsigned short data[3];
-	int	err;
+	int err;
 
-	m = kmalloc(sizeof(*m) + sizeof(data), GFP_ATOMIC);
-	if (m) {
-		memset(m, 0, sizeof(*m) + sizeof(data));
+	m = kzalloc(sizeof(*m) + sizeof(data), GFP_ATOMIC);
+	if (!m)
+		return -1; 
 
-		memcpy(&m->id, &cn_sx1snd_id, sizeof(m->id));
-		m->seq = 1;//cn_test_timer_counter;
-		m->len = sizeof(data);
+	memcpy(&m->id, &cn_sx1snd_id, sizeof(m->id));
+	m->seq = 1;
+	m->len = sizeof(data);
 
-		data[0] = (unsigned short)cmd;
-		data[1] = (unsigned short)arg1;
-		data[2] = (unsigned short)arg2;
+	data[0] = (unsigned short)cmd;
+	data[1] = (unsigned short)arg1;
+	data[2] = (unsigned short)arg2;
 
-		memcpy(m + 1, data, m->len);
+	memcpy(m + 1, data, m->len);
 
-		err = cn_netlink_send(m, CN_IDX_SX1SND, gfp_any());
-		M_DPRINTK("sent= %02X %02X %02X, err=%d\n", cmd,arg1,arg2,err);
-		kfree(m);
-		if(err == -ESRCH)
-			return -1;	// there are no listeners on socket
-		return 0;
-	}
-	return -1;	// some error
+	err = cn_netlink_send(m, CN_IDX_SX1SND, gfp_any());
+	snd_printd("sent= %02X %02X %02X, err=%d\n", cmd,arg1,arg2,err);
+	kfree(m);
+
+	if (err == -ESRCH)
+		return -1;	/* there are no listeners on socket */
+	return 0;
 }
 
-/* * Hardware capabilities */
-/*
+/* Hardware capabilities
+ *
  * DAC USB-mode sampling rates (MCLK = 12 MHz)
  * The rates and rate_reg_into MUST be in the same order
  */
@@ -142,7 +133,7 @@ static snd_pcm_hardware_t egold_snd_omap
 	.fifo_size = 0,
 };
 
-static long current_rate = -1;// current rate in egold format 0..8
+static long current_rate = -1; /* current rate in egold format 0..8 */
 /*
  * ALSA operations according to board file
  */
@@ -154,14 +145,14 @@ void egold_set_samplerate(long sample_ra
 {
 	int egold_rate = 0;
 	int clkgdv = 0;
-
 	u16 srgr1, srgr2;
-	ADEBUG();
 
 	/* Set the sample rate */
-//	clkgdv	= CODEC_CLOCK / (sample_rate * (DEFAULT_BITPERSAMPLE * 2 - 1)); //fw15: 5005E490 - divs are different !!!
-	switch(sample_rate)
-	 {
+#if 0
+	/* fw15: 5005E490 - divs are different !!! */
+	clkgdv	= CODEC_CLOCK / (sample_rate * (DEFAULT_BITPERSAMPLE * 2 - 1));
+#endif
+	switch (sample_rate) {
 		case 8000:	clkgdv = 71; egold_rate = FRQ_8000; break;
 		case 11025:	clkgdv = 51; egold_rate = FRQ_11025; break;
 		case 12000:	clkgdv = 47; egold_rate = FRQ_12000; break;
@@ -171,7 +162,7 @@ void egold_set_samplerate(long sample_ra
 		case 32000:	clkgdv = 17; egold_rate = FRQ_32000; break;
 		case 44100:	clkgdv = 12; egold_rate = FRQ_44100; break;
 		case 48000:	clkgdv = 11; egold_rate = FRQ_48000; break;
-	 }
+	}
 
 	srgr1 = (FWID(DEFAULT_BITPERSAMPLE - 1) | CLKGDV(clkgdv));
 	srgr2 = ((FSGM | FPER(DEFAULT_BITPERSAMPLE * 2 - 1)));
@@ -179,7 +170,7 @@ void egold_set_samplerate(long sample_ra
 	OMAP_MCBSP_WRITE(OMAP1510_MCBSP1_BASE, SRGR2, srgr2);
 	OMAP_MCBSP_WRITE(OMAP1510_MCBSP1_BASE, SRGR1, srgr1);
 	current_rate = egold_rate;
-	M_DPRINTK("set samplerate=%ld\n", sample_rate);
+	snd_printd("set samplerate=%ld\n", sample_rate);
 
 }
 
@@ -201,8 +192,8 @@ void egold_configure(void)
 void egold_clock_setup(void)
 {
 	omap_request_gpio(OSC_EN);
-	omap_set_gpio_direction(OSC_EN, 0); // output pin
-	M_DPRINTK("\n");
+	omap_set_gpio_direction(OSC_EN, 0); /* output */
+	snd_printd("\n");
 }
 
 /*
@@ -211,10 +202,10 @@ void egold_clock_setup(void)
 int egold_clock_on(void)
 {
 	omap_set_gpio_dataout(OSC_EN, 1);
-	egold_set_samplerate(44100);// TODO
+	egold_set_samplerate(44100); /* TODO */
 	cn_sx1snd_send(DAC_SETAUDIODEVICE, SX1_DEVICE_SPEAKER, 0);
 	cn_sx1snd_send(DAC_OPEN_DEFAULT, current_rate , 4);
-	M_DPRINTK("\n");
+	snd_printd("\n");
 	return 0;
 }
 
@@ -226,37 +217,37 @@ int egold_clock_off(void)
 	cn_sx1snd_send(DAC_CLOSE, 0 , 0);
 	cn_sx1snd_send(DAC_SETAUDIODEVICE, SX1_DEVICE_PHONE, 0);
 	omap_set_gpio_dataout(OSC_EN, 0);
-	M_DPRINTK("\n");
+	snd_printd("\n");
 	return 0;
 }
 
 int egold_get_default_samplerate(void)
 {
-	M_DPRINTK("\n");
+	snd_printd("\n");
 	return DEFAULT_SAMPLE_RATE;
 }
 
 static int __init snd_omap_alsa_egold_probe(struct platform_device *pdev)
 {
-	int	ret;
-	struct	omap_alsa_codec_config *codec_cfg;
+	int ret;
+	struct omap_alsa_codec_config *codec_cfg;
 
 	codec_cfg = pdev->dev.platform_data;
-	if (codec_cfg != NULL) {
-		codec_cfg->hw_constraints_rates	= &egold_hw_constraints_rates;
-		codec_cfg->snd_omap_alsa_playback  = &egold_snd_omap_alsa_playback;
-		codec_cfg->snd_omap_alsa_capture  = &egold_snd_omap_alsa_capture;
-		codec_cfg->codec_configure_dev	= egold_configure;
-		codec_cfg->codec_set_samplerate	= egold_set_samplerate;
-		codec_cfg->codec_clock_setup	= egold_clock_setup;
-		codec_cfg->codec_clock_on	= egold_clock_on;
-		codec_cfg->codec_clock_off	= egold_clock_off;
-		codec_cfg->get_default_samplerate = egold_get_default_samplerate;
-		ret	= snd_omap_alsa_post_probe(pdev, codec_cfg);
-	}
-	else
-		ret = -ENODEV;
-	M_DPRINTK("\n");
+	if (!codec_cfg)
+		return -ENODEV;
+
+	codec_cfg->hw_constraints_rates	= &egold_hw_constraints_rates;
+	codec_cfg->snd_omap_alsa_playback  = &egold_snd_omap_alsa_playback;
+	codec_cfg->snd_omap_alsa_capture  = &egold_snd_omap_alsa_capture;
+	codec_cfg->codec_configure_dev	= egold_configure;
+	codec_cfg->codec_set_samplerate	= egold_set_samplerate;
+	codec_cfg->codec_clock_setup	= egold_clock_setup;
+	codec_cfg->codec_clock_on	= egold_clock_on;
+	codec_cfg->codec_clock_off	= egold_clock_off;
+	codec_cfg->get_default_samplerate = egold_get_default_samplerate;
+	ret = snd_omap_alsa_post_probe(pdev, codec_cfg);
+
+	snd_printd("\n");
 	return ret;
 }
 
@@ -273,7 +264,7 @@ static struct platform_driver omap_alsa_
 static int __init omap_alsa_egold_init(void)
 {
 	int	retval;
-	ADEBUG();
+
 	retval = cn_add_callback(&cn_sx1snd_id, cn_sx1snd_name, cn_sx1snd_callback);
 	if(retval)
 		printk(KERN_WARNING "cn_sx1snd failed to register\n");
@@ -282,7 +273,6 @@ static int __init omap_alsa_egold_init(v
 
 static void __exit omap_alsa_egold_exit(void)
 {
-	ADEBUG();
 	cn_del_callback(&cn_sx1snd_id);
 	platform_driver_unregister(&omap_alsa_driver);
 }
diff --git a/sound/arm/omap/omap-alsa-sx1.h b/sound/arm/omap/omap-alsa-sx1.h
index fdf55ee..34e26fc 100644
--- a/sound/arm/omap/omap-alsa-sx1.h
+++ b/sound/arm/omap/omap-alsa-sx1.h
@@ -27,14 +27,14 @@ #define DEFAULT_BITPERSAMPLE		16
 #endif
 
 #define DEFAULT_SAMPLE_RATE		44100
-// fw15: 18356000
+/* fw15: 18356000 */
 #define CODEC_CLOCK 			18359000
-// McBSP for playing music
+/* McBSP for playing music */
 #define AUDIO_MCBSP        		OMAP_MCBSP1
-// McBSP for record/play audio from phone and mic
+/* McBSP for record/play audio from phone and mic */
 #define AUDIO_MCBSP_PCM    		OMAP_MCBSP2
-// gpio pin for enable/disable clock
-#define OSC_EN					2
+/* gpio pin for enable/disable clock */
+#define OSC_EN				2
 
 /*
  * Defines codec specific functions pointers that can be used from the
@@ -49,25 +49,25 @@ int egold_get_default_samplerate(void);
 
 /* Send IPC message to sound server */
 extern int cn_sx1snd_send(unsigned int cmd, unsigned int arg1, unsigned int arg2);
-// cmd for IPC_GROUP_DAC
+/* cmd for IPC_GROUP_DAC */
 #define DAC_VOLUME_UPDATE		0
 #define DAC_SETAUDIODEVICE		1
 #define DAC_OPEN_RING			2
 #define DAC_OPEN_DEFAULT		3
-#define DAC_CLOSE				4
+#define DAC_CLOSE			4
 #define DAC_FMRADIO_OPEN		5
 #define DAC_FMRADIO_CLOSE		6
 #define DAC_PLAYTONE			7
-// cmd for IPC_GROUP_PCM
-#define PCM_PLAY				(0+8)
+/* cmd for IPC_GROUP_PCM */
+#define PCM_PLAY			(0+8)
 #define PCM_RECORD			(1+8)
-#define PCM_CLOSE				(2+8)
+#define PCM_CLOSE			(2+8)
 
-// for DAC_SETAUDIODEVICE
+/* for DAC_SETAUDIODEVICE */
 #define SX1_DEVICE_SPEAKER		0
 #define SX1_DEVICE_HEADPHONE		4
 #define SX1_DEVICE_PHONE		3
-// frequencies for MdaDacOpenDefaultL,  MdaDacOpenRingL
+/* frequencies for MdaDacOpenDefaultL,  MdaDacOpenRingL */
 #define FRQ_8000        0
 #define FRQ_11025       1
 #define FRQ_12000       2
@@ -78,6 +78,4 @@ #define FRQ_32000       6
 #define FRQ_44100       7
 #define FRQ_48000       8
 
- /* Netlink socket defs for connection with userspace MUX */
-
-#endif /*OMAP_ALSA_SX1_H_*/
+#endif

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux