Hi Florian,
> -----Original Message-----
> From: Florian Fainelli [mailto:f.faine...@gmail.com] On Behalf Of Florian
> Fainelli
> Sent: 2012年12月5日 18:48
> To: Yang, Wenyou
> Cc: linux-arm-ker...@lists.infradead.org; Ferre, Nicolas;
> plagn...@jcrosoft.com; Lin,
> JM; w...@iguana.be; linux-watch...@vger.kernel.org;
> linux-kernel@vger.kernel.org
> Subject: Re: [v2 PATCH 2/8] watchdog/at91sam9_wdt: Convert to use the watchdog
> framework
>
> Hello Wenyou,
>
> On Wednesday 05 December 2012 09:34:21 Wenyou Yang wrote:
> > According to the kernel document: convert_drivers_to_kernel_api.txt,
> > remove the file_operations struct, miscdevice, and obsolete includes
> >
> > Since the at91sam watchdog inherent characteristics, add the watchdog
> > operations: at91wdt_start, at91wdt_stop and at91wdt_ping.
> >
>
> [snip]
>
> >
> > +static inline bool watchdog_is_open(struct watchdog_device *wddev)
> > +{
> > + return test_bit(WDOG_DEV_OPEN, &wddev->status);
> > +}
>
> This helper should be moved to include/linux/watchdog.h as it can be useful
> for other watchdog drivers as well.
Thanks, I will move it in next version.
> --
> Florian
Best Regards
Wenyou Yang
N�����r��y����b�X��ǧv�^�){.n�+����{����zX����ܨ}���Ơz�&j:+v�������zZ+��+zf���h���~����i���z� �w���?�����&�)ߢf��^jǫy�m��@A�a���
0��h���i
