-----Original Message-----
From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
Sent: 16 February 2023 16:29
To: Shradha Todi <redacted>; lpieralisi@kernel.org;
kw@linux.com; robh@kernel.org; bhelgaas@google.com;
krzysztof.kozlowski+dt@linaro.org; alim.akhtar@samsung.com;
jingoohan1@gmail.com; Sergey.Semin@baikalelectronics.ru;
lukas.bulwahn@gmail.com; hongxing.zhu@nxp.com; tglx@linutronix.de;
m.szyprowski@samsung.com; jh80.chung@samsung.co;
pankaj.dubey@samsung.com
Cc: linux-pci@vger.kernel.org; devicetree@vger.kernel.org; linux-arm-
kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org; linux-
kernel@vger.kernel.org
Subject: Re: [PATCH 02/16] PCI: exynos: Rename Exynos PCIe driver to
Samsung PCIe

On 16/02/2023 11:55, Krzysztof Kozlowski wrote:

quoted

On 14/02/2023 13:13, Shradha Todi wrote:

quoted

The current PCIe controller driver is being used for Exynos5433 SoC
only. In order to extend this driver for all SoCs manufactured by
Samsung using DWC PCIe controller, rename this driver and make it
Samsung specific instead of any Samsung SoC name.

Signed-off-by: Shradha Todi <redacted>
---
 MAINTAINERS                              |   4 +-
 drivers/pci/controller/dwc/Kconfig       |   6 +-
 drivers/pci/controller/dwc/Makefile      |   2 +-
 drivers/pci/controller/dwc/pci-samsung.c | 443
+++++++++++++++++++++++

Rename missing. I am anyway not sure if this is good. What's wrong
with old name?

OK, looking a bit at your further patches - doesn't it make sense to split a bit
the driver? Maybe keep the core as pci-samsung, but some other parts in
pci-exynso5433?

Ok agreed. So here is what I am planning, keeping in mind the next set of platform support which I am planning to send out (say FSD, ARTPEC-v8):
1: We will move samsung pci driver inside dwc/samsung/
2: pci-samsung.c shall contain common APIs, helper functions, etc
3: Platform specific driver will have their own files such as pcie-exynos.c, pcie-fsd.c, pcie-artpec-v8.c 
Let me know what you think of this.
I am not very keen on renaming Exynos SoC file as pcie-exyons5433.c as in future we may end up adding PCIe support for other Exynos which being
in same family (Exynos Series) will be very similar in design. Custom ASIC (manufactured by Samsung Foundry) is primarily driven by various
vendors and will have separate design in terms of integration of IPs in SoC and we need to have support for all such SoCs manufactured under Samsung umbrella.

Shradha

Best regards,
Krzysztof