[PATCH] Documentation: vgaarbiter: fix typos

[PATCH] Documentation: vgaarbiter: fix typos

[Nicolas Kaiser]

Fixed typos.

Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
---
 Documentation/vgaarbiter.txt |   15 +++++++--------
 1 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
index 43a9b06..8145f8f 100644
--- a/Documentation/vgaarbiter.txt
+++ b/Documentation/vgaarbiter.txt
@@ -14,11 +14,10 @@ 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
 when these devices are trying to be accessed by different userspace clients
 (e.g. two server in parallel). Their address assignments conflict. Moreover,
-ideally, being an userspace application, it is not the role of the 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 the operation of the VGA arbiter implemented for Linux
-kernel.
+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
+the operation of the VGA arbiter implemented for the Linux kernel.
 
 ----------------------------------------------------------------------------
 
@@ -39,7 +38,7 @@ I.1 vgaarb
 The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
 scans all PCI devices and adds the VGA ones inside the arbitration. The
 arbiter then enables/disables the decoding on different devices of the VGA
-legacy instructions. Device which do not want/need to use the arbiter may
+legacy instructions. Devices which do not want/need to use the arbiter may
 explicitly tell it by calling vga_set_legacy_decoding().
 
 The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
@@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
 notify them being added/removed in the system and automatically added/removed
 in the arbiter.
 
-There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
+There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
 others which may use the arbiter.
 
 
@@ -117,7 +116,7 @@ Besides it, in pci_system were added:
     struct pci_device *vga_default_dev;
 
 
-The vga_count is usually need to keep informed how many cards are being
+The vga_count is usually needed to keep informed how many cards are being
 arbitrated, so for instance if there's only one then it can totally escape the
 scheme.
 
-- 
1.7.2.2

Re: [PATCH] Documentation: vgaarbiter: fix typos

[Randy Dunlap]

On Wed, 1 Dec 2010 02:34:57 +0100 Nicolas Kaiser wrote:

> Fixed typos.
> 
> Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> ---
>  Documentation/vgaarbiter.txt |   15 +++++++--------
>  1 files changed, 7 insertions(+), 8 deletions(-)
> 
> diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> index 43a9b06..8145f8f 100644
> --- a/Documentation/vgaarbiter.txt
> +++ b/Documentation/vgaarbiter.txt
> @@ -14,11 +14,10 @@ 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
>  when these devices are trying to be accessed by different userspace clients
>  (e.g. two server in parallel). Their address assignments conflict. Moreover,
> -ideally, being an userspace application, it is not the role of the 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 the operation of the VGA arbiter implemented for Linux
> -kernel.
> +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
> +the operation of the VGA arbiter implemented for the Linux kernel.
>  
>  ----------------------------------------------------------------------------
>  
> @@ -39,7 +38,7 @@ I.1 vgaarb
>  The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
>  scans all PCI devices and adds the VGA ones inside the arbitration. The
>  arbiter then enables/disables the decoding on different devices of the VGA
> -legacy instructions. Device which do not want/need to use the arbiter may
> +legacy instructions. Devices which do not want/need to use the arbiter may
>  explicitly tell it by calling vga_set_legacy_decoding().
>  
>  The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> @@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
>  notify them being added/removed in the system and automatically added/removed
>  in the arbiter.
>  
> -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> +There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
>  others which may use the arbiter.
>  

How about:

There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
drivers want to use it.

>  
> @@ -117,7 +116,7 @@ Besides it, in pci_system were added:
>      struct pci_device *vga_default_dev;
>  
>  
> -The vga_count is usually need to keep informed how many cards are being
> +The vga_count is usually needed to keep informed how many cards are being
>  arbitrated, so for instance if there's only one then it can totally escape the
>  scheme.

Awkward.  How about:

The vga_count is used to track how many cards are being arbitrated, so for
instance, if there is only one card, then it can completely escape arbitration.

>  
> -- 


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

Re: [PATCH] Documentation: vgaarbiter: fix typos

[Jesse Barnes]

On Wed, 1 Dec 2010 16:41:41 -0800
Randy Dunlap <rdunlap@xenotime.net> wrote:

> On Wed, 1 Dec 2010 02:34:57 +0100 Nicolas Kaiser wrote:
> 
> > Fixed typos.
> > 
> > Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> > ---
> >  Documentation/vgaarbiter.txt |   15 +++++++--------
> >  1 files changed, 7 insertions(+), 8 deletions(-)
> > 
> > diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> > index 43a9b06..8145f8f 100644
> > --- a/Documentation/vgaarbiter.txt
> > +++ b/Documentation/vgaarbiter.txt
> > @@ -14,11 +14,10 @@ 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
> >  when these devices are trying to be accessed by different userspace clients
> >  (e.g. two server in parallel). Their address assignments conflict. Moreover,
> > -ideally, being an userspace application, it is not the role of the 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 the operation of the VGA arbiter implemented for Linux
> > -kernel.
> > +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
> > +the operation of the VGA arbiter implemented for the Linux kernel.
> >  
> >  ----------------------------------------------------------------------------
> >  
> > @@ -39,7 +38,7 @@ I.1 vgaarb
> >  The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
> >  scans all PCI devices and adds the VGA ones inside the arbitration. The
> >  arbiter then enables/disables the decoding on different devices of the VGA
> > -legacy instructions. Device which do not want/need to use the arbiter may
> > +legacy instructions. Devices which do not want/need to use the arbiter may
> >  explicitly tell it by calling vga_set_legacy_decoding().
> >  
> >  The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> > @@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
> >  notify them being added/removed in the system and automatically added/removed
> >  in the arbiter.
> >  
> > -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> > +There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
> >  others which may use the arbiter.
> >  
> 
> How about:
> 
> There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
> drivers want to use it.
> 
> >  
> > @@ -117,7 +116,7 @@ Besides it, in pci_system were added:
> >      struct pci_device *vga_default_dev;
> >  
> >  
> > -The vga_count is usually need to keep informed how many cards are being
> > +The vga_count is usually needed to keep informed how many cards are being
> >  arbitrated, so for instance if there's only one then it can totally escape the
> >  scheme.
> 
> Awkward.  How about:
> 
> The vga_count is used to track how many cards are being arbitrated, so for
> instance, if there is only one card, then it can completely escape arbitration.

Randy, I assume you'll pick this up once you receive a version you're
ok with?

Thanks,
-- 
Jesse Barnes, Intel Open Source Technology Center

Re: [PATCH] Documentation: vgaarbiter: fix typos

[Randy Dunlap]

On Mon, 6 Dec 2010 09:25:44 -0800 Jesse Barnes wrote:

> On Wed, 1 Dec 2010 16:41:41 -0800
> Randy Dunlap <rdunlap@xenotime.net> wrote:
> 
> > On Wed, 1 Dec 2010 02:34:57 +0100 Nicolas Kaiser wrote:
> > 
> > > Fixed typos.
> > > 
> > > Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> > > ---
> > >  Documentation/vgaarbiter.txt |   15 +++++++--------
> > >  1 files changed, 7 insertions(+), 8 deletions(-)
> > > 
> > > diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> > > index 43a9b06..8145f8f 100644
> > > --- a/Documentation/vgaarbiter.txt
> > > +++ b/Documentation/vgaarbiter.txt
> > > @@ -14,11 +14,10 @@ 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
> > >  when these devices are trying to be accessed by different userspace clients
> > >  (e.g. two server in parallel). Their address assignments conflict. Moreover,
> > > -ideally, being an userspace application, it is not the role of the 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 the operation of the VGA arbiter implemented for Linux
> > > -kernel.
> > > +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
> > > +the operation of the VGA arbiter implemented for the Linux kernel.
> > >  
> > >  ----------------------------------------------------------------------------
> > >  
> > > @@ -39,7 +38,7 @@ I.1 vgaarb
> > >  The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
> > >  scans all PCI devices and adds the VGA ones inside the arbitration. The
> > >  arbiter then enables/disables the decoding on different devices of the VGA
> > > -legacy instructions. Device which do not want/need to use the arbiter may
> > > +legacy instructions. Devices which do not want/need to use the arbiter may
> > >  explicitly tell it by calling vga_set_legacy_decoding().
> > >  
> > >  The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> > > @@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
> > >  notify them being added/removed in the system and automatically added/removed
> > >  in the arbiter.
> > >  
> > > -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> > > +There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
> > >  others which may use the arbiter.
> > >  
> > 
> > How about:
> > 
> > There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
> > drivers want to use it.
> > 
> > >  
> > > @@ -117,7 +116,7 @@ Besides it, in pci_system were added:
> > >      struct pci_device *vga_default_dev;
> > >  
> > >  
> > > -The vga_count is usually need to keep informed how many cards are being
> > > +The vga_count is usually needed to keep informed how many cards are being
> > >  arbitrated, so for instance if there's only one then it can totally escape the
> > >  scheme.
> > 
> > Awkward.  How about:
> > 
> > The vga_count is used to track how many cards are being arbitrated, so for
> > instance, if there is only one card, then it can completely escape arbitration.
> 
> Randy, I assume you'll pick this up once you receive a version you're
> ok with?

I wasn't sure who would, but I can do that.

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

[PATCH] Documentation: vgaarbiter: fix typos v2

[Nicolas Kaiser]

Fixed typos.

v2: Incorporated changes by Randy Dunlap.

Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
---
* Randy Dunlap <rdunlap@xenotime.net>:
> On Mon, 6 Dec 2010 09:25:44 -0800 Jesse Barnes wrote:

> > Randy, I assume you'll pick this up once you receive a version you're
> > ok with?
> 
> I wasn't sure who would, but I can do that.

Sorry for the delay. I completely forgot about this patch.

 Documentation/vgaarbiter.txt |   20 +++++++++-----------
 1 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
index 43a9b06..4710230 100644
--- a/Documentation/vgaarbiter.txt
+++ b/Documentation/vgaarbiter.txt
@@ -14,11 +14,10 @@ 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
 when these devices are trying to be accessed by different userspace clients
 (e.g. two server in parallel). Their address assignments conflict. Moreover,
-ideally, being an userspace application, it is not the role of the 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 the operation of the VGA arbiter implemented for Linux
-kernel.
+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
+the operation of the VGA arbiter implemented for the Linux kernel.
 
 ----------------------------------------------------------------------------
 
@@ -39,7 +38,7 @@ I.1 vgaarb
 The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
 scans all PCI devices and adds the VGA ones inside the arbitration. The
 arbiter then enables/disables the decoding on different devices of the VGA
-legacy instructions. Device which do not want/need to use the arbiter may
+legacy instructions. Devices which do not want/need to use the arbiter may
 explicitly tell it by calling vga_set_legacy_decoding().
 
 The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
@@ -95,8 +94,8 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
 notify them being added/removed in the system and automatically added/removed
 in the arbiter.
 
-There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
-others which may use the arbiter.
+There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
+drivers want to use it.
 
 
 I.2 libpciaccess
@@ -117,9 +116,8 @@ Besides it, in pci_system were added:
     struct pci_device *vga_default_dev;
 
 
-The vga_count is usually need to keep informed how many cards are being
-arbitrated, so for instance if there's only one then it can totally escape the
-scheme.
+The vga_count is used to track how many cards are being arbitrated, so for
+instance, if there is only one card, then it can completely escape arbitration.
 
 
 These functions below acquire VGA resources for the given card and mark those
-- 
1.7.5.rc3

Re: [PATCH] Documentation: vgaarbiter: fix typos v2

[Randy Dunlap]

On Sat, 14 May 2011 12:51:52 +0200 Nicolas Kaiser wrote:

> Fixed typos.
> 
> v2: Incorporated changes by Randy Dunlap.
> 
> Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> ---
> * Randy Dunlap <rdunlap@xenotime.net>:
> > On Mon, 6 Dec 2010 09:25:44 -0800 Jesse Barnes wrote:
> 
> > > Randy, I assume you'll pick this up once you receive a version you're
> > > ok with?
> > 
> > I wasn't sure who would, but I can do that.
> 
> Sorry for the delay. I completely forgot about this patch.

Applied, thanks.


>  Documentation/vgaarbiter.txt |   20 +++++++++-----------
>  1 files changed, 9 insertions(+), 11 deletions(-)
> 
> diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> index 43a9b06..4710230 100644
> --- a/Documentation/vgaarbiter.txt
> +++ b/Documentation/vgaarbiter.txt
> @@ -14,11 +14,10 @@ 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
>  when these devices are trying to be accessed by different userspace clients
>  (e.g. two server in parallel). Their address assignments conflict. Moreover,
> -ideally, being an userspace application, it is not the role of the 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 the operation of the VGA arbiter implemented for Linux
> -kernel.
> +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
> +the operation of the VGA arbiter implemented for the Linux kernel.
>  
>  ----------------------------------------------------------------------------
>  
> @@ -39,7 +38,7 @@ I.1 vgaarb
>  The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
>  scans all PCI devices and adds the VGA ones inside the arbitration. The
>  arbiter then enables/disables the decoding on different devices of the VGA
> -legacy instructions. Device which do not want/need to use the arbiter may
> +legacy instructions. Devices which do not want/need to use the arbiter may
>  explicitly tell it by calling vga_set_legacy_decoding().
>  
>  The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> @@ -95,8 +94,8 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
>  notify them being added/removed in the system and automatically added/removed
>  in the arbiter.
>  
> -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> -others which may use the arbiter.
> +There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
> +drivers want to use it.
>  
>  
>  I.2 libpciaccess
> @@ -117,9 +116,8 @@ Besides it, in pci_system were added:
>      struct pci_device *vga_default_dev;
>  
>  
> -The vga_count is usually need to keep informed how many cards are being
> -arbitrated, so for instance if there's only one then it can totally escape the
> -scheme.
> +The vga_count is used to track how many cards are being arbitrated, so for
> +instance, if there is only one card, then it can completely escape arbitration.
>  
>  
>  These functions below acquire VGA resources for the given card and mark those
> -- 


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***