From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by ash.osuosl.org (Postfix) with ESMTP id 271B51CF1C8 for ; Tue, 15 May 2018 04:54:04 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id 2315485EA3 for ; Tue, 15 May 2018 04:54:04 +0000 (UTC) Received: from fraxinus.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id pCuyetdAVH-M for ; Tue, 15 May 2018 04:54:03 +0000 (UTC) Received: from mail-wr0-f194.google.com (mail-wr0-f194.google.com [209.85.128.194]) by fraxinus.osuosl.org (Postfix) with ESMTPS id 376CB85A44 for ; Tue, 15 May 2018 04:54:03 +0000 (UTC) Received: by mail-wr0-f194.google.com with SMTP id g21-v6so14435207wrb.8 for ; Mon, 14 May 2018 21:54:03 -0700 (PDT) Date: Tue, 15 May 2018 06:53:59 +0200 From: Sergio Paracuellos Subject: Re: [PATCH 4/5] dt-bindings: gpio: add documentation for mt7621-gpio Message-ID: <20180515045359.GA17051@foobar> References: <1526306554-12168-1-git-send-email-sergio.paracuellos@gmail.com> <1526306554-12168-5-git-send-email-sergio.paracuellos@gmail.com> <87k1s57lyn.fsf@notabene.neil.brown.name> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <87k1s57lyn.fsf@notabene.neil.brown.name> List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: NeilBrown Cc: gregkh@linuxfoundation.org, driverdev-devel@linuxdriverproject.org On Tue, May 15, 2018 at 11:21:20AM +1000, NeilBrown wrote: > On Mon, May 14 2018, Sergio Paracuellos wrote: > > > This commit add missing dt bindings documentation for mt7621-gpio > > driver. After this checkpatch script complain about this > > issue dissapears. > > > > Signed-off-by: Sergio Paracuellos > > --- > > .../devicetree/bindings/gpio/mtk,mt7621-gpio.txt | 51 ++++++++++++++++++++++ > > 1 file changed, 51 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/gpio/mtk,mt7621-gpio.txt > > > > diff --git a/Documentation/devicetree/bindings/gpio/mtk,mt7621-gpio.txt b/Documentation/devicetree/bindings/gpio/mtk,mt7621-gpio.txt > > new file mode 100644 > > index 0000000..5fe4bb5 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/gpio/mtk,mt7621-gpio.txt > > @@ -0,0 +1,51 @@ > > +Mediatek SoC GPIO controller bindings > > + > > +The IP core used inside these SoCs has 3 banks of 32 GPIOs each. > > +The registers of all the banks are interwoven inside one single IO range. > > +We load one GPIO controller instance per bank. To make this possible > > +we support 2 types of nodes. The parent node defines the memory I/O range and > > +has 3 children each describing a single bank. > > + > > +Required properties for the top level node: > > +- compatible: > > + - "mtk,mt7621-gpio" for Mediatek controllers > > +- reg : Physical base address and length of the controller's registers > > + > > +Required properties for the GPIO bank node: > > +- compatible: > > + - "mtk,mt7621-gpio-bank" for Mediatek banks > > +- #gpio-cells : Should be two. > > + - first cell is the pin number > > + - second cell is used to specify optional parameters (unused) > > +- gpio-controller : Marks the device node as a GPIO controller > > +- reg : The id of the bank that the node describes. > > This is really good, but not quite complete. > Searching for "of_" in gpio-mt7621.c I find code handling everything > you've described, but also: > > mediatek_gpio_irq = irq_of_parse_and_map(np, 0); > if (mediatek_gpio_irq) { > mediatek_gpio_irq_domain = irq_domain_add_linear(np, > > The GPIO controller can receive interrupts on any of the GPIOs, > either edge or level. It then interrupts the CPU using GIC INT12. > > so > interrupt-parent = <&gic>; > interrupts = > (I think). > Then you need whatever irq_domain_add_linear() expects. > I don't know what that is... I tried following through > code and got lost in little twisty mazes. > > So if you change this patch to add the file to > drivers/staging/mt7621-gpio > then I can give it > Reviewed-by: NeilBrown I moved this to the driver directory as it is because dtsi is not including yet stuff about gpio interrupts. This should be added when make sure that interrupt works and gtsi will be updated. > > and then we can fix it when an understanding of the interrupts is > available. > But I cannot approve it for Documentation/devicetree/bindings yet. > > Thanks a lot, > NeilBrown > Best regards, Sergio PAracuellos > > > + > > +Example: > > + gpio@600 { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + compatible = "mtk,mt7621-gpio"; > > + reg = <0x600 0x100>; > > + > > + gpio0: bank@0 { > > + reg = <0>; > > + compatible = "mtk,mt7621-gpio-bank"; > > + gpio-controller; > > + #gpio-cells = <2>; > > + }; > > + > > + gpio1: bank@1 { > > + reg = <1>; > > + compatible = "mtk,mt7621-gpio-bank"; > > + gpio-controller; > > + #gpio-cells = <2>; > > + }; > > + > > + gpio2: bank@2 { > > + reg = <2>; > > + compatible = "mtk,mt7621-gpio-bank"; > > + gpio-controller; > > + #gpio-cells = <2>; > > + }; > > + }; > > -- > > 2.7.4 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel