> -----Original Message-----
> From: Randy Dunlap <[email protected]>
> Sent: Wednesday, October 29, 2025 3:47 PM
> To: Shenwei Wang <[email protected]>; Linus Walleij
> <[email protected]>; Bartosz Golaszewski <[email protected]>; Jonathan
> Corbet <[email protected]>; Bjorn Andersson <[email protected]>; Mathieu
> Poirier <[email protected]>; Rob Herring <[email protected]>; Krzysztof
> Kozlowski <[email protected]>; Conor Dooley <[email protected]>; Shawn
> Guo <[email protected]>; Sascha Hauer <[email protected]>
> Cc: Pengutronix Kernel Team <[email protected]>; Fabio Estevam
> <[email protected]>; Peng Fan <[email protected]>; linux-
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; dl-linux-imx <[email protected]>
> 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 <[email protected]>
> > ---
> >  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

Reply via email to