Hi Uwe,

I've tested your patch on my board, all ok.

> -----Original Message-----
> From: Uwe Kleine-König <[email protected]>
> Sent: martedì 1 luglio 2025 11:23
> To: FLAVIO SULIGOI <[email protected]>; Lee Jones <[email protected]>;
> Daniel Thompson <[email protected]>; Jingoo Han <[email protected]>;
> Helge Deller <[email protected]>
> Cc: [email protected]; [email protected]; linux-
> [email protected]
> Subject: EXTERNAL: [PATCH 2/2] backlight: mp3309c: Initialize backlight
> properties without memset
> 
> [Use caution with links & attachments]
> 
> 
> 
> Assigning values to a struct using a compound literal (since C99) also
> guarantees that all unspecified struct members are empty-initialized, so
> it properly replaces the memset to zero.
> 
> The code looks a bit nicer and more idiomatic (though that might be
> subjective?). The resulting binary is a bit smaller. On ARCH=arm with an
> allnoconfig + minimal changes to enable the mp3309c driver the
> difference is 12 bytes.
> 
> Signed-off-by: Uwe Kleine-König <[email protected]>

Tested-by: Flavio Suligoi <[email protected]>

Reply via email to