[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LNX.2.00.1507161530270.19651@nippy.intranet>
Date: Thu, 16 Jul 2015 16:01:41 +1000 (AEST)
From: Finn Thain <fthain@...egraphics.com.au>
To: Benjamin Herrenschmidt <benh@...nel.crashing.org>
cc: linux-kernel@...r.kernel.org, linux-m68k@...r.kernel.org,
linuxppc-dev@...ts.ozlabs.org, Paul Mackerras <paulus@...ba.org>,
Michael Ellerman <mpe@...erman.id.au>,
Arnd Bergmann <arnd@...db.de>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
Tomi Valkeinen <tomi.valkeinen@...com>,
linux-fbdev@...r.kernel.org
Subject: Re: [RFC v4 17/25] powerpc, fbdev: Use arch_nvram_ops methods instead
of nvram_read_byte() and nvram_write_byte()
On Wed, 15 Jul 2015, I wrote:
> On Tue, 14 Jul 2015, Benjamin Herrenschmidt wrote:
>
> > Maybe we should have a dedicated accessor for "mac_xpram" ...
>
> ... I can see how to implement XPRAM for matroxfb and imsttfb
I'll have to retract that. The video mode and color mode settings used by
the PowerMac framebuffer drivers don't exist in the PRAM portion of NVRAM.
Addresses 0x140F and 0x1410 are found in the partition reserved by Apple
for "Name Registry properties", according to Designing PCI Cards and
Drivers for Power Macintosh Computers. There is no equivalent on m68k
Macs, AFAIK.
This is NVRAM partition 2 on my beige g3, which begins at 0x1400. I'm not
sure that this is true on New World PowerMacs, and I suspect that the
framebuffer drivers should be calling pmac_get_partition() to determine
the offset of the beginning of the Name Registry partition.
The arch_nvram_ops methods don't deal with structures like partitions.
They treat the entire 8 KiB as unstructured, because that's how /dev/nvram
treats it.
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists