2023-12-23 14:12:20

by Dipendra Khadka

[permalink] [raw]
Subject: [PATCH v3] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h

codespell reported spelling mistakes in
ia_css_acc_types.h as below:

'''
ia_css_acc_types.h:87: cummulative ==> cumulative
ia_css_acc_types.h:411: descibes ==> describes
'''

This patch fixes these spelling mistakes.
Word "cummulative" is changed to "accumulation"
and "descibes" to "describes".

Signed-off-by: Dipendra Khadka <[email protected]>
---
v3:
- Changed "cummulative" to "accumulation" as suggested by Randy.
v2: https://lore.kernel.org/lkml/[email protected]/
- Previously only corrected spelling mistake reported by checkpatch.pl.
- All spelling mistakes reported by codespell are fixed.
v1: https://lore.kernel.org/lkml/[email protected]/

drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/media/atomisp/pci/ia_css_acc_types.h b/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
index d6e52b4971d6..f6838a8fc9d5 100644
--- a/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
+++ b/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
@@ -84,7 +84,7 @@ struct ia_css_blob_info {
memory_offsets; /** offset wrt hdr in bytes */
u32 prog_name_offset; /** offset wrt hdr in bytes */
u32 size; /** Size of blob */
- u32 padding_size; /** total cummulative of bytes added due to section alignment */
+ u32 padding_size; /** total accumulation of bytes added due to section alignment */
u32 icache_source; /** Position of icache in blob */
u32 icache_size; /** Size of icache section */
u32 icache_padding;/** bytes added due to icache section alignment */
@@ -408,7 +408,7 @@ struct ia_css_acc_sp {
};

/* Acceleration firmware descriptor.
- * This descriptor descibes either SP code (stand-alone), or
+ * This descriptor describes either SP code (stand-alone), or
* ISP code (a separate pipeline stage).
*/
struct ia_css_acc_fw_hdr {
--
2.39.2 (Apple Git-143)



2023-12-23 17:25:54

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH v3] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h



On 12/23/23 06:11, Dipendra Khadka wrote:
> codespell reported spelling mistakes in
> ia_css_acc_types.h as below:
>
> '''
> ia_css_acc_types.h:87: cummulative ==> cumulative
> ia_css_acc_types.h:411: descibes ==> describes
> '''
>
> This patch fixes these spelling mistakes.
> Word "cummulative" is changed to "accumulation"
> and "descibes" to "describes".
>
> Signed-off-by: Dipendra Khadka <[email protected]>

Reviewed-by: Randy Dunlap <[email protected]>

Thanks.

> ---
> v3:
> - Changed "cummulative" to "accumulation" as suggested by Randy.
> v2: https://lore.kernel.org/lkml/[email protected]/
> - Previously only corrected spelling mistake reported by checkpatch.pl.
> - All spelling mistakes reported by codespell are fixed.
> v1: https://lore.kernel.org/lkml/[email protected]/
>
> drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/media/atomisp/pci/ia_css_acc_types.h b/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
> index d6e52b4971d6..f6838a8fc9d5 100644
> --- a/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
> +++ b/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
> @@ -84,7 +84,7 @@ struct ia_css_blob_info {
> memory_offsets; /** offset wrt hdr in bytes */
> u32 prog_name_offset; /** offset wrt hdr in bytes */
> u32 size; /** Size of blob */
> - u32 padding_size; /** total cummulative of bytes added due to section alignment */
> + u32 padding_size; /** total accumulation of bytes added due to section alignment */
> u32 icache_source; /** Position of icache in blob */
> u32 icache_size; /** Size of icache section */
> u32 icache_padding;/** bytes added due to icache section alignment */
> @@ -408,7 +408,7 @@ struct ia_css_acc_sp {
> };
>
> /* Acceleration firmware descriptor.
> - * This descriptor descibes either SP code (stand-alone), or
> + * This descriptor describes either SP code (stand-alone), or
> * ISP code (a separate pipeline stage).
> */
> struct ia_css_acc_fw_hdr {

--
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html

2023-12-31 09:51:42

by Hans de Goede

[permalink] [raw]
Subject: Re: [PATCH v3] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h

Hi,

On 12/23/23 15:11, Dipendra Khadka wrote:
> codespell reported spelling mistakes in
> ia_css_acc_types.h as below:
>
> '''
> ia_css_acc_types.h:87: cummulative ==> cumulative
> ia_css_acc_types.h:411: descibes ==> describes
> '''
>
> This patch fixes these spelling mistakes.
> Word "cummulative" is changed to "accumulation"
> and "descibes" to "describes".
>
> Signed-off-by: Dipendra Khadka <[email protected]>

Thank you for your patch.

I have merged this in my media-atomisp branch:
https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp

And this patch will be included in my next
pull-request to Mauro (to media subsystem maintainer)

Regards,

Hans




> ---
> v3:
> - Changed "cummulative" to "accumulation" as suggested by Randy.
> v2: https://lore.kernel.org/lkml/[email protected]/
> - Previously only corrected spelling mistake reported by checkpatch.pl.
> - All spelling mistakes reported by codespell are fixed.
> v1: https://lore.kernel.org/lkml/[email protected]/
>
> drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/media/atomisp/pci/ia_css_acc_types.h b/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
> index d6e52b4971d6..f6838a8fc9d5 100644
> --- a/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
> +++ b/drivers/staging/media/atomisp/pci/ia_css_acc_types.h
> @@ -84,7 +84,7 @@ struct ia_css_blob_info {
> memory_offsets; /** offset wrt hdr in bytes */
> u32 prog_name_offset; /** offset wrt hdr in bytes */
> u32 size; /** Size of blob */
> - u32 padding_size; /** total cummulative of bytes added due to section alignment */
> + u32 padding_size; /** total accumulation of bytes added due to section alignment */
> u32 icache_source; /** Position of icache in blob */
> u32 icache_size; /** Size of icache section */
> u32 icache_padding;/** bytes added due to icache section alignment */
> @@ -408,7 +408,7 @@ struct ia_css_acc_sp {
> };
>
> /* Acceleration firmware descriptor.
> - * This descriptor descibes either SP code (stand-alone), or
> + * This descriptor describes either SP code (stand-alone), or
> * ISP code (a separate pipeline stage).
> */
> struct ia_css_acc_fw_hdr {