From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:49130)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1a3Ulf-0002Iv-91
	for qemu-devel@nongnu.org; Mon, 30 Nov 2015 15:09:48 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1a3Ulc-0003zL-3D
	for qemu-devel@nongnu.org; Mon, 30 Nov 2015 15:09:47 -0500
Message-ID: <1448914172.2666.40.camel@kernel.crashing.org>
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Tue, 01 Dec 2015 07:09:32 +1100
In-Reply-To: <565C925F.60306@fr.ibm.com>
References: <1447201710-10229-1-git-send-email-benh@kernel.crashing.org>
	<564290E1.3090205@redhat.com>
	<1447203387.31884.126.camel@kernel.crashing.org>
	<5642B59E.2070101@ozlabs.ru>
	<1447213139.31884.136.camel@kernel.crashing.org>	<5642BEF9.90406@ozlabs.ru>
	<1447215397.31884.140.camel@kernel.crashing.org>
	<5642C6F0.9040200@ozlabs.ru> <56582EAF.40103@suse.de>
	<1448697599.3172.1.camel@kernel.crashing.org>
	<565C925F.60306@fr.ibm.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 00/77] ppc: Add "native" POWER8
 platform
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: =?ISO-8859-1?Q?C=E9dric?= Le Goater <clg@fr.ibm.com>, Alexander Graf <agraf@suse.de>, Alexey Kardashevskiy <aik@ozlabs.ru>, Eric Blake <eblake@redhat.com>, qemu-ppc@nongnu.org
Cc: qemu-devel@nongnu.org

On Mon, 2015-11-30 at 19:15 +0100, C=C3=A9dric Le Goater wrote:
> The pnor file is compiled from github. The patch is below (without the =
dirty
> cut and paste I did in loader.c). The offset for the PAYLOAD and BOOTKE=
RNEL
> partitions are hard coded but I guess we don't need to read the flash p=
artition
> table in qemu, not yet.

In practice we should read the partition tables, I don't like hard
coded offsets... But we should probably create a proper "flash driver"
that does a bunch of this, and also adds the BMC style flash interface
so OPAL can write to nvram.

Cheers,
Ben.