topic Re: USB SDK STRUCT_UNPACKED in LPC Microcontrollers

USB SDK STRUCT_UNPACKED

rex_lam — Mon, 10 Jun 2019 22:46:29 GMT

In usb_misc.h, STRUCT_UNPACKED is defined as follows.

#if defined(__ICCARM__)

#ifndef STRUCT_PACKED
#define STRUCT_PACKED __packed
#endif

#ifndef STRUCT_UNPACKED
#define STRUCT_UNPACKED
#endif

#elif defined(__GNUC__)

#ifndef STRUCT_PACKED
#define STRUCT_PACKED
#endif

#ifndef STRUCT_UNPACKED
#define STRUCT_UNPACKED __attribute__((__packed__))
#endif

#elif defined(__CC_ARM) || (defined(__ARMCC_VERSION))

#ifndef STRUCT_PACKED
#define STRUCT_PACKED _Pragma("pack(1U)")
#endif

#ifndef STRUCT_UNPACKED
#define STRUCT_UNPACKED _Pragma("pack()")
#endif

#endif

Is the definition for __GNUC__ wrong? I expected STRUCT_PACKED to be __attribute__((__packed__)) instead of the other way around, i.e.

#ifndef STRUCT_PACKED
#define STRUCT_PACKED __attribute__((__packed__))
#endif

#ifndef STRUCT_UNPACKED
#define STRUCT_UNPACKED
#endif

Re: USB SDK STRUCT_UNPACKED

soledad — Mon, 24 Jun 2019 14:39:19 GMT

Hi,

We don't have any issue regarding this, however, we will check it internally. Thank you for your feedback,

Regards

Soledad

Re: USB SDK STRUCT_UNPACKED

HSL_AB — Sun, 15 Jan 2023 13:01:56 GMT

Years pass - and SDK_2_11_1_MIMXRT1052xxxxB pack still contains this old described error in the "usb_misc.h" file. What is your reason?

Re: USB SDK STRUCT_UNPACKED

frank_m — Tue, 17 Jan 2023 08:56:05 GMT

Perhaps this only potentially becomes a problem when using different platforms or different toolchains in a project/system.
And affected users don't lament loud enough ...

Re: USB SDK STRUCT_UNPACKED

HSL_AB — Tue, 17 Jan 2023 10:59:57 GMT

While STRUCT_UNPACKED is not used in SDK there is no problem: let assume that nobody will use this #define deliberately

Re: USB SDK STRUCT_UNPACKED

frank_m — Tue, 17 Jan 2023 12:11:41 GMT

That depends.

In my experience, "packed" structs are often used as overlays for serial data transfer (ethernet, Modbus, USB, CAN) and in data file interpretation.
In most cases, metadata are embedded in the byte stream, and interpreted via the "packed" overlay structure.
However, such projects usually included a manual check that the linker-generated struct offsets really matched the intended sizes/offsets.

Re: USB SDK STRUCT_UNPACKED

HSL_AB — Tue, 17 Jan 2023 12:26:52 GMT

Obviously, "packed" does be widely used, but I assume that little people know about the existence of this particular #define. I hadn't known before.