[PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h

Dipendra Khadka posted 1 patch 1 year, 12 months ago
There is a newer version of this series
drivers/staging/media/atomisp/pci/ia_css_acc_types.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h
Posted by Dipendra Khadka 1 year, 12 months ago
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(-)

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
Re: [PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h
Posted by Randy Dunlap 1 year, 12 months ago
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 */

-- 
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html
Re: [PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h
Posted by Dipendra Khadka 1 year, 12 months ago
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
Re: [PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h
Posted by Randy Dunlap 1 year, 12 months ago

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

-- 
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html
Re: [PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc_types.h
Posted by Dipendra Khadka 1 year, 12 months ago
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