[PATCH] treewide: Fix common grammar mistake "the the"

2024-04-12 Thread Thorsten Blum
Use `find . -type f -exec sed -i 's/\/the/g' {} +` to find all
occurrences of "the the" and replace them with a single "the".

Changes only comments and documentation - no code changes.

Signed-off-by: Thorsten Blum 
---
 Documentation/trace/histogram.rst | 2 +-
 arch/arm/Kconfig  | 4 ++--
 arch/arm/include/asm/unwind.h | 2 +-
 arch/arm64/Kconfig| 2 +-
 arch/arm64/kernel/entry-ftrace.S  | 2 +-
 arch/s390/kernel/perf_cpum_sf.c   | 2 +-
 arch/s390/kernel/sthyi.c  | 2 +-
 drivers/accessibility/speakup/speakup_soft.c  | 2 +-
 drivers/gpu/drm/i915/display/intel_crt.c  | 2 +-
 drivers/gpu/drm/i915/i915_request.c   | 2 +-
 drivers/mailbox/Kconfig   | 2 +-
 drivers/net/wireless/intel/iwlwifi/fw/api/tx.h| 4 ++--
 drivers/net/wireless/intel/iwlwifi/mvm/phy-ctxt.c | 2 +-
 drivers/scsi/bfa/bfa_fcs_rport.c  | 2 +-
 drivers/scsi/fcoe/fcoe_ctlr.c | 2 +-
 drivers/scsi/isci/host.h  | 2 +-
 drivers/scsi/isci/remote_device.h | 2 +-
 drivers/scsi/isci/remote_node_context.h   | 2 +-
 drivers/scsi/isci/task.c  | 2 +-
 fs/afs/flock.c| 2 +-
 fs/ecryptfs/keystore.c| 2 +-
 fs/netfs/direct_read.c| 2 +-
 fs/netfs/direct_write.c   | 2 +-
 fs/overlayfs/super.c  | 2 +-
 include/uapi/asm-generic/fcntl.h  | 2 +-
 io_uring/kbuf.c   | 2 +-
 lib/zstd/common/fse_decompress.c  | 2 +-
 lib/zstd/decompress/zstd_decompress_block.c   | 2 +-
 scripts/coccinelle/misc/badty.cocci   | 2 +-
 tools/perf/Documentation/perf-diff.txt| 2 +-
 30 files changed, 32 insertions(+), 32 deletions(-)

diff --git a/Documentation/trace/histogram.rst 
b/Documentation/trace/histogram.rst
index 3c9b263de9c2..18a419925a08 100644
--- a/Documentation/trace/histogram.rst
+++ b/Documentation/trace/histogram.rst
@@ -840,7 +840,7 @@ Extended error information
 
   The compound key examples used a key and a sum value (hitcount) to
   sort the output, but we can just as easily use two keys instead.
-  Here's an example where we use a compound key composed of the the
+  Here's an example where we use a compound key composed of the
   common_pid and size event fields.  Sorting with pid as the primary
   key and 'size' as the secondary key allows us to display an
   ordered summary of the recvfrom sizes, with counts, received by
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index b14aed3a17ab..f46fb69ff247 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1479,7 +1479,7 @@ config ARM_ATAG_DTB_COMPAT_CMDLINE_EXTEND
bool "Extend with bootloader kernel arguments"
help
  The command-line arguments provided by the boot loader will be
- appended to the the device tree bootargs property.
+ appended to the device tree bootargs property.
 
 endchoice
 
@@ -1617,7 +1617,7 @@ config DMI
  continue to boot on existing non-UEFI platforms.
 
  NOTE: This does *NOT* enable or encourage the use of DMI quirks,
- i.e., the the practice of identifying the platform via DMI to
+ i.e., the practice of identifying the platform via DMI to
  decide whether certain workarounds for buggy hardware and/or
  firmware need to be enabled. This would require the DMI subsystem
  to be enabled much earlier than we do on ARM, which is non-trivial.
diff --git a/arch/arm/include/asm/unwind.h b/arch/arm/include/asm/unwind.h
index d60b09a5acfc..a75da9a01f91 100644
--- a/arch/arm/include/asm/unwind.h
+++ b/arch/arm/include/asm/unwind.h
@@ -10,7 +10,7 @@
 
 #ifndef __ASSEMBLY__
 
-/* Unwind reason code according the the ARM EABI documents */
+/* Unwind reason code according the ARM EABI documents */
 enum unwind_reason_code {
URC_OK = 0, /* operation completed successfully */
URC_CONTINUE_UNWIND = 8,
diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
index 7b11c98b3e84..285ae4ca0b83 100644
--- a/arch/arm64/Kconfig
+++ b/arch/arm64/Kconfig
@@ -2253,7 +2253,7 @@ config CMDLINE
default ""
help
  Provide a set of default command-line options at build time by
- entering them here. As a minimum, you should specify the the
+ entering them here. As a minimum, you should specify the
  root device (e.g. root=/dev/nfs).
 
 choice
diff --git a/arch/arm64/kernel/entry-ftrace.S b/arch/arm64/kernel/entry-ftrace.S
index f0c16640ef21..e24e7d8f8b61 100644
--- a/arch/arm64/kernel/entry-ftrace.S
+++ b/arch/arm64/kernel/entry-ftrace.S
@@ -94,7 +94,7 @@ SYM_CODE_START(ftrace_caller)
stp x29, x30, [sp, 

Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-12 Thread Tyler Hicks
On 2024-04-11 17:04:40, Thorsten Blum wrote:
> Use `find . -type f -exec sed -i 's/\/the/g' {} +` to find all
> occurrences of "the the" and replace them with a single "the".
> 
> Changes only comments and documentation - no code changes.
> 
> Signed-off-by: Thorsten Blum 

Reviewed-by: Tyler Hicks 

Tyler


Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-12 Thread Thorsten Blum
On 11. Apr 2024, at 17:25, Dan Carpenter  wrote:
> 
> It's tricky to know which tree a patch like this would go through.

The patch is based on the mainline tree. Should I have sent it directly to
Linus then?

I'm relatively new here and therefore only sent it to the corresponding mailing
lists.

Thanks,
Thorsten

Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-12 Thread Jani Nikula
On Fri, 12 Apr 2024, Thorsten Blum  wrote:
> On 11. Apr 2024, at 17:25, Dan Carpenter  wrote:
>> 
>> It's tricky to know which tree a patch like this would go through.
>
> The patch is based on the mainline tree. Should I have sent it directly to
> Linus then?
>
> I'm relatively new here and therefore only sent it to the corresponding 
> mailing
> lists.

It's not just about mailing lists, it's also about changes flowing in to
mainline via a plethora of subsystem and driver repos and branches. See
MAINTAINERS. Or this old LWN page [1]. The development happens in those
branches, and if you make treewide changes like this, it's not clear who
should take it, and you also risk unnecessary conflicts if those places
get modified in the individual branches. So it might just be easiest to
split this up to smaller patches sent to the appropriate lists.

Anyway, all that said,

Acked-by: Jani Nikula 

on the i915 changes here.


BR,
Jani.


[1] https://lwn.net/Articles/737094/

-- 
Jani Nikula, Intel


Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-11 Thread Robin Murphy

On 11/04/2024 4:04 pm, Thorsten Blum wrote:

Use `find . -type f -exec sed -i 's/\/the/g' {} +` to find all
occurrences of "the the" and replace them with a single "the".


[...]

diff --git a/arch/arm/include/asm/unwind.h b/arch/arm/include/asm/unwind.h
index d60b09a5acfc..a75da9a01f91 100644
--- a/arch/arm/include/asm/unwind.h
+++ b/arch/arm/include/asm/unwind.h
@@ -10,7 +10,7 @@
  
  #ifndef __ASSEMBLY__
  
-/* Unwind reason code according the the ARM EABI documents */

+/* Unwind reason code according the ARM EABI documents */


Well, that's clearly still not right... repeated words aren't *always* 
redundant, sometimes they're meant to be other words ;)


Thanks,
Robin.


Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-11 Thread Johannes Berg
On Thu, 2024-04-11 at 17:04 +0200, Thorsten Blum wrote:
> Use `find . -type f -exec sed -i 's/\/the/g' {} +` to find all
> occurrences of "the the" and replace them with a single "the".

I estimated that this misses at least ~50 instances split across lines:

$ git grep -ih -A1 -e 'the$'|grep -vi 'the$'|grep -E -- '^[^a-zA-Z0-9]*the '|wc 
-l
51

And a bunch that have more than one space:

$ git grep -E '\'|wc -l
20

So not sure you should claim "all" ;-)

johannes


Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-11 Thread Dan Carpenter
On Thu, Apr 11, 2024 at 05:04:40PM +0200, Thorsten Blum wrote:
> Use `find . -type f -exec sed -i 's/\/the/g' {} +` to find all
> occurrences of "the the" and replace them with a single "the".
> 
> Changes only comments and documentation - no code changes.
> 
> Signed-off-by: Thorsten Blum 
> ---

It's tricky to know which tree a patch like this would go through.  We
used to have a trivial tree for this stuff but I guess that didn't work.
It's possible that it could go through linux-doc, but probably it has to
go as a set of patches through each of the trees in the CC list.

regards,
dan carpenter



Re: [PATCH] treewide: Fix common grammar mistake "the the"

2024-04-11 Thread Randy Dunlap



On 4/11/24 8:04 AM, Thorsten Blum wrote:
> Use `find . -type f -exec sed -i 's/\/the/g' {} +` to find all
> occurrences of "the the" and replace them with a single "the".
> 
> Changes only comments and documentation - no code changes.
> 
> Signed-off-by: Thorsten Blum 
> ---
>  Documentation/trace/histogram.rst | 2 +-
>  arch/arm/Kconfig  | 4 ++--
>  arch/arm/include/asm/unwind.h | 2 +-
>  arch/arm64/Kconfig| 2 +-
>  arch/arm64/kernel/entry-ftrace.S  | 2 +-
>  arch/s390/kernel/perf_cpum_sf.c   | 2 +-
>  arch/s390/kernel/sthyi.c  | 2 +-
>  drivers/accessibility/speakup/speakup_soft.c  | 2 +-
>  drivers/gpu/drm/i915/display/intel_crt.c  | 2 +-
>  drivers/gpu/drm/i915/i915_request.c   | 2 +-
>  drivers/mailbox/Kconfig   | 2 +-
>  drivers/net/wireless/intel/iwlwifi/fw/api/tx.h| 4 ++--
>  drivers/net/wireless/intel/iwlwifi/mvm/phy-ctxt.c | 2 +-
>  drivers/scsi/bfa/bfa_fcs_rport.c  | 2 +-
>  drivers/scsi/fcoe/fcoe_ctlr.c | 2 +-
>  drivers/scsi/isci/host.h  | 2 +-
>  drivers/scsi/isci/remote_device.h | 2 +-
>  drivers/scsi/isci/remote_node_context.h   | 2 +-
>  drivers/scsi/isci/task.c  | 2 +-
>  fs/afs/flock.c| 2 +-
>  fs/ecryptfs/keystore.c| 2 +-
>  fs/netfs/direct_read.c| 2 +-
>  fs/netfs/direct_write.c   | 2 +-
>  fs/overlayfs/super.c  | 2 +-
>  include/uapi/asm-generic/fcntl.h  | 2 +-
>  io_uring/kbuf.c   | 2 +-
>  lib/zstd/common/fse_decompress.c  | 2 +-
>  lib/zstd/decompress/zstd_decompress_block.c   | 2 +-
>  scripts/coccinelle/misc/badty.cocci   | 2 +-
>  tools/perf/Documentation/perf-diff.txt| 2 +-
>  30 files changed, 32 insertions(+), 32 deletions(-)
> 

Reviewed-by: Randy Dunlap 

Thanks.

-- 
#Randy