From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bin Meng Date: Mon, 10 Oct 2016 19:49:27 +0800 Subject: [U-Boot] [PATCH 8/9] pci: layerscape: rewrite pci driver based on DM In-Reply-To: <1476089269-28697-8-git-send-email-Minghuan.Lian@nxp.com> References: <1476089269-28697-1-git-send-email-Minghuan.Lian@nxp.com> <1476089269-28697-8-git-send-email-Minghuan.Lian@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Minghuan, On Mon, Oct 10, 2016 at 4:47 PM, Minghuan Lian wrote: > There are more than five kinds of Layerscape SoCs. unfortunately, > PCIe controller of each SoC is a little bit different. In order > to avoid too many macro definitions, the patch re-implement PCIe > driver based on DM. PCIe dts node is to describe the difference. > > Signed-off-by: Minghuan Lian > --- > .../include/asm/arch-fsl-layerscape/immap_lsch3.h | 8 - > drivers/pci/Kconfig | 8 + > drivers/pci/pcie_layerscape.c | 958 +++++++++++---------- > include/configs/ls1012a_common.h | 12 + > include/configs/ls1012aqds.h | 24 - > include/configs/ls1012ardb.h | 24 - > include/configs/ls1021aqds.h | 18 +- > include/configs/ls1021atwr.h | 18 +- > include/configs/ls1043a_common.h | 23 +- > include/configs/ls2080a_common.h | 27 +- > include/configs/ls2080aqds.h | 8 - > include/configs/ls2080ardb.h | 8 - These header file changes should not be put in the same commit of the layerscape PCIe driver conversion. They should be in a separate commit. So you are likely to have 3 commits: firstly add DM codes with #ifdef CONFIG_DM_PCI #endif in the layerscape PCIe driver, without breaking the existing board support. 2nd commit to update the boards configuration files (defconfig and the header), and 3rd commit to remove the #ifdef CONFIG_DM_PCI #endif, only leaving the DM version codes. > 12 files changed, 526 insertions(+), 610 deletions(-) > > diff --git a/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch3.h b/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch3.h > index 7acba27..bd07808 100644 > --- a/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch3.h > +++ b/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch3.h > @@ -104,14 +104,6 @@ > #define CONFIG_SYS_PCIE2_PHYS_ADDR 0x1200000000ULL > #define CONFIG_SYS_PCIE3_PHYS_ADDR 0x1400000000ULL > #define CONFIG_SYS_PCIE4_PHYS_ADDR 0x1600000000ULL > -/* LUT registers */ > -#define PCIE_LUT_BASE 0x80000 > -#define PCIE_LUT_LCTRL0 0x7F8 > -#define PCIE_LUT_DBG 0x7FC > -#define PCIE_LUT_UDR(n) (0x800 + (n) * 8) > -#define PCIE_LUT_LDR(n) (0x804 + (n) * 8) > -#define PCIE_LUT_ENABLE (1 << 31) > -#define PCIE_LUT_ENTRY_COUNT 32 > [snip] Regards, Bin