Re: [PATCH v4 3/5] docs: admin-guide: gpio: rpmsg: gpio over rpmsg bus

[Shenwei Wang]

> -----Original Message-----
> From: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Sent: Wednesday, October 29, 2025 3:47 PM
> To: Shenwei Wang <shenwei.wang@xxxxxxx>; Linus Walleij
> <linus.walleij@xxxxxxxxxx>; Bartosz Golaszewski <brgl@xxxxxxxx>; Jonathan
> Corbet <corbet@xxxxxxx>; Bjorn Andersson <andersson@xxxxxxxxxx>; Mathieu
> Poirier <mathieu.poirier@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof
> Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Shawn
> Guo <shawnguo@xxxxxxxxxx>; Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> Cc: Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx>; Fabio Estevam
> <festevam@xxxxxxxxx>; Peng Fan <peng.fan@xxxxxxx>; linux-
> gpio@xxxxxxxxxxxxxxx; linux-doc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> linux-remoteproc@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx;
> imx@xxxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx>
> Subject: [EXT] Re: [PATCH v4 3/5] docs: admin-guide: gpio: rpmsg: gpio over
> rpmsg bus
> 
> Hi--
> 
> On 10/29/25 12:56 PM, Shenwei Wang wrote:
> > Describes the gpio rpmsg transport protocol over the rpmsg bus between
> > the cores.
> >
> > Signed-off-by: Shenwei Wang <shenwei.wang@xxxxxxx>
> > ---
> >  Documentation/admin-guide/gpio/gpio-rpmsg.rst | 202 ++++++++++++++++++
> >  Documentation/admin-guide/gpio/index.rst      |   1 +
> >  2 files changed, 203 insertions(+)
> >  create mode 100644 Documentation/admin-guide/gpio/gpio-rpmsg.rst
> >
> 
> I don't think this should be in admin-guide/gpio/.
> How about in driver-api/gpio/?
> That may not be perfect either, but transport protocols are not typically admin
> material AFAIK.
> 

How about putting it in Documentation/staging directory?
I saw both remoteproc.rst and rpmsg.rst are in the directory.

Thanks,
Shenwei

> > diff --git a/Documentation/admin-guide/gpio/index.rst
> > b/Documentation/admin-guide/gpio/index.rst
> > index 712f379731cb..9c8f4441038a 100644
> > --- a/Documentation/admin-guide/gpio/index.rst
> > +++ b/Documentation/admin-guide/gpio/index.rst
> > @@ -9,6 +9,7 @@ GPIO
> >
> >      Character Device Userspace API <../../userspace-api/gpio/chardev>
> >      gpio-aggregator
> > +    gpio-rpmsg
> >      gpio-sim
> >      gpio-virtuser
> >      Obsolete APIs <obsolete>
> If someone thinks that it should be in admin-guide/,please explain why.
> 
> Thanks.
> --
> ~Randy