lkml.org 
[lkml]   [2013]   [Apr]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2013-04-01 23:21    [W:0.037 / U:2.972 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site