[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