Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1508538
| Path | csiph.com!news.redatomik.org!aioe.org!bofh.it!news.nic.it!robomod |
|---|---|
| From | Linus Walleij <linus.walleij@linaro.org> |
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq |
| Date | Tue, 25 Oct 2016 20:20:01 +0200 |
| Message-ID | <sweul-Lo-3@gated-at.bofh.it> (permalink) |
| References | <su0Yx-33U-25@gated-at.bofh.it> <su0Yz-33U-97@gated-at.bofh.it> <surcl-3mp-11@gated-at.bofh.it> <suDZU-3tk-23@gated-at.bofh.it> <sw63L-3ER-3@gated-at.bofh.it> <sw7jd-4nV-39@gated-at.bofh.it> <sw9El-641-1@gated-at.bofh.it> <swa7n-6eB-9@gated-at.bofh.it> <swaTM-6Oo-9@gated-at.bofh.it> <swbd8-6Vq-27@gated-at.bofh.it> |
| Dkim-Signature | v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=FtFF5c0wR/8FF4yGxW+D6KV222AusC89HSLlcjrGHv8=; b=Bw4SiW/Cb1jFiTQOGaj+/83+tCGWY8UfC6R/IH1yvOP7R3pzSyq4kt+rCEH6F6A0Ms c14cHDYXV55nkBGQp5Rv1u8wvZPnY1fnmbuN+UDj40TAlFT6aJfCXZ+3LwPZKEA63cK1 fhR0L4piqkfAZlK8o/VLlMnSxGhce16duKHng= |
| X-Google-Dkim-Signature | v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=FtFF5c0wR/8FF4yGxW+D6KV222AusC89HSLlcjrGHv8=; b=aDooltxc+HxwpGUidlCMUR4WpRBbXGS03u8IElL7dDlncuPwVeqgVN4klXG29n3lO0 85ylMZPKqnvAlsDoa6SwZlhNJgjdUaNzOhrO+bCM+Bi7DFzoXjR/fz0tZ8/6rTf1bQuP xI8HpeccOKgf3nhE+Bt1F1U8F0xcuOkYmB4YV3US46hlYG8qgWtrBZLA46EdOeuIEEha x1eFPF3iSYLwezxizDBtzo/h4cwzGbiGCeEV6Kdlb59nX31mBI2LA8UI+JuPQ13LCB7Z 3pna6XKezgAQrZAbbKl2/m73ODg249/22oLM055hhjGlwIPSH7psY/lOmqM2cNu6Ic+B cF9w== |
| X-Gm-Message-State | ABUngvfF8Y8bEq1bPf0oMMbKzJAcWroMxqpRCRA+vT6FMbECdSGQ/m524U3QstGIr6gtSVhmCnUsVLvTxDFQ0nXj |
| X-Received | by 10.55.3.67 with SMTP id 64mr1603159qkd.257.1477419001599; Tue, 25 Oct 2016 11:10:01 -0700 (PDT) |
| MIME-Version | 1.0 |
| Content-Type | text/plain; charset=UTF-8 |
| Sender | robomod@news.nic.it |
| List-ID | <linux-kernel.vger.kernel.org> |
| X-Mailing-List | linux-kernel@vger.kernel.org |
| Approved | robomod@news.nic.it |
| Lines | 29 |
| Organization | linux.* mail to news gateway |
| X-Original-Cc | Jerome Brunet <jbrunet@baylibre.com>, Carlo Caione <carlo@caione.org>, Kevin Hilman <khilman@baylibre.com>, "open list:ARM/Amlogic Meson..." <linux-amlogic@lists.infradead.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Thomas Gleixner <tglx@linutronix.de>, Jason Cooper <jason@lakedaemon.net>, Rob Herring <robh+dt@kernel.org>, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Russell King <linux@armlinux.org.uk> |
| X-Original-Date | Tue, 25 Oct 2016 20:10:00 +0200 |
| X-Original-Message-ID | <CACRpkdba_iaOK61KiQGvJZYuz7-PiBgtN4vxv__tJLsZ8ZVL=w@mail.gmail.com> |
| X-Original-References | <1476871709-8359-1-git-send-email-jbrunet@baylibre.com> <1476871709-8359-5-git-send-email-jbrunet@baylibre.com> <CACRpkdaCZPS45t3eTcjJExOu=YT_YkjNkdpg09jQqvLQOJH=Qw@mail.gmail.com> <1477040798.15560.96.camel@baylibre.com> <CACRpkdYUVXny57AC9rKX3VRAyooEk_2XLvqe9jOuT3rOaE75rg@mail.gmail.com> <ec7c6cc5-cf9a-9c75-4b07-e337064c7d6e@arm.com> <1477400900.2482.51.camel@baylibre.com> <dee26f0b-e175-f8a0-2b73-08e8e324841c@arm.com> <1477405332.2482.87.camel@baylibre.com> <bbc268c8-35c3-206a-46b1-62cebef174b2@arm.com> |
| X-Original-Sender | linux-kernel-owner@vger.kernel.org |
| Xref | csiph.com linux.kernel:1508538 |
Show key headers only | View raw
On Tue, Oct 25, 2016 at 4:47 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > On 25/10/16 15:22, Jerome Brunet wrote: >> There is a few problems to guarantee that gpio == hwirq. >> 1. We have 2 instances of pinctrl, to guarantee that the linux gpio >> number == hwirq, we would have to guarantee the order in which they are >> probed. At least this my understanding > > Maybe I wasn't clear enough, and my use of gpio is probably wrong. So > Linux has a gpio number, which is obviously an abstract number (just > like the Linux irq number). But the pad number, in the context of given > SoC, is constant. So we have: > > pad->gpio > hwirq->irq > > Why can't you have pad == hwirq, always? This is already what you have > in the irqchip driver. This would simplify a lot of things. My thought as well. We usually refer to the local numberspace on the GPIO controller as "offsets", so line offsets 0...31 on a gpiochip with 31 lines. The ngpio in struct gpio_chip is the number of lines on that controller, and should nominally map 1:1 to hwirq sources. Yours, Linus Walleij
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-19 17:30 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Linus Walleij <linus.walleij@linaro.org> - 2016-10-20 21:30 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-21 11:10 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Linus Walleij <linus.walleij@linaro.org> - 2016-10-25 11:20 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Marc Zyngier <marc.zyngier@arm.com> - 2016-10-25 12:40 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-25 15:10 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Marc Zyngier <marc.zyngier@arm.com> - 2016-10-25 15:40 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-25 16:30 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Marc Zyngier <marc.zyngier@arm.com> - 2016-10-25 16:50 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-25 17:40 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Linus Walleij <linus.walleij@linaro.org> - 2016-10-25 20:30 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-26 16:30 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Linus Walleij <linus.walleij@linaro.org> - 2016-10-26 16:40 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Kevin Hilman <khilman@baylibre.com> - 2016-10-26 18:00 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Linus Walleij <linus.walleij@linaro.org> - 2016-10-25 20:20 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Jerome Brunet <jbrunet@baylibre.com> - 2016-10-26 16:30 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Linus Walleij <linus.walleij@linaro.org> - 2016-10-26 16:50 +0200
Re: [PATCH 4/9] pinctrl: meson: allow gpio to request irq Thomas Gleixner <tglx@linutronix.de> - 2016-10-25 12:50 +0200
csiph-web