In D53413#1219714, @cperciva wrote:

Does this work with cross-builds? I suspect that WSTAGEDIR is getting set to "" and then you're setting PKG_ABI based on /usr/bin/uname in the host system.

In D52305#1219629, @jhb wrote:

Sorry I missed this earlier. Please merge into 15.0 if possible.

On Sat Oct 11, 2025 at 2:26 AM +0300, Steve Kargl wrote:

I do not have a phabricator account. That method to
review patches came after I stopped committing.

I did look at your patch, and do believe that it
is the correct fix. If you compare to remainderf(),
you'll see that the float version considers the
entire significand. I also checked remainderl().
It calls remquol() and it's implementation is
much different than the float and double versions.

Will do. I tried to add him to the review before uploading the patch but couldn't find an account.

Allocate fifo before locking priv->mtx

malloc priv->events with the correct size

I don't have the hardware to test this, but cross referencing the documentation, it looks good to me.

Bump
Userspace GPIO interrupts are currently broken because of this. (after D51932)
I believe that we do need to implement the resource allocation functions too (for cross-tree consumers), just not on the GPIO controller.

No problem :)

LGTM, just one nit.

No worries.

I got toolchain errors when running the script for amd64, but I verified that it does actually build with MK_LOADER_BIOS_TEXTONLY=no on amd64.

Bumping
This seems important, should make it to 15.0