[U-Boot] [PATCH 04/13] ppc: Remove pci config table pointer relocation fixups

Paul Gortmaker paul.gortmaker at windriver.com
Mon Sep 21 18:49:54 CEST 2009


On Mon, Sep 21, 2009 at 12:20 PM, Peter Tyser <ptyser at xes-inc.com> wrote:
> Signed-off-by: Peter Tyser <ptyser at xes-inc.com>

It looks like something happened during the send of your patches;
it seems the long log of what you are trying to fix and how the patch
fixes it is missing here and from several of the other patches
as well.

> ---
>  board/freescale/mpc8548cds/mpc8548cds.c |    7 -------
>  board/mpl/common/pci.c                  |   18 ------------------
>  board/sbc8548/sbc8548.c                 |    6 ------
>  3 files changed, 0 insertions(+), 31 deletions(-)
>
> diff --git a/board/freescale/mpc8548cds/mpc8548cds.c b/board/freescale/mpc8548cds/mpc8548cds.c
> index 80de6f8..73e7c21 100644
> --- a/board/freescale/mpc8548cds/mpc8548cds.c
> +++ b/board/freescale/mpc8548cds/mpc8548cds.c
> @@ -276,7 +276,6 @@ pci_init_board(void)
>  {
>        volatile ccsr_fsl_pci_t *pci = (ccsr_fsl_pci_t *) CONFIG_SYS_PCI1_ADDR;
>        struct pci_controller *hose = &pci1_hose;
> -       struct pci_config_table *table;
>        struct pci_region *r = hose->regions;
>
>        uint pci_32 = gur->pordevsr & MPC85xx_PORDEVSR_PCI1_PCI32;      /* PORDEVSR[15] */
> @@ -312,12 +311,6 @@ pci_init_board(void)
>                               PCI_REGION_IO);
>                hose->region_count = r - hose->regions;
>
> -               /* relocate config table pointers */
> -               hose->config_table = \
> -                       (struct pci_config_table *)((uint)hose->config_table + gd->reloc_off);
> -               for (table = hose->config_table; table && table->vendor; table++)
> -                       table->config_device += gd->reloc_off;

For the mpc8548cds, if this removal was somehow the right thing to do,
it would still be incomplete;  I am sure that there is a dummy function
related to a PCI bridge quirk associated with the above change that
would now be orphaned in the code.

> -
>                hose->first_busno=first_free_busno;
>
>                fsl_pci_init(hose, (u32)&pci->cfg_addr, (u32)&pci->cfg_data);

[...]

> diff --git a/board/sbc8548/sbc8548.c b/board/sbc8548/sbc8548.c
> index e5b21b5..5cdfef4 100644
> --- a/board/sbc8548/sbc8548.c
> +++ b/board/sbc8548/sbc8548.c
> @@ -392,12 +392,6 @@ pci_init_board(void)
>                               PCI_REGION_IO);
>                hose->region_count = r - hose->regions;
>
> -               /* relocate config table pointers */
> -               hose->config_table = \
> -                       (struct pci_config_table *)((uint)hose->config_table + gd->reloc_off);
> -               for (table = hose->config_table; table && table->vendor; table++)
> -                       table->config_device += gd->reloc_off;

This code is already gone from the sbc8548 in the 85xx branch;
the sbc8548 didn't need the bridge quirk fixup.

Paul.

> -
>                hose->first_busno=first_free_busno;
>
>                fsl_pci_init(hose, (u32)&pci->cfg_addr, (u32)&pci->cfg_data);
> --
> 1.6.2.1
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>


More information about the U-Boot mailing list