On 2022-07-11 11:13, Liviu Dudau wrote:
[...]
But nothing worrying. It does work, though doesn't compile due to:
drivers/gpu/drm/arm/display/komeda/komeda_kms.c: In function
‘komeda_kms_atomic_commit_hw_done’:
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:77:9: error: ‘for’ loop
initial declarations are only allowed in C99 or C11 mode
77 | for (int i = 0; i < kms->n_crtcs; i++) {
| ^~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:77:9: note: use option
‘-std=c9
’, ‘-std=gnu99’, ‘-std=c11’ or ‘-std=gnu11’ to compile your code
but that was a trivial fixup.
Interesting that I'm not seeing that, probably due to using GCC12? Anyway, I'll fix
that and send a proper patch.
FWIW we do use -std=gnu11 since 5.18 (see e8c07082a810), but I'm not
entirely sure what the status quo is for using the new features in fixes
which might also warrant backporting to stable. I believe Carsten's
stuck on an older kernel thanks to constraints of the rest of that
project ;)
Cheers,
Robin.