Hi Christoffer / uboot,
We are able to figure out the issue. PERIPHBASE returned on ARNDALE board was 0x10500000 and adding GIC offset was creating an incorrect address.
Setting CONFIG_ARM_GIC_BASE_ADDRESS as 0x10480000 in arndale.h solves the issue.
The u-boot code needs to be fixed for arndale right ? Please confirm
Regards
Manish
armdev Team @FTM
On 13-Mar-2014, at 11:20 am, armdev <armdev.ftm@xxxxxxxxx> wrote:
> Got a mail that Andre is not with linaro, Adding Christoffer Dall
>
> On 13-Mar-2014, at 11:17 am, armdev <armdev.ftm@xxxxxxxxx> wrote:
>
>> Hi Andrey ,
>>
>> This is wrt your patch set which enabled the switch to non-sec and hip mode in armv7.
>> We have a small piece of code which we run in hypmode using the bootm command on arndale board
>>
>> While this code was trying to access the GICD_ISENABLR0 (0x10481100), read was always returning 0, while as per the reset value it should be 0x0000ffff. Debugged more in u-boot and added prints.
>>
>> This is the log, our observation is that in _nonsec_init() there is something that is causing the issue. Now It can be an issue or a gap in our understanding. Ideally we should be able to read the GICD_ISENABLR value as 0xffff.
>> Can you please help us in find out how to read GIC values properly in hyp mode.
>>
>> armv7_switch_nonsec
>> cpsr =600001d3
>> REGS ffff
>> Before write to group regs
>> cpsr =600001d3
>> REGS ffff
>> Before _nonsec_init()
>> cpsr =600001d3
>> REGS ffff
>> After _nonsec_init()
>> cpsr =600001d3
>> REGS 0
>>
>> HYP mode: successful.
>> armv7_switch_hyp
>> cpsr =600001da
>> REGS 0
>>
>> Here is the diff of our modifications. (just added prints)
>> diff --git a/arch/arm/cpu/armv7/virt-v7.c b/arch/arm/cpu/armv7/virt-v7.c
>> index 2cd604f..f38f125 100644
>> --- a/arch/arm/cpu/armv7/virt-v7.c
>> +++ b/arch/arm/cpu/armv7/virt-v7.c
>> @@ -107,6 +107,10 @@ int armv7_switch_hyp(void)
>> printf("HYP mode: switch not successful.\n");
>> return -1;
>> }
>> + printf("HYP mode: successful.\n");
>> + printf("%s \r\n",__FUNCTION__);
>> + printf("cpsr =%x\n", read_cpsr());
>> + printf("REGS %x \n", *(uint32_t*)0x10481100);
>>
>> return 0;
>> }
>> @@ -116,6 +120,10 @@ int armv7_switch_nonsec(void)
>> unsigned int reg;
>> unsigned itlinesnr, i;
>>
>> + printf("%s \r\n",__FUNCTION__);
>> + printf("cpsr =%x\n", read_cpsr());
>> + printf("REGS %x \n", *(uint32_t*)0x10481100);
>> +
>> /* check whether the CPU supports the security extensions */
>> reg = read_id_pfr1();
>> if ((reg & 0xF0) == 0) {
>> @@ -140,6 +148,9 @@ int armv7_switch_nonsec(void)
>> /* TYPER[4:0] contains an encoded number of available interrupts */
>> itlinesnr = readl(gic_dist_addr + GICD_TYPER) & 0x1f;
>>
>> + printf("Before write to group regs \r\n");
>> + printf("cpsr =%x\n", read_cpsr());
>> + printf("REGS %x \n", *(uint32_t*)0x10481100);
>> /* set all bits in the GIC group registers to one to allow access
>> * from non-secure state. The first 32 interrupts are private per
>> * CPU and will be set later when enabling the GIC for each core
>> @@ -147,11 +158,19 @@ int armv7_switch_nonsec(void)
>> for (i = 1; i <= itlinesnr; i++)
>> writel((unsigned)-1, gic_dist_addr + GICD_IGROUPRn + 4 * i);
>>
>> - smp_set_core_boot_addr((unsigned long)_smp_pen, -1);
>> - smp_kick_all_cpus();
>> + //smp_set_core_boot_addr((unsigned long)_smp_pen, -1);
>> + //smp_kick_all_cpus();
>>
>> + printf("Before _nonsec_init() \r\n");
>> + printf("cpsr =%x\n", read_cpsr());
>> + printf("REGS %x \n", *(uint32_t*)0x10481100);
>> /* call the non-sec switching code on this CPU also */
>> _nonsec_init();
>>
>> + printf("After _nonsec_init() \r\n");
>> + printf("cpsr =%x\n", read_cpsr());
>> + printf("REGS %x \n", *(uint32_t*)0x10481100);
>> +
>> +
>> return 0;
>> }
>> diff --git a/include/configs/arndale.h b/include/configs/arndale.h
>> index 515facf..53a1212 100644
>> --- a/include/configs/arndale.h
>> +++ b/include/configs/arndale.h
>> @@ -62,7 +62,6 @@
>> /* select serial console configuration */
>> #define CONFIG_BAUDRATE 115200
>> #define EXYNOS5_DEFAULT_UART_OFFSET 0x010000
>> -#define CONFIG_SILENT_CONSOLE
>>
>> /* Console configuration */
>> #define CONFIG_CONSOLE_MUX
>> @@ -249,4 +248,7 @@
>> /* Enable Time Command */
>> #define CONFIG_CMD_TIME
>>
>> +/* Enable jump to hypervisor */
>> +#define CONFIG_ARMV7_VIRT
>> +#define CONFIG_SMP_PEN_ADDR 0x0
>> #endif /* __CONFIG_H */
>>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
