Re: [PATCH V3 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

[Leif Lindholm <leif.lindholm@linaro.org>]

On Tue, Jan 31, 2017 at 09:48:11AM +0000, Ryan Harkin wrote:
> On 26 January 2017 at 14:36, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> > On Fri, Jan 20, 2017 at 05:10:45PM +0530, Bhupesh Sharma wrote:
> >> ARM TZASC-380 IP provides a mechanism to split memory regions being
> >> protected via it into eight equal-sized sub-regions. A bit-setting
> >> allows the corresponding subregion to be disabled.
> >>
> >> Several NXP/FSL SoCs support the TZASC-380 IP block and allow
> >> the DDR connected via the TZASC to be partitioned into regions
> >> having different security settings and also allow subregions
> >> to be disabled.
> >>
> >> This patch enables this support and can be used for SoCs which
> >> support such a partition of DDR regions.
> >>
> >> Details of the 'subregion_disable' register can be viewed here:
> >> http://infocenter.arm.com/help/topic/com.arm.doc.ddi0431c/CHDIGDCI.html
> >>
> >> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> >> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >> Contributed-under: TianoCore Contribution Agreement 1.0
> >> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@nxp.com>
> >> [bhupesh.linux@gmail.com : Added my gmail ID as the NXP one is no longer valid]
> >> Signed-off-by: Bhupesh Sharma <bhupesh.linux@gmail.com>
> >
> > Thanks for the cleanup.
> > I may actually delete that CTA9x4 lib once your platform gets in...
> >
> Probably a good idea. I suspect it's not used or tested any more,
> unless QEMU is using it?

It's unused. And if we do ever resurrect the CTA9 port, it'll be using
ARM Trusted Firmware, so this code will still be irrelevant.

/
    Leif