[PATCH v7 1/3] watchdog: at91sam9_wdt: add device tree

Fabio Porcedda fabio.porcedda at gmail.com
Wed Oct 3 10:10:12 EDT 2012


On Tue, Oct 2, 2012 at 10:07 PM, Andrew Lunn <andrew at lunn.ch> wrote:
>> Date: Tue,  2 Oct 2012 18:01:00 +0200
>> From: Fabio Porcedda <fabio.porcedda at gmail.com>
>> To: Wim Van Sebroeck <wim at iguana.be>, linux-watchdog at vger.kernel.org,
>>         linux-arm-kernel at lists.infradead.org, Nicolas Ferre
>>         <nicolas.ferre at atmel.com>, Jean-Christophe PLAGNIOL-VILLARD
>>         <plagnioj at jcrosoft.com>, Andrew Victor <linux at maxim.org.za>
>> Cc: devicetree-discuss at lists.ozlabs.org
>> Subject: [PATCH v7 1/3] watchdog: at91sam9_wdt: add device tree
>>         support
>> Message-ID:
>>         <1349193662-23482-2-git-send-email-fabio.porcedda at gmail.com>
>>
>> Tested on an at91sam9260 board (evk-pro3)
>>
>> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
>> +++ b/drivers/watchdog/at91sam9_wdt.c
>> @@ -32,6 +32,7 @@
>>  #include <linux/timer.h>
>>  #include <linux/bitops.h>
>>  #include <linux/uaccess.h>
>> +#include <linux/of.h>
>>
>>  #include "at91sam9_wdt.h"
>>
>> > @@ -302,11 +303,21 @@ static int __exit at91wdt_remove(struct platform_device *pdev)
>>         return res;
>>  }
>>
>> +#if defined(CONFIG_OF)
>> +static const struct of_device_id at91_wdt_dt_ids[] = {
>> +       { .compatible = "atmel,at91sam9260-wdt" },
>> +       { /* sentinel */ }
>> +};
>
> Hi Fabio
>
> You are missing __devinitdata.

Thank you, Andrew.
I'll add __initconst, because it's const and use platform_driver_probe.

Best regards
-- 
Fabio Porcedda



More information about the linux-arm-kernel mailing list