|
From: | Mitsyanko Igor |
Subject: | Re: [Qemu-devel] [PATCH v11 5/9] ARM: exynos4210: basic Power Management Unit implementation |
Date: | Thu, 02 Feb 2012 10:06:31 +0400 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111109 Thunderbird/3.1.16 |
On 02/01/2012 08:21 PM, Peter Maydell wrote:
On 30 January 2012 07:38, Evgeny Voevodin<address@hidden> wrote:From: Maksim Kozlov<address@hidden> Patch adds basic model for Exynos4210 SoC PMU. This model implements PMU registers just as a bulk of memory. Currently, the only reason this device exists is that secondary CPU boot loader uses PMU INFORM5 register as a holding pen.Your cover letter's changelog says - hw/exynos4210_pmu.c: we do not waste space for non-existing registers in PMU state anymore; non-existing registers are now RAZ/WI; ...wrong version of this patch, or is the cover letter wrong? -- PMM
Cover letter is not wrong, when we send PMU patch for the first time (in V9 series), PMU memory was modelled as continious array of registers 0x3d0c bytes long, with every register being "read as written". This was wrong since PMU address space consists of only a handful of registers with big empty gaps between them. Starting from V10, PMU state includes only actually existing registers, and empty gaps between these registers behave as RAZ/WI.
-- Mitsyanko Igor ASWG, Moscow R&D center, Samsung Electronics email: address@hidden
[Prev in Thread] | Current Thread | [Next in Thread] |