From: Madhavan Srinivasan <maddy@linux.ibm.com>
To: Bagas Sanjaya <bagasdotme@gmail.com>,
Haren Myneni <haren@linux.ibm.com>,
linuxppc-dev@lists.ozlabs.org
Cc: sfr@canb.auug.org.au, tyreld@linux.ibm.com,
linux-next@vger.kernel.org, hbabu@us.ibm.com
Subject: Re: [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
Date: Fri, 2 May 2025 09:42:37 +0530 [thread overview]
Message-ID: <5a23e0d7-f32a-4097-b3cc-dcccb7355778@linux.ibm.com> (raw)
In-Reply-To: <aBHodTu4IjqzZeXb@archie.me>
On 4/30/25 2:38 PM, Bagas Sanjaya wrote:
> On Tue, Apr 29, 2025 at 07:28:47PM -0700, Haren Myneni wrote:
>> Fix the description format for the following build warnings:
>>
>> "Documentation/userspace-api/ioctl/ioctl-number.rst:369:
>> ERROR: Malformed table. Text in column margin in table line 301.
>>
>> 0xB2 03-05 arch/powerpc/include/uapi/asm/papr-indices.h
>> powerpc/pseries indices API
>> <mailto:linuxppc-dev>
>> 0xB2 06-07 arch/powerpc/include/uapi/asm/papr-platform-dump.h
>> powerpc/pseries Platform Dump API
>> <mailto:linuxppc-dev>
>> 0xB2 08 arch/powerpc/include/uapi/asm/papr-physical-attestation.h
>> powerpc/pseries Physical Attestation API
>> <mailto:linuxppc-dev>"
>>
>
> Hi,
>
> FYI, I've also submitted the fix earlier at [1] (but different approach).
> ppc maintainers, would you like taking this patch instead or mine?
Looked your patch (thanks for the link) and it is more of generic clean up.
I would prefer to take Haren patch now since it fixes the specific failure
case. But would encourage you to send it as generic cleanup patch.
Maddy
>
> [1]: https://lore.kernel.org/linuxppc-dev/20250429130524.33587-2-bagasdotme@gmail.com/
>
>> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
>> Signed-off-by: Haren Myneni <haren@linux.ibm.com>
>> Fixes: 43d869ac25f1 ("powerpc/pseries: Define papr_indices_io_block for papr-indices ioctls")
>> Fixes: 8aa9efc0be66 ("powerpc/pseries: Add papr-platform-dump character driver for dump retrieval")
>> Fixes: 86900ab620a4 ("powerpc/pseries: Add a char driver for physical-attestation RTAS")
>> Closes: https://lore.kernel.org/linux-next/20250429181707.7848912b@canb.auug.org.au/
>> ---
>> Documentation/userspace-api/ioctl/ioctl-number.rst | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documentation/userspace-api/ioctl/ioctl-number.rst
>> index 017a23aeadc3..fee5c4731501 100644
>> --- a/Documentation/userspace-api/ioctl/ioctl-number.rst
>> +++ b/Documentation/userspace-api/ioctl/ioctl-number.rst
>> @@ -366,11 +366,11 @@ Code Seq# Include File Comments
>> <mailto:linuxppc-dev>
>> 0xB2 01-02 arch/powerpc/include/uapi/asm/papr-sysparm.h powerpc/pseries system parameter API
>> <mailto:linuxppc-dev>
>> -0xB2 03-05 arch/powerpc/include/uapi/asm/papr-indices.h powerpc/pseries indices API
>> +0xB2 03-05 arch/powerpc/include/uapi/asm/papr-indices.h powerpc/pseries indices API
>> <mailto:linuxppc-dev>
>> -0xB2 06-07 arch/powerpc/include/uapi/asm/papr-platform-dump.h powerpc/pseries Platform Dump API
>> +0xB2 06-07 arch/powerpc/include/uapi/asm/papr-platform-dump.h powerpc/pseries Platform Dump API
>> <mailto:linuxppc-dev>
>> -0xB2 08 arch/powerpc/include/uapi/asm/papr-physical-attestation.h powerpc/pseries Physical Attestation API
>> +0xB2 08 powerpc/include/uapi/asm/papr-physical-attestation.h powerpc/pseries Physical Attestation API
>> <mailto:linuxppc-dev>
>> 0xB3 00 linux/mmc/ioctl.h
>> 0xB4 00-0F linux/gpio.h <mailto:linux-gpio@vger.kernel.org>
>
> In any case,
>
> Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
>
> Thanks.
>
next prev parent reply other threads:[~2025-05-02 4:13 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-30 2:28 [PATCH] Documentation: Fix description format for powerpc RTAS ioctls Haren Myneni
2025-04-30 8:51 ` Venkat Rao Bagalkote
2025-04-30 9:08 ` Bagas Sanjaya
2025-05-02 4:12 ` Madhavan Srinivasan [this message]
2025-05-02 7:38 ` Bagas Sanjaya
2025-05-04 3:21 ` Madhavan Srinivasan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5a23e0d7-f32a-4097-b3cc-dcccb7355778@linux.ibm.com \
--to=maddy@linux.ibm.com \
--cc=bagasdotme@gmail.com \
--cc=haren@linux.ibm.com \
--cc=hbabu@us.ibm.com \
--cc=linux-next@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=sfr@canb.auug.org.au \
--cc=tyreld@linux.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox