[PATCH] vgaarbiter: documentation grammar correction
Pranav Tyagi
pranav.tyagi03 at gmail.com
Thu Feb 13 17:24:22 UTC 2025
Hi,
Just a gentle follow-up on this patch. It has been reviewed by Bagas
Sanjaya but hasn't been applied yet.
Regards,
Pranav Tyagi
On Sat, Feb 8, 2025 at 8:36 AM Bagas Sanjaya <bagasdotme at gmail.com> wrote:
> On Fri, Feb 07, 2025 at 10:23:25PM +0530, Pranav Tyagi wrote:
> > Corrects the following grammatical issues in the VGA Arbiter
> documentation:
> > - Fixes subject-verb agreement by changing "co-exists" to "co-exist"
> > - Corrects pluralization by changing "server" to "servers"
> > - Improves sentence structure for clarity
> >
>
> +Cc: DRM folks.
>
> > Signed-off-by: Pranav Tyagi <pranav.tyagi03 at 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 bde3c0afb059..d1e953712cc2 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)::
> >
>
> The diff looks OK.
>
> Reviewed-by: Bagas Sanjaya <bagasdotme at gmail.com>
>
> --
> An old man doll... just what I always wanted! - Clara
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20250213/247b84df/attachment.htm>
More information about the dri-devel
mailing list