[PATCH v4 2/5] soc: samsung: exynos-chipid: rename method
Tudor Ambarus
tudor.ambarus at linaro.org
Tue Dec 23 02:14:49 PST 2025
On 12/23/25 11:55 AM, André Draszik wrote:
> On Mon, 2025-12-22 at 16:30 +0000, Tudor Ambarus wrote:
>> s/product_id_to_soc_id/exynos_product_id_to_name.
>> Prepend exynos_ to avoid name space pollution. The method translates the
>> product id to a name, rename the method to make that clear. While
>> touching the code where it is called, add a blank line for readability
>> purposes.
>>
>> Signed-off-by: Tudor Ambarus <tudor.ambarus at linaro.org>
>> ---
>> drivers/soc/samsung/exynos-chipid.c | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> I'm not sure this change helps with anything, in particular as the
> return value is used to assign to soc_id, but in case it gets applied:
what drove me to do the rename, and prepend exynos_ to the method name,
was that IP specific drivers should not use generic names for methods ->
those shall be left for the core.
Now about the s/soc_id/name, you're right, I could kept soc_id to align
with the soc interface. Happy to resend for this if you think it is worth
it, or drop altogether.
>
> Reviewed-by: André Draszik <andre.draszik at linaro.org>
Cheers,
ta
More information about the linux-arm-kernel
mailing list