11:57 <zapb_> Wasn't there a question on the ML regarding Artery chip support in OpenOCD? I cannot find it anymore..

12:09 <PaulFertser> zapb_: "#439 OpenOCD failed with code (-1073741819)" ?

12:10 <PaulFertser> Also, "[OpenOCD-user] Gotek with Artery AT32 programming"

12:11 <zapb_> PaulFertser, thanks! That one with -107 in the subject ;)

12:12 <zapb_> Easy to find with this subject! ;)

12:56 <zapb_> borneoa___, Why do we have different SPDX style for *.h and *.c files?

13:03 <borneoa___> zapb_: inherited from Linux. The files .h can be included by the assembler files, where // is not a valid comment

13:05 <borneoa___> zapb_: the SPDX tag should be in a single line comment, but the .h cannot follow this

13:15 <zapb_> borneoa___, I see, makes sense. Thanks!

13:17 <zapb_> borneoa___, may I ask you to help me with this error: https://build.openocd.org/job/openocd-gerrit-build/19573/TARGET=distcheck/consoleFull ?

13:18 <zapb_> It compiles on all systems but distcheck fails. If I run distcheck on my machine then it already fails with #include errors of other files

13:22 <borneoa___> zapb_: in the makefile you need to add the new file .h in the list with all the other .h

13:22 <zapb_> borneoa___, AH!

13:22 <zapb_> Damn... :D

13:24 <zapb_> Well, still fails for other files an my system. For example jtag/drivers/angie.c

13:24 <borneoa___> zapb_: but what the reason for having that new .h? It is only referenced in the associated .c, so the content can be inlined inside the .c!

13:25 <zapb_> borneoa___, it's used later for the flash loader

13:25 <borneoa___> zapb_: I see one patch in Gerrit for Angie build, but no time to check it yet

13:26 <zapb_> borneoa___, not happy with how flash drivers work at the moment but changing it needs lot of rework

13:28 <borneoa___> zapb_: you mean there would be a following commit that uses it?

13:30 <zapb_> borneoa___, I already have a patch for a flash loader that uses the #include file, but it needs more testing and I wanted to push a version without loader

13:30 <zapb_> borneoa___, I think moving everything into the *.c file and then later back to *.h is waste of time

13:30 <borneoa___> zapb_: ok!

13:36 <PaulFertser> zapb_: that looks like some non-trivial driver covering the big set of devices, looks very nice! Was it really developed a year ago? You put Copyright (C) 2023 there.

13:39 <zapb_> PaulFertser, thanks. yes, I started it in 2023 and then slowly developed it further. One reason to develop it was: https://tches.iacr.org/index.php/TCHES/article/view/11422

13:40 <zapb_> PaulFertser, of course making the driver ready for contributing was not necessary for the research and costs most of the time ;)

14:06 <PaulFertser> zapb_: it's really great to see all that research you're participating in being public and visible.

14:10 <zapb_> PaulFertser, thanks

15:28 <PaulFertser> borneoa___: do you know if someone still cares about github CI? I'm looking at https://github.com/openocd-org/openocd/actions/runs/12444414369/job/34744662313 and probably "skipIfReleaseExists: false" needs to be changed so they're not marked as failure?

20:03 <borneoa___> PaulFertser: it was started by Tarek, but I think he is currently on other projects. Maybe we should have a look at it...