<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 3, 2021, 11:10 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 3/9/21 11:33 pm, Sebastian Huber wrote:<br>
> diff --git a/bsps/arm/altera-cyclone-v/rtc/rtc.c b/bsps/arm/altera-cyclone-v/rtc/rtc.c<br>
> index 779a093459..d1c12ee874 100644<br>
> --- a/bsps/arm/altera-cyclone-v/rtc/rtc.c<br>
> +++ b/bsps/arm/altera-cyclone-v/rtc/rtc.c<br>
> @@ -353,7 +353,7 @@ static int altera_cyclone_v_ds1339_get_time(int minor, rtems_time_of_day* tod)<br>
>      temp_tod.month  = ds1339_get_month(&time);<br>
>      temp_tod.year   = ds1339_get_year(&time);<br>
>  <br>
> -    sc = _TOD_Validate(&temp_tod, TOD_ENABLE_TICKS_VALIDATION);<br>
> +    sc = _TOD_Validate(&temp_tod);<br>
<br>
This has leaked out of the internal implementation interface. Should it?<br>
<br>
I prefer this does not happen and BSPs use an API.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I noticed this also. It needs an API.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Chris<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>