[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHp75Vdm4q39hoXoodDQPwEdhi-C+jF7yaxHqHi41mrT0kXGnA@mail.gmail.com>
Date: Wed, 16 May 2018 22:53:18 +0300
From: Andy Shevchenko <andy.shevchenko@...il.com>
To: "H. Nikolaus Schaller" <hns@...delico.com>
Cc: Kumar Gala <galak@...eaurora.org>,
Rob Herring <robh+dt@...nel.org>,
Pawel Moll <pawel.moll@....com>,
Mark Rutland <mark.rutland@....com>,
Ian Campbell <ijc+devicetree@...lion.org.uk>,
Linus Walleij <linus.walleij@...aro.org>,
Alexandre Courbot <gnurou@...il.com>,
devicetree <devicetree@...r.kernel.org>,
"open list:GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Discussions about the Letux Kernel
<letux-kernel@...nphoenux.org>, kernel@...a-handheld.com
Subject: Re: [PATCH v6 0/3] pcal6524 extensions and fixes for pca953x driver
On Wed, May 16, 2018 at 8:01 PM, H. Nikolaus Schaller <hns@...delico.com> wrote:
> V6:
> * added proper attribution to the formula used for fixing the
> pcal6524 register address (changes commit message only)
> * add back missing first patch from V2 that defines the
> PCA_LATCH_INT constant
> * removed patches already merged
>
Thanks for an update.
I think we still need to address the constant representation in patch 2.
For the rest take my
Reviewed-by: Andy Shevchenko <andy.shevchenko@...il.com>
> 2018-04-28 18:33:42: V5:
> * fix wrong split up between patches 1/7 and 2/7.
>
> 2018-04-26 19:35:07: V4:
> * introduced PCA_LATCH_INT constant to make of_table more
> readable (suggested by Andy Shevchenko)
> * converted all register constants to hex in a separate
> patch (suggested by Andy Shevchenko)
> * separated additional pcal953x and pcal6524 register
> definitions into separate patches (suggested by Andy Shevchenko)
> * made special pcal6524 address adjustment more readable
> (suggested by Andy Shevchenko)
> * moved gpio-controller and interrupt-controller to the
> "required" section (reviewed by Rob Herring)
>
> 2018-04-10 18:07:07: V3:
> * add Reported-by: and Reviewed-by:
> * fix wording for bindings description and example
> * convert all register offsets to hex
> * omit the LEVEL-IRQ RFC/hack commit
>
> 2018-04-04 21:00:27: V2:
> * added PCA_PCAL flags if matched through of-table
> * fix address calculation for extended PCAL6524 registers
> * hack to map LEVEL_LOW to EDGE_FALLING to be able to
> test in combination with ts3a227e driver
> * improve description of bindings for optional vcc-supply
> and interrupt-controller;
>
> 2018-03-10 09:32:53: no initial description
>
> H. Nikolaus Schaller (3):
> gpio: pca953x: set the PCA_PCAL flag also when matching by DT
> gpio: pca953x: define masks for addressing common and extended
> registers
> gpio: pca953x: fix address calculation for pcal6524
>
> drivers/gpio/gpio-pca953x.c | 17 +++++++++++++----
> 1 file changed, 13 insertions(+), 4 deletions(-)
>
> --
> 2.12.2
>
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists