From: Jani Nikula <jani.nikula@linux.intel.com>
To: cgel.zte@gmail.com, alexander.deucher@amd.com,
christian.koenig@amd.com, Xinhui.Pan@amd.com, airlied@linux.ie
Cc: linux-rtc@vger.kernel.org, tvrtko.ursulin@linux.intel.com,
linux-scsi@vger.kernel.org, netdev@vger.kernel.org,
lv.ruyi@zte.com.cn, joonas.lahtinen@linux.intel.com,
linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
amd-gfx@lists.freedesktop.org,
platform-driver-x86@vger.kernel.org,
dri-devel@lists.freedesktop.org, daniel@ffwll.ch,
rodrigo.vivi@intel.com, Zeal Robot <zealci@zte.com.cn>,
intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] drivers: Fix spelling mistake "writting" -> "writing"
Date: Fri, 08 Apr 2022 13:17:27 +0300 [thread overview]
Message-ID: <87sfqnj2vc.fsf@intel.com> (raw)
In-Reply-To: <20220408095531.2495168-1-lv.ruyi@zte.com.cn>
On Fri, 08 Apr 2022, cgel.zte@gmail.com wrote:
> From: Lv Ruyi <lv.ruyi@zte.com.cn>
>
> There are some spelling mistakes in the comments. Fix it.
Please prefer splitting by driver. This isn't even split by subsystem. I
presume there are very few maintainers willing to pick this up as it is.
BR,
Jani.
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Lv Ruyi <lv.ruyi@zte.com.cn>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +-
> drivers/gpu/drm/i915/i915_request.c | 2 +-
> drivers/net/ethernet/sfc/mcdi_pcol.h | 4 ++--
> drivers/net/ethernet/toshiba/tc35815.c | 2 +-
> drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c | 4 ++--
> drivers/platform/x86/hp_accel.c | 2 +-
> drivers/rtc/rtc-sa1100.c | 2 +-
> drivers/scsi/pmcraid.c | 4 ++--
> 8 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 9426e252d8aa..ce361fce7155 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -7304,7 +7304,7 @@ static void gfx_v10_0_setup_grbm_cam_remapping(struct amdgpu_device *adev)
> return;
>
> /* initialize cam_index to 0
> - * index will auto-inc after each data writting */
> + * index will auto-inc after each data writing */
> WREG32_SOC15(GC, 0, mmGRBM_CAM_INDEX, 0);
>
> switch (adev->ip_versions[GC_HWIP][0]) {
> diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c
> index 582770360ad1..cf79a25cd98a 100644
> --- a/drivers/gpu/drm/i915/i915_request.c
> +++ b/drivers/gpu/drm/i915/i915_request.c
> @@ -451,7 +451,7 @@ static bool __request_in_flight(const struct i915_request *signal)
> * to avoid tearing.]
> *
> * Note that the read of *execlists->active may race with the promotion
> - * of execlists->pending[] to execlists->inflight[], overwritting
> + * of execlists->pending[] to execlists->inflight[], overwriting
> * the value at *execlists->active. This is fine. The promotion implies
> * that we received an ACK from the HW, and so the context is not
> * stuck -- if we do not see ourselves in *active, the inflight status
> diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h b/drivers/net/ethernet/sfc/mcdi_pcol.h
> index d3fcbf930dba..ff617b1b38d3 100644
> --- a/drivers/net/ethernet/sfc/mcdi_pcol.h
> +++ b/drivers/net/ethernet/sfc/mcdi_pcol.h
> @@ -73,8 +73,8 @@
> * \------------------------------ Resync (always set)
> *
> * The client writes it's request into MC shared memory, and rings the
> - * doorbell. Each request is completed by either by the MC writting
> - * back into shared memory, or by writting out an event.
> + * doorbell. Each request is completed by either by the MC writing
> + * back into shared memory, or by writing out an event.
> *
> * All MCDI commands support completion by shared memory response. Each
> * request may also contain additional data (accounted for by HEADER.LEN),
> diff --git a/drivers/net/ethernet/toshiba/tc35815.c b/drivers/net/ethernet/toshiba/tc35815.c
> index ce38f7515225..1b4c207afb66 100644
> --- a/drivers/net/ethernet/toshiba/tc35815.c
> +++ b/drivers/net/ethernet/toshiba/tc35815.c
> @@ -157,7 +157,7 @@ struct tc35815_regs {
> #define PROM_Read 0x00004000 /*10:Read operation */
> #define PROM_Write 0x00002000 /*01:Write operation */
> #define PROM_Erase 0x00006000 /*11:Erase operation */
> - /*00:Enable or Disable Writting, */
> + /*00:Enable or Disable Writing, */
> /* as specified in PROM_Addr. */
> #define PROM_Addr_Ena 0x00000030 /*11xxxx:PROM Write enable */
> /*00xxxx: disable */
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> index eaba66113328..fbb4941d0da8 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> @@ -520,7 +520,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> * 2 out-ep. Remainder pages have assigned to High queue */
> if (outepnum > 1 && txqremaininpage)
> numhq += txqremaininpage;
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = txqpageunit;
> @@ -539,7 +539,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> numlq = ischipn ? WMM_CHIP_B_PAGE_NUM_LPQ :
> WMM_CHIP_A_PAGE_NUM_LPQ;
> }
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = WMM_CHIP_B_PAGE_NUM_NPQ;
> diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c
> index e9f852f7c27f..b59b852a666f 100644
> --- a/drivers/platform/x86/hp_accel.c
> +++ b/drivers/platform/x86/hp_accel.c
> @@ -122,7 +122,7 @@ static int lis3lv02d_acpi_read(struct lis3lv02d *lis3, int reg, u8 *ret)
> static int lis3lv02d_acpi_write(struct lis3lv02d *lis3, int reg, u8 val)
> {
> struct acpi_device *dev = lis3->bus_priv;
> - unsigned long long ret; /* Not used when writting */
> + unsigned long long ret; /* Not used when writing */
> union acpi_object in_obj[2];
> struct acpi_object_list args = { 2, in_obj };
>
> diff --git a/drivers/rtc/rtc-sa1100.c b/drivers/rtc/rtc-sa1100.c
> index 1250887e4382..a52a333de8e8 100644
> --- a/drivers/rtc/rtc-sa1100.c
> +++ b/drivers/rtc/rtc-sa1100.c
> @@ -231,7 +231,7 @@ int sa1100_rtc_init(struct platform_device *pdev, struct sa1100_rtc *info)
> * initialization is unknown and could in principle happen during
> * normal processing.
> *
> - * Notice that clearing bit 1 and 0 is accomplished by writting ONES to
> + * Notice that clearing bit 1 and 0 is accomplished by writing ONES to
> * the corresponding bits in RTSR. */
> writel_relaxed(RTSR_AL | RTSR_HZ, info->rtsr);
>
> diff --git a/drivers/scsi/pmcraid.c b/drivers/scsi/pmcraid.c
> index fd674ed1febe..d7f4680f6106 100644
> --- a/drivers/scsi/pmcraid.c
> +++ b/drivers/scsi/pmcraid.c
> @@ -857,7 +857,7 @@ static void _pmcraid_fire_command(struct pmcraid_cmd *cmd)
> unsigned long lock_flags;
>
> /* Add this command block to pending cmd pool. We do this prior to
> - * writting IOARCB to ioarrin because IOA might complete the command
> + * writing IOARCB to ioarrin because IOA might complete the command
> * by the time we are about to add it to the list. Response handler
> * (isr/tasklet) looks for cmd block in the pending pending list.
> */
> @@ -2450,7 +2450,7 @@ static void pmcraid_request_sense(struct pmcraid_cmd *cmd)
>
> /* request sense might be called as part of error response processing
> * which runs in tasklets context. It is possible that mid-layer might
> - * schedule queuecommand during this time, hence, writting to IOARRIN
> + * schedule queuecommand during this time, hence, writing to IOARRIN
> * must be protect by host_lock
> */
> pmcraid_send_cmd(cmd, pmcraid_erp_done,
--
Jani Nikula, Intel Open Source Graphics Center
WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@linux.intel.com>
To: cgel.zte@gmail.com, alexander.deucher@amd.com,
christian.koenig@amd.com, Xinhui.Pan@amd.com, airlied@linux.ie
Cc: linux-rtc@vger.kernel.org, linux-scsi@vger.kernel.org,
netdev@vger.kernel.org, lv.ruyi@zte.com.cn,
linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
amd-gfx@lists.freedesktop.org,
platform-driver-x86@vger.kernel.org,
dri-devel@lists.freedesktop.org, rodrigo.vivi@intel.com,
Zeal Robot <zealci@zte.com.cn>,
intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH] drivers: Fix spelling mistake "writting" -> "writing"
Date: Fri, 08 Apr 2022 13:17:27 +0300 [thread overview]
Message-ID: <87sfqnj2vc.fsf@intel.com> (raw)
In-Reply-To: <20220408095531.2495168-1-lv.ruyi@zte.com.cn>
On Fri, 08 Apr 2022, cgel.zte@gmail.com wrote:
> From: Lv Ruyi <lv.ruyi@zte.com.cn>
>
> There are some spelling mistakes in the comments. Fix it.
Please prefer splitting by driver. This isn't even split by subsystem. I
presume there are very few maintainers willing to pick this up as it is.
BR,
Jani.
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Lv Ruyi <lv.ruyi@zte.com.cn>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +-
> drivers/gpu/drm/i915/i915_request.c | 2 +-
> drivers/net/ethernet/sfc/mcdi_pcol.h | 4 ++--
> drivers/net/ethernet/toshiba/tc35815.c | 2 +-
> drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c | 4 ++--
> drivers/platform/x86/hp_accel.c | 2 +-
> drivers/rtc/rtc-sa1100.c | 2 +-
> drivers/scsi/pmcraid.c | 4 ++--
> 8 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 9426e252d8aa..ce361fce7155 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -7304,7 +7304,7 @@ static void gfx_v10_0_setup_grbm_cam_remapping(struct amdgpu_device *adev)
> return;
>
> /* initialize cam_index to 0
> - * index will auto-inc after each data writting */
> + * index will auto-inc after each data writing */
> WREG32_SOC15(GC, 0, mmGRBM_CAM_INDEX, 0);
>
> switch (adev->ip_versions[GC_HWIP][0]) {
> diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c
> index 582770360ad1..cf79a25cd98a 100644
> --- a/drivers/gpu/drm/i915/i915_request.c
> +++ b/drivers/gpu/drm/i915/i915_request.c
> @@ -451,7 +451,7 @@ static bool __request_in_flight(const struct i915_request *signal)
> * to avoid tearing.]
> *
> * Note that the read of *execlists->active may race with the promotion
> - * of execlists->pending[] to execlists->inflight[], overwritting
> + * of execlists->pending[] to execlists->inflight[], overwriting
> * the value at *execlists->active. This is fine. The promotion implies
> * that we received an ACK from the HW, and so the context is not
> * stuck -- if we do not see ourselves in *active, the inflight status
> diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h b/drivers/net/ethernet/sfc/mcdi_pcol.h
> index d3fcbf930dba..ff617b1b38d3 100644
> --- a/drivers/net/ethernet/sfc/mcdi_pcol.h
> +++ b/drivers/net/ethernet/sfc/mcdi_pcol.h
> @@ -73,8 +73,8 @@
> * \------------------------------ Resync (always set)
> *
> * The client writes it's request into MC shared memory, and rings the
> - * doorbell. Each request is completed by either by the MC writting
> - * back into shared memory, or by writting out an event.
> + * doorbell. Each request is completed by either by the MC writing
> + * back into shared memory, or by writing out an event.
> *
> * All MCDI commands support completion by shared memory response. Each
> * request may also contain additional data (accounted for by HEADER.LEN),
> diff --git a/drivers/net/ethernet/toshiba/tc35815.c b/drivers/net/ethernet/toshiba/tc35815.c
> index ce38f7515225..1b4c207afb66 100644
> --- a/drivers/net/ethernet/toshiba/tc35815.c
> +++ b/drivers/net/ethernet/toshiba/tc35815.c
> @@ -157,7 +157,7 @@ struct tc35815_regs {
> #define PROM_Read 0x00004000 /*10:Read operation */
> #define PROM_Write 0x00002000 /*01:Write operation */
> #define PROM_Erase 0x00006000 /*11:Erase operation */
> - /*00:Enable or Disable Writting, */
> + /*00:Enable or Disable Writing, */
> /* as specified in PROM_Addr. */
> #define PROM_Addr_Ena 0x00000030 /*11xxxx:PROM Write enable */
> /*00xxxx: disable */
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> index eaba66113328..fbb4941d0da8 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> @@ -520,7 +520,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> * 2 out-ep. Remainder pages have assigned to High queue */
> if (outepnum > 1 && txqremaininpage)
> numhq += txqremaininpage;
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = txqpageunit;
> @@ -539,7 +539,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> numlq = ischipn ? WMM_CHIP_B_PAGE_NUM_LPQ :
> WMM_CHIP_A_PAGE_NUM_LPQ;
> }
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = WMM_CHIP_B_PAGE_NUM_NPQ;
> diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c
> index e9f852f7c27f..b59b852a666f 100644
> --- a/drivers/platform/x86/hp_accel.c
> +++ b/drivers/platform/x86/hp_accel.c
> @@ -122,7 +122,7 @@ static int lis3lv02d_acpi_read(struct lis3lv02d *lis3, int reg, u8 *ret)
> static int lis3lv02d_acpi_write(struct lis3lv02d *lis3, int reg, u8 val)
> {
> struct acpi_device *dev = lis3->bus_priv;
> - unsigned long long ret; /* Not used when writting */
> + unsigned long long ret; /* Not used when writing */
> union acpi_object in_obj[2];
> struct acpi_object_list args = { 2, in_obj };
>
> diff --git a/drivers/rtc/rtc-sa1100.c b/drivers/rtc/rtc-sa1100.c
> index 1250887e4382..a52a333de8e8 100644
> --- a/drivers/rtc/rtc-sa1100.c
> +++ b/drivers/rtc/rtc-sa1100.c
> @@ -231,7 +231,7 @@ int sa1100_rtc_init(struct platform_device *pdev, struct sa1100_rtc *info)
> * initialization is unknown and could in principle happen during
> * normal processing.
> *
> - * Notice that clearing bit 1 and 0 is accomplished by writting ONES to
> + * Notice that clearing bit 1 and 0 is accomplished by writing ONES to
> * the corresponding bits in RTSR. */
> writel_relaxed(RTSR_AL | RTSR_HZ, info->rtsr);
>
> diff --git a/drivers/scsi/pmcraid.c b/drivers/scsi/pmcraid.c
> index fd674ed1febe..d7f4680f6106 100644
> --- a/drivers/scsi/pmcraid.c
> +++ b/drivers/scsi/pmcraid.c
> @@ -857,7 +857,7 @@ static void _pmcraid_fire_command(struct pmcraid_cmd *cmd)
> unsigned long lock_flags;
>
> /* Add this command block to pending cmd pool. We do this prior to
> - * writting IOARCB to ioarrin because IOA might complete the command
> + * writing IOARCB to ioarrin because IOA might complete the command
> * by the time we are about to add it to the list. Response handler
> * (isr/tasklet) looks for cmd block in the pending pending list.
> */
> @@ -2450,7 +2450,7 @@ static void pmcraid_request_sense(struct pmcraid_cmd *cmd)
>
> /* request sense might be called as part of error response processing
> * which runs in tasklets context. It is possible that mid-layer might
> - * schedule queuecommand during this time, hence, writting to IOARRIN
> + * schedule queuecommand during this time, hence, writing to IOARRIN
> * must be protect by host_lock
> */
> pmcraid_send_cmd(cmd, pmcraid_erp_done,
--
Jani Nikula, Intel Open Source Graphics Center
WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@linux.intel.com>
To: cgel.zte@gmail.com, alexander.deucher@amd.com,
christian.koenig@amd.com, Xinhui.Pan@amd.com, airlied@linux.ie
Cc: daniel@ffwll.ch, joonas.lahtinen@linux.intel.com,
rodrigo.vivi@intel.com, tvrtko.ursulin@linux.intel.com,
lv.ruyi@zte.com.cn, amd-gfx@lists.freedesktop.org,
dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
intel-gfx@lists.freedesktop.org, netdev@vger.kernel.org,
linux-wireless@vger.kernel.org,
platform-driver-x86@vger.kernel.org, linux-rtc@vger.kernel.org,
linux-scsi@vger.kernel.org, Zeal Robot <zealci@zte.com.cn>
Subject: Re: [PATCH] drivers: Fix spelling mistake "writting" -> "writing"
Date: Fri, 08 Apr 2022 13:17:27 +0300 [thread overview]
Message-ID: <87sfqnj2vc.fsf@intel.com> (raw)
In-Reply-To: <20220408095531.2495168-1-lv.ruyi@zte.com.cn>
On Fri, 08 Apr 2022, cgel.zte@gmail.com wrote:
> From: Lv Ruyi <lv.ruyi@zte.com.cn>
>
> There are some spelling mistakes in the comments. Fix it.
Please prefer splitting by driver. This isn't even split by subsystem. I
presume there are very few maintainers willing to pick this up as it is.
BR,
Jani.
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Lv Ruyi <lv.ruyi@zte.com.cn>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +-
> drivers/gpu/drm/i915/i915_request.c | 2 +-
> drivers/net/ethernet/sfc/mcdi_pcol.h | 4 ++--
> drivers/net/ethernet/toshiba/tc35815.c | 2 +-
> drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c | 4 ++--
> drivers/platform/x86/hp_accel.c | 2 +-
> drivers/rtc/rtc-sa1100.c | 2 +-
> drivers/scsi/pmcraid.c | 4 ++--
> 8 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 9426e252d8aa..ce361fce7155 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -7304,7 +7304,7 @@ static void gfx_v10_0_setup_grbm_cam_remapping(struct amdgpu_device *adev)
> return;
>
> /* initialize cam_index to 0
> - * index will auto-inc after each data writting */
> + * index will auto-inc after each data writing */
> WREG32_SOC15(GC, 0, mmGRBM_CAM_INDEX, 0);
>
> switch (adev->ip_versions[GC_HWIP][0]) {
> diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c
> index 582770360ad1..cf79a25cd98a 100644
> --- a/drivers/gpu/drm/i915/i915_request.c
> +++ b/drivers/gpu/drm/i915/i915_request.c
> @@ -451,7 +451,7 @@ static bool __request_in_flight(const struct i915_request *signal)
> * to avoid tearing.]
> *
> * Note that the read of *execlists->active may race with the promotion
> - * of execlists->pending[] to execlists->inflight[], overwritting
> + * of execlists->pending[] to execlists->inflight[], overwriting
> * the value at *execlists->active. This is fine. The promotion implies
> * that we received an ACK from the HW, and so the context is not
> * stuck -- if we do not see ourselves in *active, the inflight status
> diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h b/drivers/net/ethernet/sfc/mcdi_pcol.h
> index d3fcbf930dba..ff617b1b38d3 100644
> --- a/drivers/net/ethernet/sfc/mcdi_pcol.h
> +++ b/drivers/net/ethernet/sfc/mcdi_pcol.h
> @@ -73,8 +73,8 @@
> * \------------------------------ Resync (always set)
> *
> * The client writes it's request into MC shared memory, and rings the
> - * doorbell. Each request is completed by either by the MC writting
> - * back into shared memory, or by writting out an event.
> + * doorbell. Each request is completed by either by the MC writing
> + * back into shared memory, or by writing out an event.
> *
> * All MCDI commands support completion by shared memory response. Each
> * request may also contain additional data (accounted for by HEADER.LEN),
> diff --git a/drivers/net/ethernet/toshiba/tc35815.c b/drivers/net/ethernet/toshiba/tc35815.c
> index ce38f7515225..1b4c207afb66 100644
> --- a/drivers/net/ethernet/toshiba/tc35815.c
> +++ b/drivers/net/ethernet/toshiba/tc35815.c
> @@ -157,7 +157,7 @@ struct tc35815_regs {
> #define PROM_Read 0x00004000 /*10:Read operation */
> #define PROM_Write 0x00002000 /*01:Write operation */
> #define PROM_Erase 0x00006000 /*11:Erase operation */
> - /*00:Enable or Disable Writting, */
> + /*00:Enable or Disable Writing, */
> /* as specified in PROM_Addr. */
> #define PROM_Addr_Ena 0x00000030 /*11xxxx:PROM Write enable */
> /*00xxxx: disable */
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> index eaba66113328..fbb4941d0da8 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> @@ -520,7 +520,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> * 2 out-ep. Remainder pages have assigned to High queue */
> if (outepnum > 1 && txqremaininpage)
> numhq += txqremaininpage;
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = txqpageunit;
> @@ -539,7 +539,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> numlq = ischipn ? WMM_CHIP_B_PAGE_NUM_LPQ :
> WMM_CHIP_A_PAGE_NUM_LPQ;
> }
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = WMM_CHIP_B_PAGE_NUM_NPQ;
> diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c
> index e9f852f7c27f..b59b852a666f 100644
> --- a/drivers/platform/x86/hp_accel.c
> +++ b/drivers/platform/x86/hp_accel.c
> @@ -122,7 +122,7 @@ static int lis3lv02d_acpi_read(struct lis3lv02d *lis3, int reg, u8 *ret)
> static int lis3lv02d_acpi_write(struct lis3lv02d *lis3, int reg, u8 val)
> {
> struct acpi_device *dev = lis3->bus_priv;
> - unsigned long long ret; /* Not used when writting */
> + unsigned long long ret; /* Not used when writing */
> union acpi_object in_obj[2];
> struct acpi_object_list args = { 2, in_obj };
>
> diff --git a/drivers/rtc/rtc-sa1100.c b/drivers/rtc/rtc-sa1100.c
> index 1250887e4382..a52a333de8e8 100644
> --- a/drivers/rtc/rtc-sa1100.c
> +++ b/drivers/rtc/rtc-sa1100.c
> @@ -231,7 +231,7 @@ int sa1100_rtc_init(struct platform_device *pdev, struct sa1100_rtc *info)
> * initialization is unknown and could in principle happen during
> * normal processing.
> *
> - * Notice that clearing bit 1 and 0 is accomplished by writting ONES to
> + * Notice that clearing bit 1 and 0 is accomplished by writing ONES to
> * the corresponding bits in RTSR. */
> writel_relaxed(RTSR_AL | RTSR_HZ, info->rtsr);
>
> diff --git a/drivers/scsi/pmcraid.c b/drivers/scsi/pmcraid.c
> index fd674ed1febe..d7f4680f6106 100644
> --- a/drivers/scsi/pmcraid.c
> +++ b/drivers/scsi/pmcraid.c
> @@ -857,7 +857,7 @@ static void _pmcraid_fire_command(struct pmcraid_cmd *cmd)
> unsigned long lock_flags;
>
> /* Add this command block to pending cmd pool. We do this prior to
> - * writting IOARCB to ioarrin because IOA might complete the command
> + * writing IOARCB to ioarrin because IOA might complete the command
> * by the time we are about to add it to the list. Response handler
> * (isr/tasklet) looks for cmd block in the pending pending list.
> */
> @@ -2450,7 +2450,7 @@ static void pmcraid_request_sense(struct pmcraid_cmd *cmd)
>
> /* request sense might be called as part of error response processing
> * which runs in tasklets context. It is possible that mid-layer might
> - * schedule queuecommand during this time, hence, writting to IOARRIN
> + * schedule queuecommand during this time, hence, writing to IOARRIN
> * must be protect by host_lock
> */
> pmcraid_send_cmd(cmd, pmcraid_erp_done,
--
Jani Nikula, Intel Open Source Graphics Center
WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@linux.intel.com>
To: cgel.zte@gmail.com, alexander.deucher@amd.com,
christian.koenig@amd.com, Xinhui.Pan@amd.com, airlied@linux.ie
Cc: linux-rtc@vger.kernel.org, tvrtko.ursulin@linux.intel.com,
linux-scsi@vger.kernel.org, netdev@vger.kernel.org,
lv.ruyi@zte.com.cn, linux-wireless@vger.kernel.org,
linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
platform-driver-x86@vger.kernel.org,
dri-devel@lists.freedesktop.org, rodrigo.vivi@intel.com,
Zeal Robot <zealci@zte.com.cn>,
intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] drivers: Fix spelling mistake "writting" -> "writing"
Date: Fri, 08 Apr 2022 13:17:27 +0300 [thread overview]
Message-ID: <87sfqnj2vc.fsf@intel.com> (raw)
In-Reply-To: <20220408095531.2495168-1-lv.ruyi@zte.com.cn>
On Fri, 08 Apr 2022, cgel.zte@gmail.com wrote:
> From: Lv Ruyi <lv.ruyi@zte.com.cn>
>
> There are some spelling mistakes in the comments. Fix it.
Please prefer splitting by driver. This isn't even split by subsystem. I
presume there are very few maintainers willing to pick this up as it is.
BR,
Jani.
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Lv Ruyi <lv.ruyi@zte.com.cn>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +-
> drivers/gpu/drm/i915/i915_request.c | 2 +-
> drivers/net/ethernet/sfc/mcdi_pcol.h | 4 ++--
> drivers/net/ethernet/toshiba/tc35815.c | 2 +-
> drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c | 4 ++--
> drivers/platform/x86/hp_accel.c | 2 +-
> drivers/rtc/rtc-sa1100.c | 2 +-
> drivers/scsi/pmcraid.c | 4 ++--
> 8 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 9426e252d8aa..ce361fce7155 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -7304,7 +7304,7 @@ static void gfx_v10_0_setup_grbm_cam_remapping(struct amdgpu_device *adev)
> return;
>
> /* initialize cam_index to 0
> - * index will auto-inc after each data writting */
> + * index will auto-inc after each data writing */
> WREG32_SOC15(GC, 0, mmGRBM_CAM_INDEX, 0);
>
> switch (adev->ip_versions[GC_HWIP][0]) {
> diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c
> index 582770360ad1..cf79a25cd98a 100644
> --- a/drivers/gpu/drm/i915/i915_request.c
> +++ b/drivers/gpu/drm/i915/i915_request.c
> @@ -451,7 +451,7 @@ static bool __request_in_flight(const struct i915_request *signal)
> * to avoid tearing.]
> *
> * Note that the read of *execlists->active may race with the promotion
> - * of execlists->pending[] to execlists->inflight[], overwritting
> + * of execlists->pending[] to execlists->inflight[], overwriting
> * the value at *execlists->active. This is fine. The promotion implies
> * that we received an ACK from the HW, and so the context is not
> * stuck -- if we do not see ourselves in *active, the inflight status
> diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h b/drivers/net/ethernet/sfc/mcdi_pcol.h
> index d3fcbf930dba..ff617b1b38d3 100644
> --- a/drivers/net/ethernet/sfc/mcdi_pcol.h
> +++ b/drivers/net/ethernet/sfc/mcdi_pcol.h
> @@ -73,8 +73,8 @@
> * \------------------------------ Resync (always set)
> *
> * The client writes it's request into MC shared memory, and rings the
> - * doorbell. Each request is completed by either by the MC writting
> - * back into shared memory, or by writting out an event.
> + * doorbell. Each request is completed by either by the MC writing
> + * back into shared memory, or by writing out an event.
> *
> * All MCDI commands support completion by shared memory response. Each
> * request may also contain additional data (accounted for by HEADER.LEN),
> diff --git a/drivers/net/ethernet/toshiba/tc35815.c b/drivers/net/ethernet/toshiba/tc35815.c
> index ce38f7515225..1b4c207afb66 100644
> --- a/drivers/net/ethernet/toshiba/tc35815.c
> +++ b/drivers/net/ethernet/toshiba/tc35815.c
> @@ -157,7 +157,7 @@ struct tc35815_regs {
> #define PROM_Read 0x00004000 /*10:Read operation */
> #define PROM_Write 0x00002000 /*01:Write operation */
> #define PROM_Erase 0x00006000 /*11:Erase operation */
> - /*00:Enable or Disable Writting, */
> + /*00:Enable or Disable Writing, */
> /* as specified in PROM_Addr. */
> #define PROM_Addr_Ena 0x00000030 /*11xxxx:PROM Write enable */
> /*00xxxx: disable */
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> index eaba66113328..fbb4941d0da8 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> @@ -520,7 +520,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> * 2 out-ep. Remainder pages have assigned to High queue */
> if (outepnum > 1 && txqremaininpage)
> numhq += txqremaininpage;
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = txqpageunit;
> @@ -539,7 +539,7 @@ static void _rtl92cu_init_queue_reserved_page(struct ieee80211_hw *hw,
> numlq = ischipn ? WMM_CHIP_B_PAGE_NUM_LPQ :
> WMM_CHIP_A_PAGE_NUM_LPQ;
> }
> - /* NOTE: This step done before writting REG_RQPN. */
> + /* NOTE: This step done before writing REG_RQPN. */
> if (ischipn) {
> if (queue_sel & TX_SELE_NQ)
> numnq = WMM_CHIP_B_PAGE_NUM_NPQ;
> diff --git a/drivers/platform/x86/hp_accel.c b/drivers/platform/x86/hp_accel.c
> index e9f852f7c27f..b59b852a666f 100644
> --- a/drivers/platform/x86/hp_accel.c
> +++ b/drivers/platform/x86/hp_accel.c
> @@ -122,7 +122,7 @@ static int lis3lv02d_acpi_read(struct lis3lv02d *lis3, int reg, u8 *ret)
> static int lis3lv02d_acpi_write(struct lis3lv02d *lis3, int reg, u8 val)
> {
> struct acpi_device *dev = lis3->bus_priv;
> - unsigned long long ret; /* Not used when writting */
> + unsigned long long ret; /* Not used when writing */
> union acpi_object in_obj[2];
> struct acpi_object_list args = { 2, in_obj };
>
> diff --git a/drivers/rtc/rtc-sa1100.c b/drivers/rtc/rtc-sa1100.c
> index 1250887e4382..a52a333de8e8 100644
> --- a/drivers/rtc/rtc-sa1100.c
> +++ b/drivers/rtc/rtc-sa1100.c
> @@ -231,7 +231,7 @@ int sa1100_rtc_init(struct platform_device *pdev, struct sa1100_rtc *info)
> * initialization is unknown and could in principle happen during
> * normal processing.
> *
> - * Notice that clearing bit 1 and 0 is accomplished by writting ONES to
> + * Notice that clearing bit 1 and 0 is accomplished by writing ONES to
> * the corresponding bits in RTSR. */
> writel_relaxed(RTSR_AL | RTSR_HZ, info->rtsr);
>
> diff --git a/drivers/scsi/pmcraid.c b/drivers/scsi/pmcraid.c
> index fd674ed1febe..d7f4680f6106 100644
> --- a/drivers/scsi/pmcraid.c
> +++ b/drivers/scsi/pmcraid.c
> @@ -857,7 +857,7 @@ static void _pmcraid_fire_command(struct pmcraid_cmd *cmd)
> unsigned long lock_flags;
>
> /* Add this command block to pending cmd pool. We do this prior to
> - * writting IOARCB to ioarrin because IOA might complete the command
> + * writing IOARCB to ioarrin because IOA might complete the command
> * by the time we are about to add it to the list. Response handler
> * (isr/tasklet) looks for cmd block in the pending pending list.
> */
> @@ -2450,7 +2450,7 @@ static void pmcraid_request_sense(struct pmcraid_cmd *cmd)
>
> /* request sense might be called as part of error response processing
> * which runs in tasklets context. It is possible that mid-layer might
> - * schedule queuecommand during this time, hence, writting to IOARRIN
> + * schedule queuecommand during this time, hence, writing to IOARRIN
> * must be protect by host_lock
> */
> pmcraid_send_cmd(cmd, pmcraid_erp_done,
--
Jani Nikula, Intel Open Source Graphics Center
next prev parent reply other threads:[~2022-04-08 10:17 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-08 9:55 [PATCH] drivers: Fix spelling mistake "writting" -> "writing" cgel.zte
2022-04-08 9:55 ` cgel.zte
2022-04-08 9:55 ` cgel.zte
2022-04-08 9:55 ` [Intel-gfx] " cgel.zte
2022-04-08 10:17 ` Jani Nikula [this message]
2022-04-08 10:17 ` Jani Nikula
2022-04-08 10:17 ` Jani Nikula
2022-04-08 10:17 ` [Intel-gfx] " Jani Nikula
2022-04-11 21:51 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2022-04-11 22:15 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87sfqnj2vc.fsf@intel.com \
--to=jani.nikula@linux.intel.com \
--cc=Xinhui.Pan@amd.com \
--cc=airlied@linux.ie \
--cc=alexander.deucher@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=cgel.zte@gmail.com \
--cc=christian.koenig@amd.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=joonas.lahtinen@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=lv.ruyi@zte.com.cn \
--cc=netdev@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rodrigo.vivi@intel.com \
--cc=tvrtko.ursulin@linux.intel.com \
--cc=zealci@zte.com.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.