[U-Boot] [PATCH v6 01/11] reset: add reset_request()

Marek Vasut marex at denx.de
Tue Jun 13 08:45:50 UTC 2017


On 06/12/2017 09:24 AM, Patrice CHOTARD wrote:
> Hi Tom
> 
> I think this series is mature enough to be merged.

I think there's still some discussion with Simon going on ?
Also, please don't top-post.

> Thanks
> 
> Patrice
> 
> 
> On 06/05/2017 11:42 AM, patrice.chotard at st.com wrote:
>> From: Patrice Chotard <patrice.chotard at st.com>
>>
>> This is needed in error path to assert previously deasserted
>> reset by using a saved reset_ctl reference.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard at st.com>
>> Reviewed-by: Simon Glass <sjg at chromium.org>
>> ---
>> v6:	_ none
>> v5:	_ none
>> v4:	_ none
>> v3:	_ none
>> v2:	_ none
>>
>>   drivers/reset/reset-uclass.c | 9 +++++++++
>>   include/reset.h              | 9 +++++++++
>>   2 files changed, 18 insertions(+)
>>
>> diff --git a/drivers/reset/reset-uclass.c b/drivers/reset/reset-uclass.c
>> index de3695f..4fd82b9 100644
>> --- a/drivers/reset/reset-uclass.c
>> +++ b/drivers/reset/reset-uclass.c
>> @@ -97,6 +97,15 @@ int reset_get_by_name(struct udevice *dev, const char *name,
>>   	return reset_get_by_index(dev, index, reset_ctl);
>>   }
>>   
>> +int reset_request(struct reset_ctl *reset_ctl)
>> +{
>> +	struct reset_ops *ops = reset_dev_ops(reset_ctl->dev);
>> +
>> +	debug("%s(reset_ctl=%p)\n", __func__, reset_ctl);
>> +
>> +	return ops->request(reset_ctl);
>> +}
>> +
>>   int reset_free(struct reset_ctl *reset_ctl)
>>   {
>>   	struct reset_ops *ops = reset_dev_ops(reset_ctl->dev);
>> diff --git a/include/reset.h b/include/reset.h
>> index f45fcf8..4f2e35f 100644
>> --- a/include/reset.h
>> +++ b/include/reset.h
>> @@ -100,6 +100,15 @@ int reset_get_by_name(struct udevice *dev, const char *name,
>>   		      struct reset_ctl *reset_ctl);
>>   
>>   /**
>> + * reset_request - Request a reset signal.
>> + *
>> + * @reset_ctl:	A reset control struct.
>> + *
>> + * @return 0 if OK, or a negative error code.
>> + */
>> +int reset_request(struct reset_ctl *reset_ctl);
>> +
>> +/**
>>    * reset_free - Free a previously requested reset signal.
>>    *
>>    * @reset_ctl:	A reset control struct that was previously successfully


-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list