<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 21, 2023 at 9:31 AM Karel Gardas <karel@functional.vision> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
   Folks,<br>
<br>
I'd like to propose and perform STM32 H7 HAL code update. Currently this <br>
HAL code is part of RTEMS source tree located inside bsps/arm/stm32h7 <br>
subdirectory.<br>
<br>
As the patches are too big to be sent to the mailing list, I'm providing <br>
them here for review:<br>
<br>
<a href="https://github.com/karelfv/rtems/commits/stm32h7-hal-update-2023-07" rel="noreferrer" target="_blank">https://github.com/karelfv/rtems/commits/stm32h7-hal-update-2023-07</a><br>
<br>
Few notes:<br>
<br>
- part of HAL files got relicensed by STMicro from BSD-3 to Apache-2.0. <br>
Fortunately the project does not contain NOTICE file so we do not need <br>
to organize anything about it.<br>
<a href="https://github.com/karelfv/rtems/commit/ec59116a2e8c1b637cc59d2ef619880ce507816e" rel="noreferrer" target="_blank">https://github.com/karelfv/rtems/commit/ec59116a2e8c1b637cc59d2ef619880ce507816e</a><br>
<br>
- I've tried to apply all Sebastian's and Christian's patches done to <br>
original HAL. All is done except doxygenization patch which I needed to <br>
do completely new due to a lot of hunks aborting and also a lot of new <br>
occurrences where it should be applied.<br>
<a href="https://github.com/karelfv/rtems/commit/04936e21b0daeaa43082c73919fbf3274e9482ca" rel="noreferrer" target="_blank">https://github.com/karelfv/rtems/commit/04936e21b0daeaa43082c73919fbf3274e9482ca</a><br>
<br>
- new HAL code breaks compilation of libbsd (for this particular <br>
platform! Not in general). This is due to inclusion of math.h. I've <br>
solved that by crude hack.<br>
<a href="https://github.com/karelfv/rtems/commit/a1035ba3cee4dc6967bb978aa04ef763c9d2fcfd" rel="noreferrer" target="_blank">https://github.com/karelfv/rtems/commit/a1035ba3cee4dc6967bb978aa04ef763c9d2fcfd</a><br>
<br>
<br>
Otherwise I hope the patch set is tolerable for RTEMS-6 as for RTEM-7 we <br>
plan to remove that code completely to a separate module anyway...<br></blockquote><div><br></div><div><div>I just tested this on the board I'm working on and it seems fine here after a trivial rebase to current master. I'm not very far along, so that just means that ticker works as expected.</div><div><br></div><div>Kinsey</div> </div></div></div>