[PATCH 10/12] leds-atmel-pwm.c: Make pwmled_probe() __devinit

Nicolas Ferre nicolas.ferre at atmel.com
Mon Apr 23 04:21:19 EDT 2012


On 04/20/2012 11:34 PM, Andrew Morton :
> On Mon, 16 Apr 2012 11:20:54 +0200
> Nicolas Ferre <nicolas.ferre at atmel.com> wrote:
> 
>> Commit 892a884 (leds: convert led platform drivers to module_platform_driver)
>> is omitting the section mismatch error: so change annotation of the probe
>> function to __devinit instead of __init.
>>
>> ...
>>
>> --- a/drivers/leds/leds-atmel-pwm.c
>> +++ b/drivers/leds/leds-atmel-pwm.c
>> @@ -35,7 +35,7 @@ static void pwmled_brightness(struct led_classdev *cdev, enum led_brightness b)
>>   * NOTE:  we reuse the platform_data structure of GPIO leds,
>>   * but repurpose its "gpio" number as a PWM channel number.
>>   */
>> -static int __init pwmled_probe(struct platform_device *pdev)
>> +static int __devinit pwmled_probe(struct platform_device *pdev)
>>  {
>>  	const struct gpio_led_platform_data	*pdata;
>>  	struct pwmled				*leds;
> 
> This is theoretically oopsable and the patch is probably needed in 3.4
> and 3.3.x.

Absolutely, I will request it to be included in stable tree then.

> But you've applied it to linu-next via your own tree so I won't be
> merging it.

Ok, thanks for having taken this patch into account Andrew.

Best regards,
-- 
Nicolas Ferre



More information about the linux-arm-kernel mailing list