From: Linus Walleij <linus.walleij@linaro.org> To: Mark Kettenis <mark.kettenis@xs4all.nl> Cc: kettenis@openbsd.org, Krzysztof Kozlowski <krzk@kernel.org>, Tomasz Figa <tomasz.figa@gmail.com>, Marc Zyngier <maz@kernel.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, Hector Martin <marcan@marcan.st>, Rob Herring <robh+dt@kernel.org>, Linux ARM <linux-arm-kernel@lists.infradead.org>, "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>, linux-kernel <linux-kernel@vger.kernel.org>, Bartosz Golaszewski <bgolaszewski@baylibre.com>, sven@svenpeter.dev Subject: Re: [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Date: Sun, 9 May 2021 02:27:40 +0200 [thread overview] Message-ID: <CACRpkdb=0EQN=CJqfjKS-iuAiKCvU38fw5krzEY5LvhNpyFd3w@mail.gmail.com> (raw) In-Reply-To: <c1bd678c5dc81db8@bloch.sibelius.xs4all.nl> On Sun, May 9, 2021 at 1:02 AM Mark Kettenis <mark.kettenis@xs4all.nl> wrote: > [Me] > > On Sat, May 8, 2021 at 4:20 PM Mark Kettenis <kettenis@openbsd.org> wrote: > My U-Boot driver is here: Thanks! Looks nice. > > > +description: | > > > + The Apple GPIO controller is a simple combined pin and GPIO controller > > > > spelling > > Not sure I'm seeing a spelling mistake here. Do you want a comma > inserted somewhere? Your original mail says "conroller" but the helpful google mail editor autocorrected the mistake when I hit enter after it. > > So is this an entirely Apple thing now, and not based on some Samsung > > block from S3C like what we have seen before? > > As far as I can tell, yes. This Apple controller has a single > register per pin that controls the muxing and gpio functions, whereas > the S3C controller seems to have 4 registers per pin. Fair enough. > > What I am really wondering is if these interrupts are hierarchical, > > i.e. that they match 1-to-1 to a GPIO line. > > They don't match 1-1. The GPIOs can be assigned to one of > (apparently) 7 interrupt groups. Aha so it is a 1-to-1..* thing. How delicate. > I haven't looked to closely at this > yet since U-Boot doesn't need/use the interrupt capability. But I > suspect that pins don't have to be assigned to a interrupt group and > that explains why there are only 7 interrupt groups as the 8th state > is reserved for "unasigned". The number of pins per controller > varies, but one of them has 212 pins. Wow. > Multiple pins can be assigned to the same interrupt group as far as I > can tell. So in that case the driver will have to look at status > bits. OK then this is not hierarchical. > > Marc Zyngier can probably tell the story of why it is handled > > like this, > > Ok, hopefully Marc can say something sensible here, but I'd say the > interrupts on this hardware are cascaded. Yes looks like so, it will be an interesting interrupt driver when you get to that. I have only the question in my second mail (just sent) but in any case you are not doing anything out of the ordinary (it looks very similar to the STM32) so I'm pleased with this binding. I wanna give the DT reviewers some time to look at it as well but I imagine we can soon merge this. Yours. Linus Walleij
WARNING: multiple messages have this Message-ID (diff)
From: Linus Walleij <linus.walleij@linaro.org> To: Mark Kettenis <mark.kettenis@xs4all.nl> Cc: kettenis@openbsd.org, Krzysztof Kozlowski <krzk@kernel.org>, Tomasz Figa <tomasz.figa@gmail.com>, Marc Zyngier <maz@kernel.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, Hector Martin <marcan@marcan.st>, Rob Herring <robh+dt@kernel.org>, Linux ARM <linux-arm-kernel@lists.infradead.org>, "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>, linux-kernel <linux-kernel@vger.kernel.org>, Bartosz Golaszewski <bgolaszewski@baylibre.com>, sven@svenpeter.dev Subject: Re: [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Date: Sun, 9 May 2021 02:27:40 +0200 [thread overview] Message-ID: <CACRpkdb=0EQN=CJqfjKS-iuAiKCvU38fw5krzEY5LvhNpyFd3w@mail.gmail.com> (raw) In-Reply-To: <c1bd678c5dc81db8@bloch.sibelius.xs4all.nl> On Sun, May 9, 2021 at 1:02 AM Mark Kettenis <mark.kettenis@xs4all.nl> wrote: > [Me] > > On Sat, May 8, 2021 at 4:20 PM Mark Kettenis <kettenis@openbsd.org> wrote: > My U-Boot driver is here: Thanks! Looks nice. > > > +description: | > > > + The Apple GPIO controller is a simple combined pin and GPIO controller > > > > spelling > > Not sure I'm seeing a spelling mistake here. Do you want a comma > inserted somewhere? Your original mail says "conroller" but the helpful google mail editor autocorrected the mistake when I hit enter after it. > > So is this an entirely Apple thing now, and not based on some Samsung > > block from S3C like what we have seen before? > > As far as I can tell, yes. This Apple controller has a single > register per pin that controls the muxing and gpio functions, whereas > the S3C controller seems to have 4 registers per pin. Fair enough. > > What I am really wondering is if these interrupts are hierarchical, > > i.e. that they match 1-to-1 to a GPIO line. > > They don't match 1-1. The GPIOs can be assigned to one of > (apparently) 7 interrupt groups. Aha so it is a 1-to-1..* thing. How delicate. > I haven't looked to closely at this > yet since U-Boot doesn't need/use the interrupt capability. But I > suspect that pins don't have to be assigned to a interrupt group and > that explains why there are only 7 interrupt groups as the 8th state > is reserved for "unasigned". The number of pins per controller > varies, but one of them has 212 pins. Wow. > Multiple pins can be assigned to the same interrupt group as far as I > can tell. So in that case the driver will have to look at status > bits. OK then this is not hierarchical. > > Marc Zyngier can probably tell the story of why it is handled > > like this, > > Ok, hopefully Marc can say something sensible here, but I'd say the > interrupts on this hardware are cascaded. Yes looks like so, it will be an interesting interrupt driver when you get to that. I have only the question in my second mail (just sent) but in any case you are not doing anything out of the ordinary (it looks very similar to the STM32) so I'm pleased with this binding. I wanna give the DT reviewers some time to look at it as well but I imagine we can soon merge this. Yours. Linus Walleij _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-05-09 0:27 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-08 14:19 [PATCH 0/2] Apple M1 pinctrl DT bindings Mark Kettenis 2021-05-08 14:19 ` Mark Kettenis 2021-05-08 14:19 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Mark Kettenis 2021-05-08 14:19 ` Mark Kettenis 2021-05-08 21:09 ` Linus Walleij 2021-05-08 21:09 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Linus Walleij 2021-05-08 23:02 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Mark Kettenis 2021-05-08 23:02 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Mark Kettenis 2021-05-09 0:27 ` Linus Walleij [this message] 2021-05-09 0:27 ` Linus Walleij 2021-05-09 9:50 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Tomasz Figa 2021-05-09 9:50 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Tomasz Figa 2021-05-10 13:09 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Krzysztof Kozlowski 2021-05-10 13:09 ` Krzysztof Kozlowski 2021-05-09 0:18 ` Linus Walleij 2021-05-09 0:18 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Linus Walleij 2021-05-09 9:46 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Mark Kettenis 2021-05-09 9:46 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Mark Kettenis 2021-05-09 10:49 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Linus Walleij 2021-05-09 10:49 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Linus Walleij 2021-05-10 13:03 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Krzysztof Kozlowski 2021-05-10 13:03 ` Krzysztof Kozlowski 2021-05-10 17:09 ` Mark Kettenis 2021-05-10 17:09 ` Mark Kettenis 2021-05-10 14:01 ` Rob Herring 2021-05-10 14:01 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Rob Herring 2021-05-10 18:18 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Rob Herring 2021-05-10 18:18 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Rob Herring 2021-05-10 14:19 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Rob Herring 2021-05-10 14:19 ` Rob Herring 2021-05-10 17:06 ` Mark Kettenis 2021-05-10 17:06 ` Mark Kettenis 2021-05-19 23:27 ` Linus Walleij 2021-05-19 23:27 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Linus Walleij 2021-05-20 11:22 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple,pinctrl Mark Kettenis 2021-05-20 11:22 ` [PATCH 1/2] dt-bindings: pinctrl: Add DT bindings for apple, pinctrl Mark Kettenis 2021-05-08 14:19 ` [PATCH 2/2] arm64: apple: Add pinctrl nodes Mark Kettenis 2021-05-08 14:19 ` Mark Kettenis
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CACRpkdb=0EQN=CJqfjKS-iuAiKCvU38fw5krzEY5LvhNpyFd3w@mail.gmail.com' \ --to=linus.walleij@linaro.org \ --cc=bgolaszewski@baylibre.com \ --cc=devicetree@vger.kernel.org \ --cc=kettenis@openbsd.org \ --cc=krzk@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-gpio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=marcan@marcan.st \ --cc=mark.kettenis@xs4all.nl \ --cc=maz@kernel.org \ --cc=robh+dt@kernel.org \ --cc=sven@svenpeter.dev \ --cc=tomasz.figa@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.