[native_sim] Zephyr 4.1 breaks GPIO_EMUL_SDL and INPUT_SDL_TOUCH

[PetervdPerk-NXP]

Describe the bug

I've got this application that runs natively on the NXP FRDM-MCXN947 but also has a native/posix/64 target.
https://github.com/NXPHoverGames/Doom-MCX
When upgrading from Zephyr 4.0.0 to Zephyr 4.1.0 the input (GPIO SDL emulation and SDL touch) for native/posix/64 is not working anymore.

To Reproduce

# initialize  for the example-application (main branch)
west init -m https://github.com/nxphovergames/doom-mcx --mr main doom-mcx-workspace
# update Zephyr modules
cd doom-mcx-workspace
west update
cd doom-mcx
west build -p always -b native_sim/native/64
./build/zephyr/zephyr.exe

When modifying west.yml to point to Zephyr v4.0.0 8469084dfae85f854555f0607f2c838dad097235
Then input is working again

Impact
No SDL input on native_sim

Environment (please complete the following information):

• OS: (Ubuntu 22.04 under wsl2)
• Toolchain (Zephyr sdk 0.17.0)

[faxe1008]

Does it work with CONFIG_INPUT=y?

[PetervdPerk-NXP]

CONFIG_INPUT=y already, see

Again the same code is working fine on native hardware.

[aescolar]

CC @fabiobaltieri @pdgendt

[fabiobaltieri]

Can you bisect this?

[PetervdPerk-NXP]

Can you bisect this?

I think I've narrowed it down this PR #81184
The commit 86a126d before is working fine.

EDIT: Reverting 02d562e on top Zephyr 4.1 gives me input again.

[JarmouniA]

FYI @Finomnis

[Finomnis]

Huh? I didn't change anything regarding the input, at least not that I know of...

Very weird.

I'm sadly very limited on time right now, would someone else mind dissecting that change in more detail to see which line exactly broke it?

[faxe1008]

@PetervdPerk-NXP your code does not build for me:

/tmp/doom-mcx-workspace/doom-mcx/prboom2/src/hu_stuff.c: In function ‘HU_Start’:
/tmp/doom-mcx-workspace/doom-mcx/prboom2/src/hu_stuff.c:372:19: error: passing argument 5 of ‘HUlib_initSText’ from incompatible pointer type [-Wincompatible-pointer-types]
  372 |                 _g->hu_font,
      |                 ~~^~~~~~~~~
      |                   |
      |                   const patch_t **
In file included from /tmp/doom-mcx-workspace/doom-mcx/prboom2/src/hu_stuff.c:39:
/tmp/doom-mcx-workspace/doom-mcx/prboom2/src/hu_lib.h:175:18: note: expected ‘const patch_t *’ but argument is of type ‘const patch_t **’
  175 |   const patch_t *font,
      |   ~~~~~~~~~~~~~~~^~~~
/tmp/doom-mcx-workspace/doom-mcx/prboom2/src/hu_stuff.c:384:19: error: passing argument 4 of ‘HUlib_initTextLine’ from incompatible pointer type [-Wincompatible-pointer-types]
  384 |                 _g->hu_font,
      |                 ~~^~~~~~~~~
      |                   |
      |                   const patch_t **
/tmp/doom-mcx-workspace/doom-mcx/prboom2/src/hu_lib.h:152:18: note: expected ‘const patch_t *’ but argument is of type ‘const patch_t **’
  152 |   const patch_t *f,
      |   ~~~~~~~~~~~~~~~^

Is this somehow reproducible with an intree-sample?

EDIT:// Hmm looking at the resulting autoconf.h tho it seems like CONFIG_LV_Z_POINTER_INPUT is not enabled for me, which might be the culprit. Can you check this on your end with a working build @PetervdPerk-NXP ?

EDIT-2:// @PetervdPerk-NXP try adding CONFIG_LVGL=y this is missing from your prj.conf: https://github.com/NXPHoverGames/Doom-MCX/blob/main/prj.conf
If that fixes your issue, I fail to understand what the commit you linked has to do with it.