On Wed, Jan 24, 2024 at 6:56 PM Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> wrote: > > Add some missing commas, refactor a couple small bits of code. ... > - BUILD_BUG_ON(ARRAY_SIZE(cs42l43_jack_override_modes) != > - ARRAY_SIZE(cs42l43_jack_text) - 1); > + static_assert(ARRAY_SIZE(cs42l43_jack_override_modes) == > + ARRAY_SIZE(cs42l43_jack_text) - 1); Now you can move this closer to the both array definitions (as static_assert() can be global, and basically that is the idea behind it, besides nicer error messaging). -- With Best Regards, Andy Shevchenko