Hi Antoine,
On 22/05/17 15:30, Antoine Tenart wrote:
> Hi Marc,
>
> On Wed, May 03, 2017 at 05:36:38PM +0100, Marc Zyngier wrote:
>> On 24/04/17 08:54, Antoine Tenart wrote:
>>> +
>>> + crypto: crypto@800000 {
>>> + compatible = "inside-secure,safexcel-eip197";
>>> + reg = <0x800000 0x200000>;
>>> + interrupts = <GIC_SPI 34 (IRQ_TYPE_EDGE_RISING | IRQ_TYPE_LEVEL_HIGH)>,
>>
>> I'm puzzled. How can the interrupt can be both level *and* edge? That
>> doesn't make any sense.
>
> I agree this looks odd. I took it from Russel's ICU mapping:
> http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/489040.html
This emails says:
ICU-irq => GIC-SPI-num Enable Edge/Level ICU-group
[...]
24 => 34 En Lv 0
which seems to clearly indicate Level (no Edge involved here).
> The driver does not use this interrupt (yet?). I'm not sure what is this
> interrupt exact configuration...
Surely the TRM would indicate this, wouldn't it?
> Do you want me to remove this from the documentation (and device trees)?
I'd rather you clarify it, if possible.
Thanks,
M.
--
Jazz is not dead. It just smells funny...
