Subject: Re: [PATCH] OMAP3 ROM Random Number Generator support

It could be that I am misreading something, but RX51 is OMAP3.

Regards,
Ivo


>-------- Оригинално писмо --------
>От: Tony Lindgren
>Относно: Re: [PATCH] OMAP3 ROM Random Number Generator support
>До: Ивайло Димитров
>Изпратено на: Понеделник, 2013, Април 1 19:59:50 EEST
>
>
>* Ивайло Димитров [130331 05:10]:
>>
>> Hi Pali,
>>
>> Yep, the code looks almost identical, I guess with some tweaks all SMC code from that patch could be removed and instead used the one from SMC PPA API.
>>
>> What I don't get, is why one needs to disable/enable fiqs/irqs:
>>
>> +static int call_sec_rom(u32 appl_id, u32 proc_id, u32 flag, ...)
>> +{
>> + va_list ap;
>> + u32 ret;
>> + u32 val;
>> +
>> + va_start(ap, flag);
>> + val = *(u32 *) ≈
>> + local_irq_disable();
>> + local_fiq_disable();
>> + ret = omap3_rom_rng_call(appl_id, proc_id, flag,
>> + (u32) virt_to_phys((void *) val));
>> + local_fiq_enable();
>> + local_irq_enable();
>> + va_end(ap);
>> +
>> + return ret;
>> +}
>>
>> Do you have any idea why is that needed? Any other code I've ever seen to call SM, does not disable fiqs/irqs, is RNG SMC somehow special? I know this is Nokia's code, but still, if we can get some understanding...
>
>Sounds like the fiq handling can be dropped based on what
>the TI guys posted recently about FIQs only be available in
>the secure mode starting with omap2.
>
>Regards,
>
>Tony
>


2013-04-02 16:17:26

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH] OMAP3 ROM Random Number Generator support

* Ивайло Димитров <[email protected]> [130401 13:41]:
> It could be that I am misreading something, but RX51 is OMAP3.

Yes the FIQs are only accessible in the secure mode starting with
omap2. So that also means omap3, 4, 5.

Regards,

Tony