On Thu, Dec 30, 2021 at 11:21 AM Yu Tu <yu.tu@xxxxxxxxxxx> wrote: > > All mainline .dts files have been using the stable UART since Linux > 4.16. Drop the legacy compatible strings and related clock code. > > Signed-off-by: Yu Tu <yu.tu@xxxxxxxxxxx> I have just realized that this cannot be dropped until my series "ARM: dts: meson: fix UART device-tree schema validation" [0] is merged [...] > -/* Legacy bindings, should be removed when no more used */ > -OF_EARLYCON_DECLARE(meson, "amlogic,meson-uart", > - meson_serial_early_console_setup); This part is still needed as long as above series is not merged yet. If we remove this then earlycon will stop working on the 32-bit SoCs unless [0] is merged. All other code below - except the of_device_id entry - can still be removed since meson8.dtsi and meson8b.dtsi are using the non-legacy clocks already. Sorry for only noticing this now. Best regards, Martin [0] https://patchwork.kernel.org/project/linux-amlogic/cover/20211227180026.4068352-1-martin.blumenstingl@xxxxxxxxxxxxxx/