linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
@ 2025-04-30  2:28 Haren Myneni
  2025-04-30  8:51 ` Venkat Rao Bagalkote
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Haren Myneni @ 2025-04-30  2:28 UTC (permalink / raw)
  To: linuxppc-dev; +Cc: maddy, sfr, tyreld, linux-next, hbabu, haren

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

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



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
  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-04  3:21 ` Madhavan Srinivasan
  2 siblings, 0 replies; 6+ messages in thread
From: Venkat Rao Bagalkote @ 2025-04-30  8:51 UTC (permalink / raw)
  To: Haren Myneni, linuxppc-dev; +Cc: maddy, sfr, tyreld, linux-next, hbabu


On 30/04/25 7:58 am, 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>"
>
> 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>


Tested this patch by applying on top of next-20250429 and it fixes the 
reported warning. Hence,


Tested-by: Venkat Rao Bagalkote <venkat88@linux.ibm.com>


Regards,

Venkat.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
  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
  2025-05-04  3:21 ` Madhavan Srinivasan
  2 siblings, 1 reply; 6+ messages in thread
From: Bagas Sanjaya @ 2025-04-30  9:08 UTC (permalink / raw)
  To: Haren Myneni, linuxppc-dev; +Cc: maddy, sfr, tyreld, linux-next, hbabu

[-- Attachment #1: Type: text/plain, Size: 3499 bytes --]

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?

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

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
  2025-04-30  9:08 ` Bagas Sanjaya
@ 2025-05-02  4:12   ` Madhavan Srinivasan
  2025-05-02  7:38     ` Bagas Sanjaya
  0 siblings, 1 reply; 6+ messages in thread
From: Madhavan Srinivasan @ 2025-05-02  4:12 UTC (permalink / raw)
  To: Bagas Sanjaya, Haren Myneni, linuxppc-dev; +Cc: sfr, tyreld, linux-next, hbabu



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



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
  2025-05-02  4:12   ` Madhavan Srinivasan
@ 2025-05-02  7:38     ` Bagas Sanjaya
  0 siblings, 0 replies; 6+ messages in thread
From: Bagas Sanjaya @ 2025-05-02  7:38 UTC (permalink / raw)
  To: Madhavan Srinivasan, Haren Myneni, linuxppc-dev
  Cc: sfr, tyreld, linux-next, hbabu

[-- Attachment #1: Type: text/plain, Size: 1512 bytes --]

On Fri, May 02, 2025 at 09:42:37AM +0530, Madhavan Srinivasan wrote:
> 
> 
> 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.

Do you mean submitting the cleanup patch (essentially the same
resulting table as my patch would) on top of Haren's patch?

Confused...

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Documentation: Fix description format for powerpc RTAS ioctls
  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-04  3:21 ` Madhavan Srinivasan
  2 siblings, 0 replies; 6+ messages in thread
From: Madhavan Srinivasan @ 2025-05-04  3:21 UTC (permalink / raw)
  To: linuxppc-dev, Haren Myneni; +Cc: sfr, tyreld, linux-next, hbabu

On Tue, 29 Apr 2025 19:28:47 -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>"
> 
> [...]

Applied to powerpc/next.

[1/1] Documentation: Fix description format for powerpc RTAS ioctls
      https://git.kernel.org/powerpc/c/03c9d1a5a30d93bff31b4eb0a52f030b4c7f73ea

Thanks


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-05-04  3:21 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
2025-05-02  7:38     ` Bagas Sanjaya
2025-05-04  3:21 ` Madhavan Srinivasan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).