lkml.org 
[lkml]   [2016]   [Oct]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 4/9] pinctrl: meson: allow gpio to request irq
    Date
    Hi Linus,

    Linus Walleij <linus.walleij@linaro.org> writes:

    > On Wed, Oct 26, 2016 at 4:22 PM, Jerome Brunet <jbrunet@baylibre.com> wrote:
    >> On Tue, 2016-10-25 at 20:20 +0200, Linus Walleij wrote:
    >
    >>> However the semantic is such, that it is not necessary to call
    >>> to_irq()
    >>> before using an IRQ: the irqchip and gpiochip abstractions should be
    >>> orthogonal.
    >>
    >> Linus,
    >>
    >> They are orthogonal. You can request an irq from the irqchip controller
    >> without the gpiochip, like any other irq controller.
    >
    > OK good, sorry if I'm stating the obvious.
    >
    >> irq_create_mapping (and irq_create_fwspec_mapping) internally calls
    >> irq_find_mapping. So if the mapping already exist (the irq is already
    >> used before calling to_irq), the existing mapping will be returned. The
    >> mapping will be actually created only if needed. It seems to be in line
    >> with your explanation, no ?
    >
    > Yes, but you want to call irq_create_mapping() in slowpath (irq setup)
    > and irq_find_mapping() in fastpath (irq handler). Else the first IRQ
    > may result in unwelcomed surprises.
    >
    >> There is really a *lot* of gpio drivers which use irq_create_mapping in
    >> the to_irq callback, are these all wrong ?
    >
    > Yes they are all wrong. They should all be using irq_find_mapping().

    So, dumb question from someone trying (but having a hard time) to follow
    and understand the rationale...

    If it's wrong enough to completely reject, why are changes still being
    merged that are doing it so wrong? (e.g. like this one[1], just merged
    for v4.9)

    Kevin

    [1] 0eb9f683336d pinctrl: Add IRQ support to STM32 gpios
    https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/pinctrl/stm32/pinctrl-stm32.c?id=0eb9f683336d7eb99a3b75987620417c574ffb57

    \
     
     \ /
      Last update: 2016-10-26 17:51    [W:2.646 / U:0.128 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site