From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Thu, 6 Sep 2012 11:52:09 -0500
Subject: [U-Boot] [PATCH 4/5] powerpc/85xx: Add P5040 processor support
In-Reply-To: <1346444736-24610-4-git-send-email-timur@freescale.com>
References: <1346444736-24610-1-git-send-email-timur@freescale.com>
	<1346444736-24610-4-git-send-email-timur@freescale.com>
Message-ID: <5048D4B9.7080803@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 08/31/2012 03:25 PM, Timur Tabi wrote:
> +	SET_PCI_LIODN_BASE(CONFIG_SYS_FSL_PCIE_COMPAT, 1, 193),
> +	SET_PCI_LIODN_BASE(CONFIG_SYS_FSL_PCIE_COMPAT, 2, 194),
> +	SET_PCI_LIODN_BASE(CONFIG_SYS_FSL_PCIE_COMPAT, 3, 195),

You're only allowing for one LIODN per PCIe controller, which defeats
the purpose of the new PCIe LIODN mechanism.

Please allocate several contiguous LIODNs per controller, set the base
to the first one, and enumerate the rest in a device tree property for
the OS to use.

-Scott