From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:41533)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <clg@fr.ibm.com>) id 1a3VwI-00048V-6N
	for qemu-devel@nongnu.org; Mon, 30 Nov 2015 16:24:51 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <clg@fr.ibm.com>) id 1a3VwF-0000B7-1Z
	for qemu-devel@nongnu.org; Mon, 30 Nov 2015 16:24:50 -0500
Received: from e06smtp06.uk.ibm.com ([195.75.94.102]:37756)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <clg@fr.ibm.com>) id 1a3VwE-0000Ar-O8
	for qemu-devel@nongnu.org; Mon, 30 Nov 2015 16:24:46 -0500
Received: from localhost
	by e06smtp06.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use
	Only! Violators will be prosecuted
	for <qemu-devel@nongnu.org> from <clg@fr.ibm.com>;
	Mon, 30 Nov 2015 21:24:44 -0000
Message-ID: <565CBE96.8040909@fr.ibm.com>
Date: Mon, 30 Nov 2015 22:24:38 +0100
From: =?UTF-8?B?Q8OpZHJpYyBMZSBHb2F0ZXI=?= <clg@fr.ibm.com>
MIME-Version: 1.0
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>
	<1448914172.2666.40.camel@kernel.crashing.org>
In-Reply-To: <1448914172.2666.40.camel@kernel.crashing.org>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
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: Benjamin Herrenschmidt <benh@kernel.crashing.org>, 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 11/30/2015 09:09 PM, Benjamin Herrenschmidt wrote:
> On Mon, 2015-11-30 at 19:15 +0100, Cédric 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 BOOTKERNEL
>> partitions are hard coded but I guess we don't need to read the flash partition
>> 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.

yes that would be better but I don't measure what it takes to implement 
the LPC/AHB bridge to access the PNOR.

C.