Re: [PATCH] serial: sc16is7xx: fix missing requested irq type setting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Alexander,

On 01.10.2015 10:42, Alexander Shiyan wrote:
>> Среда, 30 сентября 2015, 23:57 +03:00 от Vladimir Zapolskiy <vz@xxxxxxxxx>:
>>
>> Hardcode IC irq type to IRQF_TRIGGER_FALLING for both platform and OF
>> cases. According to the datasheet IC irq line is active low, reflect
>> this fact in the driver.
> 
> Hello.
> 
> On my opinion, the better way is use IRQ_TYPE_LEVEL_* definitions in the second
> interrupt cell in DTS. The interrupt line can be connected through external invertor,
> so we can handle all possible situations.

the external inverter you are talking about is a separate piece of hardware,
and in my opinion there is no need to describe its hypothetical presence
on board
in the sc16is7xx device driver.

--
With best wishes,
Vladimir
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux