[U-Boot] [PATCH v2] vexpress: disable cci ace slave ports when booting in non-sec/hyp mode

[Sudeep Holla]

On 26/09/16 12:35, Sudeep Holla wrote:
>
>
> On 26/09/16 12:30, Jon Medhurst (Tixy) wrote:
>> On Fri, 2016-09-23 at 17:38 +0100, Sudeep Holla wrote:
>>> Commit f225d39d3093 ("vexpress: Check TC2 firmware support before
>>> defaulting
>>> to nonsec booting") added support to check if the firmware on TC2  is
>>> configured appropriately before booting in nonsec/hyp mode.
>>>
>>> However when booting in non-secure/hyp mode, CCI control must be done in
>>> secure firmware and can't  be done in non-secure/hyp mode. In order to
>>> ensure that, this patch disables the cci slave port inteface so that it
>>> is not accessed at all.
>>>
>>> Cc: Jon Medhurst <tixy at linaro.org>
>>> Acked-by: Marc Zyngier <marc.zyngier at arm.com>
>>> Signed-off-by: Sudeep Holla <sudeep.holla at arm.com>
>>> ---
>>
>> This works for me (unsurprisingly) when booting in secure mode. What
>> kernel and firmware config do I need to use for non-sec mode? I tried
>> vexpress_defconfig, with bits 12 and 13 cleared in SCC: 0x700
>> but I get nothing out the console after "Starting kernel ..."
>>
>
> I just flipped bit 12 and 13 in SCC: 0x700 to switch between MCPM/secure
> and HYP/non-secure mode. All other images/settings remain the same.
>
> So IIUC, with vexpress_defconfig + above SCC change you are seeing a
> hand, I will check that. I am using multi_v7_defconfig.
>

+ the patches from Lorenzo fixing MCPM code in the kernel [1] as
mentioned first email carrying this patch.

-- 
Regards,
Sudeep

[1] http://www.spinics.net/lists/arm-kernel/msg533715.html