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

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


Reply via email to