From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:43631) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h2C5l-0002rM-Vt for qemu-devel@nongnu.org; Fri, 08 Mar 2019 04:47:02 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1h2C5l-0006zX-1m for qemu-devel@nongnu.org; Fri, 08 Mar 2019 04:47:01 -0500 From: Markus Armbruster References: <20190226193408.23862-1-armbru@redhat.com> <20190226193408.23862-8-armbru@redhat.com> <87va15lds9.fsf@zen.linaroharston> <87tvgfuq0g.fsf@dusky.pond.sub.org> <20190308043911.GV7722@umbus.fritz.box> Date: Fri, 08 Mar 2019 10:46:54 +0100 In-Reply-To: <20190308043911.GV7722@umbus.fritz.box> (David Gibson's message of "Fri, 8 Mar 2019 15:39:11 +1100") Message-ID: <874l8dhf5d.fsf@dusky.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v2 07/11] ppc405_boards: Don't size flash memory to match backing image List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: kwolf@redhat.com, qemu-block@nongnu.org, lersek@redhat.com, qemu-devel@nongnu.org, mreitz@redhat.com, qemu-ppc@nongnu.org, Alex =?utf-8?Q?Benn=C3=A9e?= David Gibson writes: > On Thu, Mar 07, 2019 at 08:01:35AM +0100, Markus Armbruster wrote: >> Alex Benn=C3=A9e writes: >>=20 >> > Markus Armbruster writes: >> > >> >> Machine "ref405ep" maps its flash memory at address 2^32 - image size. >> >> Image size is rounded up to the next multiple of 64KiB. Useless, >> >> because pflash_cfi02_realize() fails with "failed to read the initial >> >> flash content" unless the rounding is a no-op. >> >> >> >> If the image size exceeds 0x80000 Bytes, we overlap first SRAM, then >> >> other stuff. No idea how that would play out, but a useful outcomes >> >> seem unlikely. >> >> >> >> Map the flash memory at fixed address 0xFFF80000 with size 512KiB, >> >> regardless of image size, to match the physical hardware. >> >> >> >> Machine "taihu" maps its boot flash memory similarly. The code even >> >> has a comment /* XXX: should check that size is 2MB */, followed by >> >> disabled code to adjust the size to 2MiB regardless of image size. >> >> >> >> Its code to map its application flash memory looks the same, except >> >> there the XXX comment asks for 32MiB, and the code to adjust the size >> >> isn't disabled. Note that pflash_cfi02_realize() fails with "failed >> >> to read the initial flash content" for images smaller than 32MiB. >> >> >> >> Map the boot flash memory at fixed address 0xFFE00000 with size 2MiB, >> >> to match the physical hardware. Delete dead code from application >> >> flash mapping, and simplify some. >> > >> > It seems to me the DEBUG_BOARD_INIT code is probably out of date cruft >> > that could be excised all together. But that doesn't stop this being >> > useful: >>=20 >> David, would you like me to excise DEBUG_BOARD_INIT? > > If you have the chance to look at it, that would be great. Done in v4. Thanks!