Re: [PATCH] rtc: ab8500: replace sprintf() with sysfs_emit()

From: Linus Walleij

Date: Tue Apr 21 2026 - 07:24:21 EST


On Mon, Apr 20, 2026 at 12:36 AM Maxwell Doose <m32285159@xxxxxxxxx> wrote:

> This patch replaces sprintf() with sysfs_emit() to ensure proper
> bounds checking. It also simplifies the return logic by directly
> returning the error after logging, instead of logging, calling
> sprintf(), then returning.
>
> Signed-off-by: Maxwell Doose <m32285159@xxxxxxxxx>

Reviewed-by: Linus Walleij <linusw@xxxxxxxxxx>

Yours,
Linus Walleij