lkml.org 
[lkml]   [2014]   [Oct]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] reset: add reset_control_status helper function
Hi Philipp,

On 10/9/14, 3:44 AM, Philipp Zabel wrote:
> Hi Dinh,
>
> thank you for the patch. Just two small issues below:
>
> Am Mittwoch, den 08.10.2014, 17:45 -0500 schrieb dinguyen@opensource.altera.com:
> [...]
>> @@ -126,6 +126,19 @@ int reset_control_deassert(struct reset_control *rstc)
>> EXPORT_SYMBOL_GPL(reset_control_deassert);
>>
>> /**
>> + * reset_control_status - returns a status of a reset bit
>> + * @rstc: reset controller
>> + */
>> +unsigned int reset_control_status(struct reset_control *rstc)
>> +{
>> + if (rstc->rcdev->ops->status)
>> + return rstc->rcdev->ops->status(rstc->rcdev, rstc->id);
>> +
>> + return -ENOSYS;
>> +}
>> +EXPORT_SYMBOL_GPL(reset_control_status);
>
> Since this function can return negative error numbers, please make the
> return value of reset_control_status int. That also means that drivers
> must not set the MSB when returning status.

Ah yes...Don't why I changed it to unsigned int for the return.

>
>> +
>> +/**
>> * of_reset_control_get - Lookup and obtain a reference to a reset controller.
>> * @node: device to be reset by the controller
>> * @id: reset line name
>> diff --git a/include/linux/reset-controller.h b/include/linux/reset-controller.h
>> index 41a4695..8e659d5 100644
>> --- a/include/linux/reset-controller.h
>> +++ b/include/linux/reset-controller.h
>> @@ -17,6 +17,7 @@ struct reset_control_ops {
>> int (*reset)(struct reset_controller_dev *rcdev, unsigned long id);
>> int (*assert)(struct reset_controller_dev *rcdev, unsigned long id);
>> int (*deassert)(struct reset_controller_dev *rcdev, unsigned long id);
>> + unsigned int (*status)(struct reset_controller_dev *rcdev, unsigned long id);
>> };
>
> Please change the return value of the status callback to int and
> describe it in the kerneldoc comment above. It should probably be
> mentioned that the returned status value must be >= 0 except in error
> cases.
>
Will do for V2...

Dinh


\
 
 \ /
  Last update: 2014-10-09 16:01    [W:0.670 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site