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 <[email protected]>
---
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 */
--
2.34.1
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 <[email protected]>
> ---
> 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 01:57, Randy Dunlap <[email protected]> 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 <[email protected]>
> > ---
> > 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 <[email protected]> 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 <[email protected]>
>>> ---
>>> 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
On Sat, 23 Dec 2023 at 02:16, Randy Dunlap <[email protected]> wrote:
>
>
>
> On 12/22/23 12:29, Dipendra Khadka wrote:
> > On Sat, 23 Dec 2023 at 01:57, Randy Dunlap <[email protected]> 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 <[email protected]>
> >>> ---
> >>> 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