All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] media/video:  adding __init/__exit macros to various drivers
@ 2009-09-29  0:19 ` Peter Huewe
  0 siblings, 0 replies; 12+ messages in thread
From: Peter Huewe @ 2009-09-29  0:19 UTC (permalink / raw)
  To: Jiri Kosina
  Cc: kernel-janitors, Hans Verkuil, Andy Walls, Mauro Carvalho Chehab,
	Steven Toth, Michael Krufky, Laurent Pinchart,
	Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel, kernel-janitors

From: Peter Huewe <peterhuewe@gmx.de>

Trivial patch which adds the __init/__exit macros to the module_init/
module_exit functions of the following drivers in media video:
    drivers/media/video/ivtv/ivtv-driver.c
    drivers/media/video/cx18/cx18-driver.c
    drivers/media/video/davinci/dm355_ccdc.c
    drivers/media/video/davinci/dm644x_ccdc.c
    drivers/media/video/saa7164/saa7164-core.c
    drivers/media/video/saa7134/saa7134-core.c
    drivers/media/video/cx23885/cx23885-core.c

Please have a look at the small patch and either pull it through
your tree, or please ack' it so Jiri can pull it through the trivial tree.

linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
---
diff --git a/drivers/media/video/cx18/cx18-driver.c b/drivers/media/video/cx18/cx18-driver.c
index 6dd51e2..e12082b 100644
--- a/drivers/media/video/cx18/cx18-driver.c
+++ b/drivers/media/video/cx18/cx18-driver.c
@@ -1200,7 +1200,7 @@ static struct pci_driver cx18_pci_driver = {
       .remove =   cx18_remove,
 };
 
-static int module_start(void)
+static int __init module_start(void)
 {
 	printk(KERN_INFO "cx18:  Start initialization, version %s\n", CX18_VERSION);
 
@@ -1224,7 +1224,7 @@ static int module_start(void)
 	return 0;
 }
 
-static void module_cleanup(void)
+static void __exit module_cleanup(void)
 {
 	pci_unregister_driver(&cx18_pci_driver);
 }
diff --git a/drivers/media/video/cx23885/cx23885-core.c b/drivers/media/video/cx23885/cx23885-core.c
index c31284b..fa2d350 100644
--- a/drivers/media/video/cx23885/cx23885-core.c
+++ b/drivers/media/video/cx23885/cx23885-core.c
@@ -1957,7 +1957,7 @@ static struct pci_driver cx23885_pci_driver = {
 	.resume   = NULL,
 };
 
-static int cx23885_init(void)
+static int __init cx23885_init(void)
 {
 	printk(KERN_INFO "cx23885 driver version %d.%d.%d loaded\n",
 	       (CX23885_VERSION_CODE >> 16) & 0xff,
@@ -1970,7 +1970,7 @@ static int cx23885_init(void)
 	return pci_register_driver(&cx23885_pci_driver);
 }
 
-static void cx23885_fini(void)
+static void __exit cx23885_fini(void)
 {
 	pci_unregister_driver(&cx23885_pci_driver);
 }
diff --git a/drivers/media/video/davinci/dm355_ccdc.c b/drivers/media/video/davinci/dm355_ccdc.c
index 4629cab..56fbefe 100644
--- a/drivers/media/video/davinci/dm355_ccdc.c
+++ b/drivers/media/video/davinci/dm355_ccdc.c
@@ -959,7 +959,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
 	},
 };
 
-static int dm355_ccdc_init(void)
+static int __init dm355_ccdc_init(void)
 {
 	printk(KERN_NOTICE "dm355_ccdc_init\n");
 	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
@@ -969,7 +969,7 @@ static int dm355_ccdc_init(void)
 	return 0;
 }
 
-static void dm355_ccdc_exit(void)
+static void __exit dm355_ccdc_exit(void)
 {
 	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
 }
diff --git a/drivers/media/video/davinci/dm644x_ccdc.c b/drivers/media/video/davinci/dm644x_ccdc.c
index 2f19a91..d5fa193 100644
--- a/drivers/media/video/davinci/dm644x_ccdc.c
+++ b/drivers/media/video/davinci/dm644x_ccdc.c
@@ -859,7 +859,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
 	},
 };
 
-static int dm644x_ccdc_init(void)
+static int __init dm644x_ccdc_init(void)
 {
 	printk(KERN_NOTICE "dm644x_ccdc_init\n");
 	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
@@ -869,7 +869,7 @@ static int dm644x_ccdc_init(void)
 	return 0;
 }
 
-static void dm644x_ccdc_exit(void)
+static void __exit dm644x_ccdc_exit(void)
 {
 	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
 }
diff --git a/drivers/media/video/ivtv/ivtv-driver.c b/drivers/media/video/ivtv/ivtv-driver.c
index 463ec34..7cdbc1a 100644
--- a/drivers/media/video/ivtv/ivtv-driver.c
+++ b/drivers/media/video/ivtv/ivtv-driver.c
@@ -1361,7 +1361,7 @@ static struct pci_driver ivtv_pci_driver = {
       .remove =   ivtv_remove,
 };
 
-static int module_start(void)
+static int __init module_start(void)
 {
 	printk(KERN_INFO "ivtv: Start initialization, version %s\n", IVTV_VERSION);
 
@@ -1385,7 +1385,7 @@ static int module_start(void)
 	return 0;
 }
 
-static void module_cleanup(void)
+static void __exit module_cleanup(void)
 {
 	pci_unregister_driver(&ivtv_pci_driver);
 }
diff --git a/drivers/media/video/saa7134/saa7134-core.c b/drivers/media/video/saa7134/saa7134-core.c
index f87757f..c673901 100644
--- a/drivers/media/video/saa7134/saa7134-core.c
+++ b/drivers/media/video/saa7134/saa7134-core.c
@@ -1319,7 +1319,7 @@ static struct pci_driver saa7134_pci_driver = {
 #endif
 };
 
-static int saa7134_init(void)
+static int __init saa7134_init(void)
 {
 	INIT_LIST_HEAD(&saa7134_devlist);
 	printk(KERN_INFO "saa7130/34: v4l2 driver version %d.%d.%d loaded\n",
@@ -1333,7 +1333,7 @@ static int saa7134_init(void)
 	return pci_register_driver(&saa7134_pci_driver);
 }
 
-static void saa7134_fini(void)
+static void __exit saa7134_fini(void)
 {
 	pci_unregister_driver(&saa7134_pci_driver);
 }
diff --git a/drivers/media/video/saa7164/saa7164-core.c b/drivers/media/video/saa7164/saa7164-core.c
index 709affc..e6aa0fb 100644
--- a/drivers/media/video/saa7164/saa7164-core.c
+++ b/drivers/media/video/saa7164/saa7164-core.c
@@ -724,13 +724,13 @@ static struct pci_driver saa7164_pci_driver = {
 	.resume   = NULL,
 };
 
-static int saa7164_init(void)
+static int __init saa7164_init(void)
 {
 	printk(KERN_INFO "saa7164 driver loaded\n");
 	return pci_register_driver(&saa7164_pci_driver);
 }
 
-static void saa7164_fini(void)
+static void __exit saa7164_fini(void)
 {
 	pci_unregister_driver(&saa7164_pci_driver);
 }

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH] media/video:  adding __init/__exit macros to various drivers
@ 2009-09-29  0:19 ` Peter Huewe
  0 siblings, 0 replies; 12+ messages in thread
From: Peter Huewe @ 2009-09-29  0:19 UTC (permalink / raw)
  To: Jiri Kosina
  Cc: kernel-janitors, Hans Verkuil, Andy Walls, Mauro Carvalho Chehab,
	Steven Toth, Michael Krufky, Laurent Pinchart,
	Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel

From: Peter Huewe <peterhuewe@gmx.de>

Trivial patch which adds the __init/__exit macros to the module_init/
module_exit functions of the following drivers in media video:
    drivers/media/video/ivtv/ivtv-driver.c
    drivers/media/video/cx18/cx18-driver.c
    drivers/media/video/davinci/dm355_ccdc.c
    drivers/media/video/davinci/dm644x_ccdc.c
    drivers/media/video/saa7164/saa7164-core.c
    drivers/media/video/saa7134/saa7134-core.c
    drivers/media/video/cx23885/cx23885-core.c

Please have a look at the small patch and either pull it through
your tree, or please ack' it so Jiri can pull it through the trivial tree.

linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
---
diff --git a/drivers/media/video/cx18/cx18-driver.c b/drivers/media/video/cx18/cx18-driver.c
index 6dd51e2..e12082b 100644
--- a/drivers/media/video/cx18/cx18-driver.c
+++ b/drivers/media/video/cx18/cx18-driver.c
@@ -1200,7 +1200,7 @@ static struct pci_driver cx18_pci_driver = {
       .remove =   cx18_remove,
 };
 
-static int module_start(void)
+static int __init module_start(void)
 {
 	printk(KERN_INFO "cx18:  Start initialization, version %s\n", CX18_VERSION);
 
@@ -1224,7 +1224,7 @@ static int module_start(void)
 	return 0;
 }
 
-static void module_cleanup(void)
+static void __exit module_cleanup(void)
 {
 	pci_unregister_driver(&cx18_pci_driver);
 }
diff --git a/drivers/media/video/cx23885/cx23885-core.c b/drivers/media/video/cx23885/cx23885-core.c
index c31284b..fa2d350 100644
--- a/drivers/media/video/cx23885/cx23885-core.c
+++ b/drivers/media/video/cx23885/cx23885-core.c
@@ -1957,7 +1957,7 @@ static struct pci_driver cx23885_pci_driver = {
 	.resume   = NULL,
 };
 
-static int cx23885_init(void)
+static int __init cx23885_init(void)
 {
 	printk(KERN_INFO "cx23885 driver version %d.%d.%d loaded\n",
 	       (CX23885_VERSION_CODE >> 16) & 0xff,
@@ -1970,7 +1970,7 @@ static int cx23885_init(void)
 	return pci_register_driver(&cx23885_pci_driver);
 }
 
-static void cx23885_fini(void)
+static void __exit cx23885_fini(void)
 {
 	pci_unregister_driver(&cx23885_pci_driver);
 }
diff --git a/drivers/media/video/davinci/dm355_ccdc.c b/drivers/media/video/davinci/dm355_ccdc.c
index 4629cab..56fbefe 100644
--- a/drivers/media/video/davinci/dm355_ccdc.c
+++ b/drivers/media/video/davinci/dm355_ccdc.c
@@ -959,7 +959,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
 	},
 };
 
-static int dm355_ccdc_init(void)
+static int __init dm355_ccdc_init(void)
 {
 	printk(KERN_NOTICE "dm355_ccdc_init\n");
 	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
@@ -969,7 +969,7 @@ static int dm355_ccdc_init(void)
 	return 0;
 }
 
-static void dm355_ccdc_exit(void)
+static void __exit dm355_ccdc_exit(void)
 {
 	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
 }
diff --git a/drivers/media/video/davinci/dm644x_ccdc.c b/drivers/media/video/davinci/dm644x_ccdc.c
index 2f19a91..d5fa193 100644
--- a/drivers/media/video/davinci/dm644x_ccdc.c
+++ b/drivers/media/video/davinci/dm644x_ccdc.c
@@ -859,7 +859,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
 	},
 };
 
-static int dm644x_ccdc_init(void)
+static int __init dm644x_ccdc_init(void)
 {
 	printk(KERN_NOTICE "dm644x_ccdc_init\n");
 	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
@@ -869,7 +869,7 @@ static int dm644x_ccdc_init(void)
 	return 0;
 }
 
-static void dm644x_ccdc_exit(void)
+static void __exit dm644x_ccdc_exit(void)
 {
 	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
 }
diff --git a/drivers/media/video/ivtv/ivtv-driver.c b/drivers/media/video/ivtv/ivtv-driver.c
index 463ec34..7cdbc1a 100644
--- a/drivers/media/video/ivtv/ivtv-driver.c
+++ b/drivers/media/video/ivtv/ivtv-driver.c
@@ -1361,7 +1361,7 @@ static struct pci_driver ivtv_pci_driver = {
       .remove =   ivtv_remove,
 };
 
-static int module_start(void)
+static int __init module_start(void)
 {
 	printk(KERN_INFO "ivtv: Start initialization, version %s\n", IVTV_VERSION);
 
@@ -1385,7 +1385,7 @@ static int module_start(void)
 	return 0;
 }
 
-static void module_cleanup(void)
+static void __exit module_cleanup(void)
 {
 	pci_unregister_driver(&ivtv_pci_driver);
 }
diff --git a/drivers/media/video/saa7134/saa7134-core.c b/drivers/media/video/saa7134/saa7134-core.c
index f87757f..c673901 100644
--- a/drivers/media/video/saa7134/saa7134-core.c
+++ b/drivers/media/video/saa7134/saa7134-core.c
@@ -1319,7 +1319,7 @@ static struct pci_driver saa7134_pci_driver = {
 #endif
 };
 
-static int saa7134_init(void)
+static int __init saa7134_init(void)
 {
 	INIT_LIST_HEAD(&saa7134_devlist);
 	printk(KERN_INFO "saa7130/34: v4l2 driver version %d.%d.%d loaded\n",
@@ -1333,7 +1333,7 @@ static int saa7134_init(void)
 	return pci_register_driver(&saa7134_pci_driver);
 }
 
-static void saa7134_fini(void)
+static void __exit saa7134_fini(void)
 {
 	pci_unregister_driver(&saa7134_pci_driver);
 }
diff --git a/drivers/media/video/saa7164/saa7164-core.c b/drivers/media/video/saa7164/saa7164-core.c
index 709affc..e6aa0fb 100644
--- a/drivers/media/video/saa7164/saa7164-core.c
+++ b/drivers/media/video/saa7164/saa7164-core.c
@@ -724,13 +724,13 @@ static struct pci_driver saa7164_pci_driver = {
 	.resume   = NULL,
 };
 
-static int saa7164_init(void)
+static int __init saa7164_init(void)
 {
 	printk(KERN_INFO "saa7164 driver loaded\n");
 	return pci_register_driver(&saa7164_pci_driver);
 }
 
-static void saa7164_fini(void)
+static void __exit saa7164_fini(void)
 {
 	pci_unregister_driver(&saa7164_pci_driver);
 }

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various drivers
  2009-09-29  0:19 ` Peter Huewe
@ 2009-09-30  1:05   ` Andy Walls
  -1 siblings, 0 replies; 12+ messages in thread
From: Andy Walls @ 2009-09-30  1:05 UTC (permalink / raw)
  To: Peter Huewe
  Cc: Jiri Kosina, kernel-janitors, Hans Verkuil,
	Mauro Carvalho Chehab, Steven Toth, Michael Krufky,
	Laurent Pinchart, Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel

On Tue, 2009-09-29 at 02:19 +0200, Peter Huewe wrote:
> From: Peter Huewe <peterhuewe@gmx.de>
> 
> Trivial patch which adds the __init/__exit macros to the module_init/
> module_exit functions of the following drivers in media video:
>     drivers/media/video/ivtv/ivtv-driver.c
>     drivers/media/video/cx18/cx18-driver.c
>     drivers/media/video/davinci/dm355_ccdc.c
>     drivers/media/video/davinci/dm644x_ccdc.c
>     drivers/media/video/saa7164/saa7164-core.c
>     drivers/media/video/saa7134/saa7134-core.c
>     drivers/media/video/cx23885/cx23885-core.c
> 
> Please have a look at the small patch and either pull it through
> your tree, or please ack' it so Jiri can pull it through the trivial tree.
> 
> linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009
> 
> Signed-off-by: Peter Huewe <peterhuewe@gmx.de>

For ivtv-driver.c and cx18-driver.c:

Acked-by: Andy Walls <awalls@radix.net>

> ---
> diff --git a/drivers/media/video/cx18/cx18-driver.c b/drivers/media/video/cx18/cx18-driver.c
> index 6dd51e2..e12082b 100644
> --- a/drivers/media/video/cx18/cx18-driver.c
> +++ b/drivers/media/video/cx18/cx18-driver.c
> @@ -1200,7 +1200,7 @@ static struct pci_driver cx18_pci_driver = {
>        .remove =   cx18_remove,
>  };
>  
> -static int module_start(void)
> +static int __init module_start(void)
>  {
>  	printk(KERN_INFO "cx18:  Start initialization, version %s\n", CX18_VERSION);
>  
> @@ -1224,7 +1224,7 @@ static int module_start(void)
>  	return 0;
>  }
>  
> -static void module_cleanup(void)
> +static void __exit module_cleanup(void)
>  {
>  	pci_unregister_driver(&cx18_pci_driver);
>  }
> diff --git a/drivers/media/video/cx23885/cx23885-core.c b/drivers/media/video/cx23885/cx23885-core.c
> index c31284b..fa2d350 100644
> --- a/drivers/media/video/cx23885/cx23885-core.c
> +++ b/drivers/media/video/cx23885/cx23885-core.c
> @@ -1957,7 +1957,7 @@ static struct pci_driver cx23885_pci_driver = {
>  	.resume   = NULL,
>  };
>  
> -static int cx23885_init(void)
> +static int __init cx23885_init(void)
>  {
>  	printk(KERN_INFO "cx23885 driver version %d.%d.%d loaded\n",
>  	       (CX23885_VERSION_CODE >> 16) & 0xff,
> @@ -1970,7 +1970,7 @@ static int cx23885_init(void)
>  	return pci_register_driver(&cx23885_pci_driver);
>  }
>  
> -static void cx23885_fini(void)
> +static void __exit cx23885_fini(void)
>  {
>  	pci_unregister_driver(&cx23885_pci_driver);
>  }
> diff --git a/drivers/media/video/davinci/dm355_ccdc.c b/drivers/media/video/davinci/dm355_ccdc.c
> index 4629cab..56fbefe 100644
> --- a/drivers/media/video/davinci/dm355_ccdc.c
> +++ b/drivers/media/video/davinci/dm355_ccdc.c
> @@ -959,7 +959,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
>  	},
>  };
>  
> -static int dm355_ccdc_init(void)
> +static int __init dm355_ccdc_init(void)
>  {
>  	printk(KERN_NOTICE "dm355_ccdc_init\n");
>  	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
> @@ -969,7 +969,7 @@ static int dm355_ccdc_init(void)
>  	return 0;
>  }
>  
> -static void dm355_ccdc_exit(void)
> +static void __exit dm355_ccdc_exit(void)
>  {
>  	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
>  }
> diff --git a/drivers/media/video/davinci/dm644x_ccdc.c b/drivers/media/video/davinci/dm644x_ccdc.c
> index 2f19a91..d5fa193 100644
> --- a/drivers/media/video/davinci/dm644x_ccdc.c
> +++ b/drivers/media/video/davinci/dm644x_ccdc.c
> @@ -859,7 +859,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
>  	},
>  };
>  
> -static int dm644x_ccdc_init(void)
> +static int __init dm644x_ccdc_init(void)
>  {
>  	printk(KERN_NOTICE "dm644x_ccdc_init\n");
>  	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
> @@ -869,7 +869,7 @@ static int dm644x_ccdc_init(void)
>  	return 0;
>  }
>  
> -static void dm644x_ccdc_exit(void)
> +static void __exit dm644x_ccdc_exit(void)
>  {
>  	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
>  }
> diff --git a/drivers/media/video/ivtv/ivtv-driver.c b/drivers/media/video/ivtv/ivtv-driver.c
> index 463ec34..7cdbc1a 100644
> --- a/drivers/media/video/ivtv/ivtv-driver.c
> +++ b/drivers/media/video/ivtv/ivtv-driver.c
> @@ -1361,7 +1361,7 @@ static struct pci_driver ivtv_pci_driver = {
>        .remove =   ivtv_remove,
>  };
>  
> -static int module_start(void)
> +static int __init module_start(void)
>  {
>  	printk(KERN_INFO "ivtv: Start initialization, version %s\n", IVTV_VERSION);
>  
> @@ -1385,7 +1385,7 @@ static int module_start(void)
>  	return 0;
>  }
>  
> -static void module_cleanup(void)
> +static void __exit module_cleanup(void)
>  {
>  	pci_unregister_driver(&ivtv_pci_driver);
>  }
> diff --git a/drivers/media/video/saa7134/saa7134-core.c b/drivers/media/video/saa7134/saa7134-core.c
> index f87757f..c673901 100644
> --- a/drivers/media/video/saa7134/saa7134-core.c
> +++ b/drivers/media/video/saa7134/saa7134-core.c
> @@ -1319,7 +1319,7 @@ static struct pci_driver saa7134_pci_driver = {
>  #endif
>  };
>  
> -static int saa7134_init(void)
> +static int __init saa7134_init(void)
>  {
>  	INIT_LIST_HEAD(&saa7134_devlist);
>  	printk(KERN_INFO "saa7130/34: v4l2 driver version %d.%d.%d loaded\n",
> @@ -1333,7 +1333,7 @@ static int saa7134_init(void)
>  	return pci_register_driver(&saa7134_pci_driver);
>  }
>  
> -static void saa7134_fini(void)
> +static void __exit saa7134_fini(void)
>  {
>  	pci_unregister_driver(&saa7134_pci_driver);
>  }
> diff --git a/drivers/media/video/saa7164/saa7164-core.c b/drivers/media/video/saa7164/saa7164-core.c
> index 709affc..e6aa0fb 100644
> --- a/drivers/media/video/saa7164/saa7164-core.c
> +++ b/drivers/media/video/saa7164/saa7164-core.c
> @@ -724,13 +724,13 @@ static struct pci_driver saa7164_pci_driver = {
>  	.resume   = NULL,
>  };
>  
> -static int saa7164_init(void)
> +static int __init saa7164_init(void)
>  {
>  	printk(KERN_INFO "saa7164 driver loaded\n");
>  	return pci_register_driver(&saa7164_pci_driver);
>  }
>  
> -static void saa7164_fini(void)
> +static void __exit saa7164_fini(void)
>  {
>  	pci_unregister_driver(&saa7164_pci_driver);
>  }
> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various
@ 2009-09-30  1:05   ` Andy Walls
  0 siblings, 0 replies; 12+ messages in thread
From: Andy Walls @ 2009-09-30  1:05 UTC (permalink / raw)
  To: Peter Huewe
  Cc: Jiri Kosina, kernel-janitors, Hans Verkuil,
	Mauro Carvalho Chehab, Steven Toth, Michael Krufky,
	Laurent Pinchart, Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel

On Tue, 2009-09-29 at 02:19 +0200, Peter Huewe wrote:
> From: Peter Huewe <peterhuewe@gmx.de>
> 
> Trivial patch which adds the __init/__exit macros to the module_init/
> module_exit functions of the following drivers in media video:
>     drivers/media/video/ivtv/ivtv-driver.c
>     drivers/media/video/cx18/cx18-driver.c
>     drivers/media/video/davinci/dm355_ccdc.c
>     drivers/media/video/davinci/dm644x_ccdc.c
>     drivers/media/video/saa7164/saa7164-core.c
>     drivers/media/video/saa7134/saa7134-core.c
>     drivers/media/video/cx23885/cx23885-core.c
> 
> Please have a look at the small patch and either pull it through
> your tree, or please ack' it so Jiri can pull it through the trivial tree.
> 
> linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009
> 
> Signed-off-by: Peter Huewe <peterhuewe@gmx.de>

For ivtv-driver.c and cx18-driver.c:

Acked-by: Andy Walls <awalls@radix.net>

> ---
> diff --git a/drivers/media/video/cx18/cx18-driver.c b/drivers/media/video/cx18/cx18-driver.c
> index 6dd51e2..e12082b 100644
> --- a/drivers/media/video/cx18/cx18-driver.c
> +++ b/drivers/media/video/cx18/cx18-driver.c
> @@ -1200,7 +1200,7 @@ static struct pci_driver cx18_pci_driver = {
>        .remove =   cx18_remove,
>  };
>  
> -static int module_start(void)
> +static int __init module_start(void)
>  {
>  	printk(KERN_INFO "cx18:  Start initialization, version %s\n", CX18_VERSION);
>  
> @@ -1224,7 +1224,7 @@ static int module_start(void)
>  	return 0;
>  }
>  
> -static void module_cleanup(void)
> +static void __exit module_cleanup(void)
>  {
>  	pci_unregister_driver(&cx18_pci_driver);
>  }
> diff --git a/drivers/media/video/cx23885/cx23885-core.c b/drivers/media/video/cx23885/cx23885-core.c
> index c31284b..fa2d350 100644
> --- a/drivers/media/video/cx23885/cx23885-core.c
> +++ b/drivers/media/video/cx23885/cx23885-core.c
> @@ -1957,7 +1957,7 @@ static struct pci_driver cx23885_pci_driver = {
>  	.resume   = NULL,
>  };
>  
> -static int cx23885_init(void)
> +static int __init cx23885_init(void)
>  {
>  	printk(KERN_INFO "cx23885 driver version %d.%d.%d loaded\n",
>  	       (CX23885_VERSION_CODE >> 16) & 0xff,
> @@ -1970,7 +1970,7 @@ static int cx23885_init(void)
>  	return pci_register_driver(&cx23885_pci_driver);
>  }
>  
> -static void cx23885_fini(void)
> +static void __exit cx23885_fini(void)
>  {
>  	pci_unregister_driver(&cx23885_pci_driver);
>  }
> diff --git a/drivers/media/video/davinci/dm355_ccdc.c b/drivers/media/video/davinci/dm355_ccdc.c
> index 4629cab..56fbefe 100644
> --- a/drivers/media/video/davinci/dm355_ccdc.c
> +++ b/drivers/media/video/davinci/dm355_ccdc.c
> @@ -959,7 +959,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
>  	},
>  };
>  
> -static int dm355_ccdc_init(void)
> +static int __init dm355_ccdc_init(void)
>  {
>  	printk(KERN_NOTICE "dm355_ccdc_init\n");
>  	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
> @@ -969,7 +969,7 @@ static int dm355_ccdc_init(void)
>  	return 0;
>  }
>  
> -static void dm355_ccdc_exit(void)
> +static void __exit dm355_ccdc_exit(void)
>  {
>  	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
>  }
> diff --git a/drivers/media/video/davinci/dm644x_ccdc.c b/drivers/media/video/davinci/dm644x_ccdc.c
> index 2f19a91..d5fa193 100644
> --- a/drivers/media/video/davinci/dm644x_ccdc.c
> +++ b/drivers/media/video/davinci/dm644x_ccdc.c
> @@ -859,7 +859,7 @@ static struct ccdc_hw_device ccdc_hw_dev = {
>  	},
>  };
>  
> -static int dm644x_ccdc_init(void)
> +static int __init dm644x_ccdc_init(void)
>  {
>  	printk(KERN_NOTICE "dm644x_ccdc_init\n");
>  	if (vpfe_register_ccdc_device(&ccdc_hw_dev) < 0)
> @@ -869,7 +869,7 @@ static int dm644x_ccdc_init(void)
>  	return 0;
>  }
>  
> -static void dm644x_ccdc_exit(void)
> +static void __exit dm644x_ccdc_exit(void)
>  {
>  	vpfe_unregister_ccdc_device(&ccdc_hw_dev);
>  }
> diff --git a/drivers/media/video/ivtv/ivtv-driver.c b/drivers/media/video/ivtv/ivtv-driver.c
> index 463ec34..7cdbc1a 100644
> --- a/drivers/media/video/ivtv/ivtv-driver.c
> +++ b/drivers/media/video/ivtv/ivtv-driver.c
> @@ -1361,7 +1361,7 @@ static struct pci_driver ivtv_pci_driver = {
>        .remove =   ivtv_remove,
>  };
>  
> -static int module_start(void)
> +static int __init module_start(void)
>  {
>  	printk(KERN_INFO "ivtv: Start initialization, version %s\n", IVTV_VERSION);
>  
> @@ -1385,7 +1385,7 @@ static int module_start(void)
>  	return 0;
>  }
>  
> -static void module_cleanup(void)
> +static void __exit module_cleanup(void)
>  {
>  	pci_unregister_driver(&ivtv_pci_driver);
>  }
> diff --git a/drivers/media/video/saa7134/saa7134-core.c b/drivers/media/video/saa7134/saa7134-core.c
> index f87757f..c673901 100644
> --- a/drivers/media/video/saa7134/saa7134-core.c
> +++ b/drivers/media/video/saa7134/saa7134-core.c
> @@ -1319,7 +1319,7 @@ static struct pci_driver saa7134_pci_driver = {
>  #endif
>  };
>  
> -static int saa7134_init(void)
> +static int __init saa7134_init(void)
>  {
>  	INIT_LIST_HEAD(&saa7134_devlist);
>  	printk(KERN_INFO "saa7130/34: v4l2 driver version %d.%d.%d loaded\n",
> @@ -1333,7 +1333,7 @@ static int saa7134_init(void)
>  	return pci_register_driver(&saa7134_pci_driver);
>  }
>  
> -static void saa7134_fini(void)
> +static void __exit saa7134_fini(void)
>  {
>  	pci_unregister_driver(&saa7134_pci_driver);
>  }
> diff --git a/drivers/media/video/saa7164/saa7164-core.c b/drivers/media/video/saa7164/saa7164-core.c
> index 709affc..e6aa0fb 100644
> --- a/drivers/media/video/saa7164/saa7164-core.c
> +++ b/drivers/media/video/saa7164/saa7164-core.c
> @@ -724,13 +724,13 @@ static struct pci_driver saa7164_pci_driver = {
>  	.resume   = NULL,
>  };
>  
> -static int saa7164_init(void)
> +static int __init saa7164_init(void)
>  {
>  	printk(KERN_INFO "saa7164 driver loaded\n");
>  	return pci_register_driver(&saa7164_pci_driver);
>  }
>  
> -static void saa7164_fini(void)
> +static void __exit saa7164_fini(void)
>  {
>  	pci_unregister_driver(&saa7164_pci_driver);
>  }
> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various drivers
  2009-09-29  0:19 ` Peter Huewe
@ 2009-09-30  8:02   ` Laurent Pinchart
  -1 siblings, 0 replies; 12+ messages in thread
From: Laurent Pinchart @ 2009-09-30  8:02 UTC (permalink / raw)
  To: Peter Huewe
  Cc: Jiri Kosina, kernel-janitors, Hans Verkuil, Andy Walls,
	Mauro Carvalho Chehab, Steven Toth, Michael Krufky,
	Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel

On Tuesday 29 September 2009 02:19:00 Peter Huewe wrote:
> From: Peter Huewe <peterhuewe@gmx.de>
> 
> Trivial patch which adds the __init/__exit macros to the module_init/
> module_exit functions of the following drivers in media video:
>     drivers/media/video/ivtv/ivtv-driver.c
>     drivers/media/video/cx18/cx18-driver.c
>     drivers/media/video/davinci/dm355_ccdc.c
>     drivers/media/video/davinci/dm644x_ccdc.c
>     drivers/media/video/saa7164/saa7164-core.c
>     drivers/media/video/saa7134/saa7134-core.c
>     drivers/media/video/cx23885/cx23885-core.c

For drivers/media/video/davinci/dm355_ccdc.c and 
drivers/media/video/davinci/dm644x_ccdc.c,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> Please have a look at the small patch and either pull it through
> your tree, or please ack' it so Jiri can pull it through the trivial tree.
> 
> linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009
> 
> Signed-off-by: Peter Huewe <peterhuewe@gmx.de>

-- 
Laurent Pinchart

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various drivers
@ 2009-09-30  8:02   ` Laurent Pinchart
  0 siblings, 0 replies; 12+ messages in thread
From: Laurent Pinchart @ 2009-09-30  8:02 UTC (permalink / raw)
  To: Peter Huewe
  Cc: Jiri Kosina, kernel-janitors, Hans Verkuil, Andy Walls,
	Mauro Carvalho Chehab, Steven Toth, Michael Krufky,
	Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel

On Tuesday 29 September 2009 02:19:00 Peter Huewe wrote:
> From: Peter Huewe <peterhuewe@gmx.de>
> 
> Trivial patch which adds the __init/__exit macros to the module_init/
> module_exit functions of the following drivers in media video:
>     drivers/media/video/ivtv/ivtv-driver.c
>     drivers/media/video/cx18/cx18-driver.c
>     drivers/media/video/davinci/dm355_ccdc.c
>     drivers/media/video/davinci/dm644x_ccdc.c
>     drivers/media/video/saa7164/saa7164-core.c
>     drivers/media/video/saa7134/saa7134-core.c
>     drivers/media/video/cx23885/cx23885-core.c

For drivers/media/video/davinci/dm355_ccdc.c and 
drivers/media/video/davinci/dm644x_ccdc.c,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> Please have a look at the small patch and either pull it through
> your tree, or please ack' it so Jiri can pull it through the trivial tree.
> 
> linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009
> 
> Signed-off-by: Peter Huewe <peterhuewe@gmx.de>

-- 
Laurent Pinchart

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH] media/video:  adding __init/__exit macros to various drivers
  2009-09-30  8:02   ` Laurent Pinchart
@ 2009-09-30 13:26     ` Karicheri, Muralidharan
  -1 siblings, 0 replies; 12+ messages in thread
From: Karicheri, Muralidharan @ 2009-09-30 13:26 UTC (permalink / raw)
  To: Peter Huewe
  Cc: Jiri Kosina, kernel-janitors, Hans Verkuil, Andy Walls,
	Mauro Carvalho Chehab, Steven Toth, Michael Krufky,
	Martin Dauskardt, Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel,
	linux-media, linux-kernel, Laurent Pinchart




For drivers/media/video/davinci/dm355_ccdc.c and
drivers/media/video/davinci/dm644x_ccdc.c,
Acked-by: Muralidharan Karicheri <m-karicheri2@ti.com>

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
email: m-karicheri2@ti.com

>-----Original Message-----
>From: Laurent Pinchart [mailto:laurent.pinchart@ideasonboard.com]
>Sent: Wednesday, September 30, 2009 4:03 AM
>To: Peter Huewe
>Cc: Jiri Kosina; kernel-janitors@vger.kernel.org; Hans Verkuil; Andy Walls;
>Mauro Carvalho Chehab; Steven Toth; Michael Krufky; Karicheri,
>Muralidharan; Martin Dauskardt; Beholder Intl. Ltd. Dmitry Belimov; ivtv-
>devel@ivtvdriver.org; linux-media@vger.kernel.org; linux-
>kernel@vger.kernel.org
>Subject: Re: [PATCH] media/video: adding __init/__exit macros to various
>drivers
>
>On Tuesday 29 September 2009 02:19:00 Peter Huewe wrote:
>> From: Peter Huewe <peterhuewe@gmx.de>
>>
>> Trivial patch which adds the __init/__exit macros to the module_init/
>> module_exit functions of the following drivers in media video:
>>     drivers/media/video/ivtv/ivtv-driver.c
>>     drivers/media/video/cx18/cx18-driver.c
>>     drivers/media/video/davinci/dm355_ccdc.c
>>     drivers/media/video/davinci/dm644x_ccdc.c
>>     drivers/media/video/saa7164/saa7164-core.c
>>     drivers/media/video/saa7134/saa7134-core.c
>>     drivers/media/video/cx23885/cx23885-core.c
>
>For drivers/media/video/davinci/dm355_ccdc.c and
>drivers/media/video/davinci/dm644x_ccdc.c,
>
>Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
>> Please have a look at the small patch and either pull it through
>> your tree, or please ack' it so Jiri can pull it through the trivial tree.
>>
>> linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009
>>
>> Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
>
>--
>Laurent Pinchart


^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH] media/video:  adding __init/__exit macros to various
@ 2009-09-30 13:26     ` Karicheri, Muralidharan
  0 siblings, 0 replies; 12+ messages in thread
From: Karicheri, Muralidharan @ 2009-09-30 13:26 UTC (permalink / raw)
  To: Peter Huewe
  Cc: Jiri Kosina, kernel-janitors, Hans Verkuil, Andy Walls,
	Mauro Carvalho Chehab, Steven Toth, Michael Krufky,
	Martin Dauskardt, Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel,
	linux-media, linux-kernel, Laurent Pinchart




For drivers/media/video/davinci/dm355_ccdc.c and
drivers/media/video/davinci/dm644x_ccdc.c,
Acked-by: Muralidharan Karicheri <m-karicheri2@ti.com>

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
email: m-karicheri2@ti.com

>-----Original Message-----
>From: Laurent Pinchart [mailto:laurent.pinchart@ideasonboard.com]
>Sent: Wednesday, September 30, 2009 4:03 AM
>To: Peter Huewe
>Cc: Jiri Kosina; kernel-janitors@vger.kernel.org; Hans Verkuil; Andy Walls;
>Mauro Carvalho Chehab; Steven Toth; Michael Krufky; Karicheri,
>Muralidharan; Martin Dauskardt; Beholder Intl. Ltd. Dmitry Belimov; ivtv-
>devel@ivtvdriver.org; linux-media@vger.kernel.org; linux-
>kernel@vger.kernel.org
>Subject: Re: [PATCH] media/video: adding __init/__exit macros to various
>drivers
>
>On Tuesday 29 September 2009 02:19:00 Peter Huewe wrote:
>> From: Peter Huewe <peterhuewe@gmx.de>
>>
>> Trivial patch which adds the __init/__exit macros to the module_init/
>> module_exit functions of the following drivers in media video:
>>     drivers/media/video/ivtv/ivtv-driver.c
>>     drivers/media/video/cx18/cx18-driver.c
>>     drivers/media/video/davinci/dm355_ccdc.c
>>     drivers/media/video/davinci/dm644x_ccdc.c
>>     drivers/media/video/saa7164/saa7164-core.c
>>     drivers/media/video/saa7134/saa7134-core.c
>>     drivers/media/video/cx23885/cx23885-core.c
>
>For drivers/media/video/davinci/dm355_ccdc.c and
>drivers/media/video/davinci/dm644x_ccdc.c,
>
>Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
>> Please have a look at the small patch and either pull it through
>> your tree, or please ack' it so Jiri can pull it through the trivial tree.
>>
>> linux version v2.6.32-rc1 - linus git tree, Di 29. Sep 01:10:18 CEST 2009
>>
>> Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
>
>--
>Laurent Pinchart


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various drivers
  2009-09-30 13:26     ` [PATCH] media/video: adding __init/__exit macros to various Karicheri, Muralidharan
@ 2009-09-30 19:30       ` Steven Toth
  -1 siblings, 0 replies; 12+ messages in thread
From: Steven Toth @ 2009-09-30 19:30 UTC (permalink / raw)
  To: Karicheri, Muralidharan
  Cc: Peter Huewe, Jiri Kosina, kernel-janitors, Hans Verkuil,
	Andy Walls, Mauro Carvalho Chehab, Michael Krufky,
	Martin Dauskardt, Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel,
	linux-media, linux-kernel, Laurent Pinchart

On 9/30/09 9:26 AM, Karicheri, Muralidharan wrote:

drivers/media/video/saa7164/saa7164-core.c

drivers/media/video/cx23885/cx23885-core.c

Acked-By: Steven Toth <stoth@kernellabs.com>

-- 
Steven Toth - Kernel Labs
http://www.kernellabs.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various
@ 2009-09-30 19:30       ` Steven Toth
  0 siblings, 0 replies; 12+ messages in thread
From: Steven Toth @ 2009-09-30 19:30 UTC (permalink / raw)
  To: Karicheri, Muralidharan
  Cc: Peter Huewe, Jiri Kosina, kernel-janitors, Hans Verkuil,
	Andy Walls, Mauro Carvalho Chehab, Michael Krufky,
	Martin Dauskardt, Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel,
	linux-media, linux-kernel, Laurent Pinchart

On 9/30/09 9:26 AM, Karicheri, Muralidharan wrote:

drivers/media/video/saa7164/saa7164-core.c

drivers/media/video/cx23885/cx23885-core.c

Acked-By: Steven Toth <stoth@kernellabs.com>

-- 
Steven Toth - Kernel Labs
http://www.kernellabs.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various drivers
  2009-09-29  0:19 ` Peter Huewe
@ 2009-10-09 15:21   ` Jiri Kosina
  -1 siblings, 0 replies; 12+ messages in thread
From: Jiri Kosina @ 2009-10-09 15:21 UTC (permalink / raw)
  To: Peter Huewe
  Cc: kernel-janitors, Hans Verkuil, Andy Walls, Mauro Carvalho Chehab,
	Steven Toth, Michael Krufky, Laurent Pinchart,
	Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel, kernel-janitors

On Tue, 29 Sep 2009, Peter Huewe wrote:

> From: Peter Huewe <peterhuewe@gmx.de>
> 
> Trivial patch which adds the __init/__exit macros to the module_init/
> module_exit functions of the following drivers in media video:
>     drivers/media/video/ivtv/ivtv-driver.c
>     drivers/media/video/cx18/cx18-driver.c
>     drivers/media/video/davinci/dm355_ccdc.c
>     drivers/media/video/davinci/dm644x_ccdc.c
>     drivers/media/video/saa7164/saa7164-core.c
>     drivers/media/video/saa7134/saa7134-core.c
>     drivers/media/video/cx23885/cx23885-core.c

This doesn't seem to be present in linux-next as of today. I have queued 
in in trivial tree.

-- 
Jiri Kosina
SUSE Labs, Novell Inc.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] media/video:  adding __init/__exit macros to various
@ 2009-10-09 15:21   ` Jiri Kosina
  0 siblings, 0 replies; 12+ messages in thread
From: Jiri Kosina @ 2009-10-09 15:21 UTC (permalink / raw)
  To: Peter Huewe
  Cc: kernel-janitors, Hans Verkuil, Andy Walls, Mauro Carvalho Chehab,
	Steven Toth, Michael Krufky, Laurent Pinchart,
	Muralidharan Karicheri, Martin Dauskardt,
	Beholder Intl. Ltd. Dmitry Belimov, ivtv-devel, linux-media,
	linux-kernel

On Tue, 29 Sep 2009, Peter Huewe wrote:

> From: Peter Huewe <peterhuewe@gmx.de>
> 
> Trivial patch which adds the __init/__exit macros to the module_init/
> module_exit functions of the following drivers in media video:
>     drivers/media/video/ivtv/ivtv-driver.c
>     drivers/media/video/cx18/cx18-driver.c
>     drivers/media/video/davinci/dm355_ccdc.c
>     drivers/media/video/davinci/dm644x_ccdc.c
>     drivers/media/video/saa7164/saa7164-core.c
>     drivers/media/video/saa7134/saa7134-core.c
>     drivers/media/video/cx23885/cx23885-core.c

This doesn't seem to be present in linux-next as of today. I have queued 
in in trivial tree.

-- 
Jiri Kosina
SUSE Labs, Novell Inc.

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2009-10-09 15:22 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-09-29  0:19 [PATCH] media/video: adding __init/__exit macros to various drivers Peter Huewe
2009-09-29  0:19 ` Peter Huewe
2009-09-30  1:05 ` Andy Walls
2009-09-30  1:05   ` [PATCH] media/video: adding __init/__exit macros to various Andy Walls
2009-09-30  8:02 ` [PATCH] media/video: adding __init/__exit macros to various drivers Laurent Pinchart
2009-09-30  8:02   ` Laurent Pinchart
2009-09-30 13:26   ` Karicheri, Muralidharan
2009-09-30 13:26     ` [PATCH] media/video: adding __init/__exit macros to various Karicheri, Muralidharan
2009-09-30 19:30     ` [PATCH] media/video: adding __init/__exit macros to various drivers Steven Toth
2009-09-30 19:30       ` [PATCH] media/video: adding __init/__exit macros to various Steven Toth
2009-10-09 15:21 ` [PATCH] media/video: adding __init/__exit macros to various drivers Jiri Kosina
2009-10-09 15:21   ` [PATCH] media/video: adding __init/__exit macros to various Jiri Kosina

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.