1 files changed, 25 insertions, 25 deletions

diff --git a/sound/core/jack.c b/sound/core/jack.c
index dd4a12dc09aa..d54d1a05fe65 100644
--- a/sound/core/jack.c
+++ b/sound/core/jack.c
@@ -23,10 +23,21 @@
 #include <sound/jack.h>
 #include <sound/core.h>
 
+static int jack_types[] = {
+        SW_HEADPHONE_INSERT,
+        SW_MICROPHONE_INSERT,
+        SW_LINEOUT_INSERT,
+        SW_JACK_PHYSICAL_INSERT,
+        SW_VIDEOOUT_INSERT,
+};
+
 static int snd_jack_dev_free(struct snd_device *device)
 {
         struct snd_jack *jack = device->device_data;
 
+        if (jack->private_free)
+                jack->private_free(jack);
+
         /* If the input device is registered with the input subsystem
          * then we need to use a different deallocator. */
         if (jack->registered)
@@ -47,7 +58,7 @@ static int snd_jack_dev_register(struct snd_device *device)
         int err;
 
         snprintf(jack->name, sizeof(jack->name), "%s %s",
-                 card->longname, jack->id);
+                 card->shortname, jack->id);
         jack->input_dev->name = jack->name;
 
         /* Default to the sound card device. */
@@ -79,6 +90,7 @@ int snd_jack_new(struct snd_card *card, const char *id, int type,
 {
         struct snd_jack *jack;
         int err;
+        int i;
         static struct snd_device_ops ops = {
                 .dev_free = snd_jack_dev_free,
                 .dev_register = snd_jack_dev_register,
@@ -100,18 +112,10 @@ int snd_jack_new(struct snd_card *card, const char *id, int type,
 
         jack->type = type;
 
-        if (type & SND_JACK_HEADPHONE)
-                input_set_capability(jack->input_dev, EV_SW,
-                                     SW_HEADPHONE_INSERT);
-        if (type & SND_JACK_LINEOUT)
-                input_set_capability(jack->input_dev, EV_SW,
-                                     SW_LINEOUT_INSERT);
-        if (type & SND_JACK_MICROPHONE)
-                input_set_capability(jack->input_dev, EV_SW,
-                                     SW_MICROPHONE_INSERT);
-        if (type & SND_JACK_MECHANICAL)
-                input_set_capability(jack->input_dev, EV_SW,
-                                     SW_JACK_PHYSICAL_INSERT);
+        for (i = 0; i < ARRAY_SIZE(jack_types); i++)
+                if (type & (1 << i))
+                        input_set_capability(jack->input_dev, EV_SW,
+                                             jack_types[i]);
 
         err = snd_device_new(card, SNDRV_DEV_JACK, jack, &ops);
         if (err < 0)
@@ -154,21 +158,17 @@ EXPORT_SYMBOL(snd_jack_set_parent);
  */
 void snd_jack_report(struct snd_jack *jack, int status)
 {
+        int i;
+
         if (!jack)
                 return;
 
-        if (jack->type & SND_JACK_HEADPHONE)
-                input_report_switch(jack->input_dev, SW_HEADPHONE_INSERT,
-                                    status & SND_JACK_HEADPHONE);
-        if (jack->type & SND_JACK_LINEOUT)
-                input_report_switch(jack->input_dev, SW_LINEOUT_INSERT,
-                                    status & SND_JACK_LINEOUT);
-        if (jack->type & SND_JACK_MICROPHONE)
-                input_report_switch(jack->input_dev, SW_MICROPHONE_INSERT,
-                                    status & SND_JACK_MICROPHONE);
-        if (jack->type & SND_JACK_MECHANICAL)
-                input_report_switch(jack->input_dev, SW_JACK_PHYSICAL_INSERT,
-                                    status & SND_JACK_MECHANICAL);
+        for (i = 0; i < ARRAY_SIZE(jack_types); i++) {
+                int testbit = 1 << i;
+                if (jack->type & testbit)
+                        input_report_switch(jack->input_dev, jack_types[i],
+                                            status & testbit);
+        }
 
         input_sync(jack->input_dev);
 }