Re: [PATCH v2 03/10] serdev: Allow passing the serdev device name to serdev_device_add()

Next message: Mark Rutland: "Re: [PATCH 1/1] arm64: Print slab alloc and free paths for addresses in registers"
Previous message: Macpaul Lin: "[PATCH] clk: mediatek: set CLK_IGNORE_UNUSED to clock mt8188 adsp audio26m"
In reply to: Manivannan Sadhasivam via B4 Relay: "[PATCH v2 03/10] serdev: Allow passing the serdev device name to serdev_device_add()"
Next in thread: Manivannan Sadhasivam via B4 Relay: "[PATCH v2 04/10] serdev: Add an API to find the serdev controller associated with the devicetree node"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

From: Bartosz Golaszewski

Date: Thu Nov 27 2025 - 04:14:48 EST

On Tue, Nov 25, 2025 at 3:45 PM Manivannan Sadhasivam via B4 Relay
<devnull+manivannan.sadhasivam.oss.qualcomm.com@xxxxxxxxxx> wrote:
>
> From: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxxxxxxxx>
>
> Instead of always setting the serdev device name from 'struct device' name,
> allow the callers to pass an optional name and set it as the serdev device
> name.
>
> This will be used by the future callers passing the serdev device ID as the
> name.
>
> Acked-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxxxxxxxx>
> ---

Instead of modifying the existing callers with an unneeded argument,
I'd suggest adding a new call - serdev_device_add_full() or something
like that - that takes more arguments and make the existing function
wrap it.

Bart