[lkml]   [2005]   [Nov]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: Fwd: [RFC] IRQ type flags
    On Sun, Nov 06, 2005 at 10:59:58PM +0000, Alan Cox wrote:
    > On Sul, 2005-11-06 at 22:16 +0000, Russell King wrote:
    > > > This is actually true of some x86 hardware in the EISA space where there
    > > > is a control register for level v edge that we sort of half deal with.
    > >
    > > Thanks Alan. Can I assume you're happy with the patch, even if x86
    > > currently ignores the flags?
    > I'm certainly happy with it. Both the APIC and EISA IRQ control
    > registers could even be made to honour it if anyone ever needed to.
    > Should platforms that don't support the flags be patched to error
    > requests they don't support however ?

    Good question - I'm not sure currently. In the places where set_irq_type
    is used on ARM, we're mainly interested in setting the input according
    to rising/falling edge or high/low levels rather than switching between
    edge and level mode.

    We could do as you suggest, but my concern would be adding extra
    complexity to drivers, causing them to do something like:

    ret = request_irq(..., SA_TRIGGER_HIGH, ...);
    if (ret == -E<whatever>)
    ret = request_irq(..., SA_TRIGGER_RISING, ...);

    The alternative is:

    ret = request_irq(..., SA_TRIGGER_HIGH | SA_TRIGGER_RISING, ...);

    at which point the driver is telling the IRQ layer what it can support,
    and allowing the IRQ layer to select the most appropriate type given
    the driver and hardware constraints.

    Russell King
    Linux kernel 2.6 ARM Linux -
    maintainer of: 2.6 Serial core
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-11-06 23:44    [W:0.019 / U:72.872 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site