[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aWfNsrpC5Rgiwwp3@p14s>
Date: Wed, 14 Jan 2026 10:09:06 -0700
From: Mathieu Poirier <mathieu.poirier@...aro.org>
To: Shenwei Wang <shenwei.wang@....com>
Cc: Linus Walleij <linusw@...nel.org>,
Bartosz Golaszewski <brgl@...nel.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Jonathan Corbet <corbet@....net>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>, Peng Fan <peng.fan@....com>,
linux-gpio@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-remoteproc@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
linux-doc@...r.kernel.org, linux-imx@....com
Subject: Re: [PATCH v6 0/5] Enable Remote GPIO over RPMSG on i.MX Platform
On Fri, Dec 12, 2025 at 01:43:36PM -0600, Shenwei Wang wrote:
> Support the remote devices on the remote processor via the RPMSG bus on
> i.MX platform.
>
> Changes in v6:
> - make the driver more generic with the actions below:
> rename the driver file to gpio-rpmsg.c
> remove the imx related info in the function and variable names
> rename the imx_rpmsg.h to rpdev_info.h
> create a gpio-rpmsg.yaml and refer it in imx_rproc.yaml
> - update the gpio-rpmsg.rst according to the feedback from Andrew and
> move the source file to driver-api/gpio
> - fix the bug reported by Zhongqiu Han
> - remove the I2C related info
>
NAK - I asked to use the virtio interface (in this case virtio-gpio), something
I do not see in the above summary and the first 2 patches in this series. This
work will not move forward until that happens.
> Changes in v5:
> - move the gpio-rpmsg.rst from admin-guide to staging directory after
> discussion with Randy Dunlap.
> - add include files with some code improvements per Bartosz's comments.
>
> Changes in v4:
> - add a documentation to describe the transport protocol per Andrew's
> comments.
> - add a new handler to get the gpio direction.
>
> Changes in v3:
> - fix various format issue and return value check per Peng 's review
> comments.
> - add the logic to also populate the subnodes which are not in the
> device map per Arnaud's request. (in imx_rproc.c)
> - update the yaml per Frank's review comments.
>
> Changes in v2:
> - re-implemented the gpio driver per Linus Walleij's feedback by using
> GPIOLIB_IRQCHIP helper library.
> - fix various format issue per Mathieu/Peng 's review comments.
> - update the yaml doc per Rob's feedback
>
> Shenwei Wang (5):
> dt-bindings: remoteproc: imx_rproc: Add "rpmsg" subnode support
> remoteproc: imx_rproc: Populate devices under "rpmsg" subnode
> docs: driver-api: gpio: generic gpio driver over rpmsg bus
> gpio: rpmsg: add generic rpmsg GPIO driver
> arm64: dts: imx8ulp: Add rpmsg node under imx_rproc
>
> .../devicetree/bindings/gpio/gpio-rpmsg.yaml | 49 ++
> .../bindings/remoteproc/fsl,imx-rproc.yaml | 54 ++
> Documentation/driver-api/gpio/gpio-rpmsg.rst | 232 +++++++++
> Documentation/driver-api/gpio/index.rst | 1 +
> arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 27 +
> drivers/gpio/Kconfig | 16 +
> drivers/gpio/Makefile | 1 +
> drivers/gpio/gpio-rpmsg.c | 490 ++++++++++++++++++
> drivers/remoteproc/imx_rproc.c | 143 +++++
> include/linux/rpmsg/rpdev_info.h | 33 ++
> 10 files changed, 1046 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/gpio/gpio-rpmsg.yaml
> create mode 100644 Documentation/driver-api/gpio/gpio-rpmsg.rst
> create mode 100644 drivers/gpio/gpio-rpmsg.c
> create mode 100644 include/linux/rpmsg/rpdev_info.h
>
> --
> 2.43.0
>
Powered by blists - more mailing lists