[U-Boot] [PATCH 1/2] pci/layerscape: move pcie_layerscape.h out of arm include

Tom Rini trini at konsulko.com
Wed Mar 11 15:22:14 CET 2015


On Wed, Mar 11, 2015 at 07:09:33PM +0800, Minghuan Lian wrote:

> The patch moves pcie_layerscape.h out of arm include to top
> level include folder so that it can be shared on other Socs.
> 
> Signed-off-by: Minghuan Lian <Minghuan.Lian at freescale.com>

OK, I see a problem:

> diff --git a/arch/arm/include/asm/pcie_layerscape.h b/arch/arm/include/asm/pcie_layerscape.h
> deleted file mode 100644
> index fb08578..0000000
> --- a/arch/arm/include/asm/pcie_layerscape.h
> +++ /dev/null
> @@ -1,13 +0,0 @@
> -/*
> - * Copyright 2014 Freescale Semiconductor, Inc.
> - *
> - * SPDX-License-Identifier:	GPL-2.0+
> - */
> -
> -#ifndef __PCIE_LAYERSCAPE_H_
> -#define __PCIE_LAYERSCAPE_H_
> -
> -void pci_init_board(void);

This is already in <common.h>

> -void ft_pcie_setup(void *blob, bd_t *bd);

This belongs in <fdt_support.h> near other similar functions and frankly
might be better done as ft_pci_setup(blob, bd) which already exists.  Or
do we have both ft_pci_setup and ft_pcie_setup existing at the same
time?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150311/1f60a19b/attachment.sig>


More information about the U-Boot mailing list