lkml.org 
[lkml]   [2011]   [Jul]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/11] watchdog: WatchDog Timer Driver Core - Add basic framework
On 07/11/2011 11:32 PM, Wolfram Sang wrote:

>> +int watchdog_register_device(struct watchdog_device *wdd)
>> +{
>> + int ret;
>> +
>> + /* Make sure we have a valid watchdog_device structure */
>> + if (wdd == NULL || wdd->info == NULL || wdd->ops == NULL)
>> + return -EINVAL;
>> +
>> + /* Make sure that the mandatory operations are supported */
>> + if (wdd->ops->start == NULL || wdd->ops->stop == NULL)
>> + return -EINVAL;
>
> Check also for wdd->ops->owner? Should we mark it as mandatory?
>

If the driver is built-in owner will be NULL.



\
 
 \ /
  Last update: 2011-07-12 01:05    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site