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:   Mon, 27 Jun 2022 07:35:47 -0400
From:   William Breathitt Gray <william.gray@linaro.org>
To:     Bartosz Golaszewski <brgl@bgdev.pl>
Cc:     Linus Walleij <linus.walleij@linaro.org>,
        "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/5] gpio: Implement and utilize register structures for
 ISA drivers
Message-ID: <YrmWEwlxyJzj2zmZ@fedora>
References: <cover.1654525394.git.william.gray@linaro.org>
 <CAMRc=MfoMjFV8EsDeXuHEFjOS2m7r4fW7BqETCDrcjTHVgzDbQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="qcqNbohE9h+Wv+21"
Content-Disposition: inline
In-Reply-To: <CAMRc=MfoMjFV8EsDeXuHEFjOS2m7r4fW7BqETCDrcjTHVgzDbQ@mail.gmail.com>
Precedence: bulk


--qcqNbohE9h+Wv+21
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Jun 09, 2022 at 04:39:41PM +0200, Bartosz Golaszewski wrote:
> On Mon, Jun 6, 2022 at 4:36 PM William Breathitt Gray
> <william.gray@linaro.org> wrote:
> >
> > The PC104/ISA drivers were updated to use I/O memory accessor calls such
> > as ioread8()/iowrite8() in a previous patch series [1]. This
> > patchset is a continuation of the effort to improve the code readability
> > and reduce magic numbers by implementing and utilizing named register
> > data structures.
> >
> > One of the benefits is that we can now observe more easily similarities
> > in devices that share similar interfaces; such as the i8255 interfaces
> > used by the 104-DIO-48E, 104-IDI-48, and GPIO-MM drivers -- as well as
> > the similar interface used by the 104-IDIO-16 and PCI-IDIO-16 drivers. I
> > hope to consolidate some of these code blocks in future patchsets.
> >
> > [1] https://lore.kernel.org/all/cover.1652201921.git.william.gray@linar=
o.org/
> >
> > William Breathitt Gray (5):
> >   gpio: 104-dio-48e: Implement and utilize register structures
> >   gpio: 104-idi-48: Implement and utilize register structures
> >   gpio: 104-idio-16: Implement and utilize register structures
> >   gpio: gpio-mm: Implement and utilize register structures
> >   gpio: ws16c48: Implement and utilize register structures
> >
> >  drivers/gpio/gpio-104-dio-48e.c | 157 +++++++++++++++++++++-----------
> >  drivers/gpio/gpio-104-idi-48.c  | 128 +++++++++++++-------------
> >  drivers/gpio/gpio-104-idio-16.c |  58 ++++++++----
> >  drivers/gpio/gpio-gpio-mm.c     | 116 ++++++++++++++---------
> >  drivers/gpio/gpio-ws16c48.c     | 119 +++++++++++++++++-------
> >  5 files changed, 366 insertions(+), 212 deletions(-)
> >
> >
> > base-commit: f2906aa863381afb0015a9eb7fefad885d4e5a56
> > --
> > 2.36.1
> >
>=20
> Hi William,
>=20
> Unlike the previous patches which were relatively simple, these seem
> like there's a lot of space for breakage (even though they're
> attempting to do a good thing). Have you tested the code on real
> hardware?
>=20
> Bart

I have a tester testing the changes for to the 104-IDIO-16 and GPIO-MM
GPIO drivers on real hardware. I don't have access to the other three
devices so I've only compile tested and loaded them in a VM to verify;
I'll send out a request to ACCESIO and WinSystems to see if they have
engineers willing to test these changes for their respective devices.

I've also refactored the code in this patch series to simplify the
changes in these patches. It should make the changes much easier to
review so I'll release it in a v2 submission once the series is ready.

William Breathitt Gray

--qcqNbohE9h+Wv+21
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEARYKAB0WIQSNN83d4NIlKPjon7a1SFbKvhIjKwUCYrmWEwAKCRC1SFbKvhIj
K571AP9BVDOeWF1ppEOM7vd/GiJ31qRTdxMzoLs15UZZ0AixOwD/bJ8wxcF6ARpp
BTlCqZ06OCVeHW2v3UAaoTe2Z7eATQo=
=tR33
-----END PGP SIGNATURE-----

--qcqNbohE9h+Wv+21--