Re: [PATCH] [media] mx2_camera: use module_platform_driver_probe()

2013-04-03 Thread Fabio Porcedda
On Mon, Mar 18, 2013 at 11:09 AM, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Fabio On Mon, 18 Mar 2013, Fabio Porcedda wrote: The commit 39793c6 [media] mx2_camera: Convert it to platform driver used module_platform_driver() to make code smaller, but since the driver used

Re: [PATCH] [media] mx2_camera: use module_platform_driver_probe()

2013-04-04 Thread Fabio Porcedda
On Wed, Apr 3, 2013 at 12:30 PM, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Wed, 3 Apr 2013, Fabio Porcedda wrote: On Mon, Mar 18, 2013 at 11:09 AM, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Fabio On Mon, 18 Mar 2013, Fabio Porcedda wrote: The commit 39793c6

Re: [PATCH 08/10] drivers: mfd: use module_platform_driver_probe()

2013-04-09 Thread Fabio Porcedda
On Tue, Apr 9, 2013 at 10:06 AM, Samuel Ortiz sa...@linux.intel.com wrote: Hi Fabio, On Thu, Mar 14, 2013 at 02:11:29PM +0100, Fabio Porcedda wrote: This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off

[PATCH 04/10] drivers: hwmon: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Jean Delvare kh...@linux-fr.org Cc: Guenter Roeck li

[PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Arnd Bergmann a...@arndb.de --- drivers/misc/atmel_pwm.c

[PATCH 09/10] drivers: video: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Richard Purdie rpur...@rpsys.net Cc: Florian Tobias

[PATCH 06/10] drivers: input: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Dmitry Torokhov dmitry.torok...@gmail.com Cc: Mark Brown

[PATCH 05/10] drivers: ide: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: David S. Miller da...@davemloft.net Cc: linux

[PATCH 08/10] drivers: mfd: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Linus Walleij linus.wall...@linaro.org Cc: Samuel Ortiz sa

[PATCH 07/10] drivers: memory: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Benoit Cousson b-cous...@ti.com Cc: Aneesh V ane...@ti.com

[PATCH 00/10] Use module_platform_driver_probe() part 2

2013-03-14 Thread Fabio Porcedda
Hi all, this patch set is the second part for the conversion to use new module_platform_driver_probe() macro. I will send a part 3 for the remaining drivers. Fabio Porcedda (10): drivers: media: use module_platform_driver_probe() drivers: ata: use module_platform_driver_probe() drivers

[PATCH 01/10] drivers: media: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: Josh Wu josh

[PATCH 02/10] drivers: ata: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Jeff Garzik jgar...@pobox.com Cc: linux-...@vger.kernel.org

[PATCH 03/10] drivers: char: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Matt Mackall m...@selenic.com Cc: Herbert Xu herb

[PATCH] drivers: misc: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Arnd Bergmann a...@arndb.de --- drivers/misc/arm-charlcd.c

Re: [PATCH 01/10] drivers: media: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
On Thu, Mar 14, 2013 at 2:25 PM, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Fabio On Thu, 14 Mar 2013, Fabio Porcedda wrote: This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio

Re: [PATCH 04/10] drivers: hwmon: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
On Thu, Mar 14, 2013 at 3:01 PM, Guenter Roeck li...@roeck-us.net wrote: On Thu, Mar 14, 2013 at 02:11:25PM +0100, Fabio Porcedda wrote: This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio

[PATCH v2 0/8] Use module_platform_driver_probe() part 2

2013-03-14 Thread Fabio Porcedda
Hi all, this patch set is the second part for the conversion to use new module_platform_driver_probe() macro. v2: - rebased againg next and so dropped the already converted drivers. Fabio Porcedda (8): drivers: media: use module_platform_driver_probe() drivers: ata: use

[PATCH v2 2/8] drivers: ata: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Jeff Garzik jgar...@pobox.com Cc: linux-...@vger.kernel.org

[PATCH v2 8/8] drivers: misc: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Arnd Bergmann a...@arndb.de --- drivers/misc/atmel_pwm.c

[PATCH v2 5/8] drivers: memory: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Benoit Cousson b-cous...@ti.com Cc: Aneesh V ane...@ti.com

[PATCH v2 4/8] drivers: input: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Dmitry Torokhov dmitry.torok...@gmail.com Cc: Mark Brown

[PATCH v2 6/8] drivers: mfd: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Linus Walleij linus.wall...@linaro.org Cc: Samuel Ortiz sa

[PATCH v2 3/8] drivers: char: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Matt Mackall m...@selenic.com Cc: Herbert Xu herb

[PATCH v2 1/8] drivers: media: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: Josh Wu josh

Re: [PATCH v2 4/8] drivers: input: use module_platform_driver_probe()

2013-03-14 Thread Fabio Porcedda
On Thu, Mar 14, 2013 at 6:30 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Hi Fabio, On Thursday, March 14, 2013 06:09:34 PM Fabio Porcedda wrote: This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. I

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-15 Thread Fabio Porcedda
On Thu, Mar 14, 2013 at 3:06 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Thu, Mar 14, 2013 at 01:58:05PM +, Arnd Bergmann wrote: On Thursday 14 March 2013, Fabio Porcedda wrote: This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-15 Thread Fabio Porcedda
On Fri, Mar 15, 2013 at 12:28 PM, Arnd Bergmann a...@arndb.de wrote: On Friday 15 March 2013, Fabio Porcedda wrote: * Regarding the use of module_platform_driver_probe, I'm a little worried about the interactions with deferred probing. I don't think there are any regressions

[PATCH] [media] mx2_camera: use module_platform_driver_probe()

2013-03-18 Thread Fabio Porcedda
The commit 39793c6 [media] mx2_camera: Convert it to platform driver used module_platform_driver() to make code smaller, but since the driver used platform_driver_probe is more appropriate to use module_platform_driver_probe(). Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com Cc: Fabio

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-18 Thread Fabio Porcedda
reason to use module_platform_driver() for this driver. Best regards Fabio Porcedda I'll change this in a bit. Right now I'm trying to work out why kernel 3.8 is not booting on the ep93xx. I had 3.6.6 on my development board and 3.7 works fine but 3.8 hangs without uncompressing the kernel. Ok

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-18 Thread Fabio Porcedda
On Mon, Mar 18, 2013 at 11:58 AM, Arnd Bergmann a...@arndb.de wrote: On Monday 18 March 2013, Fabio Porcedda wrote: Since by using platform_driver_probe() the function ep93xx_pwm_probe() is freed after initialization, is better to use module_platform_drive_probe(). IMHO i don't see any good

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-19 Thread Fabio Porcedda
On Mon, Mar 18, 2013 at 12:28 PM, Arnd Bergmann a...@arndb.de wrote: On Monday 18 March 2013, Fabio Porcedda wrote: On Mon, Mar 18, 2013 at 11:58 AM, Arnd Bergmann a...@arndb.de wrote: On Monday 18 March 2013, Fabio Porcedda wrote: Since by using platform_driver_probe() the function

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-19 Thread Fabio Porcedda
platform_driver_probe() I guess we need a big fat WARN_ON(-EPROBE_DEFER) in platform_driver_probe() to catch these? Yes, very good idea. Arnd If it's fine, I'll send a patch for that. Regards -- Fabio Porcedda -- To unsubscribe from this list: send the line unsubscribe linux-media

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-20 Thread Fabio Porcedda
On Tue, Mar 19, 2013 at 6:59 PM, Arnd Bergmann a...@arndb.de wrote: On Tuesday 19 March 2013, Fabio Porcedda wrote: On Tue, Mar 19, 2013 at 5:48 PM, Arnd Bergmann a...@arndb.de wrote: On Tuesday 19 March 2013, Geert Uytterhoeven wrote: Hmm, so we may have drivers that (now) work perfectly

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-20 Thread Fabio Porcedda
On Wed, Mar 20, 2013 at 11:20 AM, Arnd Bergmann a...@arndb.de wrote: On Wednesday 20 March 2013, Fabio Porcedda wrote: I think we can check inside the deferred_probe_work_func() if the dev-probe function pointer is equal to platform_drv_probe_fail(). I think it's too late by then, because

Re: [PATCH v2 3/8] drivers: char: use module_platform_driver_probe()

2013-03-21 Thread Fabio Porcedda
On Thu, Mar 21, 2013 at 10:55 AM, Herbert Xu herb...@gondor.apana.org.au wrote: On Thu, Mar 14, 2013 at 06:09:33PM +0100, Fabio Porcedda wrote: This patch converts the drivers to use the module_platform_driver_probe() macro which makes the code smaller and a bit simpler. Signed-off-by: Fabio

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-21 Thread Fabio Porcedda
On Wed, Mar 20, 2013 at 12:46 PM, Arnd Bergmann a...@arndb.de wrote: On Wednesday 20 March 2013, Fabio Porcedda wrote: On Wed, Mar 20, 2013 at 11:20 AM, Arnd Bergmann a...@arndb.de wrote: On Wednesday 20 March 2013, Fabio Porcedda wrote: I think we can check inside