lib_otpinfo build regression

[lukehatpadl]

Recent commits in lib_otpinfo appear to break building on XS2 with an undefined reference to otp_read() from otp_read_word().

The version in b74ddabd implemented otp_read_word() by explicitly reading and writing from ports.ctrl.

As our CI pulls the latest version of develop, our build started breaking. Obviously I can fix by pinning to a commit but, wonder if this plans to be fixed?

[Berta777]

Recent commits in lib_otpinfo appear to break building on XS2 with an undefined reference to otp_read() from otp_read_word().

The version in b74ddabd implemented otp_read_word() by explicitly reading and writing from ports.ctrl.

As our CI pulls the latest version of develop, our build started breaking. Obviously I can fix by pinning to a commit but, wonder if this plans to be fixed?

Looks like the recent changes in lib_otpinfo are causing that issue with `otp_read()` on XS2. Pinning to an older commit is a quick fix, but I’d check the repo for any updates or open issues—it should be getting fixed soon. You could also ping the maintainers to see if they’re aware of it

[upav]

Hey lukehatpadl and Betra777,
In v2.2.0 we've added support for xs3a and xcommon_cmake build system which broke the building with an old xcommon build system. We've just released v2.2.1 which fixes the build for using xcommon.
However the xcommon will only support building for xs2a, so if you want to use lib_otpinfo for xs3a, you'll have to move to the xcommon_cmake.
The release has been tagged on github so you can pin your builds against v2.2.1, the website release should happen in a couple of days :)

[lukehatpadl]

Thank you, I’m stuck on XE232 so won’t be migrating to CMake in the near term.

[upav]

xcommon_cmake in lib_otpinfo does support building xs2a and xs3a, so you can switch to cmake if all the libraries you depend on support xcommon_cmake.
We've added xcommon_cmake to most of the core libraries we actively maintain, but we might have forgotten some..

[lukehatpadl]

I’ll take a look when I have time :) hopefully it fixes the dependency issues with xmake -j9, where there’s a race that occasionally causes it to fail.