Thread (20 messages) flat view 20 messages, 4 authors, 11d ago

Re: [PATCH v5 4/6] drm/ssd130x: Replace positional ssd130x_spi_id[] initialization with C99

From: Javier Martinez Canillas <javierm@redhat.com>
Date: 2026-09-10 11:18:07
Also in: dri-devel, lkml

Amit Barzilai [off-list ref] writes:
ssd130x_spi_id[] is initialized with positional initializers, which
easily create easy-to-miss bugs when changing the members of the target
struct (struct spi_device_id in this example).

Change this to C99 initializers to guarantee each member is initialized
with the correct value.

Signed-off-by: Amit Barzilai <redacted>
Reviewed-by: Andy Shevchenko <redacted>
---
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help