[U-Boot] [Patch v2, batch 2 03/23] powerpc/pcie: add PCIe version 3.x support

Andy Fleming afleming at gmail.com
Mon May 13 20:50:25 CEST 2013


This patch causes all sorts of problems. I'm NACKing it for now. Please
make sure none of these changes break 83xx before re-submitting.


On Mon, Mar 25, 2013 at 12:33 PM, York Sun <yorksun at freescale.com> wrote:

> From: Roy ZANG <tie-fei.zang at freescale.com>
>
> T4240 PCIe IP is version 3.0 and has some update comparing previous
> QorIQ products.
>
> 1.  Move Freescale specific register define
> to
> arch/powerpc/include/asm/fsl_pci.h
> and update the register offset define for T4240.
>
> 2. add the status/control register define
> use status/control register to judge the link status
>
> 3. The original code uses 'Programming Interface' field to judge if PCIE is
> EP or RC mode, however, T4240 does not support this functionality.
> According to PCIE specification, 'Header Type' offset 0x0e is used to
> indicate header type, so for PCIE controller, the patch changes code to
> use 'Header Type' field to identify if the PCIE is RC or EP mode.
>
> Signed-off-by: Roy Zang <tie-fei.zang at freescale.com>
> Signed-off-by: Minghuan Lian <Minghuan.Lian at freescale.com>
> ---
>  arch/powerpc/include/asm/config_mpc85xx.h |    1 +
>  arch/powerpc/include/asm/fsl_pci.h        |   35
> +++++++++++++++++++++++++++--
>  drivers/pci/fsl_pci_init.c                |   20 ++++++++++++-----
>  include/pci.h                             |    7 ------
>  4 files changed, 48 insertions(+), 15 deletions(-)
>
>
[...]


> index 15f583f..c0ed553 100644
> --- a/include/pci.h
> +++ b/include/pci.h
> @@ -426,13 +426,6 @@
>  #define PCI_MAX_PCI_DEVICES    32
>  #define PCI_MAX_PCI_FUNCTIONS  8
>
> -#define PCI_DCR                0x54    /* PCIe Device Control Register */
> -#define PCI_DSR                0x56    /* PCIe Device Status Register */
> -#define PCI_LSR                0x5e    /* PCIe Link Status Register */
> -#define PCI_LCR                0x5c    /* PCIe Link Control Register */
> -#define PCI_LTSSM      0x404   /* PCIe Link Training, Status State
> Machine */
> -#define  PCI_LTSSM_L0  0x16    /* L0 state */
>


These were being used by 83xx as well, and are now unavailable. Please
devise a solution that works for all of our platforms.

Andy


More information about the U-Boot mailing list