From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:52667)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <B07421@freescale.com>) id 1TLJrp-0000oK-QI
	for qemu-devel@nongnu.org; Mon, 08 Oct 2012 16:23:58 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <B07421@freescale.com>) id 1TLJro-0007lT-MV
	for qemu-devel@nongnu.org; Mon, 08 Oct 2012 16:23:57 -0400
Date: Mon, 8 Oct 2012 15:23:52 -0500
From: Scott Wood <scottwood@freescale.com>
References: <1349714816-12618-1-git-send-email-Bharat.Bhushan@freescale.com>
	<1349714816-12618-4-git-send-email-Bharat.Bhushan@freescale.com>
	<507314BD.7010001@suse.de>
	<841569A6-FF61-4AF0-91F5-7CF409E3314C@suse.de>
In-Reply-To: <841569A6-FF61-4AF0-91F5-7CF409E3314C@suse.de> (from
	agraf@suse.de on Mon Oct  8 14:04:43 2012)
Message-ID: <1349727832.3721.3@snotra>
MIME-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"; delsp=Yes; format=Flowed
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH 3/3] Adding BAR0 for e500 PCI controller
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Alexander Graf <agraf@suse.de>
Cc: qemu-devel@nongnu.org, Bharat Bhushan <Bharat.Bhushan@freescale.com>, qemu-ppc@nongnu.org, Avi Kivity <avi@redhat.com>, Bharat Bhushan <r65777@freescale.com>, Andreas =?iso-8859-1?q?F=E4rber?= <afaerber@suse.de>

On 10/08/2012 02:04:43 PM, Alexander Graf wrote:
>=20
> On 08.10.2012, at 20:00, Andreas F=E4rber wrote:
>=20
> > Am 08.10.2012 18:46, schrieb Bharat Bhushan:
> >> #define BINARY_DEVICE_TREE_FILE    "mpc8544ds.dtb"
> >> #define UIMAGE_LOAD_BASE           0
> >> -#define DTC_LOAD_PAD               0x1800000
> >> +#define DTC_LOAD_PAD               0x500000
> >> #define DTC_PAD_MASK               0xFFFFF
> >> #define INITRD_LOAD_PAD            0x2000000
> >> #define INITRD_PAD_MASK            0xFFFFFF
> >
> > Was this change intentional? I don't see it being used here, and =20
> commit
> > message doesn't seem to mention it.
>=20
> I'd assume he tried to work around a bug I fixed in between. But this =20
> change definitely is not intentional.

It looks like an accidental revert of =20
http://patchwork.ozlabs.org/patch/179475/

-Scott=