lkml.org 
[lkml]   [2017]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 2/2] rtc: stm32: add STM32H7 RTC support
From
Date
Hi,

On 07/05/2017 11:37 PM, Alexandre Belloni wrote:
> Hi,
>
> This is mostly fine, even if I'm not too happy with the
> s/ck_rtc/rtc_ck/.
>
This clock name has changed in datasheet between STM32F4/STM32F7 and
STM32H7, so, I'm not really happy too... But, as there is only one clock
on F4/F7, its name is not important for the end user. Now that pclk
appears, we need to distinguish the two clocks, and for the end user, it
is better if the clock names match the datasheet.

> On 26/06/2017 at 11:51:29 +0200, Amelie Delaunay wrote:
>> - rtc->ck_rtc = devm_clk_get(&pdev->dev, NULL);
>> - if (IS_ERR(rtc->ck_rtc)) {
>> - dev_err(&pdev->dev, "no ck_rtc clock");
>> - return PTR_ERR(rtc->ck_rtc);
>> + match = of_match_device(stm32_rtc_of_match, &pdev->dev);
>> + if (match && match->data)
>> + rtc->data = (struct stm32_rtc_data *)match->data;
>> + else
>> + return -EINVAL;
>
> This will never happen, you can remove that test.
>
>
OK, will be done in V2.

Thanks for the review,
Amelie

\
 
 \ /
  Last update: 2017-07-06 09:34    [W:0.582 / U:0.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site