Subject: Re: [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds

From: Scott Wood <[email-masked]>
Date: Wed, 14 Oct 2015 15:36:08 -0500

On Tue, 2015-10-13 at 19:29 +0800, Zhiqiang Hou wrote:
> From: Harninder Rai <[email-masked]>
> 
> Signed-off-by: Harninder Rai <[email-masked]>
> Signed-off-by: Minghuan Lian <[email-masked]>
> Change-Id: I4355add4a92d1fcf514843aea5ecadd2e2517969
> Reviewed-on: http://git.am.freescale.net:8181/2454
> Reviewed-by: Zang Tiefei-R61911 <[email-masked]>
> Reviewed-by: Kushwaha Prabhakar-B32579 <[email-masked]>
> Reviewed-by: Fleming Andrew-AFLEMING <[email-masked]>
> Tested-by: Fleming Andrew-AFLEMING <[email-masked]>

Get rid of the gerrit stuff.  And where is your signoff?

> diff --git a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi 
> b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi
> index c723071..78c8f1c 100644
> --- a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi
> @@ -40,6 +40,35 @@
>       interrupts = <16 2 0 0 20 2 0 0>;
>  };
>  
> +/* controller at 0xa000 */
> +&pci0 {
> +     compatible = "fsl,bsc9132-pcie", "fsl,qoriq-pcie-v2.2";
> +     device_type = "pci";
> +     #size-cells = <2>;
> +     #address-cells = <3>;
> +     bus-range = <0 255>;
> +     clock-frequency = <33333333>;
> +     interrupts = <16 2 0 0>;

This clock-frequency is not correct for PCIe.  Just remove it.

-Scott