> -----Original Message----- > From: Wood Scott-B07421 > Sent: Tuesday, October 09, 2012 1:54 AM > To: Alexander Graf > Cc: Andreas Färber; Bhushan Bharat-R65777; Avi Kivity; qemu-...@nongnu.org; > qemu-devel@nongnu.org; Bhushan Bharat-R65777 > Subject: Re: [Qemu-devel] [PATCH 3/3] Adding BAR0 for e500 PCI controller > > On 10/08/2012 02:04:43 PM, Alexander Graf wrote: > > > > On 08.10.2012, at 20:00, Andreas Färber wrote: > > > > > 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 > > commit > > > message doesn't seem to mention it. > > > > I'd assume he tried to work around a bug I fixed in between. But this > > change definitely is not intentional. > > It looks like an accidental revert of > http://patchwork.ozlabs.org/patch/179475/
Oops, it was not intended to be sent as patch. It was not accidental because without this revert I was not able to boot my guest. Thanks -Bharat
