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

[Ravi Patel]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk at kernel.org>
> Sent: 11 September 2025 22:52
> To: Ravi Patel <ravi.patel at samsung.com>; 'Geert Uytterhoeven' <geert at linux-m68k.org>
> Cc: gregkh at linuxfoundation.org; jirislaby at kernel.org; robh at kernel.org; krzk+dt at kernel.org; conor+dt at kernel.org;
> jesper.nilsson at axis.com; lars.persson at axis.com; alim.akhtar at samsung.com; arnd at kernel.org; andriy.shevchenko at linux.intel.com;
> geert+renesas at glider.be; thierry.bultel.yh at bp.renesas.com; dianders at chromium.org; robert.marko at sartura.hr; schnelle at linux.ibm.com;
> kkartik at nvidia.com; linux-kernel at vger.kernel.org; linux-serial at vger.kernel.org; devicetree at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; linux-samsung-soc at vger.kernel.org; linux-arm-kernel at axis.com; ksk4725 at coasia.com; kenkim at coasia.com;
> smn1196 at coasia.com; pjsin865 at coasia.com; shradha.t at samsung.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 at 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.

OK. So does that mean if someone adds the ABI then it cannot be reverted,
because of it breaks backword compatibility (users are using ABI in their local DTB) ?

Please suggest what should be the proper way.

Thanks,
Ravi

> 
> Best regards,
> Krzysztof