[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5695ADE5.7010501@linaro.org>
Date: Wed, 13 Jan 2016 09:52:37 +0800
From: Hanjun Guo <hanjun.guo@...aro.org>
To: Tomasz Nowicki <tn@...ihalf.com>,
Marc Zyngier <marc.zyngier@....com>, tglx@...utronix.de,
jason@...edaemon.net, rjw@...ysocki.net, lorenzo.pieralisi@....com,
robert.richter@...iumnetworks.com, shijie.huang@....com,
guohanjun@...wei.com, Suravee.Suthikulpanit@....com
Cc: mw@...ihalf.com, graeme.gregory@...aro.org,
Catalin.Marinas@....com, will.deacon@....com,
linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, ddaney.cavm@...il.com
Subject: Re: [PATCH V2 03/10] irqchip,GICv3,ACPI: Add redistributor support
via GICC structures.
On 2016/1/13 0:45, Tomasz Nowicki wrote:
> On 12.01.2016 13:03, Marc Zyngier wrote:
>
> > + typer = readq_relaxed(redist_base + GICR_TYPER);
> > + /* don't map reserved page as it's buggy to access it */
> > + size = (typer & GICR_TYPER_VLPIS) ? SZ_64K * 3 : SZ_64K * 2;
>
> [...]
>
>> Also, please map the whole region for the
>> redistributor as we have on the DT side (4 64kB pages for VLPIS capable
>> redistributors).
>
> Hanjun, is it something you had problem with?
Yes, for D02, it use 3 64kB pages for GICv4, it removed the reserved
page. In DT case, it has "stride" binding so it can support no standard
cases, seems that we can introduce "stride" for ACPI too.
Thanks
Hanjun
Powered by blists - more mailing lists