asoc: codecs: resolve compilation time warnings
Sanjay Singh Rawat [Fri, 6 Apr 2012 13:24:06 +0000 (18:24 +0530)]
Bug 949219

Change-Id: I9c2a0aa22432c586a7e72273ad935d42332e873f
Signed-off-by: Sanjay Singh Rawat <srawat@nvidia.com>
Reviewed-on: http://git-master/r/95087
Reviewed-by: Rohan Somvanshi <rsomvanshi@nvidia.com>
Tested-by: Rohan Somvanshi <rsomvanshi@nvidia.com>

sound/soc/codecs/aic326x_tiload.c
sound/soc/codecs/base_main_Rate48_pps_driver.h
sound/soc/codecs/rt5639.c
sound/soc/codecs/second_rate_pps_driver.h
sound/soc/codecs/tlv320aic326x.c
sound/soc/codecs/tlv320aic326x.h
sound/soc/codecs/tlv320aic326x_mini-dsp.c
sound/soc/codecs/tlv320aic326x_minidsp_config.c
sound/soc/codecs/wm8903.c

index 00aa4d4..07615dc 100644 (file)
@@ -61,7 +61,8 @@ static void aic3262_dump_page(struct i2c_client *i2c, u8 page);
 /* externs */
 extern int aic3262_change_page(struct snd_soc_codec *codec, u8 new_page);
 extern int aic3262_change_book(struct snd_soc_codec *codec, u8 new_book);
-extern int aic3262_write(struct snd_soc_codec *codec, u16 reg, u8 value);
+extern int aic3262_write(struct snd_soc_codec *codec, unsigned int reg,
+                                               unsigned int value);
 int aic3262_driver_init(struct snd_soc_codec *codec);
 /************** Dynamic aic3262 driver, TI LOAD support  ***************/
 
@@ -236,7 +237,7 @@ static ssize_t tiload_write(struct file *file, const char __user * buf,
        return i2c_master_send(i2c, wr_data, count);
 }
 
-static int tiload_ioctl( struct file *filp,
+static long tiload_ioctl(struct file *filp,
                        unsigned int cmd, unsigned long arg)
 {
        int num = 0;
index 4d6f227..dff9185 100644 (file)
@@ -16,17 +16,6 @@ static control base_speaker_SRS_VOLUME_controls[] = {
 static char * base_speaker_SRS_VOLUME_control_names[] = {\r
 };\r
 \r
-/*//INSTRUCTIONS & COEFFICIENTS\r
-typedef struct {\r
-    u8 reg_off;\r
-    u8 reg_val;\r
-} reg_value;*/\r
-\r
-static char * base_speaker_SRS_REG_Section_names[] = {\r
-    "miniDSP_A_reg_values",\r
-    "miniDSP_D_reg_values",\r
-};\r
-\r
 reg_value base_speaker_SRS_REG_init_Section_program[] = {\r
     {  0,0x0},\r
     {  0x7F,0x00},\r
index e8841e0..c9b14e9 100644 (file)
@@ -1198,8 +1198,6 @@ static int spk_event(struct snd_soc_dapm_widget *w,
 static int hp_event(struct snd_soc_dapm_widget *w, 
        struct snd_kcontrol *kcontrol, int event)
 {
-       struct snd_soc_codec *codec = w->codec;
-
        switch (event) {
        case SND_SOC_DAPM_POST_PMU:
                printk("hp_event --SND_SOC_DAPM_POST_PMU\n");
index c6c128a..9956c08 100644 (file)
@@ -14,13 +14,6 @@ static control main44_VOLUME_controls[] = {
 static char *main44_VOLUME_control_names[] = {
 };
 
-
-
-static char *main44_REG_Section_names[] = {
-    "miniDSP_A_reg_values",
-    "miniDSP_D_reg_values",
-};
-
 reg_value main44_REG_Section_init_program[] = {
     {  0,0x0},
     {  0x7F,0x00},
index ce8b5bc..5d6b929 100644 (file)
@@ -81,9 +81,6 @@ static u8 aic3262_reg_ctl;
  * This function reprograms the clock dividers etc. this flag can be used to
  * disable this when the clock dividers are programmed by pps config file
  */
-static int soc_static_freq_config = 1;
-static struct aic3262_priv *aic3262_priv_data;
-static struct i2c_client *i2c_pdev;
 static struct snd_soc_codec *aic3262_codec;
 
 /*
@@ -849,7 +846,6 @@ static const struct aic3262_rate_divs aic3262_divs[] = {
 static void aic3262_multi_i2s_dump_regs(struct snd_soc_dai *dai)
 {
        struct snd_soc_codec *codec = dai->codec;
-       struct aic3262_priv *aic3262 = snd_soc_codec_get_drvdata(codec);
        unsigned int counter;
 
        DBG(KERN_INFO "#%s: Dai Active %d ASI%d REGS DUMP\n",
@@ -1965,7 +1961,7 @@ static int aic3262_multi_i2s_hw_params(struct snd_pcm_substream *substream,
 * We can use this function to disable the DAC and ADC specific inputs from the
 * individual ASI Ports of the Audio Codec.
 */
-static int aic3262_multi_i2s_shutdown(struct snd_pcm_substream *substream,
+static void aic3262_multi_i2s_shutdown(struct snd_pcm_substream *substream,
                        struct snd_soc_dai *dai)
 {
        struct snd_soc_pcm_runtime *rtd = substream->private_data;
@@ -2106,7 +2102,7 @@ static int aic3262_multi_i2s_shutdown(struct snd_pcm_substream *substream,
                        aic3262->active_count--;
        }
 err:
-       return 0;
+       return;
 }
 
 
@@ -3565,7 +3561,7 @@ void aic3262_write_reg_cache(struct snd_soc_codec *codec,
  *----------------------------------------------------------------------------
  */
 
-u8 aic3262_read(struct snd_soc_codec *codec, u16 reg)
+unsigned int aic3262_read(struct snd_soc_codec *codec, unsigned int reg)
 {
        struct aic3262_priv *aic3262 = snd_soc_codec_get_drvdata(codec);
        u8 value;
@@ -3573,7 +3569,7 @@ u8 aic3262_read(struct snd_soc_codec *codec, u16 reg)
        u16 *cache = codec->reg_cache;
        u16 cmd;
        u8 buffer[2];
-       int rc;
+       int rc = 0;
        reg = reg % 128;
 
        if (reg >= AIC3262_CACHEREGNUM) {
@@ -3619,7 +3615,8 @@ u8 aic3262_read(struct snd_soc_codec *codec, u16 reg)
  *
  *----------------------------------------------------------------------------
  */
-int aic3262_write(struct snd_soc_codec *codec, u16 reg, u8 value)
+int aic3262_write(struct snd_soc_codec *codec, unsigned int reg,
+                                               unsigned int value)
 {
        struct aic3262_priv *aic3262 = snd_soc_codec_get_drvdata(codec);
        u8 data[2];
@@ -3855,8 +3852,6 @@ int i2c_verify_book0(struct snd_soc_codec *codec)
 static int aic3262_set_bias_level(struct snd_soc_codec *codec,
                        enum snd_soc_bias_level level)
 {
-       struct aic3262_priv *aic3262 = snd_soc_codec_get_drvdata(codec);
-       u8 value;
        switch (level) {
                /* full On */
        case SND_SOC_BIAS_ON:
@@ -4400,6 +4395,7 @@ static int aic3262_spi_write(struct spi_device *spi, const char *data, int len)
        return len;
 }
 
+#ifdef RUN_DELAYED_WORK
 /*
  * This function forces any delayed work to be queued and run.
  */
@@ -4418,6 +4414,8 @@ static int run_delayed_work(struct delayed_work *dwork)
        }
        return ret;
 }
+#endif
+
 static int __devinit aic3262_spi_probe(struct spi_device *spi)
 {
        int ret;
index a31cc9e..fe1d4f1 100644 (file)
@@ -657,11 +657,12 @@ extern int aic326x_headset_detect(struct snd_soc_codec *codec,
 extern int aic326x_headset_button_init(struct snd_soc_codec *codec,
        struct snd_soc_jack *jack, int jack_type);
 
-extern u8 aic3262_read(struct snd_soc_codec *codec, u16 reg);
+extern unsigned int aic3262_read(struct snd_soc_codec *codec, unsigned int reg);
 extern u16 aic3262_read_2byte(struct snd_soc_codec *codec, u16 reg);
 extern int aic3262_reset_cache(struct snd_soc_codec *codec);
 extern int aic3262_change_page(struct snd_soc_codec *codec, u8 new_page);
-extern int aic3262_write(struct snd_soc_codec *codec, u16 reg, u8 value);
+extern int aic3262_write(struct snd_soc_codec *codec, unsigned int reg,
+                                                       unsigned int value);
 extern void aic3262_write_reg_cache(struct snd_soc_codec *codec,
                                    u16 reg, u8 value);
 extern int aic3262_change_book(struct snd_soc_codec *codec, u8 new_book);
index 6d55abb..0ef0fd0 100644 (file)
@@ -601,25 +601,27 @@ struct process_flow{
 int
 set_minidsp_mode(struct snd_soc_codec *codec, int new_mode, int new_config)
 {
-
-       if (codec == NULL) {
-       printk(KERN_INFO "%s codec is NULL\n",__func__);
-       }
-       struct aic3262_priv *aic326x = snd_soc_codec_get_drvdata(codec);
-       struct snd_soc_dapm_context *dapm = &codec->dapm;
+       struct aic3262_priv *aic326x;
+       struct snd_soc_dapm_context *dapm;
        struct process_flow *  pflows = &miniDSP_programs[new_mode];
-       u8 reg63, reg81, pll_pow, ndac_pow, mdac_pow, nadc_pow, madc_pow;
-
+       u8 pll_pow, ndac_pow, mdac_pow, nadc_pow;
        u8 adc_status,dac_status;
-       u8 reg, val;
-       u8 shift;
-       volatile u16 counter;
 
-       int (*ptransfer)(struct snd_soc_codec *codec,
-                               reg_value *program_ptr,
-                               int size);
+       int (*ptransfer)(struct snd_soc_codec *codec, reg_value *program_ptr,
+                                                               int size);
 
        printk("%s:New Switch mode = %d New Config= %d\n", __func__, new_mode,new_config);
+
+       if (codec == NULL) {
+               printk(KERN_INFO "%s codec is NULL\n", __func__);
+               return 0;
+       }
+       aic326x = snd_soc_codec_get_drvdata(codec);
+       dapm = &codec->dapm;
+
+       printk(KERN_INFO "%s:New Switch mode = %d New Config= %d\n", __func__,
+                                                       new_mode, new_config);
+
        if (new_mode >= ARRAY_SIZE(miniDSP_programs))
                return 0; //  error condition
                if (new_config > MAXCONFIG)
@@ -1231,7 +1233,7 @@ static int __new_control_put_minidsp_mux(struct snd_kcontrol *kcontrol,
        int ret_val = -1, array_size;
        control *array;
        char **array_names;
-       char *control_name, *control_name1, *control_name2;
+       char *control_name, *control_name1;
        struct aic3262_priv *aic326x = snd_soc_codec_get_drvdata(codec);
        i2c = codec->control_data;
 
index e34ffbe..97a41c3 100644 (file)
@@ -347,17 +347,12 @@ int aic3262_add_multiconfig_controls(struct snd_soc_codec *codec)
  *---------------------------------------------------------------------------
  */
 void minidsp_multiconfig(struct snd_soc_codec *codec,
-                                                                       reg_value *a_patch, int a_size,
-                                                                       reg_value *d_patch, int d_size)
+       reg_value *a_patch, int a_size, reg_value *d_patch, int d_size)
 {
-       struct aic3262_priv *aic326x = snd_soc_codec_get_drvdata(codec);
-       int val1,val2;
        int adc_status,dac_status;
        int (*ptransfer)(struct snd_soc_codec *codec,
-                               reg_value *program_ptr,
-                               int size);
-
-printk("======in the config_multiconfiguration function==== \n");
+                       reg_value *program_ptr, int size);
+       printk(KERN_INFO "======in the config_multiconfiguration function====\n");
 #ifndef MULTIBYTE_I2C
                ptransfer = byte_i2c_array_transfer;
 #else
index df4d0ee..2a2d168 100644 (file)
@@ -1404,6 +1404,7 @@ static struct {
        { 1500, 0x9, 0x2, 2 },
 };
 
+#ifdef SYS_BCLK_RATIO
 /* CLK_SYS/BCLK ratios - multiplied by 10 due to .5s */
 static struct {
        int ratio;
@@ -1427,6 +1428,7 @@ static struct {
        { 440, 19 },
        { 480, 20 },
 };
+#endif
 
 /* Sample rates for DSP */
 static struct {