| Message ID | 20231222200350.2024-1-kdipendra88@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h | expand |
Hi-- On 12/22/23 12:03, Dipendra Khadka wrote: > The script checkpatch.pl reported a spelling error > in ia_css_acc_types.h as below: > > ''' > WARNING: 'cummulative' may be misspelled - perhaps 'cumulative'? > u32 padding_size; /** total cummulative of bytes added due to section alignment */ > ^^^^^^^^^^^ > ''' > > This patch corrects a spelling error, > changing "cummulative" to "cumulative". > 'codespell' reports this one as well: drivers/staging/media/atomisp/pci/ia_css_acc_types.h:411: descibes ==> describes Thanks. > Signed-off-by: Dipendra Khadka <kdipendra88@gmail.com> > --- > drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > 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..ac6fb0eb990a 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 cumulative 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 */
On Sat, 23 Dec 2023 at 01:57, Randy Dunlap <rdunlap@infradead.org> wrote: > > Hi-- > > On 12/22/23 12:03, Dipendra Khadka wrote: > > The script checkpatch.pl reported a spelling error > > in ia_css_acc_types.h as below: > > > > ''' > > WARNING: 'cummulative' may be misspelled - perhaps 'cumulative'? > > u32 padding_size; /** total cummulative of bytes added due to section alignment */ > > ^^^^^^^^^^^ > > ''' > > > > This patch corrects a spelling error, > > changing "cummulative" to "cumulative". > > > > 'codespell' reports this one as well: > > drivers/staging/media/atomisp/pci/ia_css_acc_types.h:411: descibes ==> describes > > Thanks. > Thanks for the reply. Do you want the spelling mistakes reported by codespell in the same patch with this change as well or you will consider it in the next patch? > > Signed-off-by: Dipendra Khadka <kdipendra88@gmail.com> > > --- > > drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > 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..ac6fb0eb990a 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 cumulative 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 */ > > -- > #Randy > https://people.kernel.org/tglx/notes-about-netiquette > https://subspace.kernel.org/etiquette.html
On 12/22/23 12:29, Dipendra Khadka wrote: > On Sat, 23 Dec 2023 at 01:57, Randy Dunlap <rdunlap@infradead.org> wrote: >> >> Hi-- >> >> On 12/22/23 12:03, Dipendra Khadka wrote: >>> The script checkpatch.pl reported a spelling error >>> in ia_css_acc_types.h as below: >>> >>> ''' >>> WARNING: 'cummulative' may be misspelled - perhaps 'cumulative'? >>> u32 padding_size; /** total cummulative of bytes added due to section alignment */ >>> ^^^^^^^^^^^ >>> ''' >>> >>> This patch corrects a spelling error, >>> changing "cummulative" to "cumulative". >>> >> >> 'codespell' reports this one as well: >> >> drivers/staging/media/atomisp/pci/ia_css_acc_types.h:411: descibes ==> describes >> >> Thanks. >> > > Thanks for the reply. Do you want the spelling mistakes reported by > codespell in the same patch with this change as well or you will > consider > it in the next patch? That's up to some maintainers, but I would prefer to see all of the spelling errors fixed in one patch. Thanks. > >>> Signed-off-by: Dipendra Khadka <kdipendra88@gmail.com> >>> --- >>> drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> 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..ac6fb0eb990a 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 cumulative 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 */ >> >> -- >> #Randy >> https://people.kernel.org/tglx/notes-about-netiquette >> https://subspace.kernel.org/etiquette.html
On Sat, 23 Dec 2023 at 02:16, Randy Dunlap <rdunlap@infradead.org> wrote: > > > > On 12/22/23 12:29, Dipendra Khadka wrote: > > On Sat, 23 Dec 2023 at 01:57, Randy Dunlap <rdunlap@infradead.org> wrote: > >> > >> Hi-- > >> > >> On 12/22/23 12:03, Dipendra Khadka wrote: > >>> The script checkpatch.pl reported a spelling error > >>> in ia_css_acc_types.h as below: > >>> > >>> ''' > >>> WARNING: 'cummulative' may be misspelled - perhaps 'cumulative'? > >>> u32 padding_size; /** total cummulative of bytes added due to section alignment */ > >>> ^^^^^^^^^^^ > >>> ''' > >>> > >>> This patch corrects a spelling error, > >>> changing "cummulative" to "cumulative". > >>> > >> > >> 'codespell' reports this one as well: > >> > >> drivers/staging/media/atomisp/pci/ia_css_acc_types.h:411: descibes ==> describes > >> > >> Thanks. > >> > > > > Thanks for the reply. Do you want the spelling mistakes reported by > > codespell in the same patch with this change as well or you will > > consider > > it in the next patch? > > That's up to some maintainers, but I would prefer to see all of the > spelling errors fixed in one patch. > > Thanks. > Ok sure,give me some time and I will send the v2 patch. Thanks. > > > >>> Signed-off-by: Dipendra Khadka <kdipendra88@gmail.com> > >>> --- > >>> drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> 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..ac6fb0eb990a 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 cumulative 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 */ > >> > >> -- > >> #Randy > >> https://people.kernel.org/tglx/notes-about-netiquette > >> https://subspace.kernel.org/etiquette.html > > -- > #Randy > https://people.kernel.org/tglx/notes-about-netiquette > https://subspace.kernel.org/etiquette.html
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..ac6fb0eb990a 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 cumulative 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 */
The script checkpatch.pl reported a spelling error in ia_css_acc_types.h as below: ''' WARNING: 'cummulative' may be misspelled - perhaps 'cumulative'? u32 padding_size; /** total cummulative of bytes added due to section alignment */ ^^^^^^^^^^^ ''' This patch corrects a spelling error, changing "cummulative" to "cumulative". Signed-off-by: Dipendra Khadka <kdipendra88@gmail.com> --- drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)