[lkml]   [2009]   [Apr]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 29/58] move omap24xxcam's probe function to .devinit.text
Hallo Sakari,

On Wed, Apr 01, 2009 at 10:40:15AM +0300, Sakari Ailus wrote:
> ext Uwe Kleine-König wrote:
>> A pointer to omap24xxcam_probe is passed to the core via
>> platform_driver_register and so the function must not disappear when the
>> .init sections are discarded. Otherwise (if also having HOTPLUG=y)
>> unbinding and binding a device to the driver via sysfs will result in an
>> oops as does a device being registered late.
>> An alternative to this patch is using platform_driver_probe instead of
>> platform_driver_register plus removing the pointer to the probe function
>> from the struct platform_driver.
>> Signed-off-by: Uwe Kleine-König <>
>> Cc: Sakari Ailus <>
>> Cc: Trilok Soni <>
>> Cc: Hans Verkuil <>
>> Cc: Mauro Carvalho Chehab <>
>> Cc: Andrew Morton <>
>> ---
>> drivers/media/video/omap24xxcam.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
> Signed-off-by: Sakari Ailus <>
Does your S-o-b means you care for inclusion in mainline?[1] Or should
I record that as an Acked-by?

Best regards

[1] Quoting Documentation/SubmittingPatches:
"The Signed-off-by: tag indicates that the signer was involved
in the development of the patch, or that he/she was in the
patch's delivery path.

If a person was not directly involved in the preparation or
handling of a patch but wishes to signify and record their
approval of it then they can arrange to have an Acked-by: line
added to the patch's changelog."
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | |
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2009-04-01 10:33    [W:0.085 / U:9.076 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site