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

[Randy Dunlap]

On 10/29/25 1:52 PM, Shenwei Wang wrote:
> 
> 
>> -----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.
> 

Yeah, maybe. That's sort of a last resort location for things
that don't have a better place.

> 
>>> 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

-- 
~Randy