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 D02D11CF1C7 for ; Tue, 15 May 2018 04:57:35 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id CCBBB85DF2 for ; Tue, 15 May 2018 04:57:35 +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 wbrj9lZOlSuZ for ; Tue, 15 May 2018 04:57:35 +0000 (UTC) Received: from mail-wm0-f67.google.com (mail-wm0-f67.google.com [74.125.82.67]) by fraxinus.osuosl.org (Postfix) with ESMTPS id E8CB485DE0 for ; Tue, 15 May 2018 04:57:34 +0000 (UTC) Received: by mail-wm0-f67.google.com with SMTP id a67-v6so16935291wmf.3 for ; Mon, 14 May 2018 21:57:34 -0700 (PDT) Date: Tue, 15 May 2018 06:57:30 +0200 From: Sergio Paracuellos Subject: Re: [PATCH 0/5] staging: mt7621-gpio: some driver cleanups Message-ID: <20180515045730.GC17051@foobar> References: <1526306554-12168-1-git-send-email-sergio.paracuellos@gmail.com> <87mux17mvz.fsf@notabene.neil.brown.name> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <87mux17mvz.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:01:20AM +1000, NeilBrown wrote: > On Mon, May 14 2018, Sergio Paracuellos wrote: > > > The following patch series fix all remaining checkpatch complains > > about this driver. Changes have not been tested and also compiled > > but it should not have any problem about them. > > Thanks for these. As you say, nothing in them could change behaviour of > the driver, but I tested them anyway and gpio still works (both in and > out) - no surprises. Thanks for testing this and review. > > I can give > Reviewed-by: NeilBrown > for all exect "dt-bindings: gpio: add documentation for mt7621-gpio". > I'll reply to the separately. I have just sent v2 for all of this series and also replied to your comments separately. Hope this helps. > > Of course the dt-binding patches will need broader review once they seem > ready to us. > > Thanks, > NeilBrown Best regards, Sergio Paracuellos > > > > > > Sergio Paracuellos (5): > > staging: mt7621-gpio: fix some warnings because of lines exceded 80 > > characters > > staging: mt7621-gpio: add SPDX identifier > > dt-bindings: add compatible string for 'mtk' MediaTek > > dt-bindings: gpio: add documentation for mt7621-gpio > > staging: mt7621-gpio: remove device tree related stuff from TODO file > > > > .../devicetree/bindings/gpio/mtk,mt7621-gpio.txt | 51 ++++++++++++++++++++++ > > .../devicetree/bindings/vendor-prefixes.txt | 1 + > > drivers/staging/mt7621-gpio/TODO | 1 - > > drivers/staging/mt7621-gpio/gpio-mt7621.c | 24 +++++----- > > 4 files changed, 66 insertions(+), 11 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/gpio/mtk,mt7621-gpio.txt > > > > -- > > 2.7.4 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel