linux-kernel - Re: [PATCH 3/3] tty: serial: samsung: Remove unused artpec-8 specific code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <abfaef6f-dd40-441d-86ec-7cd37c1e06b5@kernel.org>
Date: Thu, 11 Sep 2025 19:22:11 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Ravi Patel <ravi.patel@...sung.com>,
 'Geert Uytterhoeven' <geert@...ux-m68k.org>
Cc: gregkh@...uxfoundation.org, jirislaby@...nel.org, robh@...nel.org,
 krzk+dt@...nel.org, conor+dt@...nel.org, jesper.nilsson@...s.com,
 lars.persson@...s.com, alim.akhtar@...sung.com, arnd@...nel.org,
 andriy.shevchenko@...ux.intel.com, geert+renesas@...der.be,
 thierry.bultel.yh@...renesas.com, dianders@...omium.org,
 robert.marko@...tura.hr, schnelle@...ux.ibm.com, kkartik@...dia.com,
 linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org,
 devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-samsung-soc@...r.kernel.org, linux-arm-kernel@...s.com,
 ksk4725@...sia.com, kenkim@...sia.com, smn1196@...sia.com,
 pjsin865@...sia.com, shradha.t@...sung.com
Subject: Re: [PATCH 3/3] tty: serial: samsung: Remove unused artpec-8 specific
 code

On 11/09/2025 18:04, Ravi Patel wrote:
>>>>> -OF_EARLYCON_DECLARE(artpec8, "axis,artpec8-uart",
>>>>> -                       s5pv210_early_console_setup);
>>>>>
>>>>>  static int __init gs101_early_console_setup(struct earlycon_device *device,
>>>>>                                             const char *opt)
>>>>
>>>> Removing these breaks backwards-compatibility with existing DTBs,
>>>> which lack the new "samsung,exynos8895-uart" fallback compatible value.
>>>
>>> This was just applied, so ABI break would be fine. It should be however
>>> clearly expressed in the commit msg.
>>>
>>> I have a feeling that not much testing was happening in Samsung around
>>> this patchset and only now - after I applied it - some things happen.
>>> But it is damn too late, my tree is already closed which means this is
>>> going to be the ABI.
>>
>> Ah, no, I mixed up patches with recent DTS for Artpec-8. This serial ABI
>> was accepted three years ago (!!!), so you are Geert absolutely right -
>> that's ABI break.
> 
> Thank you for your review.
> 
> The DTS patches for ARTPEC-8 is added recently (https://lore.kernel.org/linux-samsung-soc/20250901051926.59970-1-ravi.patel@samsung.com/)
> Before that, there was no user (in DT) of "axis,artpec8-uart" compatible.
> So I am not convinced of ABI break (considering patch #1 and #2 goes first with review comment fixes)


ABI is defined by bindings and implemented by kernel. Having DTS user is
irrelevant to fact whether ABI is or is not broken.

Having DTS user determines the known impact of known ABI breakage.

Best regards,
Krzysztof