Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 31 May 2023 11:00:03 -0400
From:   Hugo Villeneuve <hugo@hugovil.com>
To:     Hugo Villeneuve <hugo@hugovil.com>
Cc:     Lech Perczak <lech.perczak@camlingroup.com>,
        gregkh@linuxfoundation.org, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
        jirislaby@kernel.org, jringle@gridpoint.com,
        l.perczak@camlintechnologies.com, tomasz.mon@camlingroup.com,
        linux-serial@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org,
        Hugo Villeneuve <hvilleneuve@dimonoff.com>
Message-Id: <20230531110003.86b96815986db3bef7b72635@hugovil.com>
In-Reply-To: <20230531095608.e1b4358549562b8e1bdf5cb4@hugovil.com>
References: <20230529140711.896830-1-hugo@hugovil.com>
        <c15a90d6-b3c1-e432-9216-c4c1e2c44ce6@camlingroup.com>
        <20230530090836.27b8d080d6b6c022b303ac9e@hugovil.com>
        <c691858d-31af-2892-c0a3-89a37b19af86@camlingroup.com>
        <20230531095608.e1b4358549562b8e1bdf5cb4@hugovil.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PATCH v4 0/9] serial: sc16is7xx: fix GPIO regression and rs485
 improvements
Precedence: bulk

On Wed, 31 May 2023 09:56:08 -0400
Hugo Villeneuve <hugo@hugovil.com> wrote:

> On Wed, 31 May 2023 12:43:48 +0200
> Lech Perczak <lech.perczak@camlingroup.com> wrote:
>=20
> > W dniu 30.05.2023 o=A015:08, Hugo Villeneuve pisze:
> > > On Tue, 30 May 2023 11:30:07 +0200
> > > Lech Perczak <lech.perczak@camlingroup.com> wrote:
> > >
> > > > W dniu 29.05.2023 o=A016:07, Hugo Villeneuve pisze:
> > > > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com>
> > > > >
> > > > > Hello,
> > > > > this patch series mainly fixes a GPIO regression and improve RS48=
5 flags and
> > > > > properties detection from DT.
> > > > >
> > > > > It now also includes various small fixes and improvements that we=
re previously
> > > > > sent as separate patches, but that made testing everything diffic=
ult.
> > > > >
> > > > > Patch 1 fixes an issue when debugging IOcontrol register. After t=
esting the GPIO
> > > > > regression patches (patches 6 and 7, tests done by Lech Perczak),=
 it appers that
> > > > > this patch is also necessary for having the correct IOcontrol reg=
ister values.
> > > > >
> > > > > Patch 2 introduces a delay after a reset operation to respect dat=
asheet
> > > > > timing recommandations.
> > > > >
> > > > > Patch 3 fixes an issue with init of first port during probing.
> > > > >
> > > > > Patch 4 fixes a bug with the output value when first setting the =
GPIO direction.
> > > > >
> > > > > Patch 5 is a refactor of GPIO registration code.
> > > > >
> > > > > Patches 6 and 7 fix a GPIO regression by (re)allowing to choose G=
PIO function
> > > > > for GPIO pins shared with modem status lines.
> > > > >
> > > > > Patch 8 allows to read common rs485 device-tree flags and propert=
ies.
> > > > >
> > > > > Patch 9 improves comments about chip variants.
> > > > >
> > > > > I have tested the changes on a custom board with two SC16IS752 DU=
ART using a
> > > > > Variscite IMX8MN NANO SOM.
> > > > >
> > > > > Thank you.
> > > > >
> > > > > Link: [v1] https://lkml.org/lkml/2023/5/17/967 <https://lkml.org/=
lkml/2023/5/17/967> <https://lkml.org/lkml/2023/5/17/967 <https://lkml.org/=
lkml/2023/5/17/967>>
> > > > > [v1] https://lkml.org/lkml/2023/5/17/777 <https://lkml.org/lkml/2=
023/5/17/777> <https://lkml.org/lkml/2023/5/17/777 <https://lkml.org/lkml/2=
023/5/17/777>>
> > > > > [v1] https://lkml.org/lkml/2023/5/17/780 <https://lkml.org/lkml/2=
023/5/17/780> <https://lkml.org/lkml/2023/5/17/780 <https://lkml.org/lkml/2=
023/5/17/780>>
> > > > > [v1] https://lkml.org/lkml/2023/5/17/785 <https://lkml.org/lkml/2=
023/5/17/785> <https://lkml.org/lkml/2023/5/17/785 <https://lkml.org/lkml/2=
023/5/17/785>>
> > > > > [v1] https://lkml.org/lkml/2023/5/17/1311 <https://lkml.org/lkml/=
2023/5/17/1311> <https://lkml.org/lkml/2023/5/17/1311 <https://lkml.org/lkm=
l/2023/5/17/1311>>
> > > > > [v2] https://lkml.org/lkml/2023/5/18/516 <https://lkml.org/lkml/2=
023/5/18/516> <https://lkml.org/lkml/2023/5/18/516 <https://lkml.org/lkml/2=
023/5/18/516>>
> > > > > [v3] https://lkml.org/lkml/2023/5/25/7 <https://lkml.org/lkml/202=
3/5/25/7> <https://lkml.org/lkml/2023/5/25/7 <https://lkml.org/lkml/2023/5/=
25/7>>
> > > > >
> > > > > Changes for V3:
> > > > > - Integrated all patches into single serie to facilitate debuggin=
g and tests.
> > > > > - Reduce number of exported GPIOs depending on new property
> > > > > nxp,modem-control-line-ports
> > > > > - Added additional example in DT bindings
> > > > >
> > > > > Changes for V4:
> > > > > - Increase reset post delay to relax scheduler.
> > > > > - Put comments patches at the end.
> > > > > - Remove Fixes tag for patch "mark IOCONTROL register as volatile=
".
> > > > > - Improve commit messages after reviews.
> > > > > - Fix coding style issues after reviews.
> > > > > - Change GPIO registration to always register the maximum number =
of GPIOs
> > > > > supported by the chip, but maks-out GPIOs declared as modem contr=
ol lines.
> > > > > - Add patch to refactor GPIO registration.
> > > > > - Remove patch "serial: sc16is7xx: fix syntax error in comments".
> > > > > - Remove patch "add dump registers function"
> > > > >
> > > > > Hugo Villeneuve (9):
> > > > > serial: sc16is7xx: mark IOCONTROL register as volatile
> > > > > serial: sc16is7xx: add post reset delay
> > > > > serial: sc16is7xx: fix broken port 0 uart init
> > > > > serial: sc16is7xx: fix bug when first setting GPIO direction
> > > > > serial: sc16is7xx: refactor GPIO controller registration
> > > > > dt-bindings: sc16is7xx: Add property to change GPIO function
> > > > > serial: sc16is7xx: fix regression with GPIO configuration
> > > > > serial: sc16is7xx: add call to get rs485 DT flags and properties
> > > > > serial: sc16is7xx: improve comments about variants
> > > > >
> > > > > .../bindings/serial/nxp,sc16is7xx.txt | 46 ++++++
> > > > > drivers/tty/serial/sc16is7xx.c | 150 +++++++++++++-----
> > > > > 2 files changed, 156 insertions(+), 40 deletions(-)
> > > > >
> > > > >
> > > > > base-commit: 8b817fded42d8fe3a0eb47b1149d907851a3c942
> > > >
> > > > It would be a lot of sending, to do that for every patch separately=
, so for whole series:
> > > > Reviewed-by: Lech Perczak <lech.perczak@camlingroup.com>
> > > >
> > > > And where applicable - for code patches:
> > > > Tested-by: Lech Perczak <lech.perczak@camlingroup.com>
> > > >
> > > > I tested whole series at the same time.
> > > > I did my tests on an i.MX6 board with SC16IS760 over SPI, which dif=
fers a tiny bit from SC16IS752,
> > > > and everything works as it should.
> > > > Thank you for fixing this!
> > >
> > > Hi Lech,
> > > thank for your feedback.
> > >
> > > You mentioned before that without the patch "mark IOCONTROL register =
as volatile", things were not working properly for you. Could you retest by=
 removing this patch and see if things are still working?
> > >
> > > Thank you, Hugo.
> >=20
> > Hello Hugo,
> >=20
> > Just checked - this patch is required, reverting it causes things to fa=
il, so this patch should be marked as a pre-requisite for the actual fix an=
d included in backports.
> > Perhaps using direct write to this register made it work, but it was li=
kely by accident.
>=20
> Hi Lech,
> thank you for the test, I will mark it as such in upcoming series V5.
>=20
> Hugo.

Since I reworked a bit patch 5/9 in series 5, I removed your "Tested-by" an=
d "Reviewed-by" tags only for this patch. Please reconfirm these tags when =
you have tested series 5.

Thank you,
Hugo.

--=20
Hugo Villeneuve <hugo@hugovil.com>