Re: [PATCH v12 03/11] PCI: kirin: Add support for a PHY layer

From: Pali Rohár
Date: Tue Oct 05 2021 - 16:31:54 EST


Hello!

On Tuesday 28 September 2021 09:34:13 Mauro Carvalho Chehab wrote:
> The pcie-kirin driver contains both PHY and generic PCI driver
> on it.
>
> The best would be, instead, to support a PCI PHY driver, making
> the driver more generic.
>
> However, it is too late to remove the Kirin 960 PHY, as a change
> like that would make the DT schema incompatible with past versions.

I have not looked deeply at it. But is not it really possible to declare
PHY node in DTS file with backward compatible manner? Or cannot Rob help
with it (maybe there was similar issue in past with other driver)?

I was fixing something similar, address space defined in DTS was used by
two HW blocks: clock and UART. And I was able to make both DTS file and
driver backward compatible.