169 files changed, 555 insertions, 2073 deletions
diff --git a/drivers/media/common/tuners/mxl5007t.c b/drivers/media/common/tuners/mxl5007t.c
index 7eb1bf75cd0..5d02221e99d 100644
--- a/drivers/media/common/tuners/mxl5007t.c
+++ b/drivers/media/common/tuners/mxl5007t.c
@@ -488,9 +488,10 @@ static int mxl5007t_write_regs(struct mxl5007t_state *state,
 static int mxl5007t_read_reg(struct mxl5007t_state *state, u8 reg, u8 *val)
 {
+        u8 buf[2] = { 0xfb, reg };
        struct i2c_msg msg[] = {
                { .addr = state->i2c_props.addr, .flags = 0,
-                  .buf = &reg, .len = 1 },
+                  .buf = buf, .len = 2 },
                { .addr = state->i2c_props.addr, .flags = I2C_M_RD,
                  .buf = val, .len = 1 },
        };
diff --git a/drivers/media/common/tuners/tda18218.c b/drivers/media/common/tuners/tda18218.c
index aacfe2387e2..4fc29730a12 100644
--- a/drivers/media/common/tuners/tda18218.c
+++ b/drivers/media/common/tuners/tda18218.c
@@ -141,7 +141,7 @@ static int tda18218_set_params(struct dvb_frontend *fe,
        switch (params->u.ofdm.bandwidth) {
        case BANDWIDTH_6_MHZ:
                LP_Fc = 0;
-                LO_Frac = params->frequency + 4000000;
+                LO_Frac = params->frequency + 3000000;
                break;
        case BANDWIDTH_7_MHZ:
                LP_Fc = 1;
diff --git a/drivers/media/dvb/b2c2/flexcop-usb.c b/drivers/media/dvb/b2c2/flexcop-usb.c
index bedcfb67162..26c666dd351 100644
--- a/drivers/media/dvb/b2c2/flexcop-usb.c
+++ b/drivers/media/dvb/b2c2/flexcop-usb.c
@@ -583,25 +583,7 @@ static struct usb_driver flexcop_usb_driver = {
        .id_table       = flexcop_usb_table,
 };
-/* module stuff */
+module_usb_driver(flexcop_usb_driver);
-static int __init flexcop_usb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&flexcop_usb_driver))) {
-                err("usb_register failed. (%d)", result);
-                return result;
-        }
-        return 0;
-}
-static void __exit flexcop_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&flexcop_usb_driver);
-}
-module_init(flexcop_usb_module_init);
-module_exit(flexcop_usb_module_exit);
 MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_NAME);
diff --git a/drivers/media/dvb/ddbridge/ddbridge-core.c b/drivers/media/dvb/ddbridge/ddbridge-core.c
index ba9a643b9c6..d1e91bc80e7 100644
--- a/drivers/media/dvb/ddbridge/ddbridge-core.c
+++ b/drivers/media/dvb/ddbridge/ddbridge-core.c
@@ -1480,7 +1480,7 @@ static const struct file_operations ddb_fops = {
        .open           = ddb_open,
 };
-static char *ddb_devnode(struct device *device, mode_t *mode)
+static char *ddb_devnode(struct device *device, umode_t *mode)
 {
        struct ddb *dev = dev_get_drvdata(device);
diff --git a/drivers/media/dvb/dvb-core/dvbdev.c b/drivers/media/dvb/dvb-core/dvbdev.c
index f7328777595..00a67326c19 100644
--- a/drivers/media/dvb/dvb-core/dvbdev.c
+++ b/drivers/media/dvb/dvb-core/dvbdev.c
@@ -450,7 +450,7 @@ static int dvb_uevent(struct device *dev, struct kobj_uevent_env *env)
        return 0;
 }
-static char *dvb_devnode(struct device *dev, mode_t *mode)
+static char *dvb_devnode(struct device *dev, umode_t *mode)
 {
        struct dvb_device *dvbdev = dev_get_drvdata(dev);
diff --git a/drivers/media/dvb/dvb-usb/a800.c b/drivers/media/dvb/dvb-usb/a800.c
index 2aef3c89e9f..8d7fef84afd 100644
--- a/drivers/media/dvb/dvb-usb/a800.c
+++ b/drivers/media/dvb/dvb-usb/a800.c
@@ -183,26 +183,7 @@ static struct usb_driver a800_driver = {
        .id_table       = a800_table,
 };
-/* module stuff */
+module_usb_driver(a800_driver);
-static int __init a800_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&a800_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit a800_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&a800_driver);
-}
-module_init (a800_module_init);
-module_exit (a800_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("AVerMedia AverTV DVB-T USB 2.0 (A800)");
diff --git a/drivers/media/dvb/dvb-usb/af9005.c b/drivers/media/dvb/dvb-usb/af9005.c
index bd51a764351..4fc024d7704 100644
--- a/drivers/media/dvb/dvb-usb/af9005.c
+++ b/drivers/media/dvb/dvb-usb/af9005.c
@@ -30,7 +30,7 @@ MODULE_PARM_DESC(debug,
                 "set debugging level (1=info,xfer=2,rc=4,reg=8,i2c=16,fw=32 (or-able))."
                 DVB_USB_DEBUG_STATUS);
 /* enable obnoxious led */
-int dvb_usb_af9005_led = 1;
+bool dvb_usb_af9005_led = 1;
 module_param_named(led, dvb_usb_af9005_led, bool, 0644);
 MODULE_PARM_DESC(led, "enable led (default: 1).");
diff --git a/drivers/media/dvb/dvb-usb/af9005.h b/drivers/media/dvb/dvb-usb/af9005.h
index c71c77bd7f4..6a2bf3de845 100644
--- a/drivers/media/dvb/dvb-usb/af9005.h
+++ b/drivers/media/dvb/dvb-usb/af9005.h
@@ -35,7 +35,7 @@ extern int dvb_usb_af9005_debug;
 #define deb_i2c(args...)  dprintk(dvb_usb_af9005_debug,0x10,args)
 #define deb_fw(args...)   dprintk(dvb_usb_af9005_debug,0x20,args)
-extern int dvb_usb_af9005_led;
+extern bool dvb_usb_af9005_led;
 /* firmware */
 #define FW_BULKOUT_SIZE 250
diff --git a/drivers/media/dvb/dvb-usb/af9015.c b/drivers/media/dvb/dvb-usb/af9015.c
index c6c275bac08..56cbd3636c3 100644
--- a/drivers/media/dvb/dvb-usb/af9015.c
+++ b/drivers/media/dvb/dvb-usb/af9015.c
@@ -1713,25 +1713,7 @@ static struct usb_driver af9015_usb_driver = {
        .id_table = af9015_usb_table,
 };
-/* module stuff */
+module_usb_driver(af9015_usb_driver);
-static int __init af9015_usb_module_init(void)
-{
-        int ret;
-        ret = usb_register(&af9015_usb_driver);
-        if (ret)
-                err("module init failed:%d", ret);
-        return ret;
-}
-static void __exit af9015_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&af9015_usb_driver);
-}
-module_init(af9015_usb_module_init);
-module_exit(af9015_usb_module_exit);
 MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>");
 MODULE_DESCRIPTION("Driver for Afatech AF9015 DVB-T");
diff --git a/drivers/media/dvb/dvb-usb/anysee.c b/drivers/media/dvb/dvb-usb/anysee.c
index 5f2278b73ee..b39f14f85e7 100644
--- a/drivers/media/dvb/dvb-usb/anysee.c
+++ b/drivers/media/dvb/dvb-usb/anysee.c
@@ -1091,26 +1091,7 @@ static struct usb_driver anysee_driver = {
        .id_table   = anysee_table,
 };
-/* module stuff */
+module_usb_driver(anysee_driver);
-static int __init anysee_module_init(void)
-{
-        int ret;
-        ret = usb_register(&anysee_driver);
-        if (ret)
-                err("%s: usb_register failed. Error number %d", __func__, ret);
-        return ret;
-}
-static void __exit anysee_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&anysee_driver);
-}
-module_init(anysee_module_init);
-module_exit(anysee_module_exit);
 MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>");
 MODULE_DESCRIPTION("Driver Anysee E30 DVB-C & DVB-T USB2.0");
diff --git a/drivers/media/dvb/dvb-usb/au6610.c b/drivers/media/dvb/dvb-usb/au6610.c
index b77994967b9..16210c06030 100644
--- a/drivers/media/dvb/dvb-usb/au6610.c
+++ b/drivers/media/dvb/dvb-usb/au6610.c
@@ -244,26 +244,7 @@ static struct usb_driver au6610_driver = {
        .id_table   = au6610_table,
 };
-/* module stuff */
+module_usb_driver(au6610_driver);
-static int __init au6610_module_init(void)
-{
-        int ret;
-        ret = usb_register(&au6610_driver);
-        if (ret)
-                err("usb_register failed. Error number %d", ret);
-        return ret;
-}
-static void __exit au6610_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&au6610_driver);
-}
-module_init(au6610_module_init);
-module_exit(au6610_module_exit);
 MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>");
 MODULE_DESCRIPTION("Driver for Alcor Micro AU6610 DVB-T USB2.0");
diff --git a/drivers/media/dvb/dvb-usb/az6027.c b/drivers/media/dvb/dvb-usb/az6027.c
index bf67b4dfd82..5e45ae60542 100644
--- a/drivers/media/dvb/dvb-usb/az6027.c
+++ b/drivers/media/dvb/dvb-usb/az6027.c
@@ -1174,28 +1174,7 @@ static struct usb_driver az6027_usb_driver = {
        .id_table       = az6027_usb_table,
 };
-/* module stuff */
+module_usb_driver(az6027_usb_driver);
-static int __init az6027_usb_module_init(void)
-{
-        int result;
-        result = usb_register(&az6027_usb_driver);
-        if (result) {
-                err("usb_register failed. (%d)", result);
-                return result;
-        }
-        return 0;
-}
-static void __exit az6027_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&az6027_usb_driver);
-}
-module_init(az6027_usb_module_init);
-module_exit(az6027_usb_module_exit);
 MODULE_AUTHOR("Adams Xu <Adams.xu@azwave.com.cn>");
 MODULE_DESCRIPTION("Driver for AZUREWAVE DVB-S/S2 USB2.0 (AZ6027)");
diff --git a/drivers/media/dvb/dvb-usb/ce6230.c b/drivers/media/dvb/dvb-usb/ce6230.c
index 57afb5a9157..fa637255729 100644
--- a/drivers/media/dvb/dvb-usb/ce6230.c
+++ b/drivers/media/dvb/dvb-usb/ce6230.c
@@ -317,27 +317,7 @@ static struct usb_driver ce6230_driver = {
        .id_table   = ce6230_table,
 };
-/* module stuff */
+module_usb_driver(ce6230_driver);
-static int __init ce6230_module_init(void)
-{
-        int ret;
-        deb_info("%s:\n", __func__);
-        ret = usb_register(&ce6230_driver);
-        if (ret)
-                err("usb_register failed with error:%d", ret);
-        return ret;
-}
-static void __exit ce6230_module_exit(void)
-{
-        deb_info("%s:\n", __func__);
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&ce6230_driver);
-}
-module_init(ce6230_module_init);
-module_exit(ce6230_module_exit);
 MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>");
 MODULE_DESCRIPTION("Driver for Intel CE6230 DVB-T USB2.0");
diff --git a/drivers/media/dvb/dvb-usb/cinergyT2-core.c b/drivers/media/dvb/dvb-usb/cinergyT2-core.c
index f9d905002ec..0a98548ecd1 100644
--- a/drivers/media/dvb/dvb-usb/cinergyT2-core.c
+++ b/drivers/media/dvb/dvb-usb/cinergyT2-core.c
@@ -247,25 +247,7 @@ static struct usb_driver cinergyt2_driver = {
        .id_table       = cinergyt2_usb_table
 };
-static int __init cinergyt2_usb_init(void)
+module_usb_driver(cinergyt2_driver);
-{
-        int err;
-        err = usb_register(&cinergyt2_driver);
-        if (err) {
-                err("usb_register() failed! (err %i)\n", err);
-                return err;
-        }
-        return 0;
-}
-static void __exit cinergyt2_usb_exit(void)
-{
-        usb_deregister(&cinergyt2_driver);
-}
-module_init(cinergyt2_usb_init);
-module_exit(cinergyt2_usb_exit);
 MODULE_DESCRIPTION("Terratec Cinergy T2 DVB-T driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/media/dvb/dvb-usb/cxusb.c b/drivers/media/dvb/dvb-usb/cxusb.c
index 9f2a02c4837..949ea1bc0aa 100644
--- a/drivers/media/dvb/dvb-usb/cxusb.c
+++ b/drivers/media/dvb/dvb-usb/cxusb.c
@@ -2034,26 +2034,7 @@ static struct usb_driver cxusb_driver = {
        .id_table       = cxusb_table,
 };
-/* module stuff */
+module_usb_driver(cxusb_driver);
-static int __init cxusb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&cxusb_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit cxusb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&cxusb_driver);
-}
-module_init (cxusb_module_init);
-module_exit (cxusb_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_AUTHOR("Michael Krufky <mkrufky@linuxtv.org>");
diff --git a/drivers/media/dvb/dvb-usb/dib0700_core.c b/drivers/media/dvb/dvb-usb/dib0700_core.c
index 156cbfc9c79..206999476f0 100644
--- a/drivers/media/dvb/dvb-usb/dib0700_core.c
+++ b/drivers/media/dvb/dvb-usb/dib0700_core.c
@@ -832,27 +832,7 @@ static struct usb_driver dib0700_driver = {
        .id_table   = dib0700_usb_id_table,
 };
-/* module stuff */
+module_usb_driver(dib0700_driver);
-static int __init dib0700_module_init(void)
-{
-        int result;
-        info("loaded with support for %d different device-types", dib0700_device_count);
-        if ((result = usb_register(&dib0700_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit dib0700_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&dib0700_driver);
-}
-module_init (dib0700_module_init);
-module_exit (dib0700_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <pboettcher@dibcom.fr>");
 MODULE_DESCRIPTION("Driver for devices based on DiBcom DiB0700 - USB bridge");
diff --git a/drivers/media/dvb/dvb-usb/dibusb-mb.c b/drivers/media/dvb/dvb-usb/dibusb-mb.c
index 7270791f834..a4ac37e0e98 100644
--- a/drivers/media/dvb/dvb-usb/dibusb-mb.c
+++ b/drivers/media/dvb/dvb-usb/dibusb-mb.c
@@ -463,26 +463,7 @@ static struct usb_driver dibusb_driver = {
        .id_table       = dibusb_dib3000mb_table,
 };
-/* module stuff */
+module_usb_driver(dibusb_driver);
-static int __init dibusb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&dibusb_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit dibusb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&dibusb_driver);
-}
-module_init (dibusb_module_init);
-module_exit (dibusb_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for DiBcom USB DVB-T devices (DiB3000M-B based)");
diff --git a/drivers/media/dvb/dvb-usb/dibusb-mc.c b/drivers/media/dvb/dvb-usb/dibusb-mc.c
index 9c165e2569d..9d1a59d09c5 100644
--- a/drivers/media/dvb/dvb-usb/dibusb-mc.c
+++ b/drivers/media/dvb/dvb-usb/dibusb-mc.c
@@ -141,26 +141,7 @@ static struct usb_driver dibusb_mc_driver = {
        .id_table       = dibusb_dib3000mc_table,
 };
-/* module stuff */
+module_usb_driver(dibusb_mc_driver);
-static int __init dibusb_mc_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&dibusb_mc_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit dibusb_mc_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&dibusb_mc_driver);
-}
-module_init (dibusb_mc_module_init);
-module_exit (dibusb_mc_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for DiBcom USB2.0 DVB-T (DiB3000M-C/P based) devices");
diff --git a/drivers/media/dvb/dvb-usb/digitv.c b/drivers/media/dvb/dvb-usb/digitv.c
index f7184111aa6..0a9a79820f2 100644
--- a/drivers/media/dvb/dvb-usb/digitv.c
+++ b/drivers/media/dvb/dvb-usb/digitv.c
@@ -346,26 +346,7 @@ static struct usb_driver digitv_driver = {
        .id_table       = digitv_table,
 };
-/* module stuff */
+module_usb_driver(digitv_driver);
-static int __init digitv_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&digitv_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit digitv_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&digitv_driver);
-}
-module_init (digitv_module_init);
-module_exit (digitv_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for Nebula Electronics uDigiTV DVB-T USB2.0");
diff --git a/drivers/media/dvb/dvb-usb/dtt200u.c b/drivers/media/dvb/dvb-usb/dtt200u.c
index 106dfd55ff9..66f205c112b 100644
--- a/drivers/media/dvb/dvb-usb/dtt200u.c
+++ b/drivers/media/dvb/dvb-usb/dtt200u.c
@@ -360,26 +360,7 @@ static struct usb_driver dtt200u_usb_driver = {
        .id_table       = dtt200u_usb_table,
 };
-/* module stuff */
+module_usb_driver(dtt200u_usb_driver);
-static int __init dtt200u_usb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&dtt200u_usb_driver))) {
-                err("usb_register failed. (%d)",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit dtt200u_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&dtt200u_usb_driver);
-}
-module_init(dtt200u_usb_module_init);
-module_exit(dtt200u_usb_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for the WideView/Yakumo/Hama/Typhoon/Club3D/Miglia DVB-T USB2.0 devices");
diff --git a/drivers/media/dvb/dvb-usb/dtv5100.c b/drivers/media/dvb/dvb-usb/dtv5100.c
index 7373132163d..3d11df41cac 100644
--- a/drivers/media/dvb/dvb-usb/dtv5100.c
+++ b/drivers/media/dvb/dvb-usb/dtv5100.c
@@ -217,26 +217,7 @@ static struct usb_driver dtv5100_driver = {
        .id_table       = dtv5100_table,
 };
-/* module stuff */
+module_usb_driver(dtv5100_driver);
-static int __init dtv5100_module_init(void)
-{
-        int ret;
-        ret = usb_register(&dtv5100_driver);
-        if (ret)
-                err("usb_register failed. Error number %d", ret);
-        return ret;
-}
-static void __exit dtv5100_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&dtv5100_driver);
-}
-module_init(dtv5100_module_init);
-module_exit(dtv5100_module_exit);
 MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_DESC);
diff --git a/drivers/media/dvb/dvb-usb/dw2102.c b/drivers/media/dvb/dvb-usb/dw2102.c
index f103ec1fe82..ff941d20e6b 100644
--- a/drivers/media/dvb/dvb-usb/dw2102.c
+++ b/drivers/media/dvb/dvb-usb/dw2102.c
@@ -1928,22 +1928,7 @@ static struct usb_driver dw2102_driver = {
        .id_table = dw2102_table,
 };
-static int __init dw2102_module_init(void)
+module_usb_driver(dw2102_driver);
-{
-        int ret =  usb_register(&dw2102_driver);
-        if (ret)
-                err("usb_register failed. Error number %d", ret);
-        return ret;
-}
-static void __exit dw2102_module_exit(void)
-{
-        usb_deregister(&dw2102_driver);
-}
-module_init(dw2102_module_init);
-module_exit(dw2102_module_exit);
 MODULE_AUTHOR("Igor M. Liplianin (c) liplianin@me.by");
 MODULE_DESCRIPTION("Driver for DVBWorld DVB-S 2101, 2102, DVB-S2 2104,"
diff --git a/drivers/media/dvb/dvb-usb/ec168.c b/drivers/media/dvb/dvb-usb/ec168.c
index 78442fe4aa5..b4989ba8897 100644
--- a/drivers/media/dvb/dvb-usb/ec168.c
+++ b/drivers/media/dvb/dvb-usb/ec168.c
@@ -428,27 +428,7 @@ static struct usb_driver ec168_driver = {
        .id_table   = ec168_id,
 };
-/* module stuff */
+module_usb_driver(ec168_driver);
-static int __init ec168_module_init(void)
-{
-        int ret;
-        deb_info("%s:\n", __func__);
-        ret = usb_register(&ec168_driver);
-        if (ret)
-                err("module init failed:%d", ret);
-        return ret;
-}
-static void __exit ec168_module_exit(void)
-{
-        deb_info("%s:\n", __func__);
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&ec168_driver);
-}
-module_init(ec168_module_init);
-module_exit(ec168_module_exit);
 MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>");
 MODULE_DESCRIPTION("E3C EC168 DVB-T USB2.0 driver");
diff --git a/drivers/media/dvb/dvb-usb/friio.c b/drivers/media/dvb/dvb-usb/friio.c
index b092dc2137c..474a17e4db0 100644
--- a/drivers/media/dvb/dvb-usb/friio.c
+++ b/drivers/media/dvb/dvb-usb/friio.c
@@ -514,28 +514,7 @@ static struct usb_driver friio_driver = {
        .id_table       = friio_table,
 };
+module_usb_driver(friio_driver);
-/* module stuff */
-static int __init friio_module_init(void)
-{
-        int ret;
-        ret = usb_register(&friio_driver);
-        if (ret)
-                err("usb_register failed. Error number %d", ret);
-        return ret;
-}
-static void __exit friio_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&friio_driver);
-}
-module_init(friio_module_init);
-module_exit(friio_module_exit);
 MODULE_AUTHOR("Akihiro Tsukada <tskd2@yahoo.co.jp>");
 MODULE_DESCRIPTION("Driver for Friio ISDB-T USB2.0 Receiver");
diff --git a/drivers/media/dvb/dvb-usb/gl861.c b/drivers/media/dvb/dvb-usb/gl861.c
index 63681df244c..c1f5582e1cd 100644
--- a/drivers/media/dvb/dvb-usb/gl861.c
+++ b/drivers/media/dvb/dvb-usb/gl861.c
@@ -209,26 +209,7 @@ static struct usb_driver gl861_driver = {
        .id_table       = gl861_table,
 };
-/* module stuff */
+module_usb_driver(gl861_driver);
-static int __init gl861_module_init(void)
-{
-        int ret;
-        ret = usb_register(&gl861_driver);
-        if (ret)
-                err("usb_register failed. Error number %d", ret);
-        return ret;
-}
-static void __exit gl861_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&gl861_driver);
-}
-module_init(gl861_module_init);
-module_exit(gl861_module_exit);
 MODULE_AUTHOR("Carl Lundqvist <comabug@gmail.com>");
 MODULE_DESCRIPTION("Driver MSI Mega Sky 580 DVB-T USB2.0 / GL861");
diff --git a/drivers/media/dvb/dvb-usb/gp8psk.c b/drivers/media/dvb/dvb-usb/gp8psk.c
index 5f71284703d..5d0384dd45b 100644
--- a/drivers/media/dvb/dvb-usb/gp8psk.c
+++ b/drivers/media/dvb/dvb-usb/gp8psk.c
@@ -320,26 +320,7 @@ static struct usb_driver gp8psk_usb_driver = {
        .id_table       = gp8psk_usb_table,
 };
-/* module stuff */
+module_usb_driver(gp8psk_usb_driver);
-static int __init gp8psk_usb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&gp8psk_usb_driver))) {
-                err("usb_register failed. (%d)",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit gp8psk_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&gp8psk_usb_driver);
-}
-module_init(gp8psk_usb_module_init);
-module_exit(gp8psk_usb_module_exit);
 MODULE_AUTHOR("Alan Nisota <alannisota@gamil.com>");
 MODULE_DESCRIPTION("Driver for Genpix DVB-S");
diff --git a/drivers/media/dvb/dvb-usb/it913x.c b/drivers/media/dvb/dvb-usb/it913x.c
index c4622618714..67094b879bb 100644
--- a/drivers/media/dvb/dvb-usb/it913x.c
+++ b/drivers/media/dvb/dvb-usb/it913x.c
@@ -675,26 +675,7 @@ static struct usb_driver it913x_driver = {
        .id_table       = it913x_table,
 };
-/* module stuff */
+module_usb_driver(it913x_driver);
-static int __init it913x_module_init(void)
-{
-        int result = usb_register(&it913x_driver);
-        if (result) {
-                err("usb_register failed. Error number %d", result);
-                return result;
-        }
-        return 0;
-}
-static void __exit it913x_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&it913x_driver);
-}
-module_init(it913x_module_init);
-module_exit(it913x_module_exit);
 MODULE_AUTHOR("Malcolm Priestley <tvboxspy@gmail.com>");
 MODULE_DESCRIPTION("it913x USB 2 Driver");
diff --git a/drivers/media/dvb/dvb-usb/lmedm04.c b/drivers/media/dvb/dvb-usb/lmedm04.c
index b9228240f5c..1a876a65ed5 100644
--- a/drivers/media/dvb/dvb-usb/lmedm04.c
+++ b/drivers/media/dvb/dvb-usb/lmedm04.c
@@ -1289,26 +1289,7 @@ static struct usb_driver lme2510_driver = {
        .id_table       = lme2510_table,
 };
-/* module stuff */
+module_usb_driver(lme2510_driver);
-static int __init lme2510_module_init(void)
-{
-        int result = usb_register(&lme2510_driver);
-        if (result) {
-                err("usb_register failed. Error number %d", result);
-                return result;
-        }
-        return 0;
-}
-static void __exit lme2510_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&lme2510_driver);
-}
-module_init(lme2510_module_init);
-module_exit(lme2510_module_exit);
 MODULE_AUTHOR("Malcolm Priestley <tvboxspy@gmail.com>");
 MODULE_DESCRIPTION("LME2510(C) DVB-S USB2.0");
diff --git a/drivers/media/dvb/dvb-usb/m920x.c b/drivers/media/dvb/dvb-usb/m920x.c
index a1e1287c949..288af29a8bb 100644
--- a/drivers/media/dvb/dvb-usb/m920x.c
+++ b/drivers/media/dvb/dvb-usb/m920x.c
@@ -1086,27 +1086,7 @@ static struct usb_driver m920x_driver = {
        .id_table       = m920x_table,
 };
-/* module stuff */
+module_usb_driver(m920x_driver);
-static int __init m920x_module_init(void)
-{
-        int ret;
-        if ((ret = usb_register(&m920x_driver))) {
-                err("usb_register failed. Error number %d", ret);
-                return ret;
-        }
-        return 0;
-}
-static void __exit m920x_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&m920x_driver);
-}
-module_init (m920x_module_init);
-module_exit (m920x_module_exit);
 MODULE_AUTHOR("Aapo Tahkola <aet@rasterburn.org>");
 MODULE_DESCRIPTION("DVB Driver for ULI M920x");
diff --git a/drivers/media/dvb/dvb-usb/mxl111sf.c b/drivers/media/dvb/dvb-usb/mxl111sf.c
index b5c98da5d9e..825a8b242e0 100644
--- a/drivers/media/dvb/dvb-usb/mxl111sf.c
+++ b/drivers/media/dvb/dvb-usb/mxl111sf.c
@@ -1055,24 +1055,7 @@ static struct usb_driver mxl111sf_driver = {
        .id_table       = mxl111sf_table,
 };
-static int __init mxl111sf_module_init(void)
+module_usb_driver(mxl111sf_driver);
-{
-        int result = usb_register(&mxl111sf_driver);
-        if (result) {
-                err("usb_register failed. Error number %d", result);
-                return result;
-        }
-        return 0;
-}
-static void __exit mxl111sf_module_exit(void)
-{
-        usb_deregister(&mxl111sf_driver);
-}
-module_init(mxl111sf_module_init);
-module_exit(mxl111sf_module_exit);
 MODULE_AUTHOR("Michael Krufky <mkrufky@kernellabs.com>");
 MODULE_DESCRIPTION("Driver for MaxLinear MxL111SF");
diff --git a/drivers/media/dvb/dvb-usb/nova-t-usb2.c b/drivers/media/dvb/dvb-usb/nova-t-usb2.c
index 21384da6570..6c55384e2fc 100644
--- a/drivers/media/dvb/dvb-usb/nova-t-usb2.c
+++ b/drivers/media/dvb/dvb-usb/nova-t-usb2.c
@@ -225,26 +225,7 @@ static struct usb_driver nova_t_driver = {
        .id_table       = nova_t_table,
 };
-/* module stuff */
+module_usb_driver(nova_t_driver);
-static int __init nova_t_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&nova_t_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit nova_t_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&nova_t_driver);
-}
-module_init (nova_t_module_init);
-module_exit (nova_t_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Hauppauge WinTV-NOVA-T usb2");
diff --git a/drivers/media/dvb/dvb-usb/opera1.c b/drivers/media/dvb/dvb-usb/opera1.c
index 98fd9a6092b..c8a95042dfb 100644
--- a/drivers/media/dvb/dvb-usb/opera1.c
+++ b/drivers/media/dvb/dvb-usb/opera1.c
@@ -574,22 +574,7 @@ static struct usb_driver opera1_driver = {
        .id_table = opera1_table,
 };
-static int __init opera1_module_init(void)
+module_usb_driver(opera1_driver);
-{
-        int result = 0;
-        if ((result = usb_register(&opera1_driver))) {
-                err("usb_register failed. Error number %d", result);
-        }
-        return result;
-}
-static void __exit opera1_module_exit(void)
-{
-        usb_deregister(&opera1_driver);
-}
-module_init(opera1_module_init);
-module_exit(opera1_module_exit);
 MODULE_AUTHOR("Mario Hlawitschka (c) dh1pa@amsat.org");
 MODULE_AUTHOR("Marco Gittler (c) g.marco@freenet.de");
diff --git a/drivers/media/dvb/dvb-usb/pctv452e.c b/drivers/media/dvb/dvb-usb/pctv452e.c
index f9aec5cb6e7..f526eb05cc7 100644
--- a/drivers/media/dvb/dvb-usb/pctv452e.c
+++ b/drivers/media/dvb/dvb-usb/pctv452e.c
@@ -1055,22 +1055,7 @@ static struct usb_driver pctv452e_usb_driver = {
        .id_table   = pctv452e_usb_table,
 };
-static int __init pctv452e_usb_init(void)
+module_usb_driver(pctv452e_usb_driver);
-{
-        int ret = usb_register(&pctv452e_usb_driver);
-        if (ret)
-                err("%s: usb_register failed! Error %d", __FILE__, ret);
-        return ret;
-}
-static void __exit pctv452e_usb_exit(void)
-{
-        usb_deregister(&pctv452e_usb_driver);
-}
-module_init(pctv452e_usb_init);
-module_exit(pctv452e_usb_exit);
 MODULE_AUTHOR("Dominik Kuhlen <dkuhlen@gmx.net>");
 MODULE_AUTHOR("Andre Weidemann <Andre.Weidemann@web.de>");
diff --git a/drivers/media/dvb/dvb-usb/technisat-usb2.c b/drivers/media/dvb/dvb-usb/technisat-usb2.c
index 0998fe96195..acefaa89cc5 100644
--- a/drivers/media/dvb/dvb-usb/technisat-usb2.c
+++ b/drivers/media/dvb/dvb-usb/technisat-usb2.c
@@ -781,25 +781,7 @@ static struct usb_driver technisat_usb2_driver = {
        .id_table   = technisat_usb2_id_table,
 };
-/* module stuff */
+module_usb_driver(technisat_usb2_driver);
-static int __init technisat_usb2_module_init(void)
-{
-        int result = usb_register(&technisat_usb2_driver);
-        if (result) {
-                err("usb_register failed. Code %d", result);
-                return result;
-        }
-        return 0;
-}
-static void __exit technisat_usb2_module_exit(void)
-{
-        usb_deregister(&technisat_usb2_driver);
-}
-module_init(technisat_usb2_module_init);
-module_exit(technisat_usb2_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <pboettcher@kernellabs.com>");
 MODULE_DESCRIPTION("Driver for Technisat DVB-S/S2 USB 2.0 device");
diff --git a/drivers/media/dvb/dvb-usb/ttusb2.c b/drivers/media/dvb/dvb-usb/ttusb2.c
index ea4eab8b396..56acf8e55d5 100644
--- a/drivers/media/dvb/dvb-usb/ttusb2.c
+++ b/drivers/media/dvb/dvb-usb/ttusb2.c
@@ -799,26 +799,7 @@ static struct usb_driver ttusb2_driver = {
        .id_table       = ttusb2_table,
 };
-/* module stuff */
+module_usb_driver(ttusb2_driver);
-static int __init ttusb2_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&ttusb2_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit ttusb2_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&ttusb2_driver);
-}
-module_init (ttusb2_module_init);
-module_exit (ttusb2_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for Pinnacle PCTV 400e DVB-S USB2.0");
diff --git a/drivers/media/dvb/dvb-usb/umt-010.c b/drivers/media/dvb/dvb-usb/umt-010.c
index 463673a5c2b..9b042292e78 100644
--- a/drivers/media/dvb/dvb-usb/umt-010.c
+++ b/drivers/media/dvb/dvb-usb/umt-010.c
@@ -143,26 +143,7 @@ static struct usb_driver umt_driver = {
        .id_table       = umt_table,
 };
-/* module stuff */
+module_usb_driver(umt_driver);
-static int __init umt_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&umt_driver))) {
-                err("usb_register failed. Error number %d",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit umt_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&umt_driver);
-}
-module_init (umt_module_init);
-module_exit (umt_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for HanfTek UMT 010 USB2.0 DVB-T device");
diff --git a/drivers/media/dvb/dvb-usb/vp702x.c b/drivers/media/dvb/dvb-usb/vp702x.c
index 45e31f22481..07c673a6e76 100644
--- a/drivers/media/dvb/dvb-usb/vp702x.c
+++ b/drivers/media/dvb/dvb-usb/vp702x.c
@@ -436,26 +436,7 @@ static struct usb_driver vp702x_usb_driver = {
        .id_table       = vp702x_usb_table,
 };
-/* module stuff */
+module_usb_driver(vp702x_usb_driver);
-static int __init vp702x_usb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&vp702x_usb_driver))) {
-                err("usb_register failed. (%d)",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit vp702x_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&vp702x_usb_driver);
-}
-module_init(vp702x_usb_module_init);
-module_exit(vp702x_usb_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for Twinhan StarBox DVB-S USB2.0 and clones");
diff --git a/drivers/media/dvb/dvb-usb/vp7045.c b/drivers/media/dvb/dvb-usb/vp7045.c
index 90873af5682..d750724132e 100644
--- a/drivers/media/dvb/dvb-usb/vp7045.c
+++ b/drivers/media/dvb/dvb-usb/vp7045.c
@@ -294,26 +294,7 @@ static struct usb_driver vp7045_usb_driver = {
        .id_table       = vp7045_usb_table,
 };
-/* module stuff */
+module_usb_driver(vp7045_usb_driver);
-static int __init vp7045_usb_module_init(void)
-{
-        int result;
-        if ((result = usb_register(&vp7045_usb_driver))) {
-                err("usb_register failed. (%d)",result);
-                return result;
-        }
-        return 0;
-}
-static void __exit vp7045_usb_module_exit(void)
-{
-        /* deregister this driver from the USB subsystem */
-        usb_deregister(&vp7045_usb_driver);
-}
-module_init(vp7045_usb_module_init);
-module_exit(vp7045_usb_module_exit);
 MODULE_AUTHOR("Patrick Boettcher <patrick.boettcher@desy.de>");
 MODULE_DESCRIPTION("Driver for Twinhan MagicBox/Alpha and DNTV tinyUSB2 DVB-T USB2.0");
diff --git a/drivers/media/dvb/siano/smsusb.c b/drivers/media/dvb/siano/smsusb.c
index 51c7121b321..b1fe5137df0 100644
--- a/drivers/media/dvb/siano/smsusb.c
+++ b/drivers/media/dvb/siano/smsusb.c
@@ -557,26 +557,7 @@ static struct usb_driver smsusb_driver = {
        .resume                 = smsusb_resume,
 };
-static int __init smsusb_module_init(void)
+module_usb_driver(smsusb_driver);
-{
-        int rc = usb_register(&smsusb_driver);
-        if (rc)
-                sms_err("usb_register failed. Error number %d", rc);
-        sms_debug("");
-        return rc;
-}
-static void __exit smsusb_module_exit(void)
-{
-        /* Regular USB Cleanup */
-        usb_deregister(&smsusb_driver);
-        sms_info("end");
-}
-module_init(smsusb_module_init);
-module_exit(smsusb_module_exit);
 MODULE_DESCRIPTION("Driver for the Siano SMS1xxx USB dongle");
 MODULE_AUTHOR("Siano Mobile Silicon, INC. (uris@siano-ms.com)");
diff --git a/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c b/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c
index 420bb42d523..e90192fdde1 100644
--- a/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c
+++ b/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c
@@ -1794,26 +1794,7 @@ static struct usb_driver ttusb_driver = {
      .id_table         = ttusb_table,
 };
-static int __init ttusb_init(void)
+module_usb_driver(ttusb_driver);
-{
-        int err;
-        if ((err = usb_register(&ttusb_driver)) < 0) {
-                printk("%s: usb_register failed! Error number %d",
-                       __FILE__, err);
-                return err;
-        }
-        return 0;
-}
-static void __exit ttusb_exit(void)
-{
-        usb_deregister(&ttusb_driver);
-}
-module_init(ttusb_init);
-module_exit(ttusb_exit);
 MODULE_AUTHOR("Holger Waechtler <holger@convergence.de>");
 MODULE_DESCRIPTION("TTUSB DVB Driver");
diff --git a/drivers/media/dvb/ttusb-dec/ttusb_dec.c b/drivers/media/dvb/ttusb-dec/ttusb_dec.c
index f893bffa08a..504c8123033 100644
--- a/drivers/media/dvb/ttusb-dec/ttusb_dec.c
+++ b/drivers/media/dvb/ttusb-dec/ttusb_dec.c
@@ -1756,26 +1756,7 @@ static struct usb_driver ttusb_dec_driver = {
        .id_table       = ttusb_dec_table,
 };
-static int __init ttusb_dec_init(void)
+module_usb_driver(ttusb_dec_driver);
-{
-        int result;
-        if ((result = usb_register(&ttusb_dec_driver)) < 0) {
-                printk("%s: initialisation failed: error %d.\n", __func__,
-                       result);
-                return result;
-        }
-        return 0;
-}
-static void __exit ttusb_dec_exit(void)
-{
-        usb_deregister(&ttusb_dec_driver);
-}
-module_init(ttusb_dec_init);
-module_exit(ttusb_dec_exit);
 MODULE_AUTHOR("Alex Woods <linux-dvb@giblets.org>");
 MODULE_DESCRIPTION(DRIVER_NAME);
diff --git a/drivers/media/radio/dsbr100.c b/drivers/media/radio/dsbr100.c
index 25e58cbf35f..f36905b6364 100644
--- a/drivers/media/radio/dsbr100.c
+++ b/drivers/media/radio/dsbr100.c
@@ -624,21 +624,7 @@ static int usb_dsbr100_probe(struct usb_interface *intf,
        return 0;
 }
-static int __init dsbr100_init(void)
+module_usb_driver(usb_dsbr100_driver);
-{
-        int retval = usb_register(&usb_dsbr100_driver);
-        printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_VERSION ":"
-               DRIVER_DESC "\n");
-        return retval;
-}
-static void __exit dsbr100_exit(void)
-{
-        usb_deregister(&usb_dsbr100_driver);
-}
-module_init (dsbr100_init);
-module_exit (dsbr100_exit);
 MODULE_AUTHOR( DRIVER_AUTHOR );
 MODULE_DESCRIPTION( DRIVER_DESC );
diff --git a/drivers/media/radio/radio-gemtek.c b/drivers/media/radio/radio-gemtek.c
index edadc8449a3..36ce0611c03 100644
--- a/drivers/media/radio/radio-gemtek.c
+++ b/drivers/media/radio/radio-gemtek.c
@@ -47,11 +47,11 @@ MODULE_VERSION("0.0.4");
 #endif
 static int io           = CONFIG_RADIO_GEMTEK_PORT;
-static int probe        = CONFIG_RADIO_GEMTEK_PROBE;
+static bool probe       = CONFIG_RADIO_GEMTEK_PROBE;
-static int hardmute;
+static bool hardmute;
-static int shutdown     = 1;
+static bool shutdown    = 1;
-static int keepmuted    = 1;
+static bool keepmuted   = 1;
-static int initmute     = 1;
+static bool initmute    = 1;
 static int radio_nr     = -1;
 module_param(io, int, 0444);
diff --git a/drivers/media/radio/radio-miropcm20.c b/drivers/media/radio/radio-miropcm20.c
index 3fb76e3834c..87c1ee13b05 100644
--- a/drivers/media/radio/radio-miropcm20.c
+++ b/drivers/media/radio/radio-miropcm20.c
@@ -23,7 +23,7 @@ static int radio_nr = -1;
 module_param(radio_nr, int, 0);
 MODULE_PARM_DESC(radio_nr, "Set radio device number (/dev/radioX).  Default: -1 (autodetect)");
-static int mono;
+static bool mono;
 module_param(mono, bool, 0);
 MODULE_PARM_DESC(mono, "Force tuner into mono mode.");
diff --git a/drivers/media/radio/radio-mr800.c b/drivers/media/radio/radio-mr800.c
index 1742bd8110b..a860a72a58e 100644
--- a/drivers/media/radio/radio-mr800.c
+++ b/drivers/media/radio/radio-mr800.c
@@ -659,25 +659,4 @@ err:
        return retval;
 }
-static int __init amradio_init(void)
+module_usb_driver(usb_amradio_driver);
-{
-        int retval = usb_register(&usb_amradio_driver);
-        pr_info(KBUILD_MODNAME
-                ": version " DRIVER_VERSION " " DRIVER_DESC "\n");
-        if (retval)
-                pr_err(KBUILD_MODNAME
-                        ": usb_register failed. Error number %d\n", retval);
-        return retval;
-}
-static void __exit amradio_exit(void)
-{
-        usb_deregister(&usb_amradio_driver);
-}
-module_init(amradio_init);
-module_exit(amradio_exit);
diff --git a/drivers/media/radio/si470x/radio-si470x-usb.c b/drivers/media/radio/si470x/radio-si470x-usb.c
index a6ad707fae9..b7debb67932 100644
--- a/drivers/media/radio/si470x/radio-si470x-usb.c
+++ b/drivers/media/radio/si470x/radio-si470x-usb.c
@@ -861,33 +861,7 @@ static struct usb_driver si470x_usb_driver = {
        .supports_autosuspend   = 1,
 };
+module_usb_driver(si470x_usb_driver);
-/**************************************************************************
- * Module Interface
- **************************************************************************/
-/*
- * si470x_module_init - module init
- */
-static int __init si470x_module_init(void)
-{
-        printk(KERN_INFO DRIVER_DESC ", Version " DRIVER_VERSION "\n");
-        return usb_register(&si470x_usb_driver);
-}
-/*
- * si470x_module_exit - module exit
- */
-static void __exit si470x_module_exit(void)
-{
-        usb_deregister(&si470x_usb_driver);
-}
-module_init(si470x_module_init);
-module_exit(si470x_module_exit);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR(DRIVER_AUTHOR);
diff --git a/drivers/media/rc/ati_remote.c b/drivers/media/rc/ati_remote.c
index 303f22ea04c..baf907b3ce7 100644
--- a/drivers/media/rc/ati_remote.c
+++ b/drivers/media/rc/ati_remote.c
@@ -189,7 +189,7 @@ struct ati_remote {
        dma_addr_t inbuf_dma;
        dma_addr_t outbuf_dma;
-        unsigned char old_data[2];  /* Detect duplicate events */
+        unsigned char old_data;     /* Detect duplicate events */
        unsigned long old_jiffies;
        unsigned long acc_jiffies;  /* handle acceleration */
        unsigned long first_jiffies;
@@ -221,35 +221,35 @@ struct ati_remote {
 /* Translation table from hardware messages to input events. */
 static const struct {
        short kind;
-        unsigned char data1, data2;
+        unsigned char data;
        int type;
        unsigned int code;
        int value;
 }  ati_remote_tbl[] = {
        /* Directional control pad axes */
-        {KIND_ACCEL,   0x35, 0x70, EV_REL, REL_X, -1},   /* left */
+        {KIND_ACCEL,   0x70, EV_REL, REL_X, -1},   /* left */
-        {KIND_ACCEL,   0x36, 0x71, EV_REL, REL_X, 1},    /* right */
+        {KIND_ACCEL,   0x71, EV_REL, REL_X, 1},    /* right */
-        {KIND_ACCEL,   0x37, 0x72, EV_REL, REL_Y, -1},   /* up */
+        {KIND_ACCEL,   0x72, EV_REL, REL_Y, -1},   /* up */
-        {KIND_ACCEL,   0x38, 0x73, EV_REL, REL_Y, 1},    /* down */
+        {KIND_ACCEL,   0x73, EV_REL, REL_Y, 1},    /* down */
        /* Directional control pad diagonals */
-        {KIND_LU,      0x39, 0x74, EV_REL, 0, 0},        /* left up */
+        {KIND_LU,      0x74, EV_REL, 0, 0},        /* left up */
-        {KIND_RU,      0x3a, 0x75, EV_REL, 0, 0},        /* right up */
+        {KIND_RU,      0x75, EV_REL, 0, 0},        /* right up */
-        {KIND_LD,      0x3c, 0x77, EV_REL, 0, 0},        /* left down */
+        {KIND_LD,      0x77, EV_REL, 0, 0},        /* left down */
-        {KIND_RD,      0x3b, 0x76, EV_REL, 0, 0},        /* right down */
+        {KIND_RD,      0x76, EV_REL, 0, 0},        /* right down */
        /* "Mouse button" buttons */
-        {KIND_LITERAL, 0x3d, 0x78, EV_KEY, BTN_LEFT, 1}, /* left btn down */
+        {KIND_LITERAL, 0x78, EV_KEY, BTN_LEFT, 1}, /* left btn down */
-        {KIND_LITERAL, 0x3e, 0x79, EV_KEY, BTN_LEFT, 0}, /* left btn up */
+        {KIND_LITERAL, 0x79, EV_KEY, BTN_LEFT, 0}, /* left btn up */
-        {KIND_LITERAL, 0x41, 0x7c, EV_KEY, BTN_RIGHT, 1},/* right btn down */
+        {KIND_LITERAL, 0x7c, EV_KEY, BTN_RIGHT, 1},/* right btn down */
-        {KIND_LITERAL, 0x42, 0x7d, EV_KEY, BTN_RIGHT, 0},/* right btn up */
+        {KIND_LITERAL, 0x7d, EV_KEY, BTN_RIGHT, 0},/* right btn up */
        /* Artificial "doubleclick" events are generated by the hardware.
         * They are mapped to the "side" and "extra" mouse buttons here. */
-        {KIND_FILTERED, 0x3f, 0x7a, EV_KEY, BTN_SIDE, 1}, /* left dblclick */
+        {KIND_FILTERED, 0x7a, EV_KEY, BTN_SIDE, 1}, /* left dblclick */
-        {KIND_FILTERED, 0x43, 0x7e, EV_KEY, BTN_EXTRA, 1},/* right dblclick */
+        {KIND_FILTERED, 0x7e, EV_KEY, BTN_EXTRA, 1},/* right dblclick */
        /* Non-mouse events are handled by rc-core */
-        {KIND_END, 0x00, 0x00, EV_MAX + 1, 0, 0}
+        {KIND_END, 0x00, EV_MAX + 1, 0, 0}
 };
 /* Local function prototypes */
@@ -397,25 +397,6 @@ static int ati_remote_sendpacket(struct ati_remote *ati_remote, u16 cmd, unsigne
 }
 /*
- *      ati_remote_event_lookup
- */
-static int ati_remote_event_lookup(int rem, unsigned char d1, unsigned char d2)
-{
-        int i;
-        for (i = 0; ati_remote_tbl[i].kind != KIND_END; i++) {
-                /*
-                 * Decide if the table entry matches the remote input.
-                 */
-                if (ati_remote_tbl[i].data1 == d1 &&
-                    ati_remote_tbl[i].data2 == d2)
-                        return i;
-        }
-        return -1;
-}
-/*
 *      ati_remote_compute_accel
 *
 * Implements acceleration curve for directional control pad
@@ -463,7 +444,15 @@ static void ati_remote_input_report(struct urb *urb)
        int index = -1;
        int acc;
        int remote_num;
-        unsigned char scancode[2];
+        unsigned char scancode;
+        int i;
+        /*
+         * data[0] = 0x14
+         * data[1] = data[2] + data[3] + 0xd5 (a checksum byte)
+         * data[2] = the key code (with toggle bit in MSB with some models)
+         * data[3] = channel << 4 (the low 4 bits must be zero)
+         */
        /* Deal with strange looking inputs */
        if ( (urb->actual_length != 4) || (data[0] != 0x14) ||
@@ -472,6 +461,13 @@ static void ati_remote_input_report(struct urb *urb)
                return;
        }
+        if (data[1] != ((data[2] + data[3] + 0xd5) & 0xff)) {
+                dbginfo(&ati_remote->interface->dev,
+                        "wrong checksum in input: %02x %02x %02x %02x\n",
+                        data[0], data[1], data[2], data[3]);
+                return;
+        }
        /* Mask unwanted remote channels.  */
        /* note: remote_num is 0-based, channel 1 on remote == 0 here */
        remote_num = (data[3] >> 4) & 0x0f;
@@ -482,31 +478,30 @@ static void ati_remote_input_report(struct urb *urb)
                return;
        }
-        scancode[0] = (((data[1] - ((remote_num + 1) << 4)) & 0xf0) | (data[1] & 0x0f));
        /*
-         * Some devices (e.g. SnapStream Firefly) use 8080 as toggle code,
+         * MSB is a toggle code, though only used by some devices
-         * so we have to clear them. The first bit is a bit tricky as the
+         * (e.g. SnapStream Firefly)
-         * "non-toggled" state depends on remote_num, so we xor it with the
-         * second bit which is only used for toggle.
         */
-        scancode[0] ^= (data[2] & 0x80);
+        scancode = data[2] & 0x7f;
-        scancode[1] = data[2] & ~0x80;
+        /* Look up event code index in the mouse translation table. */
+        for (i = 0; ati_remote_tbl[i].kind != KIND_END; i++) {
-        /* Look up event code index in mouse translation table. */
+                if (scancode == ati_remote_tbl[i].data) {
-        index = ati_remote_event_lookup(remote_num, scancode[0], scancode[1]);
+                        index = i;
+                        break;
+                }
+        }
        if (index >= 0) {
                dbginfo(&ati_remote->interface->dev,
-                        "channel 0x%02x; mouse data %02x,%02x; index %d; keycode %d\n",
+                        "channel 0x%02x; mouse data %02x; index %d; keycode %d\n",
-                        remote_num, data[1], data[2], index, ati_remote_tbl[index].code);
+                        remote_num, data[2], index, ati_remote_tbl[index].code);
                if (!dev)
                        return; /* no mouse device */
        } else
                dbginfo(&ati_remote->interface->dev,
-                        "channel 0x%02x; key data %02x,%02x, scancode %02x,%02x\n",
+                        "channel 0x%02x; key data %02x, scancode %02x\n",
-                        remote_num, data[1], data[2], scancode[0], scancode[1]);
+                        remote_num, data[2], scancode);
        if (index >= 0 && ati_remote_tbl[index].kind == KIND_LITERAL) {
@@ -523,8 +518,7 @@ static void ati_remote_input_report(struct urb *urb)
                unsigned long now = jiffies;
                /* Filter duplicate events which happen "too close" together. */
-                if (ati_remote->old_data[0] == data[1] &&
+                if (ati_remote->old_data == data[2] &&
-                    ati_remote->old_data[1] == data[2] &&
                    time_before(now, ati_remote->old_jiffies +
                                     msecs_to_jiffies(repeat_filter))) {
                        ati_remote->repeat_count++;
@@ -533,8 +527,7 @@ static void ati_remote_input_report(struct urb *urb)
                        ati_remote->first_jiffies = now;
                }
-                ati_remote->old_data[0] = data[1];
+                ati_remote->old_data = data[2];
-                ati_remote->old_data[1] = data[2];
                ati_remote->old_jiffies = now;
                /* Ensure we skip at least the 4 first duplicate events (generated
@@ -549,14 +542,13 @@ static void ati_remote_input_report(struct urb *urb)
                if (index < 0) {
                        /* Not a mouse event, hand it to rc-core. */
-                        u32 rc_code = (scancode[0] << 8) | scancode[1];
                        /*
                         * We don't use the rc-core repeat handling yet as
                         * it would cause ghost repeats which would be a
                         * regression for this driver.
                         */
-                        rc_keydown_notimeout(ati_remote->rdev, rc_code,
+                        rc_keydown_notimeout(ati_remote->rdev, scancode,
                                             data[2]);
                        rc_keyup(ati_remote->rdev);
                        return;
@@ -607,8 +599,7 @@ static void ati_remote_input_report(struct urb *urb)
                input_sync(dev);
                ati_remote->old_jiffies = jiffies;
-                ati_remote->old_data[0] = data[1];
+                ati_remote->old_data = data[2];
-                ati_remote->old_data[1] = data[2];
        }
 }
@@ -908,38 +899,7 @@ static void ati_remote_disconnect(struct usb_interface *interface)
        kfree(ati_remote);
 }
-/*
+module_usb_driver(ati_remote_driver);
- *      ati_remote_init
- */
-static int __init ati_remote_init(void)
-{
-        int result;
-        result = usb_register(&ati_remote_driver);
-        if (result)
-                printk(KERN_ERR KBUILD_MODNAME
-                       ": usb_register error #%d\n", result);
-        else
-                printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_VERSION ":"
-                       DRIVER_DESC "\n");
-        return result;
-}
-/*
- *      ati_remote_exit
- */
-static void __exit ati_remote_exit(void)
-{
-        usb_deregister(&ati_remote_driver);
-}
-/*
- *      module specification
- */
-module_init(ati_remote_init);
-module_exit(ati_remote_exit);
 MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_DESC);
diff --git a/drivers/media/rc/ene_ir.c b/drivers/media/rc/ene_ir.c
index cf10ecf5ace..860c112e0fd 100644
--- a/drivers/media/rc/ene_ir.c
+++ b/drivers/media/rc/ene_ir.c
@@ -324,7 +324,7 @@ static int ene_rx_get_sample_reg(struct ene_device *dev)
                return dev->extra_buf2_address + r_pointer;
        }
-        dbg("attempt to read beyong ring bufer end");
+        dbg("attempt to read beyond ring buffer end");
        return 0;
 }
diff --git a/drivers/media/rc/ene_ir.h b/drivers/media/rc/ene_ir.h
index fd108d90f75..6f978e85db8 100644
--- a/drivers/media/rc/ene_ir.h
+++ b/drivers/media/rc/ene_ir.h
@@ -227,7 +227,7 @@ struct ene_device {
        /* TX buffer */
        unsigned *tx_buffer;                    /* input samples buffer*/
-        int tx_pos;                             /* position in that bufer */
+        int tx_pos;                             /* position in that buffer */
        int tx_len;                             /* current len of tx buffer */
        int tx_done;                            /* done transmitting */
                                                /* one more sample pending*/
diff --git a/drivers/media/rc/imon.c b/drivers/media/rc/imon.c
index 6ed96465137..3aeb29a7ce1 100644
--- a/drivers/media/rc/imon.c
+++ b/drivers/media/rc/imon.c
@@ -2458,23 +2458,4 @@ static int imon_resume(struct usb_interface *intf)
        return rc;
 }
-static int __init imon_init(void)
+module_usb_driver(imon_driver);
-{
-        int rc;
-        rc = usb_register(&imon_driver);
-        if (rc) {
-                pr_err("usb register failed(%d)\n", rc);
-                rc = -ENODEV;
-        }
-        return rc;
-}
-static void __exit imon_exit(void)
-{
-        usb_deregister(&imon_driver);
-}
-module_init(imon_init);
-module_exit(imon_exit);
diff --git a/drivers/media/rc/keymaps/rc-ati-x10.c b/drivers/media/rc/keymaps/rc-ati-x10.c
index e1b8b2605c4..81506440ede 100644
--- a/drivers/media/rc/keymaps/rc-ati-x10.c
+++ b/drivers/media/rc/keymaps/rc-ati-x10.c
@@ -27,55 +27,55 @@
 #include <media/rc-map.h>
 static struct rc_map_table ati_x10[] = {
-        { 0xd20d, KEY_1 },
+        { 0x0d, KEY_1 },
-        { 0xd30e, KEY_2 },
+        { 0x0e, KEY_2 },
-        { 0xd40f, KEY_3 },
+        { 0x0f, KEY_3 },
-        { 0xd510, KEY_4 },
+        { 0x10, KEY_4 },
-        { 0xd611, KEY_5 },
+        { 0x11, KEY_5 },
-        { 0xd712, KEY_6 },
+        { 0x12, KEY_6 },
-        { 0xd813, KEY_7 },
+        { 0x13, KEY_7 },
-        { 0xd914, KEY_8 },
+        { 0x14, KEY_8 },
-        { 0xda15, KEY_9 },
+        { 0x15, KEY_9 },
-        { 0xdc17, KEY_0 },
+        { 0x17, KEY_0 },
-        { 0xc500, KEY_A },
+        { 0x00, KEY_A },
-        { 0xc601, KEY_B },
+        { 0x01, KEY_B },
-        { 0xde19, KEY_C },
+        { 0x19, KEY_C },
-        { 0xe01b, KEY_D },
+        { 0x1b, KEY_D },
-        { 0xe621, KEY_E },
+        { 0x21, KEY_E },
-        { 0xe823, KEY_F },
+        { 0x23, KEY_F },
-        { 0xdd18, KEY_KPENTER },    /* "check" */
+        { 0x18, KEY_KPENTER },    /* "check" */
-        { 0xdb16, KEY_MENU },       /* "menu" */
+        { 0x16, KEY_MENU },       /* "menu" */
-        { 0xc702, KEY_POWER },      /* Power */
+        { 0x02, KEY_POWER },      /* Power */
-        { 0xc803, KEY_TV },         /* TV */
+        { 0x03, KEY_TV },         /* TV */
-        { 0xc904, KEY_DVD },        /* DVD */
+        { 0x04, KEY_DVD },        /* DVD */
-        { 0xca05, KEY_WWW },        /* WEB */
+        { 0x05, KEY_WWW },        /* WEB */
-        { 0xcb06, KEY_BOOKMARKS },  /* "book" */
+        { 0x06, KEY_BOOKMARKS },  /* "book" */
-        { 0xcc07, KEY_EDIT },       /* "hand" */
+        { 0x07, KEY_EDIT },       /* "hand" */
-        { 0xe11c, KEY_COFFEE },     /* "timer" */
+        { 0x1c, KEY_COFFEE },     /* "timer" */
-        { 0xe520, KEY_FRONT },      /* "max" */
+        { 0x20, KEY_FRONT },      /* "max" */
-        { 0xe21d, KEY_LEFT },       /* left */
+        { 0x1d, KEY_LEFT },       /* left */
-        { 0xe41f, KEY_RIGHT },      /* right */
+        { 0x1f, KEY_RIGHT },      /* right */
-        { 0xe722, KEY_DOWN },       /* down */
+        { 0x22, KEY_DOWN },       /* down */
-        { 0xdf1a, KEY_UP },         /* up */
+        { 0x1a, KEY_UP },         /* up */
-        { 0xe31e, KEY_OK },         /* "OK" */
+        { 0x1e, KEY_OK },         /* "OK" */
-        { 0xce09, KEY_VOLUMEDOWN }, /* VOL + */
+        { 0x09, KEY_VOLUMEDOWN }, /* VOL + */
-        { 0xcd08, KEY_VOLUMEUP },   /* VOL - */
+        { 0x08, KEY_VOLUMEUP },   /* VOL - */
-        { 0xcf0a, KEY_MUTE },       /* MUTE  */
+        { 0x0a, KEY_MUTE },       /* MUTE  */
-        { 0xd00b, KEY_CHANNELUP },  /* CH + */
+        { 0x0b, KEY_CHANNELUP },  /* CH + */
-        { 0xd10c, KEY_CHANNELDOWN },/* CH - */
+        { 0x0c, KEY_CHANNELDOWN },/* CH - */
-        { 0xec27, KEY_RECORD },     /* ( o) red */
+        { 0x27, KEY_RECORD },     /* ( o) red */
-        { 0xea25, KEY_PLAY },       /* ( >) */
+        { 0x25, KEY_PLAY },       /* ( >) */
-        { 0xe924, KEY_REWIND },     /* (<<) */
+        { 0x24, KEY_REWIND },     /* (<<) */
-        { 0xeb26, KEY_FORWARD },    /* (>>) */
+        { 0x26, KEY_FORWARD },    /* (>>) */
-        { 0xed28, KEY_STOP },       /* ([]) */
+        { 0x28, KEY_STOP },       /* ([]) */
-        { 0xee29, KEY_PAUSE },      /* ('') */
+        { 0x29, KEY_PAUSE },      /* ('') */
-        { 0xf02b, KEY_PREVIOUS },   /* (<-) */
+        { 0x2b, KEY_PREVIOUS },   /* (<-) */
-        { 0xef2a, KEY_NEXT },       /* (>+) */
+        { 0x2a, KEY_NEXT },       /* (>+) */
-        { 0xf22d, KEY_INFO },       /* PLAYING */
+        { 0x2d, KEY_INFO },       /* PLAYING */
-        { 0xf32e, KEY_HOME },       /* TOP */
+        { 0x2e, KEY_HOME },       /* TOP */
-        { 0xf42f, KEY_END },        /* END */
+        { 0x2f, KEY_END },        /* END */
-        { 0xf530, KEY_SELECT },     /* SELECT */
+        { 0x30, KEY_SELECT },     /* SELECT */
 };
 static struct rc_map_list ati_x10_map = {
diff --git a/drivers/media/rc/keymaps/rc-medion-x10.c b/drivers/media/rc/keymaps/rc-medion-x10.c
index 09e2cc01d11..479cdb89781 100644
--- a/drivers/media/rc/keymaps/rc-medion-x10.c
+++ b/drivers/media/rc/keymaps/rc-medion-x10.c
@@ -25,70 +25,70 @@
 #include <media/rc-map.h>
 static struct rc_map_table medion_x10[] = {
-        { 0xf12c, KEY_TV },    /* TV */
+        { 0x2c, KEY_TV },    /* TV */
-        { 0xf22d, KEY_VCR },   /* VCR */
+        { 0x2d, KEY_VCR },   /* VCR */
-        { 0xc904, KEY_DVD },   /* DVD */
+        { 0x04, KEY_DVD },   /* DVD */
-        { 0xcb06, KEY_AUDIO }, /* MUSIC */
+        { 0x06, KEY_AUDIO }, /* MUSIC */
-        { 0xf32e, KEY_RADIO },     /* RADIO */
+        { 0x2e, KEY_RADIO },     /* RADIO */
-        { 0xca05, KEY_DIRECTORY }, /* PHOTO */
+        { 0x05, KEY_DIRECTORY }, /* PHOTO */
-        { 0xf42f, KEY_INFO },      /* TV-PREVIEW */
+        { 0x2f, KEY_INFO },      /* TV-PREVIEW */
-        { 0xf530, KEY_LIST },      /* CHANNEL-LST */
+        { 0x30, KEY_LIST },      /* CHANNEL-LST */
-        { 0xe01b, KEY_SETUP }, /* SETUP */
+        { 0x1b, KEY_SETUP }, /* SETUP */
-        { 0xf631, KEY_VIDEO }, /* VIDEO DESKTOP */
+        { 0x31, KEY_VIDEO }, /* VIDEO DESKTOP */
-        { 0xcd08, KEY_VOLUMEDOWN },  /* VOL - */
+        { 0x08, KEY_VOLUMEDOWN },  /* VOL - */
-        { 0xce09, KEY_VOLUMEUP },    /* VOL + */
+        { 0x09, KEY_VOLUMEUP },    /* VOL + */
-        { 0xd00b, KEY_CHANNELUP },   /* CHAN + */
+        { 0x0b, KEY_CHANNELUP },   /* CHAN + */
-        { 0xd10c, KEY_CHANNELDOWN }, /* CHAN - */
+        { 0x0c, KEY_CHANNELDOWN }, /* CHAN - */
-        { 0xc500, KEY_MUTE },        /* MUTE */
+        { 0x00, KEY_MUTE },        /* MUTE */
-        { 0xf732, KEY_RED }, /* red */
+        { 0x32, KEY_RED }, /* red */
-        { 0xf833, KEY_GREEN }, /* green */
+        { 0x33, KEY_GREEN }, /* green */
-        { 0xf934, KEY_YELLOW }, /* yellow */
+        { 0x34, KEY_YELLOW }, /* yellow */
-        { 0xfa35, KEY_BLUE }, /* blue */
+        { 0x35, KEY_BLUE }, /* blue */
-        { 0xdb16, KEY_TEXT }, /* TXT */
+        { 0x16, KEY_TEXT }, /* TXT */
-        { 0xd20d, KEY_1 },
+        { 0x0d, KEY_1 },
-        { 0xd30e, KEY_2 },
+        { 0x0e, KEY_2 },
-        { 0xd40f, KEY_3 },
+        { 0x0f, KEY_3 },
-        { 0xd510, KEY_4 },
+        { 0x10, KEY_4 },
-        { 0xd611, KEY_5 },
+        { 0x11, KEY_5 },
-        { 0xd712, KEY_6 },
+        { 0x12, KEY_6 },
-        { 0xd813, KEY_7 },
+        { 0x13, KEY_7 },
-        { 0xd914, KEY_8 },
+        { 0x14, KEY_8 },
-        { 0xda15, KEY_9 },
+        { 0x15, KEY_9 },
-        { 0xdc17, KEY_0 },
+        { 0x17, KEY_0 },
-        { 0xe11c, KEY_SEARCH }, /* TV/RAD, CH SRC */
+        { 0x1c, KEY_SEARCH }, /* TV/RAD, CH SRC */
-        { 0xe520, KEY_DELETE }, /* DELETE */
+        { 0x20, KEY_DELETE }, /* DELETE */
-        { 0xfb36, KEY_KEYBOARD }, /* RENAME */
+        { 0x36, KEY_KEYBOARD }, /* RENAME */
-        { 0xdd18, KEY_SCREEN },   /* SNAPSHOT */
+        { 0x18, KEY_SCREEN },   /* SNAPSHOT */
-        { 0xdf1a, KEY_UP },    /* up */
+        { 0x1a, KEY_UP },    /* up */
-        { 0xe722, KEY_DOWN },  /* down */
+        { 0x22, KEY_DOWN },  /* down */
-        { 0xe21d, KEY_LEFT },  /* left */
+        { 0x1d, KEY_LEFT },  /* left */
-        { 0xe41f, KEY_RIGHT }, /* right */
+        { 0x1f, KEY_RIGHT }, /* right */
-        { 0xe31e, KEY_OK },    /* OK */
+        { 0x1e, KEY_OK },    /* OK */
-        { 0xfc37, KEY_SELECT }, /* ACQUIRE IMAGE */
+        { 0x37, KEY_SELECT }, /* ACQUIRE IMAGE */
-        { 0xfd38, KEY_EDIT },   /* EDIT IMAGE */
+        { 0x38, KEY_EDIT },   /* EDIT IMAGE */
-        { 0xe924, KEY_REWIND },   /* rewind  (<<) */
+        { 0x24, KEY_REWIND },   /* rewind  (<<) */
-        { 0xea25, KEY_PLAY },     /* play    ( >) */
+        { 0x25, KEY_PLAY },     /* play    ( >) */
-        { 0xeb26, KEY_FORWARD },  /* forward (>>) */
+        { 0x26, KEY_FORWARD },  /* forward (>>) */
-        { 0xec27, KEY_RECORD },   /* record  ( o) */
+        { 0x27, KEY_RECORD },   /* record  ( o) */
-        { 0xed28, KEY_STOP },     /* stop    ([]) */
+        { 0x28, KEY_STOP },     /* stop    ([]) */
-        { 0xee29, KEY_PAUSE },    /* pause   ('') */
+        { 0x29, KEY_PAUSE },    /* pause   ('') */
-        { 0xe621, KEY_PREVIOUS },        /* prev */
+        { 0x21, KEY_PREVIOUS },        /* prev */
-        { 0xfe39, KEY_SWITCHVIDEOMODE }, /* F SCR */
+        { 0x39, KEY_SWITCHVIDEOMODE }, /* F SCR */
-        { 0xe823, KEY_NEXT },            /* next */
+        { 0x23, KEY_NEXT },            /* next */
-        { 0xde19, KEY_MENU },            /* MENU */
+        { 0x19, KEY_MENU },            /* MENU */
-        { 0xff3a, KEY_LANGUAGE },        /* AUDIO */
+        { 0x3a, KEY_LANGUAGE },        /* AUDIO */
-        { 0xc702, KEY_POWER }, /* POWER */
+        { 0x02, KEY_POWER }, /* POWER */
 };
 static struct rc_map_list medion_x10_map = {
diff --git a/drivers/media/rc/keymaps/rc-snapstream-firefly.c b/drivers/media/rc/keymaps/rc-snapstream-firefly.c
index ef146520931..c7f33ec719b 100644
--- a/drivers/media/rc/keymaps/rc-snapstream-firefly.c
+++ b/drivers/media/rc/keymaps/rc-snapstream-firefly.c
@@ -22,63 +22,63 @@
 #include <media/rc-map.h>
 static struct rc_map_table snapstream_firefly[] = {
-        { 0xf12c, KEY_ZOOM },       /* Maximize */
+        { 0x2c, KEY_ZOOM },       /* Maximize */
-        { 0xc702, KEY_CLOSE },
+        { 0x02, KEY_CLOSE },
-        { 0xd20d, KEY_1 },
+        { 0x0d, KEY_1 },
-        { 0xd30e, KEY_2 },
+        { 0x0e, KEY_2 },
-        { 0xd40f, KEY_3 },
+        { 0x0f, KEY_3 },
-        { 0xd510, KEY_4 },
+        { 0x10, KEY_4 },
-        { 0xd611, KEY_5 },
+        { 0x11, KEY_5 },
-        { 0xd712, KEY_6 },
+        { 0x12, KEY_6 },
-        { 0xd813, KEY_7 },
+        { 0x13, KEY_7 },
-        { 0xd914, KEY_8 },
+        { 0x14, KEY_8 },
-        { 0xda15, KEY_9 },
+        { 0x15, KEY_9 },
-        { 0xdc17, KEY_0 },
+        { 0x17, KEY_0 },
-        { 0xdb16, KEY_BACK },
+        { 0x16, KEY_BACK },
-        { 0xdd18, KEY_KPENTER },    /* ent */
+        { 0x18, KEY_KPENTER },    /* ent */
-        { 0xce09, KEY_VOLUMEUP },
+        { 0x09, KEY_VOLUMEUP },
-        { 0xcd08, KEY_VOLUMEDOWN },
+        { 0x08, KEY_VOLUMEDOWN },
-        { 0xcf0a, KEY_MUTE },
+        { 0x0a, KEY_MUTE },
-        { 0xd00b, KEY_CHANNELUP },
+        { 0x0b, KEY_CHANNELUP },
-        { 0xd10c, KEY_CHANNELDOWN },
+        { 0x0c, KEY_CHANNELDOWN },
-        { 0xc500, KEY_VENDOR },     /* firefly */
+        { 0x00, KEY_VENDOR },     /* firefly */
-        { 0xf32e, KEY_INFO },
+        { 0x2e, KEY_INFO },
-        { 0xf42f, KEY_OPTION },
+        { 0x2f, KEY_OPTION },
-        { 0xe21d, KEY_LEFT },
+        { 0x1d, KEY_LEFT },
-        { 0xe41f, KEY_RIGHT },
+        { 0x1f, KEY_RIGHT },
-        { 0xe722, KEY_DOWN },
+        { 0x22, KEY_DOWN },
-        { 0xdf1a, KEY_UP },
+        { 0x1a, KEY_UP },
-        { 0xe31e, KEY_OK },
+        { 0x1e, KEY_OK },
-        { 0xe11c, KEY_MENU },
+        { 0x1c, KEY_MENU },
-        { 0xe520, KEY_EXIT },
+        { 0x20, KEY_EXIT },
-        { 0xec27, KEY_RECORD },
+        { 0x27, KEY_RECORD },
-        { 0xea25, KEY_PLAY },
+        { 0x25, KEY_PLAY },
-        { 0xed28, KEY_STOP },
+        { 0x28, KEY_STOP },
-        { 0xe924, KEY_REWIND },
+        { 0x24, KEY_REWIND },
-        { 0xeb26, KEY_FORWARD },
+        { 0x26, KEY_FORWARD },
-        { 0xee29, KEY_PAUSE },
+        { 0x29, KEY_PAUSE },
-        { 0xf02b, KEY_PREVIOUS },
+        { 0x2b, KEY_PREVIOUS },
-        { 0xef2a, KEY_NEXT },
+        { 0x2a, KEY_NEXT },
-        { 0xcb06, KEY_AUDIO },      /* Music */
+        { 0x06, KEY_AUDIO },      /* Music */
-        { 0xca05, KEY_IMAGES },     /* Photos */
+        { 0x05, KEY_IMAGES },     /* Photos */
-        { 0xc904, KEY_DVD },
+        { 0x04, KEY_DVD },
-        { 0xc803, KEY_TV },
+        { 0x03, KEY_TV },
-        { 0xcc07, KEY_VIDEO },
+        { 0x07, KEY_VIDEO },
-        { 0xc601, KEY_HELP },
+        { 0x01, KEY_HELP },
-        { 0xf22d, KEY_MODE },       /* Mouse */
+        { 0x2d, KEY_MODE },       /* Mouse */
-        { 0xde19, KEY_A },
+        { 0x19, KEY_A },
-        { 0xe01b, KEY_B },
+        { 0x1b, KEY_B },
-        { 0xe621, KEY_C },
+        { 0x21, KEY_C },
-        { 0xe823, KEY_D },
+        { 0x23, KEY_D },
 };
 static struct rc_map_list snapstream_firefly_map = {
diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c
index 27997a9ceb0..ca12d3289bf 100644
--- a/drivers/media/rc/lirc_dev.c
+++ b/drivers/media/rc/lirc_dev.c
@@ -38,7 +38,7 @@
 #include <media/lirc.h>
 #include <media/lirc_dev.h>
-static int debug;
+static bool debug;
 #define IRCTL_DEV_NAME  "BaseRemoteCtl"
 #define NOPLUG          -1
diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c
index 60d3c1e0971..21105bf9594 100644
--- a/drivers/media/rc/mceusb.c
+++ b/drivers/media/rc/mceusb.c
@@ -156,9 +156,9 @@
 /* module parameters */
 #ifdef CONFIG_USB_DEBUG
-static int debug = 1;
+static bool debug = 1;
 #else
-static int debug;
+static bool debug;
 #endif
 #define mce_dbg(dev, fmt, ...)                                  \
@@ -1448,25 +1448,7 @@ static struct usb_driver mceusb_dev_driver = {
        .id_table =     mceusb_dev_table
 };
-static int __init mceusb_dev_init(void)
+module_usb_driver(mceusb_dev_driver);
-{
-        int ret;
-        ret = usb_register(&mceusb_dev_driver);
-        if (ret < 0)
-                printk(KERN_ERR DRIVER_NAME
-                       ": usb register failed, result = %d\n", ret);
-        return ret;
-}
-static void __exit mceusb_dev_exit(void)
-{
-        usb_deregister(&mceusb_dev_driver);
-}
-module_init(mceusb_dev_init);
-module_exit(mceusb_dev_exit);
 MODULE_DESCRIPTION(DRIVER_DESC);
 MODULE_AUTHOR(DRIVER_AUTHOR);
diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c
index 29f900065d8..f5db8b949bc 100644
--- a/drivers/media/rc/rc-main.c
+++ b/drivers/media/rc/rc-main.c
@@ -715,7 +715,7 @@ static void ir_close(struct input_dev *idev)
 }
 /* class for /sys/class/rc */
-static char *ir_devnode(struct device *dev, mode_t *mode)
+static char *ir_devnode(struct device *dev, umode_t *mode)
 {
        return kasprintf(GFP_KERNEL, "rc/%s", dev_name(dev));
 }
diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c
index 61287fcca61..07322fb75ef 100644
--- a/drivers/media/rc/redrat3.c
+++ b/drivers/media/rc/redrat3.c
@@ -1300,25 +1300,7 @@ static struct usb_driver redrat3_dev_driver = {
        .id_table       = redrat3_dev_table
 };
-static int __init redrat3_dev_init(void)
+module_usb_driver(redrat3_dev_driver);
-{
-        int ret;
-        ret = usb_register(&redrat3_dev_driver);
-        if (ret < 0)
-                pr_err(DRIVER_NAME
-                       ": usb register failed, result = %d\n", ret);
-        return ret;
-}
-static void __exit redrat3_dev_exit(void)
-{
-        usb_deregister(&redrat3_dev_driver);
-}
-module_init(redrat3_dev_init);
-module_exit(redrat3_dev_exit);
 MODULE_DESCRIPTION(DRIVER_DESC);
 MODULE_AUTHOR(DRIVER_AUTHOR);
diff --git a/drivers/media/rc/streamzap.c b/drivers/media/rc/streamzap.c
index e435d94c077..d6f4bfe0939 100644
--- a/drivers/media/rc/streamzap.c
+++ b/drivers/media/rc/streamzap.c
@@ -43,9 +43,9 @@
 #define DRIVER_DESC     "Streamzap Remote Control driver"
 #ifdef CONFIG_USB_DEBUG
-static int debug = 1;
+static bool debug = 1;
 #else
-static int debug;
+static bool debug;
 #endif
 #define USB_STREAMZAP_VENDOR_ID         0x0e9c
@@ -523,33 +523,7 @@ static int streamzap_resume(struct usb_interface *intf)
        return 0;
 }
-/**
+module_usb_driver(streamzap_driver);
- *      streamzap_init
- */
-static int __init streamzap_init(void)
-{
-        int ret;
-        /* register this driver with the USB subsystem */
-        ret = usb_register(&streamzap_driver);
-        if (ret < 0)
-                printk(KERN_ERR DRIVER_NAME ": usb register failed, "
-                       "result = %d\n", ret);
-        return ret;
-}
-/**
- *      streamzap_exit
- */
-static void __exit streamzap_exit(void)
-{
-        usb_deregister(&streamzap_driver);
-}
-module_init(streamzap_init);
-module_exit(streamzap_exit);
 MODULE_AUTHOR("Jarod Wilson <jarod@wilsonet.com>");
 MODULE_DESCRIPTION(DRIVER_DESC);
diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond-cir.c
index 13f54b51194..b09c5fae489 100644
--- a/drivers/media/rc/winbond-cir.c
+++ b/drivers/media/rc/winbond-cir.c
@@ -226,11 +226,11 @@ module_param(protocol, uint, 0444);
 MODULE_PARM_DESC(protocol, "IR protocol to use for the power-on command "
                 "(0 = RC5, 1 = NEC, 2 = RC6A, default)");
-static int invert; /* default = 0 */
+static bool invert; /* default = 0 */
 module_param(invert, bool, 0444);
 MODULE_PARM_DESC(invert, "Invert the signal from the IR receiver");
-static int txandrx; /* default = 0 */
+static bool txandrx; /* default = 0 */
 module_param(txandrx, bool, 0444);
 MODULE_PARM_DESC(invert, "Allow simultaneous TX and RX");
@@ -1176,6 +1176,6 @@ wbcir_exit(void)
 module_init(wbcir_init);
 module_exit(wbcir_exit);
-MODULE_AUTHOR("David H�rdeman <david@hardeman.nu>");
+MODULE_AUTHOR("David Härdeman <david@hardeman.nu>");
 MODULE_DESCRIPTION("Winbond SuperI/O Consumer IR Driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/media/video/au0828/au0828-cards.c b/drivers/media/video/au0828/au0828-cards.c
index 39fc923fc46..1c6015a04f9 100644
--- a/drivers/media/video/au0828/au0828-cards.c
+++ b/drivers/media/video/au0828/au0828-cards.c
@@ -162,11 +162,14 @@ static void hauppauge_eeprom(struct au0828_dev *dev, u8 *eeprom_data)
        switch (tv.model) {
        case 72000: /* WinTV-HVR950q (Retail, IR, ATSC/QAM */
        case 72001: /* WinTV-HVR950q (Retail, IR, ATSC/QAM and analog video */
+        case 72101: /* WinTV-HVR950q (Retail, IR, ATSC/QAM and analog video */
+        case 72201: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
        case 72211: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
        case 72221: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
        case 72231: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
        case 72241: /* WinTV-HVR950q (OEM, No IR, ATSC/QAM and analog video */
        case 72251: /* WinTV-HVR950q (Retail, IR, ATSC/QAM and analog video */
+        case 72261: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
        case 72301: /* WinTV-HVR850 (Retail, IR, ATSC and analog video */
        case 72500: /* WinTV-HVR950q (OEM, No IR, ATSC/QAM */
                break;
@@ -324,6 +327,10 @@ struct usb_device_id au0828_usb_id_table[] = {
                .driver_info = AU0828_BOARD_HAUPPAUGE_HVR950Q_MXL },
        { USB_DEVICE(0x2040, 0x8200),
                .driver_info = AU0828_BOARD_HAUPPAUGE_WOODBURY },
+        { USB_DEVICE(0x2040, 0x7260),
+                .driver_info = AU0828_BOARD_HAUPPAUGE_HVR950Q },
+        { USB_DEVICE(0x2040, 0x7213),
+                .driver_info = AU0828_BOARD_HAUPPAUGE_HVR950Q },
        { },
 };
diff --git a/drivers/media/video/c-qcam.c b/drivers/media/video/c-qcam.c
index cd8ff047318..fda32f52554 100644
--- a/drivers/media/video/c-qcam.c
+++ b/drivers/media/video/c-qcam.c
@@ -72,7 +72,7 @@ struct qcam {
 static int parport[MAX_CAMS] = { [1 ... MAX_CAMS-1] = -1 };
 static int probe = 2;
-static int force_rgb;
+static bool force_rgb;
 static int video_nr = -1;
 /* FIXME: parport=auto would never have worked, surely? --RR */
diff --git a/drivers/media/video/cs5345.c b/drivers/media/video/cs5345.c
index 5909f2557ab..1d64af9adf7 100644
--- a/drivers/media/video/cs5345.c
+++ b/drivers/media/video/cs5345.c
@@ -31,7 +31,7 @@ MODULE_DESCRIPTION("i2c device driver for cs5345 Audio ADC");
 MODULE_AUTHOR("Hans Verkuil");
 MODULE_LICENSE("GPL");
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
diff --git a/drivers/media/video/cs53l32a.c b/drivers/media/video/cs53l32a.c
index d93e5ab45fd..51c5b9ad67d 100644
--- a/drivers/media/video/cs53l32a.c
+++ b/drivers/media/video/cs53l32a.c
@@ -35,7 +35,7 @@ MODULE_DESCRIPTION("i2c device driver for cs53l32a Audio ADC");
 MODULE_AUTHOR("Martin Vaughan");
 MODULE_LICENSE("GPL");
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
diff --git a/drivers/media/video/cx18/cx18-driver.c b/drivers/media/video/cx18/cx18-driver.c
index c6ff32a6137..349bd9c2aff 100644
--- a/drivers/media/video/cx18/cx18-driver.c
+++ b/drivers/media/video/cx18/cx18-driver.c
@@ -75,7 +75,7 @@ static int radio[CX18_MAX_CARDS] = { -1, -1, -1, -1, -1, -1, -1, -1,
                                     -1, -1, -1, -1, -1, -1, -1, -1 };
 static unsigned cardtype_c = 1;
 static unsigned tuner_c = 1;
-static unsigned radio_c = 1;
+static bool radio_c = 1;
 static char pal[] = "--";
 static char secam[] = "--";
 static char ntsc[] = "-";
diff --git a/drivers/media/video/cx231xx/cx231xx-cards.c b/drivers/media/video/cx231xx/cx231xx-cards.c
index 53dae2a8272..60b021e7986 100644
--- a/drivers/media/video/cx231xx/cx231xx-cards.c
+++ b/drivers/media/video/cx231xx/cx231xx-cards.c
@@ -1385,26 +1385,4 @@ static struct usb_driver cx231xx_usb_driver = {
        .id_table = cx231xx_id_table,
 };
-static int __init cx231xx_module_init(void)
+module_usb_driver(cx231xx_usb_driver);
-{
-        int result;
-        printk(KERN_INFO DRIVER_NAME " v4l2 driver loaded.\n");
-        /* register this driver with the USB subsystem */
-        result = usb_register(&cx231xx_usb_driver);
-        if (result)
-                cx231xx_err(DRIVER_NAME
-                            " usb_register failed. Error number %d.\n", result);
-        return result;
-}
-static void __exit cx231xx_module_exit(void)
-{
-        /* deregister this driver with the USB subsystem */
-        usb_deregister(&cx231xx_usb_driver);
-}
-module_init(cx231xx_module_init);
-module_exit(cx231xx_module_exit);
diff --git a/drivers/media/video/cx25821/cx25821-alsa.c b/drivers/media/video/cx25821/cx25821-alsa.c
index 09e99de5fd2..58be4f3bb3c 100644
--- a/drivers/media/video/cx25821/cx25821-alsa.c
+++ b/drivers/media/video/cx25821/cx25821-alsa.c
@@ -102,7 +102,7 @@ struct cx25821_audio_dev {
 static int index[SNDRV_CARDS] = SNDRV_DEFAULT_IDX;      /* Index 0-MAX */
 static char *id[SNDRV_CARDS] = SNDRV_DEFAULT_STR;       /* ID for this card */
-static int enable[SNDRV_CARDS] = { 1, [1 ... (SNDRV_CARDS - 1)] = 1 };
+static bool enable[SNDRV_CARDS] = { 1, [1 ... (SNDRV_CARDS - 1)] = 1 };
 module_param_array(enable, bool, NULL, 0444);
 MODULE_PARM_DESC(enable, "Enable cx25821 soundcard. default enabled.");
diff --git a/drivers/media/video/cx88/cx88-alsa.c b/drivers/media/video/cx88/cx88-alsa.c
index 68d1240f493..04bf6627d36 100644
--- a/drivers/media/video/cx88/cx88-alsa.c
+++ b/drivers/media/video/cx88/cx88-alsa.c
@@ -96,7 +96,7 @@ typedef struct cx88_audio_dev snd_cx88_card_t;
 static int index[SNDRV_CARDS] = SNDRV_DEFAULT_IDX;      /* Index 0-MAX */
 static const char *id[SNDRV_CARDS] = SNDRV_DEFAULT_STR; /* ID for this card */
-static int enable[SNDRV_CARDS] = {1, [1 ... (SNDRV_CARDS - 1)] = 1};
+static bool enable[SNDRV_CARDS] = {1, [1 ... (SNDRV_CARDS - 1)] = 1};
 module_param_array(enable, bool, NULL, 0444);
 MODULE_PARM_DESC(enable, "Enable cx88x soundcard. default enabled.");
diff --git a/drivers/media/video/davinci/vpif.h b/drivers/media/video/davinci/vpif.h
index 10550bd93b0..25036cb11be 100644
--- a/drivers/media/video/davinci/vpif.h
+++ b/drivers/media/video/davinci/vpif.h
@@ -20,6 +20,7 @@
 #include <linux/videodev2.h>
 #include <mach/hardware.h>
 #include <mach/dm646x.h>
+#include <media/davinci/vpif_types.h>
 /* Maximum channel allowed */
 #define VPIF_NUM_CHANNELS               (4)
diff --git a/drivers/media/video/davinci/vpif_capture.h b/drivers/media/video/davinci/vpif_capture.h
index 064550f5ce4..a693d4ebda5 100644
--- a/drivers/media/video/davinci/vpif_capture.h
+++ b/drivers/media/video/davinci/vpif_capture.h
@@ -27,7 +27,7 @@
 #include <media/v4l2-device.h>
 #include <media/videobuf-core.h>
 #include <media/videobuf-dma-contig.h>
-#include <mach/dm646x.h>
+#include <media/davinci/vpif_types.h>
 #include "vpif.h"
diff --git a/drivers/media/video/davinci/vpif_display.h b/drivers/media/video/davinci/vpif_display.h
index 5d1936dafed..56879d1a068 100644
--- a/drivers/media/video/davinci/vpif_display.h
+++ b/drivers/media/video/davinci/vpif_display.h
@@ -22,6 +22,7 @@
 #include <media/v4l2-device.h>
 #include <media/videobuf-core.h>
 #include <media/videobuf-dma-contig.h>
+#include <media/davinci/vpif_types.h>
 #include "vpif.h"
diff --git a/drivers/media/video/em28xx/em28xx-cards.c b/drivers/media/video/em28xx/em28xx-cards.c
index 9b747c266af..93807dcf944 100644
--- a/drivers/media/video/em28xx/em28xx-cards.c
+++ b/drivers/media/video/em28xx/em28xx-cards.c
@@ -3325,26 +3325,4 @@ static struct usb_driver em28xx_usb_driver = {
        .id_table = em28xx_id_table,
 };
-static int __init em28xx_module_init(void)
+module_usb_driver(em28xx_usb_driver);
-{
-        int result;
-        /* register this driver with the USB subsystem */
-        result = usb_register(&em28xx_usb_driver);
-        if (result)
-                em28xx_err(DRIVER_NAME
-                           " usb_register failed. Error number %d.\n", result);
-        printk(KERN_INFO DRIVER_NAME " driver loaded\n");
-        return result;
-}
-static void __exit em28xx_module_exit(void)
-{
-        /* deregister this driver with the USB subsystem */
-        usb_deregister(&em28xx_usb_driver);
-}
-module_init(em28xx_module_init);
-module_exit(em28xx_module_exit);
diff --git a/drivers/media/video/et61x251/et61x251_core.c b/drivers/media/video/et61x251/et61x251_core.c
index d3777c86e1d..5539f09440a 100644
--- a/drivers/media/video/et61x251/et61x251_core.c
+++ b/drivers/media/video/et61x251/et61x251_core.c
@@ -76,8 +76,8 @@ MODULE_PARM_DESC(video_nr,
                 "\none and for every other camera."
                 "\n");
-static short force_munmap[] = {[0 ... ET61X251_MAX_DEVICES-1] =
+static bool force_munmap[] = {[0 ... ET61X251_MAX_DEVICES-1] =
-                               ET61X251_FORCE_MUNMAP};
+                              ET61X251_FORCE_MUNMAP};
 module_param_array(force_munmap, bool, NULL, 0444);
 MODULE_PARM_DESC(force_munmap,
                 "\n<0|1[,...]> Force the application to unmap previously"
@@ -2680,27 +2680,4 @@ static struct usb_driver et61x251_usb_driver = {
        .disconnect = et61x251_usb_disconnect,
 };
-/*****************************************************************************/
+module_usb_driver(et61x251_usb_driver);
-static int __init et61x251_module_init(void)
-{
-        int err = 0;
-        KDBG(2, ET61X251_MODULE_NAME " v" ET61X251_MODULE_VERSION);
-        KDBG(3, ET61X251_MODULE_AUTHOR);
-        if ((err = usb_register(&et61x251_usb_driver)))
-                KDBG(1, "usb_register() failed");
-        return err;
-}
-static void __exit et61x251_module_exit(void)
-{
-        usb_deregister(&et61x251_usb_driver);
-}
-module_init(et61x251_module_init);
-module_exit(et61x251_module_exit);
diff --git a/drivers/media/video/gspca/benq.c b/drivers/media/video/gspca/benq.c
index 6ae26160b81..636627b57dc 100644
--- a/drivers/media/video/gspca/benq.c
+++ b/drivers/media/video/gspca/benq.c
@@ -288,15 +288,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/conex.c b/drivers/media/video/gspca/conex.c
index 4c56dbef6d9..ea17b5d94ea 100644
--- a/drivers/media/video/gspca/conex.c
+++ b/drivers/media/video/gspca/conex.c
@@ -1067,15 +1067,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/cpia1.c b/drivers/media/video/gspca/cpia1.c
index f9b86b2484b..8f33bbd091a 100644
--- a/drivers/media/video/gspca/cpia1.c
+++ b/drivers/media/video/gspca/cpia1.c
@@ -2132,15 +2132,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/etoms.c b/drivers/media/video/gspca/etoms.c
index 0357d6d461d..81a4adbd9f7 100644
--- a/drivers/media/video/gspca/etoms.c
+++ b/drivers/media/video/gspca/etoms.c
@@ -895,16 +895,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/finepix.c b/drivers/media/video/gspca/finepix.c
index ea48200fd3a..0107513cd72 100644
--- a/drivers/media/video/gspca/finepix.c
+++ b/drivers/media/video/gspca/finepix.c
@@ -290,16 +290,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/gl860/gl860.c b/drivers/media/video/gspca/gl860/gl860.c
index 2ced3b73994..a8f54c20e58 100644
--- a/drivers/media/video/gspca/gl860/gl860.c
+++ b/drivers/media/video/gspca/gl860/gl860.c
@@ -524,22 +524,7 @@ static struct usb_driver sd_driver = {
 /*====================== Init and Exit module functions ====================*/
-static int __init sd_mod_init(void)
+module_usb_driver(sd_driver);
-{
-        PDEBUG(D_PROBE, "driver startup - version %s", DRIVER_VERSION);
-        if (usb_register(&sd_driver) < 0)
-                return -1;
-        return 0;
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
 /*==========================================================================*/
diff --git a/drivers/media/video/gspca/gspca.c b/drivers/media/video/gspca/gspca.c
index 881e04c7ffe..2ca10dfec91 100644
--- a/drivers/media/video/gspca/gspca.c
+++ b/drivers/media/video/gspca/gspca.c
@@ -838,13 +838,13 @@ static int gspca_init_transfer(struct gspca_dev *gspca_dev)
        gspca_dev->usb_err = 0;
        /* do the specific subdriver stuff before endpoint selection */
-        gspca_dev->alt = 0;
+        intf = usb_ifnum_to_if(gspca_dev->dev, gspca_dev->iface);
+        gspca_dev->alt = gspca_dev->cam.bulk ? intf->num_altsetting : 0;
        if (gspca_dev->sd_desc->isoc_init) {
                ret = gspca_dev->sd_desc->isoc_init(gspca_dev);
                if (ret < 0)
                        goto unlock;
        }
-        intf = usb_ifnum_to_if(gspca_dev->dev, gspca_dev->iface);
        xfer = gspca_dev->cam.bulk ? USB_ENDPOINT_XFER_BULK
                                   : USB_ENDPOINT_XFER_ISOC;
@@ -957,7 +957,7 @@ retry:
                                ret = -EIO;
                                goto out;
                        }
-                        alt = ep_tb[--alt_idx].alt;
+                        gspca_dev->alt = ep_tb[--alt_idx].alt;
                }
        }
 out:
diff --git a/drivers/media/video/gspca/jeilinj.c b/drivers/media/video/gspca/jeilinj.c
index 8e3dabe3007..5ab3f7e1276 100644
--- a/drivers/media/video/gspca/jeilinj.c
+++ b/drivers/media/video/gspca/jeilinj.c
@@ -582,16 +582,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/kinect.c b/drivers/media/video/gspca/kinect.c
index 4fe51fda80f..e8e8f2fe916 100644
--- a/drivers/media/video/gspca/kinect.c
+++ b/drivers/media/video/gspca/kinect.c
@@ -413,16 +413,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/konica.c b/drivers/media/video/gspca/konica.c
index f3f7fe0ec4b..b1da7f4096c 100644
--- a/drivers/media/video/gspca/konica.c
+++ b/drivers/media/video/gspca/konica.c
@@ -634,15 +634,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/m5602/m5602_core.c b/drivers/media/video/gspca/m5602/m5602_core.c
index 67533e5582a..0c449367543 100644
--- a/drivers/media/video/gspca/m5602/m5602_core.c
+++ b/drivers/media/video/gspca/m5602/m5602_core.c
@@ -27,8 +27,8 @@
 /* Kernel module parameters */
 int force_sensor;
-static int dump_bridge;
+static bool dump_bridge;
-int dump_sensor;
+bool dump_sensor;
 static const struct usb_device_id m5602_table[] = {
        {USB_DEVICE(0x0402, 0x5602)},
@@ -404,19 +404,7 @@ static struct usb_driver sd_driver = {
        .disconnect = m5602_disconnect
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init mod_m5602_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit mod_m5602_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(mod_m5602_init);
-module_exit(mod_m5602_exit);
 MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_DESC);
diff --git a/drivers/media/video/gspca/m5602/m5602_mt9m111.h b/drivers/media/video/gspca/m5602/m5602_mt9m111.h
index b1f0c492036..8c672b5c8c6 100644
--- a/drivers/media/video/gspca/m5602/m5602_mt9m111.h
+++ b/drivers/media/video/gspca/m5602/m5602_mt9m111.h
@@ -106,7 +106,7 @@
 /* Kernel module parameters */
 extern int force_sensor;
-extern int dump_sensor;
+extern bool dump_sensor;
 int mt9m111_probe(struct sd *sd);
 int mt9m111_init(struct sd *sd);
diff --git a/drivers/media/video/gspca/m5602/m5602_ov7660.h b/drivers/media/video/gspca/m5602/m5602_ov7660.h
index 2efd607987e..2b6a13b508f 100644
--- a/drivers/media/video/gspca/m5602/m5602_ov7660.h
+++ b/drivers/media/video/gspca/m5602/m5602_ov7660.h
@@ -86,7 +86,7 @@
 /* Kernel module parameters */
 extern int force_sensor;
-extern int dump_sensor;
+extern bool dump_sensor;
 int ov7660_probe(struct sd *sd);
 int ov7660_init(struct sd *sd);
diff --git a/drivers/media/video/gspca/m5602/m5602_ov9650.h b/drivers/media/video/gspca/m5602/m5602_ov9650.h
index da9a129b739..f7aa5bf6898 100644
--- a/drivers/media/video/gspca/m5602/m5602_ov9650.h
+++ b/drivers/media/video/gspca/m5602/m5602_ov9650.h
@@ -135,7 +135,7 @@
 /* Kernel module parameters */
 extern int force_sensor;
-extern int dump_sensor;
+extern bool dump_sensor;
 int ov9650_probe(struct sd *sd);
 int ov9650_init(struct sd *sd);
diff --git a/drivers/media/video/gspca/m5602/m5602_po1030.h b/drivers/media/video/gspca/m5602/m5602_po1030.h
index 33835959639..81a2bcb88fe 100644
--- a/drivers/media/video/gspca/m5602/m5602_po1030.h
+++ b/drivers/media/video/gspca/m5602/m5602_po1030.h
@@ -147,7 +147,7 @@
 /* Kernel module parameters */
 extern int force_sensor;
-extern int dump_sensor;
+extern bool dump_sensor;
 int po1030_probe(struct sd *sd);
 int po1030_init(struct sd *sd);
diff --git a/drivers/media/video/gspca/m5602/m5602_s5k4aa.h b/drivers/media/video/gspca/m5602/m5602_s5k4aa.h
index 8cc7a3f6da7..8e0035e731c 100644
--- a/drivers/media/video/gspca/m5602/m5602_s5k4aa.h
+++ b/drivers/media/video/gspca/m5602/m5602_s5k4aa.h
@@ -65,7 +65,7 @@
 /* Kernel module parameters */
 extern int force_sensor;
-extern int dump_sensor;
+extern bool dump_sensor;
 int s5k4aa_probe(struct sd *sd);
 int s5k4aa_init(struct sd *sd);
diff --git a/drivers/media/video/gspca/m5602/m5602_s5k83a.h b/drivers/media/video/gspca/m5602/m5602_s5k83a.h
index 80a63a236e2..79952247b53 100644
--- a/drivers/media/video/gspca/m5602/m5602_s5k83a.h
+++ b/drivers/media/video/gspca/m5602/m5602_s5k83a.h
@@ -41,7 +41,7 @@
 /* Kernel module parameters */
 extern int force_sensor;
-extern int dump_sensor;
+extern bool dump_sensor;
 int s5k83a_probe(struct sd *sd);
 int s5k83a_init(struct sd *sd);
diff --git a/drivers/media/video/gspca/mars.c b/drivers/media/video/gspca/mars.c
index ef45fa57575..5c2ea05c46b 100644
--- a/drivers/media/video/gspca/mars.c
+++ b/drivers/media/video/gspca/mars.c
@@ -517,15 +517,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/mr97310a.c b/drivers/media/video/gspca/mr97310a.c
index 473e813b680..d73e5bd3dbf 100644
--- a/drivers/media/video/gspca/mr97310a.c
+++ b/drivers/media/video/gspca/mr97310a.c
@@ -1259,15 +1259,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/nw80x.c b/drivers/media/video/gspca/nw80x.c
index 7681814e594..d4bec932177 100644
--- a/drivers/media/video/gspca/nw80x.c
+++ b/drivers/media/video/gspca/nw80x.c
@@ -2118,18 +2118,7 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
 module_param(webcam, int, 0644);
 MODULE_PARM_DESC(webcam,
diff --git a/drivers/media/video/gspca/ov519.c b/drivers/media/video/gspca/ov519.c
index 6a01b35a947..08b8ce1dee1 100644
--- a/drivers/media/video/gspca/ov519.c
+++ b/drivers/media/video/gspca/ov519.c
@@ -5056,18 +5056,7 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
 module_param(frame_rate, int, 0644);
 MODULE_PARM_DESC(frame_rate, "Frame rate (5, 10, 15, 20 or 30 fps)");
diff --git a/drivers/media/video/gspca/ov534.c b/drivers/media/video/gspca/ov534.c
index 76907eced4a..04753391de3 100644
--- a/drivers/media/video/gspca/ov534.c
+++ b/drivers/media/video/gspca/ov534.c
@@ -1533,16 +1533,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/ov534_9.c b/drivers/media/video/gspca/ov534_9.c
index b3b1ea60a84..f30060d5063 100644
--- a/drivers/media/video/gspca/ov534_9.c
+++ b/drivers/media/video/gspca/ov534_9.c
@@ -1432,16 +1432,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/pac207.c b/drivers/media/video/gspca/pac207.c
index 1600df152fd..ece8b1e82a1 100644
--- a/drivers/media/video/gspca/pac207.c
+++ b/drivers/media/video/gspca/pac207.c
@@ -569,15 +569,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/pac7302.c b/drivers/media/video/gspca/pac7302.c
index 1c44f78ff9e..2811195258c 100644
--- a/drivers/media/video/gspca/pac7302.c
+++ b/drivers/media/video/gspca/pac7302.c
@@ -1220,15 +1220,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/pac7311.c b/drivers/media/video/gspca/pac7311.c
index 7509d05dc06..1ac111176ff 100644
--- a/drivers/media/video/gspca/pac7311.c
+++ b/drivers/media/video/gspca/pac7311.c
@@ -868,15 +868,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/se401.c b/drivers/media/video/gspca/se401.c
index 3b71bbcd977..1494e1829d3 100644
--- a/drivers/media/video/gspca/se401.c
+++ b/drivers/media/video/gspca/se401.c
@@ -766,15 +766,4 @@ static struct usb_driver sd_driver = {
        .post_reset = sd_post_reset,
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sn9c2028.c b/drivers/media/video/gspca/sn9c2028.c
index 48aae3926a3..478533cb115 100644
--- a/drivers/media/video/gspca/sn9c2028.c
+++ b/drivers/media/video/gspca/sn9c2028.c
@@ -737,16 +737,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sn9c20x.c b/drivers/media/video/gspca/sn9c20x.c
index 86e07a139a1..33cabc342dc 100644
--- a/drivers/media/video/gspca/sn9c20x.c
+++ b/drivers/media/video/gspca/sn9c20x.c
@@ -2554,15 +2554,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sonixb.c b/drivers/media/video/gspca/sonixb.c
index 146b459b08d..ddb392dc4f2 100644
--- a/drivers/media/video/gspca/sonixb.c
+++ b/drivers/media/video/gspca/sonixb.c
@@ -1527,15 +1527,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sonixj.c b/drivers/media/video/gspca/sonixj.c
index c746bf19ca1..afa3186b803 100644
--- a/drivers/media/video/gspca/sonixj.c
+++ b/drivers/media/video/gspca/sonixj.c
@@ -3104,15 +3104,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca1528.c b/drivers/media/video/gspca/spca1528.c
index 695673106e7..070b9c33b51 100644
--- a/drivers/media/video/gspca/spca1528.c
+++ b/drivers/media/video/gspca/spca1528.c
@@ -590,15 +590,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca500.c b/drivers/media/video/gspca/spca500.c
index bb82c94ece1..103984708c7 100644
--- a/drivers/media/video/gspca/spca500.c
+++ b/drivers/media/video/gspca/spca500.c
@@ -1092,15 +1092,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca501.c b/drivers/media/video/gspca/spca501.c
index 7aaac72aee9..9c16821addd 100644
--- a/drivers/media/video/gspca/spca501.c
+++ b/drivers/media/video/gspca/spca501.c
@@ -2188,15 +2188,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca505.c b/drivers/media/video/gspca/spca505.c
index 16722dc6039..1320f35e39f 100644
--- a/drivers/media/video/gspca/spca505.c
+++ b/drivers/media/video/gspca/spca505.c
@@ -815,15 +815,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca506.c b/drivers/media/video/gspca/spca506.c
index 89fec4c500a..54eed87672d 100644
--- a/drivers/media/video/gspca/spca506.c
+++ b/drivers/media/video/gspca/spca506.c
@@ -714,21 +714,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        int ret;
-        ret = usb_register(&sd_driver);
-        if (ret < 0)
-                return ret;
-        PDEBUG(D_PROBE, "registered");
-        return 0;
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-        PDEBUG(D_PROBE, "deregistered");
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca508.c b/drivers/media/video/gspca/spca508.c
index a44fe3d2596..df4e1699646 100644
--- a/drivers/media/video/gspca/spca508.c
+++ b/drivers/media/video/gspca/spca508.c
@@ -1544,15 +1544,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/spca561.c b/drivers/media/video/gspca/spca561.c
index c82fd53cef9..259a0c73c66 100644
--- a/drivers/media/video/gspca/spca561.c
+++ b/drivers/media/video/gspca/spca561.c
@@ -1106,15 +1106,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sq905.c b/drivers/media/video/gspca/sq905.c
index df805f79828..2fe3c29bd6b 100644
--- a/drivers/media/video/gspca/sq905.c
+++ b/drivers/media/video/gspca/sq905.c
@@ -432,16 +432,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sq905c.c b/drivers/media/video/gspca/sq905c.c
index c2c056056e0..ae783634712 100644
--- a/drivers/media/video/gspca/sq905c.c
+++ b/drivers/media/video/gspca/sq905c.c
@@ -339,16 +339,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/sq930x.c b/drivers/media/video/gspca/sq930x.c
index e4255b4905e..1a8ba9b3550 100644
--- a/drivers/media/video/gspca/sq930x.c
+++ b/drivers/media/video/gspca/sq930x.c
@@ -1197,15 +1197,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/stk014.c b/drivers/media/video/gspca/stk014.c
index 42a7a28a6c8..4ae7cc8f463 100644
--- a/drivers/media/video/gspca/stk014.c
+++ b/drivers/media/video/gspca/stk014.c
@@ -519,15 +519,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/stv0680.c b/drivers/media/video/gspca/stv0680.c
index 4dcc7e37f9f..461ed645f30 100644
--- a/drivers/media/video/gspca/stv0680.c
+++ b/drivers/media/video/gspca/stv0680.c
@@ -355,15 +355,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/stv06xx/stv06xx.c b/drivers/media/video/gspca/stv06xx/stv06xx.c
index b1fca7db101..6f878f6c6e9 100644
--- a/drivers/media/video/gspca/stv06xx/stv06xx.c
+++ b/drivers/media/video/gspca/stv06xx/stv06xx.c
@@ -36,8 +36,8 @@ MODULE_AUTHOR("Erik Andrén");
 MODULE_DESCRIPTION("STV06XX USB Camera Driver");
 MODULE_LICENSE("GPL");
-static int dump_bridge;
+static bool dump_bridge;
-static int dump_sensor;
+static bool dump_sensor;
 int stv06xx_write_bridge(struct sd *sd, u16 address, u16 i2c_data)
 {
@@ -612,18 +612,7 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
 module_param(dump_bridge, bool, S_IRUGO | S_IWUSR);
 MODULE_PARM_DESC(dump_bridge, "Dumps all usb bridge registers at startup");
diff --git a/drivers/media/video/gspca/sunplus.c b/drivers/media/video/gspca/sunplus.c
index c8909772435..c80f0c0c75b 100644
--- a/drivers/media/video/gspca/sunplus.c
+++ b/drivers/media/video/gspca/sunplus.c
@@ -1211,15 +1211,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/t613.c b/drivers/media/video/gspca/t613.c
index 90f0877eb59..ea44deb66af 100644
--- a/drivers/media/video/gspca/t613.c
+++ b/drivers/media/video/gspca/t613.c
@@ -1438,15 +1438,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/topro.c b/drivers/media/video/gspca/topro.c
index 29596c59837..b2695b1dc60 100644
--- a/drivers/media/video/gspca/topro.c
+++ b/drivers/media/video/gspca/topro.c
@@ -4971,18 +4971,7 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
 module_param(force_sensor, int, 0644);
 MODULE_PARM_DESC(force_sensor,
diff --git a/drivers/media/video/gspca/tv8532.c b/drivers/media/video/gspca/tv8532.c
index 933ef2ca658..c8922c5ffbf 100644
--- a/drivers/media/video/gspca/tv8532.c
+++ b/drivers/media/video/gspca/tv8532.c
@@ -418,16 +418,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/vc032x.c b/drivers/media/video/gspca/vc032x.c
index 7ee2c8271dc..208f6b2d512 100644
--- a/drivers/media/video/gspca/vc032x.c
+++ b/drivers/media/video/gspca/vc032x.c
@@ -4230,15 +4230,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/vicam.c b/drivers/media/video/gspca/vicam.c
index 81dd4c99d02..d12ea1518ac 100644
--- a/drivers/media/video/gspca/vicam.c
+++ b/drivers/media/video/gspca/vicam.c
@@ -368,16 +368,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/xirlink_cit.c b/drivers/media/video/gspca/xirlink_cit.c
index 3aed42acdb5..fbb6ed25ec3 100644
--- a/drivers/media/video/gspca/xirlink_cit.c
+++ b/drivers/media/video/gspca/xirlink_cit.c
@@ -3325,15 +3325,4 @@ static struct usb_driver sd_driver = {
 #endif
 };
-/* -- module insert / remove -- */
+module_usb_driver(sd_driver);
-static int __init sd_mod_init(void)
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
diff --git a/drivers/media/video/gspca/zc3xx.c b/drivers/media/video/gspca/zc3xx.c
index 30ea1e47949..0202fead6b9 100644
--- a/drivers/media/video/gspca/zc3xx.c
+++ b/drivers/media/video/gspca/zc3xx.c
@@ -7050,18 +7050,7 @@ static struct usb_driver sd_driver = {
 #endif
 };
-static int __init sd_mod_init(void)
+module_usb_driver(sd_driver);
-{
-        return usb_register(&sd_driver);
-}
-static void __exit sd_mod_exit(void)
-{
-        usb_deregister(&sd_driver);
-}
-module_init(sd_mod_init);
-module_exit(sd_mod_exit);
 module_param(force_sensor, int, 0644);
 MODULE_PARM_DESC(force_sensor,
diff --git a/drivers/media/video/hdpvr/hdpvr-core.c b/drivers/media/video/hdpvr/hdpvr-core.c
index 441dacf642b..e5eb56a5b61 100644
--- a/drivers/media/video/hdpvr/hdpvr-core.c
+++ b/drivers/media/video/hdpvr/hdpvr-core.c
@@ -49,7 +49,7 @@ module_param(default_audio_input, uint, S_IRUGO|S_IWUSR);
 MODULE_PARM_DESC(default_audio_input, "default audio input: 0=RCA back / "
                 "1=RCA front / 2=S/PDIF");
-static int boost_audio;
+static bool boost_audio;
 module_param(boost_audio, bool, S_IRUGO|S_IWUSR);
 MODULE_PARM_DESC(boost_audio, "boost the audio signal");
@@ -452,26 +452,7 @@ static struct usb_driver hdpvr_usb_driver = {
        .id_table =     hdpvr_table,
 };
-static int __init hdpvr_init(void)
+module_usb_driver(hdpvr_usb_driver);
-{
-        int result;
-        /* register this driver with the USB subsystem */
-        result = usb_register(&hdpvr_usb_driver);
-        if (result)
-                err("usb_register failed. Error number %d", result);
-        return result;
-}
-static void __exit hdpvr_exit(void)
-{
-        /* deregister this driver with the USB subsystem */
-        usb_deregister(&hdpvr_usb_driver);
-}
-module_init(hdpvr_init);
-module_exit(hdpvr_exit);
 MODULE_LICENSE("GPL");
 MODULE_VERSION("0.2.1");
diff --git a/drivers/media/video/ivtv/ivtv-driver.c b/drivers/media/video/ivtv/ivtv-driver.c
index 41108a9a195..544af91cbdc 100644
--- a/drivers/media/video/ivtv/ivtv-driver.c
+++ b/drivers/media/video/ivtv/ivtv-driver.c
@@ -99,7 +99,7 @@ static int i2c_clock_period[IVTV_MAX_CARDS] = { -1, -1, -1, -1, -1, -1, -1, -1,
 static unsigned int cardtype_c = 1;
 static unsigned int tuner_c = 1;
-static unsigned int radio_c = 1;
+static bool radio_c = 1;
 static unsigned int i2c_clock_period_c = 1;
 static char pal[] = "---";
 static char secam[] = "--";
diff --git a/drivers/media/video/ivtv/ivtvfb.c b/drivers/media/video/ivtv/ivtvfb.c
index 6b7c9c82333..d0fbfcf7133 100644
--- a/drivers/media/video/ivtv/ivtvfb.c
+++ b/drivers/media/video/ivtv/ivtvfb.c
@@ -58,7 +58,7 @@
 /* card parameters */
 static int ivtvfb_card_id = -1;
 static int ivtvfb_debug = 0;
-static int osd_laced;
+static bool osd_laced;
 static int osd_depth;
 static int osd_upper;
 static int osd_left;
diff --git a/drivers/media/video/m5mols/m5mols.h b/drivers/media/video/m5mols/m5mols.h
index 89d09a8914f..82c8817bd32 100644
--- a/drivers/media/video/m5mols/m5mols.h
+++ b/drivers/media/video/m5mols/m5mols.h
@@ -162,7 +162,6 @@ struct m5mols_version {
 * @pad: media pad
 * @ffmt: current fmt according to resolution type
 * @res_type: current resolution type
- * @code: current code
 * @irq_waitq: waitqueue for the capture
 * @work_irq: workqueue for the IRQ
 * @flags: state variable for the interrupt handler
@@ -192,7 +191,6 @@ struct m5mols_info {
        struct media_pad pad;
        struct v4l2_mbus_framefmt ffmt[M5MOLS_RESTYPE_MAX];
        int res_type;
-        enum v4l2_mbus_pixelcode code;
        wait_queue_head_t irq_waitq;
        struct work_struct work_irq;
        unsigned long flags;
diff --git a/drivers/media/video/m5mols/m5mols_core.c b/drivers/media/video/m5mols/m5mols_core.c
index 05ab3700647..e0f09e53180 100644
--- a/drivers/media/video/m5mols/m5mols_core.c
+++ b/drivers/media/video/m5mols/m5mols_core.c
@@ -334,7 +334,7 @@ int m5mols_mode(struct m5mols_info *info, u8 mode)
        int ret = -EINVAL;
        u8 reg;
-        if (mode < REG_PARAMETER && mode > REG_CAPTURE)
+        if (mode < REG_PARAMETER || mode > REG_CAPTURE)
                return ret;
        ret = m5mols_read_u8(sd, SYSTEM_SYSMODE, &reg);
@@ -511,9 +511,6 @@ static int m5mols_get_fmt(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh,
        struct m5mols_info *info = to_m5mols(sd);
        struct v4l2_mbus_framefmt *format;
-        if (fmt->pad != 0)
-                return -EINVAL;
        format = __find_format(info, fh, fmt->which, info->res_type);
        if (!format)
                return -EINVAL;
@@ -532,9 +529,6 @@ static int m5mols_set_fmt(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh,
        u32 resolution = 0;
        int ret;
-        if (fmt->pad != 0)
-                return -EINVAL;
        ret = __find_resolution(sd, format, &type, &resolution);
        if (ret < 0)
                return ret;
@@ -543,13 +537,14 @@ static int m5mols_set_fmt(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh,
        if (!sfmt)
                return 0;
-        *sfmt           = m5mols_default_ffmt[type];
-        sfmt->width     = format->width;
+        format->code = m5mols_default_ffmt[type].code;
-        sfmt->height    = format->height;
+        format->colorspace = V4L2_COLORSPACE_JPEG;
+        format->field = V4L2_FIELD_NONE;
        if (fmt->which == V4L2_SUBDEV_FORMAT_ACTIVE) {
+                *sfmt = *format;
                info->resolution = resolution;
-                info->code = format->code;
                info->res_type = type;
        }
@@ -626,13 +621,14 @@ static int m5mols_start_monitor(struct m5mols_info *info)
 static int m5mols_s_stream(struct v4l2_subdev *sd, int enable)
 {
        struct m5mols_info *info = to_m5mols(sd);
+        u32 code = info->ffmt[info->res_type].code;
        if (enable) {
                int ret = -EINVAL;
-                if (is_code(info->code, M5MOLS_RESTYPE_MONITOR))
+                if (is_code(code, M5MOLS_RESTYPE_MONITOR))
                        ret = m5mols_start_monitor(info);
-                if (is_code(info->code, M5MOLS_RESTYPE_CAPTURE))
+                if (is_code(code, M5MOLS_RESTYPE_CAPTURE))
                        ret = m5mols_start_capture(info);
                return ret;
diff --git a/drivers/media/video/marvell-ccic/mcam-core.c b/drivers/media/video/marvell-ccic/mcam-core.c
index 80ec64d2d6d..2c8fc0f6d69 100644
--- a/drivers/media/video/marvell-ccic/mcam-core.c
+++ b/drivers/media/video/marvell-ccic/mcam-core.c
@@ -51,7 +51,7 @@ static int delivered;
 * sense.
 */
-static int alloc_bufs_at_read;
+static bool alloc_bufs_at_read;
 module_param(alloc_bufs_at_read, bool, 0444);
 MODULE_PARM_DESC(alloc_bufs_at_read,
                "Non-zero value causes DMA buffers to be allocated when the "
@@ -73,11 +73,11 @@ MODULE_PARM_DESC(dma_buf_size,
                "parameters require larger buffers, an attempt to reallocate "
                "will be made.");
 #else /* MCAM_MODE_VMALLOC */
-static const int alloc_bufs_at_read = 0;
+static const bool alloc_bufs_at_read = 0;
 static const int n_dma_bufs = 3;  /* Used by S/G_PARM */
 #endif /* MCAM_MODE_VMALLOC */
-static int flip;
+static bool flip;
 module_param(flip, bool, 0444);
 MODULE_PARM_DESC(flip,
                "If set, the sensor will be instructed to flip the image "
diff --git a/drivers/media/video/msp3400-driver.c b/drivers/media/video/msp3400-driver.c
index d0f53885728..d7cd0f633f6 100644
--- a/drivers/media/video/msp3400-driver.c
+++ b/drivers/media/video/msp3400-driver.c
@@ -69,12 +69,12 @@ MODULE_LICENSE("GPL");
 /* module parameters */
 static int opmode   = OPMODE_AUTO;
 int msp_debug;           /* msp_debug output */
-int msp_once;            /* no continuous stereo monitoring */
+bool msp_once;           /* no continuous stereo monitoring */
-int msp_amsound;         /* hard-wire AM sound at 6.5 Hz (france),
+bool msp_amsound;        /* hard-wire AM sound at 6.5 Hz (france),
                            the autoscan seems work well only with FM... */
 int msp_standard = 1;    /* Override auto detect of audio msp_standard,
                            if needed. */
-int msp_dolby;
+bool msp_dolby;
 int msp_stereo_thresh = 0x190; /* a2 threshold for stereo/bilingual
                                        (msp34xxg only) 0x00a0-0x03c0 */
diff --git a/drivers/media/video/msp3400-driver.h b/drivers/media/video/msp3400-driver.h
index 831e8db4368..fbe5e0715f9 100644
--- a/drivers/media/video/msp3400-driver.h
+++ b/drivers/media/video/msp3400-driver.h
@@ -44,10 +44,10 @@
 /* module parameters */
 extern int msp_debug;
-extern int msp_once;
+extern bool msp_once;
-extern int msp_amsound;
+extern bool msp_amsound;
 extern int msp_standard;
-extern int msp_dolby;
+extern bool msp_dolby;
 extern int msp_stereo_thresh;
 struct msp_state {
diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c
index cf2c0fb95f2..398f96ffd35 100644
--- a/drivers/media/video/mt9m111.c
+++ b/drivers/media/video/mt9m111.c
@@ -955,6 +955,7 @@ static int mt9m111_probe(struct i2c_client *client,
        mt9m111->rect.height    = MT9M111_MAX_HEIGHT;
        mt9m111->fmt            = &mt9m111_colour_fmts[0];
        mt9m111->lastpage       = -1;
+        mutex_init(&mt9m111->power_lock);
        ret = mt9m111_video_probe(client);
        if (ret) {
diff --git a/drivers/media/video/mt9t112.c b/drivers/media/video/mt9t112.c
index 32114a3c0ca..7b34b11daf2 100644
--- a/drivers/media/video/mt9t112.c
+++ b/drivers/media/video/mt9t112.c
@@ -1083,8 +1083,10 @@ static int mt9t112_probe(struct i2c_client *client,
        v4l2_i2c_subdev_init(&priv->subdev, client, &mt9t112_subdev_ops);
        ret = mt9t112_camera_probe(client);
-        if (ret)
+        if (ret) {
                kfree(priv);
+                return ret;
+        }
        /* Cannot fail: using the default supported pixel code */
        mt9t112_set_params(priv, &rect, V4L2_MBUS_FMT_UYVY8_2X8);
diff --git a/drivers/media/video/omap/omap_vout.c b/drivers/media/video/omap/omap_vout.c
index 9c5c19f142d..a378c2ce127 100644
--- a/drivers/media/video/omap/omap_vout.c
+++ b/drivers/media/video/omap/omap_vout.c
@@ -38,6 +38,7 @@
 #include <linux/irq.h>
 #include <linux/videodev2.h>
 #include <linux/dma-mapping.h>
+#include <linux/slab.h>
 #include <media/videobuf-dma-contig.h>
 #include <media/v4l2-device.h>
@@ -69,9 +70,9 @@ static u32 video1_numbuffers = 3;
 static u32 video2_numbuffers = 3;
 static u32 video1_bufsize = OMAP_VOUT_MAX_BUF_SIZE;
 static u32 video2_bufsize = OMAP_VOUT_MAX_BUF_SIZE;
-static u32 vid1_static_vrfb_alloc;
+static bool vid1_static_vrfb_alloc;
-static u32 vid2_static_vrfb_alloc;
+static bool vid2_static_vrfb_alloc;
-static int debug;
+static bool debug;
 /* Module parameters */
 module_param(video1_numbuffers, uint, S_IRUGO);
@@ -423,7 +424,7 @@ static int omapvid_setup_overlay(struct omap_vout_device *vout,
                "%s enable=%d addr=%x width=%d\n height=%d color_mode=%d\n"
                "rotation=%d mirror=%d posx=%d posy=%d out_width = %d \n"
                "out_height=%d rotation_type=%d screen_width=%d\n",
-                __func__, info.enabled, info.paddr, info.width, info.height,
+                __func__, ovl->is_enabled(ovl), info.paddr, info.width, info.height,
                info.color_mode, info.rotation, info.mirror, info.pos_x,
                info.pos_y, info.out_width, info.out_height, info.rotation_type,
                info.screen_width);
@@ -942,12 +943,8 @@ static int omap_vout_release(struct file *file)
        /* Disable all the overlay managers connected with this interface */
        for (i = 0; i < ovid->num_overlays; i++) {
                struct omap_overlay *ovl = ovid->overlays[i];
-                if (ovl->manager && ovl->manager->device) {
+                if (ovl->manager && ovl->manager->device)
-                        struct omap_overlay_info info;
+                        ovl->disable(ovl);
-                        ovl->get_overlay_info(ovl, &info);
-                        info.enabled = 0;
-                        ovl->set_overlay_info(ovl, &info);
-                }
        }
        /* Turn off the pipeline */
        ret = omapvid_apply_changes(vout);
@@ -1667,7 +1664,6 @@ static int vidioc_streamon(struct file *file, void *fh, enum v4l2_buf_type i)
                if (ovl->manager && ovl->manager->device) {
                        struct omap_overlay_info info;
                        ovl->get_overlay_info(ovl, &info);
-                        info.enabled = 1;
                        info.paddr = addr;
                        if (ovl->set_overlay_info(ovl, &info)) {
                                ret = -EINVAL;
@@ -1686,6 +1682,16 @@ static int vidioc_streamon(struct file *file, void *fh, enum v4l2_buf_type i)
        if (ret)
                v4l2_err(&vout->vid_dev->v4l2_dev, "failed to change mode\n");
+        for (j = 0; j < ovid->num_overlays; j++) {
+                struct omap_overlay *ovl = ovid->overlays[j];
+                if (ovl->manager && ovl->manager->device) {
+                        ret = ovl->enable(ovl);
+                        if (ret)
+                                goto streamon_err1;
+                }
+        }
        ret = 0;
 streamon_err1:
@@ -1715,16 +1721,8 @@ static int vidioc_streamoff(struct file *file, void *fh, enum v4l2_buf_type i)
        for (j = 0; j < ovid->num_overlays; j++) {
                struct omap_overlay *ovl = ovid->overlays[j];
-                if (ovl->manager && ovl->manager->device) {
+                if (ovl->manager && ovl->manager->device)
-                        struct omap_overlay_info info;
+                        ovl->disable(ovl);
-                        ovl->get_overlay_info(ovl, &info);
-                        info.enabled = 0;
-                        ret = ovl->set_overlay_info(ovl, &info);
-                        if (ret)
-                                v4l2_err(&vout->vid_dev->v4l2_dev,
-                                "failed to update overlay info in streamoff\n");
-                }
        }
        /* Turn of the pipeline */
@@ -2169,6 +2167,14 @@ static int __init omap_vout_probe(struct platform_device *pdev)
        vid_dev->num_displays = 0;
        for_each_dss_dev(dssdev) {
                omap_dss_get_device(dssdev);
+                if (!dssdev->driver) {
+                        dev_warn(&pdev->dev, "no driver for display: %s\n",
+                                        dssdev->name);
+                        omap_dss_put_device(dssdev);
+                        continue;
+                }
                vid_dev->displays[vid_dev->num_displays++] = dssdev;
        }
diff --git a/drivers/media/video/omap/omap_vout_vrfb.c b/drivers/media/video/omap/omap_vout_vrfb.c
index ebebcac4922..4be26abf6ce 100644
--- a/drivers/media/video/omap/omap_vout_vrfb.c
+++ b/drivers/media/video/omap/omap_vout_vrfb.c
@@ -84,7 +84,7 @@ void omap_vout_free_vrfb_buffers(struct omap_vout_device *vout)
 }
 int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
-                        u32 static_vrfb_allocation)
+                              bool static_vrfb_allocation)
 {
        int ret = 0, i, j;
        struct omap_vout_device *vout;
diff --git a/drivers/media/video/omap1_camera.c b/drivers/media/video/omap1_camera.c
index e87ae2f634b..6a6cf388bae 100644
--- a/drivers/media/video/omap1_camera.c
+++ b/drivers/media/video/omap1_camera.c
@@ -24,6 +24,7 @@
 #include <linux/clk.h>
 #include <linux/dma-mapping.h>
 #include <linux/interrupt.h>
+#include <linux/module.h>
 #include <linux/platform_device.h>
 #include <linux/slab.h>
diff --git a/drivers/media/video/omap24xxcam-dma.c b/drivers/media/video/omap24xxcam-dma.c
index 1d54b86c936..3ea38a8def8 100644
--- a/drivers/media/video/omap24xxcam-dma.c
+++ b/drivers/media/video/omap24xxcam-dma.c
@@ -506,7 +506,7 @@ int omap24xxcam_sgdma_queue(struct omap24xxcam_sgdma *sgdma,
        unsigned long flags;
        struct sgdma_state *sg_state;
-        if ((sglen < 0) || ((sglen > 0) & !sglist))
+        if ((sglen < 0) || ((sglen > 0) && !sglist))
                return -EINVAL;
        spin_lock_irqsave(&sgdma->lock, flags);
diff --git a/drivers/media/video/omap3isp/isp.c b/drivers/media/video/omap3isp/isp.c
index b818cacf420..d4c48ef227f 100644
--- a/drivers/media/video/omap3isp/isp.c
+++ b/drivers/media/video/omap3isp/isp.c
@@ -80,13 +80,6 @@
 #include "isph3a.h"
 #include "isphist.h"
-/*
- * this is provided as an interim solution until omap3isp doesn't need
- * any omap-specific iommu API
- */
-#define to_iommu(dev)                                                   \
-        (struct omap_iommu *)platform_get_drvdata(to_platform_device(dev))
 static unsigned int autoidle;
 module_param(autoidle, int, 0444);
 MODULE_PARM_DESC(autoidle, "Enable OMAP3ISP AUTOIDLE support");
@@ -1114,8 +1107,7 @@ isp_restore_context(struct isp_device *isp, struct isp_reg *reg_list)
 static void isp_save_ctx(struct isp_device *isp)
 {
        isp_save_context(isp, isp_reg_list);
-        if (isp->iommu)
+        omap_iommu_save_ctx(isp->dev);
-                omap_iommu_save_ctx(isp->iommu);
 }
 /*
@@ -1128,8 +1120,7 @@ static void isp_save_ctx(struct isp_device *isp)
 static void isp_restore_ctx(struct isp_device *isp)
 {
        isp_restore_context(isp, isp_reg_list);
-        if (isp->iommu)
+        omap_iommu_restore_ctx(isp->dev);
-                omap_iommu_restore_ctx(isp->iommu);
        omap3isp_ccdc_restore_context(isp);
        omap3isp_preview_restore_context(isp);
 }
@@ -1983,7 +1974,7 @@ static int isp_remove(struct platform_device *pdev)
        isp_cleanup_modules(isp);
        omap3isp_get(isp);
-        iommu_detach_device(isp->domain, isp->iommu_dev);
+        iommu_detach_device(isp->domain, &pdev->dev);
        iommu_domain_free(isp->domain);
        omap3isp_put(isp);
@@ -2131,17 +2122,6 @@ static int isp_probe(struct platform_device *pdev)
                }
        }
-        /* IOMMU */
-        isp->iommu_dev = omap_find_iommu_device("isp");
-        if (!isp->iommu_dev) {
-                dev_err(isp->dev, "omap_find_iommu_device failed\n");
-                ret = -ENODEV;
-                goto error_isp;
-        }
-        /* to be removed once iommu migration is complete */
-        isp->iommu = to_iommu(isp->iommu_dev);
        isp->domain = iommu_domain_alloc(pdev->dev.bus);
        if (!isp->domain) {
                dev_err(isp->dev, "can't alloc iommu domain\n");
@@ -2149,7 +2129,7 @@ static int isp_probe(struct platform_device *pdev)
                goto error_isp;
        }
-        ret = iommu_attach_device(isp->domain, isp->iommu_dev);
+        ret = iommu_attach_device(isp->domain, &pdev->dev);
        if (ret) {
                dev_err(&pdev->dev, "can't attach iommu device: %d\n", ret);
                goto free_domain;
@@ -2188,7 +2168,7 @@ error_modules:
 error_irq:
        free_irq(isp->irq_num, isp);
 detach_dev:
-        iommu_detach_device(isp->domain, isp->iommu_dev);
+        iommu_detach_device(isp->domain, &pdev->dev);
 free_domain:
        iommu_domain_free(isp->domain);
 error_isp:
diff --git a/drivers/media/video/omap3isp/isp.h b/drivers/media/video/omap3isp/isp.h
index 705946ef4d6..d96603eb0d1 100644
--- a/drivers/media/video/omap3isp/isp.h
+++ b/drivers/media/video/omap3isp/isp.h
@@ -212,9 +212,7 @@ struct isp_device {
        unsigned int sbl_resources;
        unsigned int subclk_resources;
-        struct omap_iommu *iommu;
        struct iommu_domain *domain;
-        struct device *iommu_dev;
        struct isp_platform_callback platform_cb;
 };
diff --git a/drivers/media/video/omap3isp/ispccdc.c b/drivers/media/video/omap3isp/ispccdc.c
index b0b0fa5a357..d341ba12593 100644
--- a/drivers/media/video/omap3isp/ispccdc.c
+++ b/drivers/media/video/omap3isp/ispccdc.c
@@ -366,7 +366,7 @@ static void ccdc_lsc_free_request(struct isp_ccdc_device *ccdc,
                dma_unmap_sg(isp->dev, req->iovm->sgt->sgl,
                             req->iovm->sgt->nents, DMA_TO_DEVICE);
        if (req->table)
-                omap_iommu_vfree(isp->domain, isp->iommu, req->table);
+                omap_iommu_vfree(isp->domain, isp->dev, req->table);
        kfree(req);
 }
@@ -438,7 +438,7 @@ static int ccdc_lsc_config(struct isp_ccdc_device *ccdc,
                req->enable = 1;
-                req->table = omap_iommu_vmalloc(isp->domain, isp->iommu, 0,
+                req->table = omap_iommu_vmalloc(isp->domain, isp->dev, 0,
                                        req->config.size, IOMMU_FLAG);
                if (IS_ERR_VALUE(req->table)) {
                        req->table = 0;
@@ -446,7 +446,7 @@ static int ccdc_lsc_config(struct isp_ccdc_device *ccdc,
                        goto done;
                }
-                req->iovm = omap_find_iovm_area(isp->iommu, req->table);
+                req->iovm = omap_find_iovm_area(isp->dev, req->table);
                if (req->iovm == NULL) {
                        ret = -ENOMEM;
                        goto done;
@@ -462,7 +462,7 @@ static int ccdc_lsc_config(struct isp_ccdc_device *ccdc,
                dma_sync_sg_for_cpu(isp->dev, req->iovm->sgt->sgl,
                                    req->iovm->sgt->nents, DMA_TO_DEVICE);
-                table = omap_da_to_va(isp->iommu, req->table);
+                table = omap_da_to_va(isp->dev, req->table);
                if (copy_from_user(table, config->lsc, req->config.size)) {
                        ret = -EFAULT;
                        goto done;
@@ -734,15 +734,15 @@ static int ccdc_config(struct isp_ccdc_device *ccdc,
                         * already done by omap_iommu_vmalloc().
                         */
                        size = ccdc->fpc.fpnum * 4;
-                        table_new = omap_iommu_vmalloc(isp->domain, isp->iommu,
+                        table_new = omap_iommu_vmalloc(isp->domain, isp->dev,
                                                        0, size, IOMMU_FLAG);
                        if (IS_ERR_VALUE(table_new))
                                return -ENOMEM;
-                        if (copy_from_user(omap_da_to_va(isp->iommu, table_new),
+                        if (copy_from_user(omap_da_to_va(isp->dev, table_new),
                                           (__force void __user *)
                                           ccdc->fpc.fpcaddr, size)) {
-                                omap_iommu_vfree(isp->domain, isp->iommu,
+                                omap_iommu_vfree(isp->domain, isp->dev,
                                                                table_new);
                                return -EFAULT;
                        }
@@ -753,7 +753,7 @@ static int ccdc_config(struct isp_ccdc_device *ccdc,
                ccdc_configure_fpc(ccdc);
                if (table_old != 0)
-                        omap_iommu_vfree(isp->domain, isp->iommu, table_old);
+                        omap_iommu_vfree(isp->domain, isp->dev, table_old);
        }
        return ccdc_lsc_config(ccdc, ccdc_struct);
@@ -1408,7 +1408,7 @@ static void ccdc_hs_vs_isr(struct isp_ccdc_device *ccdc)
 {
        struct isp_pipeline *pipe =
                to_isp_pipeline(&ccdc->video_out.video.entity);
-        struct video_device *vdev = &ccdc->subdev.devnode;
+        struct video_device *vdev = ccdc->subdev.devnode;
        struct v4l2_event event;
        memset(&event, 0, sizeof(event));
@@ -2309,7 +2309,7 @@ void omap3isp_ccdc_cleanup(struct isp_device *isp)
        ccdc_lsc_free_queue(ccdc, &ccdc->lsc.free_queue);
        if (ccdc->fpc.fpcaddr != 0)
-                omap_iommu_vfree(isp->domain, isp->iommu, ccdc->fpc.fpcaddr);
+                omap_iommu_vfree(isp->domain, isp->dev, ccdc->fpc.fpcaddr);
        mutex_destroy(&ccdc->ioctl_lock);
 }
diff --git a/drivers/media/video/omap3isp/ispstat.c b/drivers/media/video/omap3isp/ispstat.c
index 68d539456c5..11871ecc6d2 100644
--- a/drivers/media/video/omap3isp/ispstat.c
+++ b/drivers/media/video/omap3isp/ispstat.c
@@ -366,7 +366,7 @@ static void isp_stat_bufs_free(struct ispstat *stat)
                                dma_unmap_sg(isp->dev, buf->iovm->sgt->sgl,
                                             buf->iovm->sgt->nents,
                                             DMA_FROM_DEVICE);
-                        omap_iommu_vfree(isp->domain, isp->iommu,
+                        omap_iommu_vfree(isp->domain, isp->dev,
                                                        buf->iommu_addr);
                } else {
                        if (!buf->virt_addr)
@@ -400,7 +400,7 @@ static int isp_stat_bufs_alloc_iommu(struct ispstat *stat, unsigned int size)
                struct iovm_struct *iovm;
                WARN_ON(buf->dma_addr);
-                buf->iommu_addr = omap_iommu_vmalloc(isp->domain, isp->iommu, 0,
+                buf->iommu_addr = omap_iommu_vmalloc(isp->domain, isp->dev, 0,
                                                        size, IOMMU_FLAG);
                if (IS_ERR((void *)buf->iommu_addr)) {
                        dev_err(stat->isp->dev,
@@ -410,7 +410,7 @@ static int isp_stat_bufs_alloc_iommu(struct ispstat *stat, unsigned int size)
                        return -ENOMEM;
                }
-                iovm = omap_find_iovm_area(isp->iommu, buf->iommu_addr);
+                iovm = omap_find_iovm_area(isp->dev, buf->iommu_addr);
                if (!iovm ||
                    !dma_map_sg(isp->dev, iovm->sgt->sgl, iovm->sgt->nents,
                                DMA_FROM_DEVICE)) {
@@ -419,7 +419,7 @@ static int isp_stat_bufs_alloc_iommu(struct ispstat *stat, unsigned int size)
                }
                buf->iovm = iovm;
-                buf->virt_addr = omap_da_to_va(stat->isp->iommu,
+                buf->virt_addr = omap_da_to_va(stat->isp->dev,
                                          (u32)buf->iommu_addr);
                buf->empty = 1;
                dev_dbg(stat->isp->dev, "%s: buffer[%d] allocated."
@@ -496,7 +496,7 @@ static int isp_stat_bufs_alloc(struct ispstat *stat, u32 size)
 static void isp_stat_queue_event(struct ispstat *stat, int err)
 {
-        struct video_device *vdev = &stat->subdev.devnode;
+        struct video_device *vdev = stat->subdev.devnode;
        struct v4l2_event event;
        struct omap3isp_stat_event_status *status = (void *)event.u.data;
diff --git a/drivers/media/video/omap3isp/ispvideo.c b/drivers/media/video/omap3isp/ispvideo.c
index d1000723c5a..bd3aebafafa 100644
--- a/drivers/media/video/omap3isp/ispvideo.c
+++ b/drivers/media/video/omap3isp/ispvideo.c
@@ -26,6 +26,7 @@
 #include <asm/cacheflush.h>
 #include <linux/clk.h>
 #include <linux/mm.h>
+#include <linux/module.h>
 #include <linux/pagemap.h>
 #include <linux/scatterlist.h>
 #include <linux/sched.h>
@@ -452,7 +453,7 @@ ispmmu_vmap(struct isp_device *isp, const struct scatterlist *sglist, int sglen)
        sgt->nents = sglen;
        sgt->orig_nents = sglen;
-        da = omap_iommu_vmap(isp->domain, isp->iommu, 0, sgt, IOMMU_FLAG);
+        da = omap_iommu_vmap(isp->domain, isp->dev, 0, sgt, IOMMU_FLAG);
        if (IS_ERR_VALUE(da))
                kfree(sgt);
@@ -468,7 +469,7 @@ static void ispmmu_vunmap(struct isp_device *isp, dma_addr_t da)
 {
        struct sg_table *sgt;
-        sgt = omap_iommu_vunmap(isp->domain, isp->iommu, (u32)da);
+        sgt = omap_iommu_vunmap(isp->domain, isp->dev, (u32)da);
        kfree(sgt);
 }
diff --git a/drivers/media/video/ov6650.c b/drivers/media/video/ov6650.c
index 9f2d26b1d4c..6806345ec2f 100644
--- a/drivers/media/video/ov6650.c
+++ b/drivers/media/video/ov6650.c
@@ -540,7 +540,7 @@ static u8 to_clkrc(struct v4l2_fract *timeperframe,
 static int ov6650_s_fmt(struct v4l2_subdev *sd, struct v4l2_mbus_framefmt *mf)
 {
        struct i2c_client *client = v4l2_get_subdevdata(sd);
-        struct soc_camera_device *icd = (struct soc_camera_device *)sd->grp_id;
+        struct soc_camera_device *icd = v4l2_get_subdev_hostdata(sd);
        struct soc_camera_sense *sense = icd->sense;
        struct ov6650 *priv = to_ov6650(client);
        bool half_scale = !is_unscaled_ok(mf->width, mf->height, &priv->rect);
diff --git a/drivers/media/video/ov7670.c b/drivers/media/video/ov7670.c
index 8aa05853128..6a564964853 100644
--- a/drivers/media/video/ov7670.c
+++ b/drivers/media/video/ov7670.c
@@ -25,7 +25,7 @@ MODULE_AUTHOR("Jonathan Corbet <corbet@lwn.net>");
 MODULE_DESCRIPTION("A low-level driver for OmniVision ov7670 sensors");
 MODULE_LICENSE("GPL");
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
 MODULE_PARM_DESC(debug, "Debug level (0-1)");
diff --git a/drivers/media/video/s2255drv.c b/drivers/media/video/s2255drv.c
index 803c9c82e49..c1bef618766 100644
--- a/drivers/media/video/s2255drv.c
+++ b/drivers/media/video/s2255drv.c
@@ -2682,25 +2682,7 @@ static struct usb_driver s2255_driver = {
        .id_table = s2255_table,
 };
-static int __init usb_s2255_init(void)
+module_usb_driver(s2255_driver);
-{
-        int result;
-        /* register this driver with the USB subsystem */
-        result = usb_register(&s2255_driver);
-        if (result)
-                pr_err(KBUILD_MODNAME
-                       ": usb_register failed. Error number %d\n", result);
-        dprintk(2, "%s\n", __func__);
-        return result;
-}
-static void __exit usb_s2255_exit(void)
-{
-        usb_deregister(&s2255_driver);
-}
-module_init(usb_s2255_init);
-module_exit(usb_s2255_exit);
 MODULE_DESCRIPTION("Sensoray 2255 Video for Linux driver");
 MODULE_AUTHOR("Dean Anderson (Sensoray Company Inc.)");
diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-fimc/fimc-capture.c
index c8d91b0cd9b..2cc3b916672 100644
--- a/drivers/media/video/s5p-fimc/fimc-capture.c
+++ b/drivers/media/video/s5p-fimc/fimc-capture.c
@@ -98,6 +98,10 @@ static int fimc_capture_state_cleanup(struct fimc_dev *fimc, bool suspend)
                        vb2_buffer_done(&buf->vb, VB2_BUF_STATE_ERROR);
        }
        set_bit(ST_CAPT_SUSPENDED, &fimc->state);
+        fimc_hw_reset(fimc);
+        cap->buf_index = 0;
        spin_unlock_irqrestore(&fimc->slock, flags);
        if (streaming)
@@ -137,7 +141,7 @@ int fimc_capture_config_update(struct fimc_ctx *ctx)
        struct fimc_dev *fimc = ctx->fimc_dev;
        int ret;
-        if (test_bit(ST_CAPT_APPLY_CFG, &fimc->state))
+        if (!test_bit(ST_CAPT_APPLY_CFG, &fimc->state))
                return 0;
        spin_lock(&ctx->slock);
@@ -150,7 +154,7 @@ int fimc_capture_config_update(struct fimc_ctx *ctx)
                fimc_hw_set_rotation(ctx);
                fimc_prepare_dma_offset(ctx, &ctx->d_frame);
                fimc_hw_set_out_dma(ctx);
-                set_bit(ST_CAPT_APPLY_CFG, &fimc->state);
+                clear_bit(ST_CAPT_APPLY_CFG, &fimc->state);
        }
        spin_unlock(&ctx->slock);
        return ret;
@@ -164,7 +168,6 @@ static int start_streaming(struct vb2_queue *q, unsigned int count)
        int min_bufs;
        int ret;
-        fimc_hw_reset(fimc);
        vid_cap->frame_count = 0;
        ret = fimc_init_capture(fimc);
@@ -523,7 +526,7 @@ static struct fimc_fmt *fimc_capture_try_format(struct fimc_ctx *ctx,
        max_w = rotation ? pl->out_rot_en_w : pl->out_rot_dis_w;
        min_w = ctx->state & FIMC_DST_CROP ? dst->width : var->min_out_pixsize;
        min_h = ctx->state & FIMC_DST_CROP ? dst->height : var->min_out_pixsize;
-        if (fimc->id == 1 && var->pix_hoff)
+        if (var->min_vsize_align == 1 && !rotation)
                align_h = fimc_fmt_is_rgb(ffmt->color) ? 0 : 1;
        depth = fimc_get_format_depth(ffmt);
@@ -1239,6 +1242,7 @@ static int fimc_subdev_set_fmt(struct v4l2_subdev *sd,
        mutex_lock(&fimc->lock);
        set_frame_bounds(ff, mf->width, mf->height);
+        fimc->vid_cap.mf = *mf;
        ff->fmt = ffmt;
        /* Reset the crop rectangle if required. */
@@ -1375,7 +1379,7 @@ static void fimc_destroy_capture_subdev(struct fimc_dev *fimc)
        media_entity_cleanup(&sd->entity);
        v4l2_device_unregister_subdev(sd);
        kfree(sd);
-        sd = NULL;
+        fimc->vid_cap.subdev = NULL;
 }
 /* Set default format at the sensor and host interface */
diff --git a/drivers/media/video/s5p-fimc/fimc-core.c b/drivers/media/video/s5p-fimc/fimc-core.c
index 19ca6db38b2..07c6254faee 100644
--- a/drivers/media/video/s5p-fimc/fimc-core.c
+++ b/drivers/media/video/s5p-fimc/fimc-core.c
@@ -37,7 +37,7 @@ static char *fimc_clocks[MAX_FIMC_CLOCKS] = {
 static struct fimc_fmt fimc_formats[] = {
        {
                .name           = "RGB565",
-                .fourcc         = V4L2_PIX_FMT_RGB565X,
+                .fourcc         = V4L2_PIX_FMT_RGB565,
                .depth          = { 16 },
                .color          = S5P_FIMC_RGB565,
                .memplanes      = 1,
@@ -1038,12 +1038,11 @@ static int fimc_try_fmt_mplane(struct fimc_ctx *ctx, struct v4l2_format *f)
                mod_x = 6; /* 64 x 32 pixels tile */
                mod_y = 5;
        } else {
-                if (fimc->id == 1 && variant->pix_hoff)
+                if (variant->min_vsize_align == 1)
                        mod_y = fimc_fmt_is_rgb(fmt->color) ? 0 : 1;
                else
-                        mod_y = mod_x;
+                        mod_y = ffs(variant->min_vsize_align) - 1;
        }
-        dbg("mod_x: %d, mod_y: %d, max_w: %d", mod_x, mod_y, max_w);
        v4l_bound_align_image(&pix->width, 16, max_w, mod_x,
                &pix->height, 8, variant->pix_limit->scaler_dis_w, mod_y, 0);
@@ -1226,10 +1225,10 @@ static int fimc_m2m_try_crop(struct fimc_ctx *ctx, struct v4l2_crop *cr)
                fimc->variant->min_inp_pixsize : fimc->variant->min_out_pixsize;
        /* Get pixel alignment constraints. */
-        if (fimc->id == 1 && fimc->variant->pix_hoff)
+        if (fimc->variant->min_vsize_align == 1)
                halign = fimc_fmt_is_rgb(f->fmt->color) ? 0 : 1;
        else
-                halign = ffs(min_size) - 1;
+                halign = ffs(fimc->variant->min_vsize_align) - 1;
        for (i = 0; i < f->fmt->colplanes; i++)
                depth += f->fmt->depth[i];
@@ -1615,7 +1614,6 @@ static int fimc_probe(struct platform_device *pdev)
        pdata = pdev->dev.platform_data;
        fimc->pdata = pdata;
-        set_bit(ST_LPM, &fimc->state);
        init_waitqueue_head(&fimc->irq_queue);
        spin_lock_init(&fimc->slock);
@@ -1707,8 +1705,6 @@ static int fimc_runtime_resume(struct device *dev)
        /* Enable clocks and perform basic initalization */
        clk_enable(fimc->clock[CLK_GATE]);
        fimc_hw_reset(fimc);
-        if (fimc->variant->out_buf_count > 4)
-                fimc_hw_set_dma_seq(fimc, 0xF);
        /* Resume the capture or mem-to-mem device */
        if (fimc_capture_busy(fimc))
@@ -1750,8 +1746,6 @@ static int fimc_resume(struct device *dev)
                return 0;
        }
        fimc_hw_reset(fimc);
-        if (fimc->variant->out_buf_count > 4)
-                fimc_hw_set_dma_seq(fimc, 0xF);
        spin_unlock_irqrestore(&fimc->slock, flags);
        if (fimc_capture_busy(fimc))
@@ -1780,7 +1774,6 @@ static int __devexit fimc_remove(struct platform_device *pdev)
        struct fimc_dev *fimc = platform_get_drvdata(pdev);
        pm_runtime_disable(&pdev->dev);
-        fimc_runtime_suspend(&pdev->dev);
        pm_runtime_set_suspended(&pdev->dev);
        vb2_dma_contig_cleanup_ctx(fimc->alloc_ctx);
@@ -1840,6 +1833,7 @@ static struct samsung_fimc_variant fimc0_variant_s5p = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 8,
+        .min_vsize_align = 16,
        .out_buf_count   = 4,
        .pix_limit       = &s5p_pix_limit[0],
 };
@@ -1849,6 +1843,7 @@ static struct samsung_fimc_variant fimc2_variant_s5p = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 8,
+        .min_vsize_align = 16,
        .out_buf_count   = 4,
        .pix_limit = &s5p_pix_limit[1],
 };
@@ -1861,6 +1856,7 @@ static struct samsung_fimc_variant fimc0_variant_s5pv210 = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 8,
+        .min_vsize_align = 16,
        .out_buf_count   = 4,
        .pix_limit       = &s5p_pix_limit[1],
 };
@@ -1874,6 +1870,7 @@ static struct samsung_fimc_variant fimc1_variant_s5pv210 = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 1,
+        .min_vsize_align = 1,
        .out_buf_count   = 4,
        .pix_limit       = &s5p_pix_limit[2],
 };
@@ -1884,6 +1881,7 @@ static struct samsung_fimc_variant fimc2_variant_s5pv210 = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 8,
+        .min_vsize_align = 16,
        .out_buf_count   = 4,
        .pix_limit       = &s5p_pix_limit[2],
 };
@@ -1898,6 +1896,7 @@ static struct samsung_fimc_variant fimc0_variant_exynos4 = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 2,
+        .min_vsize_align = 1,
        .out_buf_count   = 32,
        .pix_limit       = &s5p_pix_limit[1],
 };
@@ -1910,6 +1909,7 @@ static struct samsung_fimc_variant fimc3_variant_exynos4 = {
        .min_inp_pixsize = 16,
        .min_out_pixsize = 16,
        .hor_offs_align  = 2,
+        .min_vsize_align = 1,
        .out_buf_count   = 32,
        .pix_limit       = &s5p_pix_limit[3],
 };
diff --git a/drivers/media/video/s5p-fimc/fimc-core.h b/drivers/media/video/s5p-fimc/fimc-core.h
index a6936dad5b1..c7f01c47b20 100644
--- a/drivers/media/video/s5p-fimc/fimc-core.h
+++ b/drivers/media/video/s5p-fimc/fimc-core.h
@@ -377,6 +377,7 @@ struct fimc_pix_limit {
 * @min_inp_pixsize: minimum input pixel size
 * @min_out_pixsize: minimum output pixel size
 * @hor_offs_align: horizontal pixel offset aligment
+ * @min_vsize_align: minimum vertical pixel size alignment
 * @out_buf_count: the number of buffers in output DMA sequence
 */
 struct samsung_fimc_variant {
@@ -390,6 +391,7 @@ struct samsung_fimc_variant {
        u16             min_inp_pixsize;
        u16             min_out_pixsize;
        u16             hor_offs_align;
+        u16             min_vsize_align;
        u16             out_buf_count;
 };
diff --git a/drivers/media/video/s5p-fimc/fimc-mdevice.c b/drivers/media/video/s5p-fimc/fimc-mdevice.c
index cc337b1de91..615c862f036 100644
--- a/drivers/media/video/s5p-fimc/fimc-mdevice.c
+++ b/drivers/media/video/s5p-fimc/fimc-mdevice.c
@@ -220,6 +220,7 @@ static struct v4l2_subdev *fimc_md_register_sensor(struct fimc_md *fmd,
        sd = v4l2_i2c_new_subdev_board(&fmd->v4l2_dev, adapter,
                                       s_info->pdata->board_info, NULL);
        if (IS_ERR_OR_NULL(sd)) {
+                i2c_put_adapter(adapter);
                v4l2_err(&fmd->v4l2_dev, "Failed to acquire subdev\n");
                return NULL;
        }
@@ -234,12 +235,15 @@ static struct v4l2_subdev *fimc_md_register_sensor(struct fimc_md *fmd,
 static void fimc_md_unregister_sensor(struct v4l2_subdev *sd)
 {
        struct i2c_client *client = v4l2_get_subdevdata(sd);
+        struct i2c_adapter *adapter;
        if (!client)
                return;
        v4l2_device_unregister_subdev(sd);
+        adapter = client->adapter;
        i2c_unregister_device(client);
-        i2c_put_adapter(client->adapter);
+        if (adapter)
+                i2c_put_adapter(adapter);
 }
 static int fimc_md_register_sensor_entities(struct fimc_md *fmd)
@@ -381,20 +385,28 @@ static void fimc_md_unregister_entities(struct fimc_md *fmd)
 static int fimc_md_register_video_nodes(struct fimc_md *fmd)
 {
+        struct video_device *vdev;
        int i, ret = 0;
        for (i = 0; i < FIMC_MAX_DEVS && !ret; i++) {
                if (!fmd->fimc[i])
                        continue;
-                if (fmd->fimc[i]->m2m.vfd)
+                vdev = fmd->fimc[i]->m2m.vfd;
-                        ret = video_register_device(fmd->fimc[i]->m2m.vfd,
+                if (vdev) {
-                                                    VFL_TYPE_GRABBER, -1);
+                        ret = video_register_device(vdev, VFL_TYPE_GRABBER, -1);
-                if (ret)
+                        if (ret)
-                        break;
+                                break;
-                if (fmd->fimc[i]->vid_cap.vfd)
+                        v4l2_info(&fmd->v4l2_dev, "Registered %s as /dev/%s\n",
-                        ret = video_register_device(fmd->fimc[i]->vid_cap.vfd,
+                                  vdev->name, video_device_node_name(vdev));
-                                                    VFL_TYPE_GRABBER, -1);
+                }
+                vdev = fmd->fimc[i]->vid_cap.vfd;
+                if (vdev == NULL)
+                        continue;
+                ret = video_register_device(vdev, VFL_TYPE_GRABBER, -1);
+                v4l2_info(&fmd->v4l2_dev, "Registered %s as /dev/%s\n",
+                          vdev->name, video_device_node_name(vdev));
        }
        return ret;
@@ -502,7 +514,7 @@ static int fimc_md_create_links(struct fimc_md *fmd)
                        if (WARN(csis == NULL,
                                 "MIPI-CSI interface specified "
                                 "but s5p-csis module is not loaded!\n"))
-                                continue;
+                                return -EINVAL;
                        ret = media_entity_create_link(&sensor->entity, 0,
                                              &csis->entity, CSIS_PAD_SINK,
@@ -742,9 +754,6 @@ static int __devinit fimc_md_probe(struct platform_device *pdev)
        struct fimc_md *fmd;
        int ret;
-        if (WARN(!pdev->dev.platform_data, "Platform data not specified!\n"))
-                return -EINVAL;
        fmd = kzalloc(sizeof(struct fimc_md), GFP_KERNEL);
        if (!fmd)
                return -ENOMEM;
@@ -782,9 +791,11 @@ static int __devinit fimc_md_probe(struct platform_device *pdev)
        if (ret)
                goto err3;
-        ret = fimc_md_register_sensor_entities(fmd);
+        if (pdev->dev.platform_data) {
-        if (ret)
+                ret = fimc_md_register_sensor_entities(fmd);
-                goto err3;
+                if (ret)
+                        goto err3;
+        }
        ret = fimc_md_create_links(fmd);
        if (ret)
                goto err3;
diff --git a/drivers/media/video/s5p-fimc/fimc-reg.c b/drivers/media/video/s5p-fimc/fimc-reg.c
index 20e664e3416..44f5c2d1920 100644
--- a/drivers/media/video/s5p-fimc/fimc-reg.c
+++ b/drivers/media/video/s5p-fimc/fimc-reg.c
@@ -35,6 +35,9 @@ void fimc_hw_reset(struct fimc_dev *dev)
        cfg = readl(dev->regs + S5P_CIGCTRL);
        cfg &= ~S5P_CIGCTRL_SWRST;
        writel(cfg, dev->regs + S5P_CIGCTRL);
+        if (dev->variant->out_buf_count > 4)
+                fimc_hw_set_dma_seq(dev, 0xF);
 }
 static u32 fimc_hw_get_in_flip(struct fimc_ctx *ctx)
@@ -251,7 +254,14 @@ static void fimc_hw_set_scaler(struct fimc_ctx *ctx)
        struct fimc_scaler *sc = &ctx->scaler;
        struct fimc_frame *src_frame = &ctx->s_frame;
        struct fimc_frame *dst_frame = &ctx->d_frame;
-        u32 cfg = 0;
+        u32 cfg = readl(dev->regs + S5P_CISCCTRL);
+        cfg &= ~(S5P_CISCCTRL_CSCR2Y_WIDE | S5P_CISCCTRL_CSCY2R_WIDE |
+                 S5P_CISCCTRL_SCALEUP_H | S5P_CISCCTRL_SCALEUP_V |
+                 S5P_CISCCTRL_SCALERBYPASS | S5P_CISCCTRL_ONE2ONE |
+                 S5P_CISCCTRL_INRGB_FMT_MASK | S5P_CISCCTRL_OUTRGB_FMT_MASK |
+                 S5P_CISCCTRL_INTERLACE | S5P_CISCCTRL_RGB_EXT);
        if (!(ctx->flags & FIMC_COLOR_RANGE_NARROW))
                cfg |= (S5P_CISCCTRL_CSCR2Y_WIDE | S5P_CISCCTRL_CSCY2R_WIDE);
@@ -308,9 +318,9 @@ void fimc_hw_set_mainscaler(struct fimc_ctx *ctx)
        fimc_hw_set_scaler(ctx);
        cfg = readl(dev->regs + S5P_CISCCTRL);
+        cfg &= ~(S5P_CISCCTRL_MHRATIO_MASK | S5P_CISCCTRL_MVRATIO_MASK);
        if (variant->has_mainscaler_ext) {
-                cfg &= ~(S5P_CISCCTRL_MHRATIO_MASK | S5P_CISCCTRL_MVRATIO_MASK);
                cfg |= S5P_CISCCTRL_MHRATIO_EXT(sc->main_hratio);
                cfg |= S5P_CISCCTRL_MVRATIO_EXT(sc->main_vratio);
                writel(cfg, dev->regs + S5P_CISCCTRL);
@@ -323,7 +333,6 @@ void fimc_hw_set_mainscaler(struct fimc_ctx *ctx)
                cfg |= S5P_CIEXTEN_MVRATIO_EXT(sc->main_vratio);
                writel(cfg, dev->regs + S5P_CIEXTEN);
        } else {
-                cfg &= ~(S5P_CISCCTRL_MHRATIO_MASK | S5P_CISCCTRL_MVRATIO_MASK);
                cfg |= S5P_CISCCTRL_MHRATIO(sc->main_hratio);
                cfg |= S5P_CISCCTRL_MVRATIO(sc->main_vratio);
                writel(cfg, dev->regs + S5P_CISCCTRL);
diff --git a/drivers/media/video/s5p-mfc/s5p_mfc_enc.c b/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
index 1e8cdb77d4b..dff9dc79879 100644
--- a/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
+++ b/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
@@ -61,7 +61,7 @@ static struct s5p_mfc_fmt formats[] = {
                .num_planes = 1,
        },
        {
-                .name = "H264 Encoded Stream",
+                .name = "H263 Encoded Stream",
                .fourcc = V4L2_PIX_FMT_H263,
                .codec_mode = S5P_FIMV_CODEC_H263_ENC,
                .type = MFC_FMT_ENC,
diff --git a/drivers/media/video/s5p-tv/mixer_video.c b/drivers/media/video/s5p-tv/mixer_video.c
index e16d3a4bc1d..b47d0c06ecf 100644
--- a/drivers/media/video/s5p-tv/mixer_video.c
+++ b/drivers/media/video/s5p-tv/mixer_video.c
@@ -16,6 +16,7 @@
 #include <media/v4l2-ioctl.h>
 #include <linux/videodev2.h>
 #include <linux/mm.h>
+#include <linux/module.h>
 #include <linux/version.h>
 #include <linux/timer.h>
 #include <media/videobuf2-dma-contig.h>
diff --git a/drivers/media/video/saa7115.c b/drivers/media/video/saa7115.c
index 5cfdbc78b91..0ef5484696b 100644
--- a/drivers/media/video/saa7115.c
+++ b/drivers/media/video/saa7115.c
@@ -57,7 +57,7 @@ MODULE_AUTHOR(  "Maxim Yevtyushkin, Kevin Thayer, Chris Kennedy, "
                "Hans Verkuil, Mauro Carvalho Chehab");
 MODULE_LICENSE("GPL");
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
 MODULE_PARM_DESC(debug, "Debug level (0-1)");
diff --git a/drivers/media/video/sh_mobile_ceu_camera.c b/drivers/media/video/sh_mobile_ceu_camera.c
index f390682629c..c51decfcae1 100644
--- a/drivers/media/video/sh_mobile_ceu_camera.c
+++ b/drivers/media/video/sh_mobile_ceu_camera.c
@@ -566,8 +566,10 @@ static int sh_mobile_ceu_add_device(struct soc_camera_device *icd)
        ret = sh_mobile_ceu_soft_reset(pcdev);
        csi2_sd = find_csi2(pcdev);
-        if (csi2_sd)
+        if (csi2_sd) {
-                csi2_sd->grp_id = (long)icd;
+                csi2_sd->grp_id = soc_camera_grp_id(icd);
+                v4l2_set_subdev_hostdata(csi2_sd, icd);
+        }
        ret = v4l2_subdev_call(csi2_sd, core, s_power, 1);
        if (ret < 0 && ret != -ENOIOCTLCMD && ret != -ENODEV) {
@@ -768,7 +770,7 @@ static struct v4l2_subdev *find_bus_subdev(struct sh_mobile_ceu_dev *pcdev,
 {
        if (pcdev->csi2_pdev) {
                struct v4l2_subdev *csi2_sd = find_csi2(pcdev);
-                if (csi2_sd && csi2_sd->grp_id == (u32)icd)
+                if (csi2_sd && csi2_sd->grp_id == soc_camera_grp_id(icd))
                        return csi2_sd;
        }
@@ -1089,8 +1091,9 @@ static int sh_mobile_ceu_get_formats(struct soc_camera_device *icd, unsigned int
                        /* Try 2560x1920, 1280x960, 640x480, 320x240 */
                        mf.width        = 2560 >> shift;
                        mf.height       = 1920 >> shift;
-                        ret = v4l2_device_call_until_err(sd->v4l2_dev, (long)icd, video,
+                        ret = v4l2_device_call_until_err(sd->v4l2_dev,
-                                                         s_mbus_fmt, &mf);
+                                        soc_camera_grp_id(icd), video,
+                                        s_mbus_fmt, &mf);
                        if (ret < 0)
                                return ret;
                        shift++;
@@ -1389,7 +1392,8 @@ static int client_s_fmt(struct soc_camera_device *icd,
        bool ceu_1to1;
        int ret;
-        ret = v4l2_device_call_until_err(sd->v4l2_dev, (long)icd, video,
+        ret = v4l2_device_call_until_err(sd->v4l2_dev,
+                                         soc_camera_grp_id(icd), video,
                                         s_mbus_fmt, mf);
        if (ret < 0)
                return ret;
@@ -1426,8 +1430,9 @@ static int client_s_fmt(struct soc_camera_device *icd,
                tmp_h = min(2 * tmp_h, max_height);
                mf->width = tmp_w;
                mf->height = tmp_h;
-                ret = v4l2_device_call_until_err(sd->v4l2_dev, (long)icd, video,
+                ret = v4l2_device_call_until_err(sd->v4l2_dev,
-                                                 s_mbus_fmt, mf);
+                                        soc_camera_grp_id(icd), video,
+                                        s_mbus_fmt, mf);
                dev_geo(dev, "Camera scaled to %ux%u\n",
                        mf->width, mf->height);
                if (ret < 0) {
@@ -1580,8 +1585,9 @@ static int sh_mobile_ceu_set_crop(struct soc_camera_device *icd,
        }
        if (interm_width < icd->user_width || interm_height < icd->user_height) {
-                ret = v4l2_device_call_until_err(sd->v4l2_dev, (int)icd, video,
+                ret = v4l2_device_call_until_err(sd->v4l2_dev,
-                                                 s_mbus_fmt, &mf);
+                                        soc_camera_grp_id(icd), video,
+                                        s_mbus_fmt, &mf);
                if (ret < 0)
                        return ret;
@@ -1867,7 +1873,8 @@ static int sh_mobile_ceu_try_fmt(struct soc_camera_device *icd,
        mf.code         = xlate->code;
        mf.colorspace   = pix->colorspace;
-        ret = v4l2_device_call_until_err(sd->v4l2_dev, (long)icd, video, try_mbus_fmt, &mf);
+        ret = v4l2_device_call_until_err(sd->v4l2_dev, soc_camera_grp_id(icd),
+                                         video, try_mbus_fmt, &mf);
        if (ret < 0)
                return ret;
@@ -1891,8 +1898,9 @@ static int sh_mobile_ceu_try_fmt(struct soc_camera_device *icd,
                         */
                        mf.width = 2560;
                        mf.height = 1920;
-                        ret = v4l2_device_call_until_err(sd->v4l2_dev, (long)icd, video,
+                        ret = v4l2_device_call_until_err(sd->v4l2_dev,
-                                                         try_mbus_fmt, &mf);
+                                        soc_camera_grp_id(icd), video,
+                                        try_mbus_fmt, &mf);
                        if (ret < 0) {
                                /* Shouldn't actually happen... */
                                dev_err(icd->parent,
diff --git a/drivers/media/video/sh_mobile_csi2.c b/drivers/media/video/sh_mobile_csi2.c
index ea4f0473ed3..8a652b53ff7 100644
--- a/drivers/media/video/sh_mobile_csi2.c
+++ b/drivers/media/video/sh_mobile_csi2.c
@@ -143,7 +143,7 @@ static int sh_csi2_s_mbus_config(struct v4l2_subdev *sd,
                                 const struct v4l2_mbus_config *cfg)
 {
        struct sh_csi2 *priv = container_of(sd, struct sh_csi2, subdev);
-        struct soc_camera_device *icd = (struct soc_camera_device *)sd->grp_id;
+        struct soc_camera_device *icd = v4l2_get_subdev_hostdata(sd);
        struct v4l2_subdev *client_sd = soc_camera_to_subdev(icd);
        struct v4l2_mbus_config client_cfg = {.type = V4L2_MBUS_CSI2,
                                              .flags = priv->mipi_flags};
@@ -202,7 +202,7 @@ static void sh_csi2_hwinit(struct sh_csi2 *priv)
 static int sh_csi2_client_connect(struct sh_csi2 *priv)
 {
        struct sh_csi2_pdata *pdata = priv->pdev->dev.platform_data;
-        struct soc_camera_device *icd = (struct soc_camera_device *)priv->subdev.grp_id;
+        struct soc_camera_device *icd = v4l2_get_subdev_hostdata(&priv->subdev);
        struct v4l2_subdev *client_sd = soc_camera_to_subdev(icd);
        struct device *dev = v4l2_get_subdevdata(&priv->subdev);
        struct v4l2_mbus_config cfg;
diff --git a/drivers/media/video/sn9c102/sn9c102_core.c b/drivers/media/video/sn9c102/sn9c102_core.c
index 16cb07c5c27..c2882fa5be8 100644
--- a/drivers/media/video/sn9c102/sn9c102_core.c
+++ b/drivers/media/video/sn9c102/sn9c102_core.c
@@ -75,8 +75,8 @@ MODULE_PARM_DESC(video_nr,
                 "\none and for every other camera."
                 "\n");
-static short force_munmap[] = {[0 ... SN9C102_MAX_DEVICES-1] =
+static bool force_munmap[] = {[0 ... SN9C102_MAX_DEVICES-1] =
-                               SN9C102_FORCE_MUNMAP};
+                              SN9C102_FORCE_MUNMAP};
 module_param_array(force_munmap, bool, NULL, 0444);
 MODULE_PARM_DESC(force_munmap,
                 " <0|1[,...]>"
@@ -3420,27 +3420,4 @@ static struct usb_driver sn9c102_usb_driver = {
        .disconnect = sn9c102_usb_disconnect,
 };
-/*****************************************************************************/
+module_usb_driver(sn9c102_usb_driver);
-static int __init sn9c102_module_init(void)
-{
-        int err = 0;
-        KDBG(2, SN9C102_MODULE_NAME " v" SN9C102_MODULE_VERSION);
-        KDBG(3, SN9C102_MODULE_AUTHOR);
-        if ((err = usb_register(&sn9c102_usb_driver)))
-                KDBG(1, "usb_register() failed");
-        return err;
-}
-static void __exit sn9c102_module_exit(void)
-{
-        usb_deregister(&sn9c102_usb_driver);
-}
-module_init(sn9c102_module_init);
-module_exit(sn9c102_module_exit);
diff --git a/drivers/media/video/soc_camera.c b/drivers/media/video/soc_camera.c
index b72580c3895..62e4312515c 100644
--- a/drivers/media/video/soc_camera.c
+++ b/drivers/media/video/soc_camera.c
@@ -1103,7 +1103,8 @@ static int soc_camera_probe(struct soc_camera_device *icd)
        }
        sd = soc_camera_to_subdev(icd);
-        sd->grp_id = (long)icd;
+        sd->grp_id = soc_camera_grp_id(icd);
+        v4l2_set_subdev_hostdata(sd, icd);
        if (v4l2_ctrl_add_handler(&icd->ctrl_handler, sd->ctrl_handler))
                goto ectrl;
diff --git a/drivers/media/video/stk-webcam.c b/drivers/media/video/stk-webcam.c
index cbc105f975d..3c61aec517a 100644
--- a/drivers/media/video/stk-webcam.c
+++ b/drivers/media/video/stk-webcam.c
@@ -38,11 +38,11 @@
 #include "stk-webcam.h"
-static int hflip = 1;
+static bool hflip = 1;
 module_param(hflip, bool, 0444);
 MODULE_PARM_DESC(hflip, "Horizontal image flip (mirror). Defaults to 1");
-static int vflip = 1;
+static bool vflip = 1;
 module_param(vflip, bool, 0444);
 MODULE_PARM_DESC(vflip, "Vertical image flip. Defaults to 1");
@@ -1377,25 +1377,4 @@ static struct usb_driver stk_camera_driver = {
 #endif
 };
+module_usb_driver(stk_camera_driver);
-static int __init stk_camera_init(void)
-{
-        int result;
-        result = usb_register(&stk_camera_driver);
-        if (result)
-                STK_ERROR("usb_register failed ! Error number %d\n", result);
-        return result;
-}
-static void __exit stk_camera_exit(void)
-{
-        usb_deregister(&stk_camera_driver);
-}
-module_init(stk_camera_init);
-module_exit(stk_camera_exit);
diff --git a/drivers/media/video/tm6000/tm6000-alsa.c b/drivers/media/video/tm6000/tm6000-alsa.c
index 7d675c72fd4..bb2047c1035 100644
--- a/drivers/media/video/tm6000/tm6000-alsa.c
+++ b/drivers/media/video/tm6000/tm6000-alsa.c
@@ -42,7 +42,7 @@
 static int index[SNDRV_CARDS] = SNDRV_DEFAULT_IDX;      /* Index 0-MAX */
-static int enable[SNDRV_CARDS] = {1, [1 ... (SNDRV_CARDS - 1)] = 1};
+static bool enable[SNDRV_CARDS] = {1, [1 ... (SNDRV_CARDS - 1)] = 1};
 module_param_array(enable, bool, NULL, 0444);
 MODULE_PARM_DESC(enable, "Enable tm6000x soundcard. default enabled.");
diff --git a/drivers/media/video/tm6000/tm6000-cards.c b/drivers/media/video/tm6000/tm6000-cards.c
index ec2578a0fdf..ff939bc0e0b 100644
--- a/drivers/media/video/tm6000/tm6000-cards.c
+++ b/drivers/media/video/tm6000/tm6000-cards.c
@@ -1371,31 +1371,7 @@ static struct usb_driver tm6000_usb_driver = {
                .id_table = tm6000_id_table,
 };
-static int __init tm6000_module_init(void)
+module_usb_driver(tm6000_usb_driver);
-{
-        int result;
-        printk(KERN_INFO "tm6000" " v4l2 driver version %d.%d.%d loaded\n",
-               (TM6000_VERSION  >> 16) & 0xff,
-               (TM6000_VERSION  >> 8) & 0xff, TM6000_VERSION  & 0xff);
-        /* register this driver with the USB subsystem */
-        result = usb_register(&tm6000_usb_driver);
-        if (result)
-                printk(KERN_ERR "tm6000"
-                           " usb_register failed. Error number %d.\n", result);
-        return result;
-}
-static void __exit tm6000_module_exit(void)
-{
-        /* deregister at USB subsystem */
-        usb_deregister(&tm6000_usb_driver);
-}
-module_init(tm6000_module_init);
-module_exit(tm6000_module_exit);
 MODULE_DESCRIPTION("Trident TVMaster TM5600/TM6000/TM6010 USB2 adapter");
 MODULE_AUTHOR("Mauro Carvalho Chehab");
diff --git a/drivers/media/video/tvp514x.c b/drivers/media/video/tvp514x.c
index 926f0393115..dd26cacd055 100644
--- a/drivers/media/video/tvp514x.c
+++ b/drivers/media/video/tvp514x.c
@@ -52,7 +52,7 @@
 #define LOCK_RETRY_DELAY                (200)
 /* Debug functions */
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
 MODULE_PARM_DESC(debug, "Debug level (0-1)");
diff --git a/drivers/media/video/tvp7002.c b/drivers/media/video/tvp7002.c
index 7875e80cb2f..236c559d5f5 100644
--- a/drivers/media/video/tvp7002.c
+++ b/drivers/media/video/tvp7002.c
@@ -63,7 +63,7 @@ MODULE_LICENSE("GPL");
 #define TVP7002_CL_MASK         0x0f
 /* Debug functions */
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
 MODULE_PARM_DESC(debug, "Debug level (0-2)");
diff --git a/drivers/media/video/upd64083.c b/drivers/media/video/upd64083.c
index 9bbe61700fd..65d065aa609 100644
--- a/drivers/media/video/upd64083.c
+++ b/drivers/media/video/upd64083.c
@@ -34,7 +34,7 @@ MODULE_DESCRIPTION("uPD64083 driver");
 MODULE_AUTHOR("T. Adachi, Takeru KOMORIYA, Hans Verkuil");
 MODULE_LICENSE("GPL");
-static int debug;
+static bool debug;
 module_param(debug, bool, 0644);
 MODULE_PARM_DESC(debug, "Debug level (0-1)");
diff --git a/drivers/media/video/via-camera.c b/drivers/media/video/via-camera.c
index cbf13d09b4a..bfae41ba53c 100644
--- a/drivers/media/video/via-camera.c
+++ b/drivers/media/video/via-camera.c
@@ -34,13 +34,13 @@ MODULE_AUTHOR("Jonathan Corbet <corbet@lwn.net>");
 MODULE_DESCRIPTION("VIA framebuffer-based camera controller driver");
 MODULE_LICENSE("GPL");
-static int flip_image;
+static bool flip_image;
 module_param(flip_image, bool, 0444);
 MODULE_PARM_DESC(flip_image,
                "If set, the sensor will be instructed to flip the image "
                "vertically.");
-static int override_serial;
+static bool override_serial;
 module_param(override_serial, bool, 0444);
 MODULE_PARM_DESC(override_serial,
                "The camera driver will normally refuse to load if "
diff --git a/drivers/media/video/zoran/zoran_device.c b/drivers/media/video/zoran/zoran_device.c
index e8a27844bf3..e86173bd132 100644
--- a/drivers/media/video/zoran/zoran_device.c
+++ b/drivers/media/video/zoran/zoran_device.c
@@ -57,7 +57,7 @@
                   ZR36057_ISR_GIRQ1 | \
                   ZR36057_ISR_JPEGRepIRQ )
-static int lml33dpath;          /* default = 0
+static bool lml33dpath;         /* default = 0
                                 * 1 will use digital path in capture
                                 * mode instead of analog. It can be
                                 * used for picture adjustments using
diff --git a/drivers/media/video/zoran/zoran_driver.c b/drivers/media/video/zoran/zoran_driver.c
index d4d05d2ace6..f6d26419445 100644
--- a/drivers/media/video/zoran/zoran_driver.c
+++ b/drivers/media/video/zoran/zoran_driver.c
@@ -1550,7 +1550,7 @@ static int zoran_enum_fmt(struct zoran *zr, struct v4l2_fmtdesc *fmt, int flag)
                if (zoran_formats[i].flags & flag && num++ == fmt->index) {
                        strncpy(fmt->description, zoran_formats[i].name,
                                sizeof(fmt->description) - 1);
-                        /* fmt struct pre-zeroed, so adding '\0' not neeed */
+                        /* fmt struct pre-zeroed, so adding '\0' not needed */
                        fmt->pixelformat = zoran_formats[i].fourcc;
                        if (zoran_formats[i].flags & ZORAN_FORMAT_COMPRESSED)
                                fmt->flags |= V4L2_FMT_FLAG_COMPRESSED;
diff --git a/drivers/media/video/zoran/zr36060.c b/drivers/media/video/zoran/zr36060.c
index 5e4f57cbf31..f08546fe223 100644
--- a/drivers/media/video/zoran/zr36060.c
+++ b/drivers/media/video/zoran/zr36060.c
@@ -50,7 +50,7 @@
 /* amount of chips attached via this driver */
 static int zr36060_codecs;
-static int low_bitrate;
+static bool low_bitrate;
 module_param(low_bitrate, bool, 0);
 MODULE_PARM_DESC(low_bitrate, "Buz compatibility option, halves bitrate");
diff --git a/drivers/media/video/zr364xx.c b/drivers/media/video/zr364xx.c
index e78cf94f491..cd2e39fc4bf 100644
--- a/drivers/media/video/zr364xx.c
+++ b/drivers/media/video/zr364xx.c
@@ -1695,28 +1695,7 @@ static struct usb_driver zr364xx_driver = {
        .id_table = device_table
 };
+module_usb_driver(zr364xx_driver);
-static int __init zr364xx_init(void)
-{
-        int retval;
-        retval = usb_register(&zr364xx_driver);
-        if (retval)
-                printk(KERN_ERR KBUILD_MODNAME ": usb_register failed!\n");
-        else
-                printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_DESC "\n");
-        return retval;
-}
-static void __exit zr364xx_exit(void)
-{
-        printk(KERN_INFO KBUILD_MODNAME ": " DRIVER_DESC " module unloaded\n");
-        usb_deregister(&zr364xx_driver);
-}
-module_init(zr364xx_init);
-module_exit(zr364xx_exit);
 MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_DESC);