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

On Tue, Apr 9, 2013 at 10:06 AM, Samuel Ortiz 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.

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

On Wed, Apr 3, 2013 at 12:30 PM, Guennadi Liakhovetski wrote: > On Wed, 3 Apr 2013, Fabio Porcedda wrote: > >> On Mon, Mar 18, 2013 at 11:09 AM, Guennadi Liakhovetski >> wrote: >> > Hi Fabio >> > >> > On Mon, 18 Mar 2013, Fabio Porcedda wrote: >

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

On Mon, Mar 18, 2013 at 11:09 AM, Guennadi Liakhovetski 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 sma

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

On Wed, Mar 20, 2013 at 12:46 PM, Arnd Bergmann wrote: > On Wednesday 20 March 2013, Fabio Porcedda wrote: >> >> On Wed, Mar 20, 2013 at 11:20 AM, Arnd Bergmann wrote: >> > On Wednesday 20 March 2013, Fabio Porcedda wrote: >> >> I think we can check

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

On Thu, Mar 21, 2013 at 10:55 AM, Herbert Xu 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. >> >> Sig

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

On Wed, Mar 20, 2013 at 11:20 AM, Arnd Bergmann 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 t

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

On Tue, Mar 19, 2013 at 6:59 PM, Arnd Bergmann wrote: > On Tuesday 19 March 2013, Fabio Porcedda wrote: >> On Tue, Mar 19, 2013 at 5:48 PM, Arnd Bergmann wrote: >> > On Tuesday 19 March 2013, Geert Uytterhoeven wrote: >> >> Hmm, so we may have drivers that

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

alized, except when they are > using 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

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

On Mon, Mar 18, 2013 at 12:28 PM, Arnd Bergmann wrote: > On Monday 18 March 2013, Fabio Porcedda wrote: >> >> On Mon, Mar 18, 2013 at 11:58 AM, Arnd Bergmann wrote: >> > On Monday 18 March 2013, Fabio Porcedda wrote: >> >> Since by using p

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

On Mon, Mar 18, 2013 at 11:58 AM, Arnd Bergmann 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(). >>

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

king it would save a couple lines of code. 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 reason to use module_platform_driver() for this driver. Best regards Fabio Porcedda >> I&#x

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

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 Cc: Fabio Estevam Cc

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

On Fri, Mar 15, 2013 at 12:28 PM, Arnd Bergmann 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

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

On Thu, Mar 14, 2013 at 3:06 PM, Sascha Hauer 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

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

On Thu, Mar 14, 2013 at 6:30 PM, Dmitry Torokhov 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.

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

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 Cc: Greg Kroah-Hartman Cc: Mauro Carvalho Chehab Cc: Josh Wu Cc: Guennadi Liakhovetski Cc: Hans Verkuil Cc: linux-media

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

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 Cc: Greg Kroah-Hartman Cc: Matt Mackall Cc: Herbert Xu Cc: Fabio Estevam Cc: Sascha Hauer --- drivers/char/hw_random/mxc-rnga.c

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

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 Cc: Greg Kroah-Hartman Cc: Linus Walleij Cc: Samuel Ortiz Cc: linux-arm-ker...@lists.infradead.org --- drivers/mfd

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

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 Cc: Greg Kroah-Hartman Cc: Dmitry Torokhov Cc: Mark Brown Cc: Liam Girdwood Cc: Bill Pemberton Cc: Linus Walleij Cc: David Howells

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

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 Cc: Greg Kroah-Hartman Cc: Benoit Cousson Cc: Aneesh V --- drivers/memory/emif.c | 12 +--- 1 file changed, 1 insertion(+), 11

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

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 Cc: Greg Kroah-Hartman Cc: Arnd Bergmann --- drivers/misc/atmel_pwm.c | 12 +--- drivers/misc/ep93xx_pwm.c | 13

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

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 Cc: Greg Kroah-Hartman Cc: Jeff Garzik Cc: linux-...@vger.kernel.org --- drivers/ata/pata_at32.c | 13 + 1 file changed, 1

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

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

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

On Thu, Mar 14, 2013 at 3:01 PM, Guenter Roeck 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. >> >&

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

On Thu, Mar 14, 2013 at 2:25 PM, Guennadi Liakhovetski 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. >&

[PATCH] drivers: misc: use module_platform_driver_probe()

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 Cc: Greg Kroah-Hartman Cc: Arnd Bergmann --- drivers/misc/arm-charlcd.c | 13 + drivers/misc/atmel_pwm.c | 12

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

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 Cc: Greg Kroah-Hartman Cc: Matt Mackall Cc: Herbert Xu Cc: Fabio Estevam Cc: Sascha Hauer --- drivers/char/hw_random/mxc-rnga.c

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

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 Cc: Greg Kroah-Hartman Cc: Jeff Garzik Cc: linux-...@vger.kernel.org --- drivers/ata/pata_at32.c | 13 + drivers/ata

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

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 Cc: Greg Kroah-Hartman Cc: Mauro Carvalho Chehab Cc: Josh Wu Cc: Guennadi Liakhovetski Cc: Hans Verkuil Cc: linux-media

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

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 07/10] drivers: memory: use module_platform_driver_probe()

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 Cc: Greg Kroah-Hartman Cc: Benoit Cousson Cc: Aneesh V --- drivers/memory/emif.c | 12 +--- 1 file changed, 1 insertion(+), 11

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

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 Cc: Greg Kroah-Hartman Cc: Linus Walleij Cc: Samuel Ortiz Cc: linux-arm-ker...@lists.infradead.org --- drivers/mfd

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

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 Cc: Greg Kroah-Hartman Cc: "David S. Miller" Cc: linux-...@vger.kernel.org --- drivers/ide/gayle.c | 15 +-

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

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 Cc: Greg Kroah-Hartman Cc: Dmitry Torokhov Cc: Mark Brown Cc: Liam Girdwood Cc: Bill Pemberton Cc: Linus Walleij Cc: David Howells

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

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 Cc: Greg Kroah-Hartman Cc: Richard Purdie Cc: Florian Tobias Schandinat Cc: linux-fb...@vger.kernel.org --- drivers/video/backlight

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

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 Cc: Greg Kroah-Hartman Cc: Arnd Bergmann --- drivers/misc/atmel_pwm.c | 12 +--- drivers/misc/ep93xx_pwm.c | 13

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

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 Cc: Greg Kroah-Hartman Cc: Jean Delvare Cc: Guenter Roeck Cc: lm-sens...@lm-sensors.org --- drivers/hwmon/mc13783-adc.c | 13