| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <3cd38c59-708e-76d3-52dc-2adf250c2d14@conchuod.ie>
Date: Mon, 29 Aug 2022 18:23:05 +0100
From: Conor Dooley <mail@...chuod.ie>
To: Jules Maselbas <jmaselbas@...ray.eu>, linux-kernel@...r.kernel.org
Cc: Conor.Dooley@...rochip.com, Randy Dunlap <rdunlap@...radead.org>,
Bagas Sanjaya <bagasdotme@...il.com>, linux-doc@...r.kernel.org
Subject: Re: [PATCH v5] Remove duplicated words across the whole documentation
On 29/08/2022 10:29, Jules Maselbas wrote:
> Remove duplicated words (the, at, be ...) across the whole documentation.
> In some cases the duplicated words are replaced by something that makes
> more sense, for instance: "be be" is replaced by "can be" when possible.
>
> There are likely more duplicated words.
>
> CC: Conor.Dooley@...rochip.com
> CC: Randy Dunlap <rdunlap@...radead.org>
> CC: Bagas Sanjaya <bagasdotme@...il.com>
> CC: linux-doc@...r.kernel.org
> Signed-off-by: Jules Maselbas <jmaselbas@...ray.eu>
> ---
> in v5:
> - remove spurious changes reported by Randy Dunlap
> in v4:
> - remove spurious changes reported by Conor Dooley
> in v3:
> as suggested by Randy Dunlap:
> - remove duplicated 'The the '
> - remove duplicated 'at at ' (some are replaced by 'at a ')
> - remove duplicated be, is, to, we, on ...
> in v2:
> - also remove the second 'the' in one sentence as suggested by
> Bagas Sanjaya
> ---
> Documentation/RCU/checklist.rst | 2 +-
> Documentation/admin-guide/kdump/vmcoreinfo.rst | 2 +-
> Documentation/bpf/instruction-set.rst | 2 +-
> Documentation/bpf/map_cgroup_storage.rst | 4 ++--
> Documentation/core-api/cpu_hotplug.rst | 3 +--
---8<---
> Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml | 4 ++--
> .../devicetree/bindings/arm/tegra/nvidia,tegra20-ahb.txt | 2 +-
> Documentation/devicetree/bindings/dma/ti-dma-crossbar.txt | 2 +-
> Documentation/devicetree/bindings/fpga/fpga-region.txt | 4 ++--
> Documentation/devicetree/bindings/mfd/ti,lp87524-q1.yaml | 2 +-
> Documentation/devicetree/bindings/mfd/ti,lp87561-q1.yaml | 2 +-
> Documentation/devicetree/bindings/mfd/ti,lp87565-q1.yaml | 2 +-
> .../devicetree/bindings/net/amlogic,meson-dwmac.yaml | 2 +-
> .../devicetree/bindings/net/can/microchip,mcp251xfd.yaml | 2 +-
---8<---
I think I saw Jon pointing out against the v3 that these are bindings
and need to be sent separately via the subsystem trees rather than his
docs tree.
Other than that, I read back through these and all seem correct now,
so FWIW:
Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
Thanks.
> Documentation/driver-api/isa.rst | 2 +-
> Documentation/filesystems/caching/backend-api.rst | 2 +-
> Documentation/filesystems/journalling.rst | 2 +-
> Documentation/hwmon/f71882fg.rst | 2 +-
> Documentation/locking/seqlock.rst | 2 +-
> Documentation/sphinx/cdomain.py | 2 +-
> Documentation/trace/histogram.rst | 2 +-
> Documentation/userspace-api/media/dvb/dmx-reqbufs.rst | 2 +-
> Documentation/userspace-api/media/dvb/frontend_f_open.rst | 2 +-
> 23 files changed, 26 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/RCU/checklist.rst b/Documentation/RCU/checklist.rst
> index 42cc5d891bd2..7b1c85a16dc3 100644
> --- a/Documentation/RCU/checklist.rst
> +++ b/Documentation/RCU/checklist.rst
> @@ -477,6 +477,6 @@ over a rather long period of time, but improvements are always welcome!
> So if you need to wait for both an RCU grace period and for
> all pre-existing call_rcu() callbacks, you will need to execute
> both rcu_barrier() and synchronize_rcu(), if necessary, using
> - something like workqueues to to execute them concurrently.
> + something like workqueues to execute them concurrently.
>
> See rcubarrier.rst for more information.
> diff --git a/Documentation/admin-guide/kdump/vmcoreinfo.rst b/Documentation/admin-guide/kdump/vmcoreinfo.rst
> index 8419019b6a88..6726f439958c 100644
> --- a/Documentation/admin-guide/kdump/vmcoreinfo.rst
> +++ b/Documentation/admin-guide/kdump/vmcoreinfo.rst
> @@ -200,7 +200,7 @@ prb
>
> A pointer to the printk ringbuffer (struct printk_ringbuffer). This
> may be pointing to the static boot ringbuffer or the dynamically
> -allocated ringbuffer, depending on when the the core dump occurred.
> +allocated ringbuffer, depending on when the core dump occurred.
> Used by user-space tools to read the active kernel log buffer.
>
> printk_rb_static
> diff --git a/Documentation/bpf/instruction-set.rst b/Documentation/bpf/instruction-set.rst
> index 1b0e6711dec9..0ac7ae40be37 100644
> --- a/Documentation/bpf/instruction-set.rst
> +++ b/Documentation/bpf/instruction-set.rst
> @@ -133,7 +133,7 @@ code field of ``BPF_END``.
> The byte swap instructions operate on the destination register
> only and do not use a separate source register or immediate value.
>
> -The 1-bit source operand field in the opcode is used to to select what byte
> +The 1-bit source operand field in the opcode is used to select what byte
> order the operation convert from or to:
>
> ========= ===== =================================================
> diff --git a/Documentation/bpf/map_cgroup_storage.rst b/Documentation/bpf/map_cgroup_storage.rst
> index cab9543017bf..8e5fe532c07e 100644
> --- a/Documentation/bpf/map_cgroup_storage.rst
> +++ b/Documentation/bpf/map_cgroup_storage.rst
> @@ -31,7 +31,7 @@ The map uses key of type of either ``__u64 cgroup_inode_id`` or
> };
>
> ``cgroup_inode_id`` is the inode id of the cgroup directory.
> -``attach_type`` is the the program's attach type.
> +``attach_type`` is the program's attach type.
>
> Linux 5.9 added support for type ``__u64 cgroup_inode_id`` as the key type.
> When this key type is used, then all attach types of the particular cgroup and
> @@ -155,7 +155,7 @@ However, the BPF program can still only associate with one map of each type
> ``BPF_MAP_TYPE_CGROUP_STORAGE`` or more than one
> ``BPF_MAP_TYPE_PERCPU_CGROUP_STORAGE``.
>
> -In all versions, userspace may use the the attach parameters of cgroup and
> +In all versions, userspace may use the attach parameters of cgroup and
> attach type pair in ``struct bpf_cgroup_storage_key`` as the key to the BPF map
> APIs to read or update the storage for a given attachment. For Linux 5.9
> attach type shared storages, only the first value in the struct, cgroup inode
> diff --git a/Documentation/core-api/cpu_hotplug.rst b/Documentation/core-api/cpu_hotplug.rst
> index c6f4ba2fb32d..c326f4a86d34 100644
> --- a/Documentation/core-api/cpu_hotplug.rst
> +++ b/Documentation/core-api/cpu_hotplug.rst
> @@ -560,8 +560,7 @@ available:
> * cpuhp_state_remove_instance(state, node)
> * cpuhp_state_remove_instance_nocalls(state, node)
>
> -The arguments are the same as for the the cpuhp_state_add_instance*()
> -variants above.
> +The arguments are the same as for cpuhp_state_add_instance*() variants above.
>
> The functions differ in the way how the installed callbacks are treated:
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> index a4b4452afc1d..e5b56ee500bc 100644
> --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> @@ -121,7 +121,7 @@ properties:
>
> arm,vexpress,position:
> description: When daughterboards are stacked on one site, their position
> - in the stack be be described this attribute.
> + in the stack can be described with this attribute.
> $ref: '/schemas/types.yaml#/definitions/uint32'
> minimum: 0
> maximum: 3
> @@ -139,7 +139,7 @@ patternProperties:
> the connection between the motherboard and any tiles. Sometimes the
> compatible is placed directly under this node, sometimes it is placed
> in a subnode named "motherboard-bus". Sometimes the compatible includes
> - "arm,vexpress,v2?-p1" sometimes (on software models) is is just
> + "arm,vexpress,v2?-p1" sometimes (on software models) it is just
> "simple-bus". If the compatible is placed in the "motherboard-bus" node,
> it is stricter and always has two compatibles.
> type: object
> diff --git a/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-ahb.txt b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-ahb.txt
> index 9a4295b54539..b300c42c52d7 100644
> --- a/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-ahb.txt
> +++ b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-ahb.txt
> @@ -8,7 +8,7 @@ Required properties:
> - reg : Should contain 1 register ranges(address and length). For
> Tegra20, Tegra30, and Tegra114 chips, the value must be <0x6000c004
> 0x10c>. For Tegra124, Tegra132 and Tegra210 chips, the value should
> - be be <0x6000c000 0x150>.
> + be <0x6000c000 0x150>.
>
> Example (for a Tegra20 chip):
> ahb: ahb@...0c004 {
> diff --git a/Documentation/devicetree/bindings/dma/ti-dma-crossbar.txt b/Documentation/devicetree/bindings/dma/ti-dma-crossbar.txt
> index b849a1ed389d..47e477cce6d2 100644
> --- a/Documentation/devicetree/bindings/dma/ti-dma-crossbar.txt
> +++ b/Documentation/devicetree/bindings/dma/ti-dma-crossbar.txt
> @@ -4,7 +4,7 @@ Required properties:
> - compatible: "ti,dra7-dma-crossbar" for DRA7xx DMA crossbar
> "ti,am335x-edma-crossbar" for AM335x and AM437x
> - reg: Memory map for accessing module
> -- #dma-cells: Should be set to to match with the DMA controller's dma-cells
> +- #dma-cells: Should be set to match with the DMA controller's dma-cells
> for ti,dra7-dma-crossbar and <3> for ti,am335x-edma-crossbar.
> - dma-requests: Number of DMA requests the crossbar can receive
> - dma-masters: phandle pointing to the DMA controller
> diff --git a/Documentation/devicetree/bindings/fpga/fpga-region.txt b/Documentation/devicetree/bindings/fpga/fpga-region.txt
> index 6694ef29a267..095b5e728dff 100644
> --- a/Documentation/devicetree/bindings/fpga/fpga-region.txt
> +++ b/Documentation/devicetree/bindings/fpga/fpga-region.txt
> @@ -50,7 +50,7 @@ Partial Reconfiguration Region (PRR)
> Persona
> * Also called a "partial bit stream"
> * An FPGA image that is designed to be loaded into a PRR. There may be
> - any number of personas designed to fit into a PRR, but only one at at time
> + any number of personas designed to fit into a PRR, but only one at a time
> may be loaded.
> * A persona may create more regions.
>
> @@ -127,7 +127,7 @@ add the child devices:
>
> * FPGA Manager
> * FPGA Bridges
> - * image-specific information needed to to the programming.
> + * image-specific information needed to do the programming.
> * child nodes
>
> The intended use is that a Device Tree overlay (DTO) can be used to reprogram an
> diff --git a/Documentation/devicetree/bindings/mfd/ti,lp87524-q1.yaml b/Documentation/devicetree/bindings/mfd/ti,lp87524-q1.yaml
> index f6cac4b1079c..3549a32452ec 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,lp87524-q1.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,lp87524-q1.yaml
> @@ -26,7 +26,7 @@ properties:
> '#gpio-cells':
> description:
> The first cell is the pin number.
> - The second cell is is used to specify flags.
> + The second cell is used to specify flags.
> See ../gpio/gpio.txt for more information.
> const: 2
>
> diff --git a/Documentation/devicetree/bindings/mfd/ti,lp87561-q1.yaml b/Documentation/devicetree/bindings/mfd/ti,lp87561-q1.yaml
> index dc5a29b5ef7d..43a3f7ccaf36 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,lp87561-q1.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,lp87561-q1.yaml
> @@ -26,7 +26,7 @@ properties:
> '#gpio-cells':
> description:
> The first cell is the pin number.
> - The second cell is is used to specify flags.
> + The second cell is used to specify flags.
> See ../gpio/gpio.txt for more information.
> const: 2
>
> diff --git a/Documentation/devicetree/bindings/mfd/ti,lp87565-q1.yaml b/Documentation/devicetree/bindings/mfd/ti,lp87565-q1.yaml
> index 012d25111054..373c4f89c4ea 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,lp87565-q1.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,lp87565-q1.yaml
> @@ -28,7 +28,7 @@ properties:
> '#gpio-cells':
> description:
> The first cell is the pin number.
> - The second cell is is used to specify flags.
> + The second cell is used to specify flags.
> See ../gpio/gpio.txt for more information.
> const: 2
>
> diff --git a/Documentation/devicetree/bindings/net/amlogic,meson-dwmac.yaml b/Documentation/devicetree/bindings/net/amlogic,meson-dwmac.yaml
> index 608e1d62bed5..3eb0513d824c 100644
> --- a/Documentation/devicetree/bindings/net/amlogic,meson-dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/amlogic,meson-dwmac.yaml
> @@ -149,7 +149,7 @@ properties:
> - description:
> The first register range should be the one of the DWMAC controller
> - description:
> - The second range is is for the Amlogic specific configuration
> + The second range is for the Amlogic specific configuration
> (for example the PRG_ETHERNET register range on Meson8b and newer)
>
> required:
> diff --git a/Documentation/devicetree/bindings/net/can/microchip,mcp251xfd.yaml b/Documentation/devicetree/bindings/net/can/microchip,mcp251xfd.yaml
> index 7a73057707b4..0415c3a886ca 100644
> --- a/Documentation/devicetree/bindings/net/can/microchip,mcp251xfd.yaml
> +++ b/Documentation/devicetree/bindings/net/can/microchip,mcp251xfd.yaml
> @@ -42,7 +42,7 @@ properties:
>
> microchip,rx-int-gpios:
> description:
> - GPIO phandle of GPIO connected to to INT1 pin of the MCP251XFD, which
> + GPIO phandle of GPIO connected to INT1 pin of the MCP251XFD, which
> signals a pending RX interrupt.
> maxItems: 1
>
> diff --git a/Documentation/driver-api/isa.rst b/Documentation/driver-api/isa.rst
> index def4a7b690b5..3df1b1696524 100644
> --- a/Documentation/driver-api/isa.rst
> +++ b/Documentation/driver-api/isa.rst
> @@ -100,7 +100,7 @@ I believe platform_data is available for this, but if rather not, moving
> the isa_driver pointer to the private struct isa_dev is ofcourse fine as
> well.
>
> -Then, if the the driver did not provide a .match, it matches. If it did,
> +Then, if the driver did not provide a .match, it matches. If it did,
> the driver match() method is called to determine a match.
>
> If it did **not** match, dev->platform_data is reset to indicate this to
> diff --git a/Documentation/filesystems/caching/backend-api.rst b/Documentation/filesystems/caching/backend-api.rst
> index d7507becf674..3a199fc50828 100644
> --- a/Documentation/filesystems/caching/backend-api.rst
> +++ b/Documentation/filesystems/caching/backend-api.rst
> @@ -122,7 +122,7 @@ volumes, calling::
> to tell fscache that a volume has been withdrawn. This waits for all
> outstanding accesses on the volume to complete before returning.
>
> -When the the cache is completely withdrawn, fscache should be notified by
> +When the cache is completely withdrawn, fscache should be notified by
> calling::
>
> void fscache_relinquish_cache(struct fscache_cache *cache);
> diff --git a/Documentation/filesystems/journalling.rst b/Documentation/filesystems/journalling.rst
> index e18f90ffc6fd..2a69bd2d95ed 100644
> --- a/Documentation/filesystems/journalling.rst
> +++ b/Documentation/filesystems/journalling.rst
> @@ -93,7 +93,7 @@ easily as on jbd2_journal_start().
>
> Try to reserve the right number of blocks the first time. ;-). This will
> be the maximum number of blocks you are going to touch in this
> -transaction. I advise having a look at at least ext4_jbd.h to see the
> +transaction. I advise having at least a look at ext4_jbd.h to see the
> basis on which ext4 uses to make these decisions.
>
> Another wriggle to watch out for is your on-disk block allocation
> diff --git a/Documentation/hwmon/f71882fg.rst b/Documentation/hwmon/f71882fg.rst
> index 38e30fbd4806..ab83bc7bbbdf 100644
> --- a/Documentation/hwmon/f71882fg.rst
> +++ b/Documentation/hwmon/f71882fg.rst
> @@ -179,7 +179,7 @@ Writing an unsupported mode will result in an invalid parameter error.
>
> * 2: Normal auto mode
> You can define a number of temperature/fan speed trip points, which % the
> - fan should run at at this temp and which temp a fan should follow using the
> + fan should run at this temp and which temp a fan should follow using the
> standard sysfs interface. The number and type of trip points is chip
> depended, see which files are available in sysfs.
> Fan/PWM channel 3 of the F8000 is always in this mode!
> diff --git a/Documentation/locking/seqlock.rst b/Documentation/locking/seqlock.rst
> index 64405e5da63e..bfda1a5fecad 100644
> --- a/Documentation/locking/seqlock.rst
> +++ b/Documentation/locking/seqlock.rst
> @@ -39,7 +39,7 @@ as the writer can invalidate a pointer that the reader is following.
> Sequence counters (``seqcount_t``)
> ==================================
>
> -This is the the raw counting mechanism, which does not protect against
> +This is the raw counting mechanism, which does not protect against
> multiple writers. Write side critical sections must thus be serialized
> by an external lock.
>
> diff --git a/Documentation/sphinx/cdomain.py b/Documentation/sphinx/cdomain.py
> index ca8ac9e59ded..a7d1866e72ff 100644
> --- a/Documentation/sphinx/cdomain.py
> +++ b/Documentation/sphinx/cdomain.py
> @@ -151,7 +151,7 @@ class CObject(Base_CObject):
> def handle_func_like_macro(self, sig, signode):
> u"""Handles signatures of function-like macros.
>
> - If the objtype is 'function' and the the signature ``sig`` is a
> + If the objtype is 'function' and the signature ``sig`` is a
> function-like macro, the name of the macro is returned. Otherwise
> ``False`` is returned. """
>
> diff --git a/Documentation/trace/histogram.rst b/Documentation/trace/histogram.rst
> index 859fd1b76c63..c1b685a38f6b 100644
> --- a/Documentation/trace/histogram.rst
> +++ b/Documentation/trace/histogram.rst
> @@ -412,7 +412,7 @@ Extended error information
> Because the default sort key above is 'hitcount', the above shows a
> the list of call_sites by increasing hitcount, so that at the bottom
> we see the functions that made the most kmalloc calls during the
> - run. If instead we we wanted to see the top kmalloc callers in
> + run. If instead we wanted to see the top kmalloc callers in
> terms of the number of bytes requested rather than the number of
> calls, and we wanted the top caller to appear at the top, we can use
> the 'sort' parameter, along with the 'descending' modifier::
> diff --git a/Documentation/userspace-api/media/dvb/dmx-reqbufs.rst b/Documentation/userspace-api/media/dvb/dmx-reqbufs.rst
> index d2bb1909ec98..18810f0bbca8 100644
> --- a/Documentation/userspace-api/media/dvb/dmx-reqbufs.rst
> +++ b/Documentation/userspace-api/media/dvb/dmx-reqbufs.rst
> @@ -72,4 +72,4 @@ appropriately. The generic error codes are described at the
> :ref:`Generic Error Codes <gen-errors>` chapter.
>
> EOPNOTSUPP
> - The the requested I/O method is not supported.
> + The requested I/O method is not supported.
> diff --git a/Documentation/userspace-api/media/dvb/frontend_f_open.rst b/Documentation/userspace-api/media/dvb/frontend_f_open.rst
> index bb37eded0870..70e169b8f601 100644
> --- a/Documentation/userspace-api/media/dvb/frontend_f_open.rst
> +++ b/Documentation/userspace-api/media/dvb/frontend_f_open.rst
> @@ -91,7 +91,7 @@ appropriately.
> - The caller has no permission to access the device.
>
> - - ``EBUSY``
> - - The the device driver is already in use.
> + - The device driver is already in use.
>
> - - ``EMFILE``
> - The process already has the maximum number of files open.
Powered by blists - more mailing lists