samples: usb: sample.usb_device_next.midi: build failure on NXP series platforms

[hakehuang]

Describe the bug

• frdm_mcxa156
• frdm_mcxn236
• frdm_k64f
• lpcxpresso55sxx

4b04b74 is the first bad commit
commit 4b04b74
Author: Titouan Christophe titouan.christophe@mind.be
Date: Thu Jul 31 11:47:56 2025 +0200

audio: midi: improve API documentation

Improve the API documentation for Universal MIDI Packets definitions:

- Add link to the reference document, and make it referenceable in doxygen
- Use BIT_MASK macro instead of hexadecimal litterals to better convey the
  length of various fields within UMP packets
- Add more cross-references where possible
- Use @remark when applicable

Signed-off-by: Titouan Christophe <titouan.christophe@mind.be>

Regression

• This is a regression.

Steps to reproduce

west build -b frdm_mcxa156 samples/subsys/usb/midi

Relevant log output

/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/subsys/usb/device_next/class/usbd_midi2.c
In file included from /home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/include/zephyr/logging/log.h:11,
                 from /home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/include/zephyr/usb/usbd.h:24,
                 from /home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/subsys/usb/device_next/class/usbd_midi2.c:13:
/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/subsys/usb/device_next/class/usbd_midi2.c: In function 'usbd_midi_send':
/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/subsys/usb/device_next/class/usbd_midi2.c:472:17: error: format '%X' expects argument of type 'unsigned int', but argument 3 has type 'long unsigned int' [-Werror=format=]
  472 |         LOG_DBG("Send MT=%X group=%X", UMP_MT(ump), UMP_GROUP(ump));
      |                 ^~~~~~~~~~~~~~~~~~~~~
/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/include/zephyr/logging/log_core.h:315:50: note: in definition of macro 'Z_LOG2'
  315 |                         z_log_printf_arg_checker(__VA_ARGS__);                                     \
      |                                                  ^~~~~~~~~~~
/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/include/zephyr/logging/log.h:93:22: note: in expansion of macro 'Z_LOG'
   93 | #define LOG_DBG(...) Z_LOG(LOG_LEVEL_DBG, __VA_ARGS__)
      |                      ^~~~~
/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/subsys/usb/device_next/class/usbd_midi2.c:472:9: note: in expansion of macro 'LOG_DBG'
  472 |         LOG_DBG("Send MT=%X group=%X", UMP_MT(ump), UMP_GROUP(ump));
      |         ^~~~~~~
/home/jenkins/agent/workspace/bitbucket_build_platform/zephyr/subsys/usb/device_next/class/usbd_midi2.c:472:36: note: format string is defined here
  472 |         LOG_DBG("Send MT=%X group=%X", UMP_MT(ump), UMP_GROUP(ump));
      |                                   ~^
      |                                    |
      |                                    unsigned int
      |                                   %lX

Impact

Annoyance – Minor irritation; no significant impact on usability or functionality.

Environment

• OS: ubuntu
• toolchain: Zephyr
• version: v4.2.0-5335-g57df5abaa675

Additional Context

No response

[nordic-piks]

I also got error at this sample, beside those listed in description, there is:

zephyr/samples/subsys/usb/midi/src/main.c:49:16: error: initialization discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
   49 |         .dev = midi,
      |                ^~~~
In file included from ../../../../../../../../../zephyr/include/zephyr/logging/log.h:11,
                 from ../../../../../../../../../zephyr/include/zephyr/usb/usbd.h:24,
                 from ../../../../../../../../../zephyr/samples/subsys/usb/common/sample_usbd.h:11,
                 from ../../../../../../../../../zephyr/samples/subsys/usb/midi/src/main.c:10:

[nordic-piks]

Commit introducing my issues is b1d6252

[dleach02]

The reason for the error is because UMP_GROUP was changed to use BIT_MASK() macro. That macro is defined with a 1UL which changes the size of the value returned to be a unsigned long int.

#define BIT_MASK(n) (BIT(n) - 1UL)

The fix is to be specific in the size specifiers in the format print strings.