> -----Original Message-----
> From: Bill Wendling <[email protected]>
> Sent: Tuesday, September 15, 2026 10:19 AM
> To: [email protected]
> Cc: Bill Wendling <[email protected]>; codemender-
> [email protected]; Russell King <[email protected]>;
> Huacai Chen <[email protected]>; WANG Xuerui <[email protected]>;
> Thomas Bogendoerfer <[email protected]>; James E.J. Bottomley
> <[email protected]>; Helge Deller <[email protected]>;
> Thomas Gleixner <[email protected]>; Ingo Molnar <[email protected]>;
> Borislav Petkov <[email protected]>; Dave Hansen
> <[email protected]>; [email protected]; H. Peter Anvin
> <[email protected]>; Ian Abbott <[email protected]>; H Hartley Sweeten
> <[email protected]>; Luck, Tony <[email protected]>;
> Maarten Lankhorst <[email protected]>; Maxime Ripard
> <[email protected]>; Thomas Zimmermann <[email protected]>; David
> Airlie <[email protected]>; Simona Vetter <[email protected]>; Brost,
> Matthew <[email protected]>; Thomas Hellström
> <[email protected]>; Vivi, Rodrigo
> <[email protected]>; Dmitry Torokhov <[email protected]>;
> Matthias Schwarzott <[email protected]>; Mauro Carvalho Chehab
> <[email protected]>; Nguyen, Anthony L <[email protected]>;
> Kitszel, Przemyslaw <[email protected]>; Andrew Lunn
> <[email protected]>; David S. Miller <[email protected]>; Eric
> Dumazet <[email protected]>; Jakub Kicinski <[email protected]>; Paolo
> Abeni <[email protected]>; Arend van Spriel
> <[email protected]>; Rob Herring <[email protected]>;
> Saravana Kannan <[email protected]>; Krzysztof Kozlowski
> <[email protected]>; Sylwester Nawrocki <[email protected]>; Peter
> Griffin <[email protected]>; Alim Akhtar
> <[email protected]>; Linus Walleij <[email protected]>; Anil
> Gurumurthy <[email protected]>; Sudarsana Kalluru
> <[email protected]>; Martin K. Petersen <[email protected]>;
> Trond Myklebust <[email protected]>; Anna Schumaker
> <[email protected]>; Mike Marshall <[email protected]>; Martin
> Brandenburg <[email protected]>; Kees Cook <[email protected]>; Jiri
> Pirko <[email protected]>; Simon Horman <[email protected]>; Chuck Lever
> <[email protected]>; Jeff Layton <[email protected]>; NeilBrown
> <[email protected]>; Olga Kornievskaia <[email protected]>; Dai Ngo
> <[email protected]>; Tom Talpey <[email protected]>; Jaroslav Kysela
> <[email protected]>; Takashi Iwai <[email protected]>; Andrew Morton
> <[email protected]>; Mike Rapoport (Microsoft)
> <[email protected]>; Kanglong Wang <[email protected]>; Tiezhu
> Yang <[email protected]>; Qiang Ma <[email protected]>;
> Randy Dunlap <[email protected]>; Pengpeng Hou
> <[email protected]>; Ard Biesheuvel <[email protected]>; Breno Leitao
> <[email protected]>; Thorsten Blum <[email protected]>; Harshit
> Mogalapalli <[email protected]>; Greg Kroah-Hartman
> <[email protected]>; Lyude Paul <[email protected]>; Ashutosh
> Desai <[email protected]>; Deak, Imre <[email protected]>;
> Dmitry Baryshkov <[email protected]>; Johan Hovold
> <[email protected]>; Berg, Johannes <[email protected]>;
> Korenblit, Miriam Rachel <[email protected]>;
> Alexander Stein <[email protected]>; Cryolitia PukNgae
> <[email protected]>; Jiaming Zhang <[email protected]>; Will
> Porter <[email protected]>; Cen Zhang <[email protected]>;
> Cássio Gabriel <[email protected]>; Rong Zhang
> <[email protected]>; Arun Raghavan <[email protected]>; linux-arm-
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; intel-
> [email protected]; [email protected]; linux-
> [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]
> Subject: [PATCH 22/28] media: si2165: Use snprintf() to format
> frontend name
> 
> si2165_probe() constructs the frontend name by appending the chip name
> and supported delivery system suffixes (" DVB-T", " DVB-C") to
> state->fe.ops.info.name using multiple strlcat() calls.
> 
> In preparation for removing the deprecated strlcat() API[1], format
> the complete frontend name in a single snprintf() call.
> 
> Link: https://github.com/KSPP/linux/issues/370 [1]
> Cc: [email protected]
> Assisted-by: Developer:Gemini [editor_update_file,
> make_kernel_build_single_file, shell_command_run]
> Signed-off-by: Bill Wendling <[email protected]>
> ---
> Cc: Russell King <[email protected]>
> Cc: Huacai Chen <[email protected]>
> Cc: WANG Xuerui <[email protected]>
> Cc: Thomas Bogendoerfer <[email protected]>
> Cc: "James E.J. Bottomley" <[email protected]>
> Cc: Helge Deller <[email protected]>
> Cc: Thomas Gleixner <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: Borislav Petkov <[email protected]>
> Cc: Dave Hansen <[email protected]>
> Cc: [email protected]
> Cc: "H. Peter Anvin" <[email protected]>
> Cc: Ian Abbott <[email protected]>
> Cc: H Hartley Sweeten <[email protected]>
> Cc: Tony Luck <[email protected]>
> Cc: Maarten Lankhorst <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Thomas Zimmermann <[email protected]>
> Cc: David Airlie <[email protected]>
> Cc: Simona Vetter <[email protected]>
> Cc: Matthew Brost <[email protected]>
> Cc: "Thomas Hellström" <[email protected]>
> Cc: Rodrigo Vivi <[email protected]>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Matthias Schwarzott <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Tony Nguyen <[email protected]>
> Cc: Przemek Kitszel <[email protected]>
> Cc: Andrew Lunn <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Eric Dumazet <[email protected]>
> Cc: Jakub Kicinski <[email protected]>
> Cc: Paolo Abeni <[email protected]>
> Cc: Arend van Spriel <[email protected]>
> Cc: Rob Herring <[email protected]>
> Cc: Saravana Kannan <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Sylwester Nawrocki <[email protected]>
> Cc: Peter Griffin <[email protected]>
> Cc: Alim Akhtar <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Anil Gurumurthy <[email protected]>
> Cc: Sudarsana Kalluru <[email protected]>
> Cc: "Martin K. Petersen" <[email protected]>
> Cc: Trond Myklebust <[email protected]>
> Cc: Anna Schumaker <[email protected]>
> Cc: Mike Marshall <[email protected]>
> Cc: Martin Brandenburg <[email protected]>
> Cc: Kees Cook <[email protected]>
> Cc: Jiri Pirko <[email protected]>
> Cc: Simon Horman <[email protected]>
> Cc: Chuck Lever <[email protected]>
> Cc: Jeff Layton <[email protected]>
> Cc: NeilBrown <[email protected]>
> Cc: Olga Kornievskaia <[email protected]>
> Cc: Dai Ngo <[email protected]>
> Cc: Tom Talpey <[email protected]>
> Cc: Jaroslav Kysela <[email protected]>
> Cc: Takashi Iwai <[email protected]>
> Cc: Bill Wendling <[email protected]>
> Cc: Andrew Morton <[email protected]>
> Cc: "Mike Rapoport (Microsoft)" <[email protected]>
> Cc: Kanglong Wang <[email protected]>
> Cc: Tiezhu Yang <[email protected]>
> Cc: Qiang Ma <[email protected]>
> Cc: Randy Dunlap <[email protected]>
> Cc: Pengpeng Hou <[email protected]>
> Cc: Ard Biesheuvel <[email protected]>
> Cc: Breno Leitao <[email protected]>
> Cc: Thorsten Blum <[email protected]>
> Cc: Harshit Mogalapalli <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Lyude Paul <[email protected]>
> Cc: Ashutosh Desai <[email protected]>
> Cc: Imre Deak <[email protected]>
> Cc: Dmitry Baryshkov <[email protected]>
> Cc: Johan Hovold <[email protected]>
> Cc: Johannes Berg <[email protected]>
> Cc: Miri Korenblit <[email protected]>
> Cc: Alexander Stein <[email protected]>
> Cc: Cryolitia PukNgae <[email protected]>
> Cc: Jiaming Zhang <[email protected]>
> Cc: Will Porter <[email protected]>
> Cc: Cen Zhang <[email protected]>
> Cc: "Cássio Gabriel" <[email protected]>
> Cc: Rong Zhang <[email protected]>
> Cc: Arun Raghavan <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> ---
>  drivers/media/dvb-frontends/si2165.c | 17 +++++++----------
>  1 file changed, 7 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/media/dvb-frontends/si2165.c b/drivers/media/dvb-
> frontends/si2165.c
> index f1241b63aa5c..bd15eb58fc84 100644
> --- a/drivers/media/dvb-frontends/si2165.c
> +++ b/drivers/media/dvb-frontends/si2165.c
> @@ -1243,20 +1243,17 @@ static int si2165_probe(struct i2c_client
> *client)
>               chip_name, rev_char, state->chip_type,
>               state->chip_revcode);
> 
> -     strlcat(state->fe.ops.info.name, chip_name,
> -             sizeof(state->fe.ops.info.name));
> +     snprintf(state->fe.ops.info.name, sizeof(state-
> >fe.ops.info.name),
> +              "Silicon Labs %s%s%s",
> +              chip_name,
> +              state->has_dvbt ? " DVB-T" : "",
> +              state->has_dvbc ? " DVB-C" : "");
> 
>       n = 0;
> -     if (state->has_dvbt) {
> +     if (state->has_dvbt)
>               state->fe.ops.delsys[n++] = SYS_DVBT;
> -             strlcat(state->fe.ops.info.name, " DVB-T",
> -                     sizeof(state->fe.ops.info.name));
> -     }
> -     if (state->has_dvbc) {
> +     if (state->has_dvbc)
>               state->fe.ops.delsys[n++] = SYS_DVBC_ANNEX_A;
> -             strlcat(state->fe.ops.info.name, " DVB-C",
> -                     sizeof(state->fe.ops.info.name));
> -     }
> 
>       /* return fe pointer */
>       *pdata->fe = &state->fe;
> --
> 2.55.0.1032.g73a4cd73de-goog

Reviewed-by: Aleksandr Loktionov <[email protected]>

Reply via email to