[PATCH] Documentation: vgaarbiter: Fix grammar

Bagas Sanjaya posted 1 patch 9 months ago
Documentation/gpu/vgaarbiter.rst | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
[PATCH] Documentation: vgaarbiter: Fix grammar
Posted by Bagas Sanjaya 9 months ago
From: Pranav Tyagi <pranav.tyagi03@gmail.com>

Correct grammar issues:

- Fix "co-exist" subject-verb agreement
- Correct plural form of "server" in context of more than one legacy
  devices
- Use passive mood for intro sentence of libpciaccess section

Signed-off-by: Pranav Tyagi <pranav.tyagi03@gmail.com>
Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
Link: https://lore.kernel.org/r/20250220164946.18007-1-pranav.tyagi03@gmail.com
[Bagas: massage commit message]
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---

I have to carry the original patch [1] because DRM maintainers doesn't seem
to apply that AFAIK. Jon, would you like to apply this patch before the
merge window in case DRM people don't apply this either?

This patch is based on docs-next as gitlab.freedesktop.org is in migration
maintenance.

[1]: https://lore.kernel.org/linux-doc/20250220164946.18007-1-pranav.tyagi03@gmail.com/

 Documentation/gpu/vgaarbiter.rst | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/gpu/vgaarbiter.rst b/Documentation/gpu/vgaarbiter.rst
index bde3c0afb0590f..d1e953712cc293 100644
--- a/Documentation/gpu/vgaarbiter.rst
+++ b/Documentation/gpu/vgaarbiter.rst
@@ -11,9 +11,9 @@ Section 7, Legacy Devices.
 
 The Resource Access Control (RAC) module inside the X server [0] existed for
 the legacy VGA arbitration task (besides other bus management tasks) when more
-than one legacy device co-exists on the same machine. But the problem happens
+than one legacy device co-exist on the same machine. But the problem happens
 when these devices are trying to be accessed by different userspace clients
-(e.g. two server in parallel). Their address assignments conflict. Moreover,
+(e.g. two servers in parallel). Their address assignments conflict. Moreover,
 ideally, being a userspace application, it is not the role of the X server to
 control bus resources. Therefore an arbitration scheme outside of the X server
 is needed to control the sharing of these resources. This document introduces
@@ -106,7 +106,7 @@ In-kernel interface
 libpciaccess
 ------------
 
-To use the vga arbiter char device it was implemented an API inside the
+To use the vga arbiter char device, an API was implemented inside the
 libpciaccess library. One field was added to struct pci_device (each device
 on the system)::
 

base-commit: 5b8f85d081da449ab35e4bd009d7c00afaab2fab
-- 
An old man doll... just what I always wanted! - Clara
Re: [PATCH] Documentation: vgaarbiter: Fix grammar
Posted by Maxime Ripard 9 months ago
On Tue, 18 Mar 2025 11:12:50 +0700, Bagas Sanjaya wrote:
> Correct grammar issues:
> 
> - Fix "co-exist" subject-verb agreement
> - Correct plural form of "server" in context of more than one legacy
>   devices
> - Use passive mood for intro sentence of libpciaccess section
> 
> [...]

Applied to misc/kernel.git (drm-misc-next).

Thanks!
Maxime
Re: [PATCH] Documentation: vgaarbiter: Fix grammar
Posted by Jonathan Corbet 9 months ago
Bagas Sanjaya <bagasdotme@gmail.com> writes:

> From: Pranav Tyagi <pranav.tyagi03@gmail.com>
>
> Correct grammar issues:
>
> - Fix "co-exist" subject-verb agreement
> - Correct plural form of "server" in context of more than one legacy
>   devices
> - Use passive mood for intro sentence of libpciaccess section
>
> Signed-off-by: Pranav Tyagi <pranav.tyagi03@gmail.com>
> Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
> Link: https://lore.kernel.org/r/20250220164946.18007-1-pranav.tyagi03@gmail.com
> [Bagas: massage commit message]
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>
> I have to carry the original patch [1] because DRM maintainers doesn't seem
> to apply that AFAIK. Jon, would you like to apply this patch before the
> merge window in case DRM people don't apply this either?

The DRM folks manage their own docs.  I'm certainly not going to
circumvent them during -rc7 for a patch like this.

jon
Re: [PATCH] Documentation: vgaarbiter: Fix grammar
Posted by Bagas Sanjaya 9 months ago
On Tue, Mar 18, 2025 at 08:43:31AM -0600, Jonathan Corbet wrote:
> Bagas Sanjaya <bagasdotme@gmail.com> writes:
> 
> > From: Pranav Tyagi <pranav.tyagi03@gmail.com>
> >
> > Correct grammar issues:
> >
> > - Fix "co-exist" subject-verb agreement
> > - Correct plural form of "server" in context of more than one legacy
> >   devices
> > - Use passive mood for intro sentence of libpciaccess section
> >
> > Signed-off-by: Pranav Tyagi <pranav.tyagi03@gmail.com>
> > Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
> > Link: https://lore.kernel.org/r/20250220164946.18007-1-pranav.tyagi03@gmail.com
> > [Bagas: massage commit message]
> > Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> > ---
> >
> > I have to carry the original patch [1] because DRM maintainers doesn't seem
> > to apply that AFAIK. Jon, would you like to apply this patch before the
> > merge window in case DRM people don't apply this either?
> 
> The DRM folks manage their own docs.  I'm certainly not going to
> circumvent them during -rc7 for a patch like this.

OK, thanks! I should've post this after 6.15 merge window closes then...

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