[media] em28xx: remove some unnecessary fields from struct em28xx_audio_mode
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Sat, 13 Sep 2014 08:52:19 +0000 (05:52 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 23 Sep 2014 19:13:41 +0000 (16:13 -0300)
Fields "ac97_feat", "ac97_vendor_id" and "i2s_samplerates" of struct
em28xx_audio_mode are used nowhere, except in function
em28xx_audio_setup(). So get rid of them and use local variables
instead.

Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/usb/em28xx/em28xx-core.c
drivers/media/usb/em28xx/em28xx.h

index 2f82e929a574a90197546bb9b89998201c1363a3..3bbab0e468de5d88b4d32d1bbe6665b345d38faf 100644 (file)
@@ -505,6 +505,7 @@ int em28xx_audio_setup(struct em28xx *dev)
 {
        int vid1, vid2, feat, cfg;
        u32 vid;
+       u8 i2s_samplerates;
 
        if (dev->chip_id == CHIP_ID_EM2870 ||
            dev->chip_id == CHIP_ID_EM2874 ||
@@ -534,15 +535,15 @@ int em28xx_audio_setup(struct em28xx *dev)
                if (dev->chip_id < CHIP_ID_EM2860 &&
                    (cfg & EM28XX_CHIPCFG_AUDIOMASK) ==
                    EM2820_CHIPCFG_I2S_1_SAMPRATE)
-                       dev->audio_mode.i2s_samplerates = 1;
+                       i2s_samplerates = 1;
                else if (dev->chip_id >= CHIP_ID_EM2860 &&
                         (cfg & EM28XX_CHIPCFG_AUDIOMASK) ==
                         EM2860_CHIPCFG_I2S_5_SAMPRATES)
-                       dev->audio_mode.i2s_samplerates = 5;
+                       i2s_samplerates = 5;
                else
-                       dev->audio_mode.i2s_samplerates = 3;
+                       i2s_samplerates = 3;
                em28xx_info("I2S Audio (%d sample rate(s))\n",
-                                              dev->audio_mode.i2s_samplerates);
+                                              i2s_samplerates);
                /* Skip the code that does AC97 vendor detection */
                dev->audio_mode.ac97 = EM28XX_NO_AC97;
                goto init_audio;
@@ -569,15 +570,12 @@ int em28xx_audio_setup(struct em28xx *dev)
                goto init_audio;
 
        vid = vid1 << 16 | vid2;
-
-       dev->audio_mode.ac97_vendor_id = vid;
        em28xx_warn("AC97 vendor ID = 0x%08x\n", vid);
 
        feat = em28xx_read_ac97(dev, AC97_RESET);
        if (feat < 0)
                goto init_audio;
 
-       dev->audio_mode.ac97_feat = feat;
        em28xx_warn("AC97 features = 0x%04x\n", feat);
 
        /* Try to identify what audio processor we have */
@@ -596,8 +594,8 @@ init_audio:
                em28xx_info("Empia 202 AC97 audio processor detected\n");
                break;
        case EM28XX_AC97_SIGMATEL:
-               em28xx_info("Sigmatel audio processor detected(stac 97%02x)\n",
-                           dev->audio_mode.ac97_vendor_id & 0xff);
+               em28xx_info("Sigmatel audio processor detected (stac 97%02x)\n",
+                           vid & 0xff);
                break;
        case EM28XX_AC97_OTHER:
                em28xx_warn("Unknown AC97 audio processor detected!\n");
index 84ef8efdb148c1dcf81d5be799a2f819bde3d3f5..1f38163352b668e4723051a9af5cf9a3d088c816 100644 (file)
@@ -309,13 +309,7 @@ enum em28xx_ac97_mode {
 
 struct em28xx_audio_mode {
        enum em28xx_ac97_mode ac97;
-
-       u16 ac97_feat;
-       u32 ac97_vendor_id;
-
        unsigned int has_audio:1;
-
-       u8 i2s_samplerates;
 };
 
 /* em28xx has two audio inputs: tuner and line in.