[lkml]   [2009]   [Nov]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] mfd/mc13783: near complete rewrite
    On Fri, Oct 23, 2009 at 10:38:43PM +0200, Uwe Kleine-K??nig wrote:

    > -int mc13783_register_irq(struct mc13783 *mc13783, int irq,
    > - void (*handler) (int, void *), void *data)
    > +int mc13783_irq_request_nounmask(struct mc13783 *mc13783, unsigned int irq,
    > + irqreturn_t (*handler)(struct mc13783 *, unsigned int, void *),
    > + const char *name, void *dev)

    If you're changing the signature of the IRQ handler functions it'd be
    nice to change them to irq_handler_t - that way it'll be much easier to
    transition the driver to using genirq in future since it should end up
    being possible to just stub out the Atlas-specific calls in the header
    with calls to the standard IRQ functions when the core is transitioned,
    reducing cross-tree issues.

    This might create issues with a request_nounmask() function, though a
    request plus mask is probably enough - I guess you're using this for
    your the RTC driver in which case a spurious periodic interrupt is
    unlikely to be an issue.

     \ /
      Last update: 2009-11-02 12:53    [W:0.021 / U:110.324 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site