Hi Lothar

On 06/20/2017 02:11 PM, Lothar Waßmann wrote:
> Hi,
> 
> On Tue, 20 Jun 2017 11:59:04 +0200 patrice.chot...@st.com wrote:
>> From: Patrice Chotard <patrice.chot...@st.com>
>>
>> Add clk_disable_all() method which Request/Disable/Free an
>> array of clocks that has been previously requested by
>> clk_request/get_by_*()
>>
>> Signed-off-by: Patrice Chotard <patrice.chot...@st.com>
>> Reviewed-by: Simon Glass <s...@chromium.org>
>> ---
>>
>> v7:  _ none
>> v6:  _ none
>> v5:  _ none
>> v4:  _ none
>> v3:  _ add commit message
>> v2:  _ create this independant path for printf() replacement
>>
>>   drivers/clk/clk-uclass.c | 22 ++++++++++++++++++++++
>>   include/clk.h            | 10 ++++++++++
>>   2 files changed, 32 insertions(+)
>>
>> diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c
>> index 83b6328..e732514 100644
>> --- a/drivers/clk/clk-uclass.c
>> +++ b/drivers/clk/clk-uclass.c
>> @@ -187,6 +187,28 @@ int clk_disable(struct clk *clk)
>>      return ops->disable(clk);
>>   }
>>   
>> +int clk_disable_all(struct clk *clk, int count)
>> +{
>> +    int i, ret;
>> +
>> +    debug("%s(clk=%p count=%d)\n", __func__, clk, count);
>> +
>> +    for (i = 0; i < count; i++) {
>> +            ret = clk_request(clk->dev, &clk[i]);
>>
> Shouldn't this be:
>               ret = clk_request(clk[i].dev, &clk[i]);

Yes, i will fix it

Thanks

Patrice
> 
> 
> 
> Lothar Waßmann
> 
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to