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.
Signed-off-by: Dipendra Khadka <[email protected]>
---
v2:
- 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..1dc2085ecd61 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 */
@@ -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)
Hi Dipendra,
On 12/22/23 21: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.
>
> Signed-off-by: Dipendra Khadka <[email protected]>
This patch is an improvement so it could be merged as is IMO.
But...
> ---
> v2:
> - 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..1dc2085ecd61 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 */
I apologize for not looking at your v1 patch carefully.
The comment above would be much better as
/** total accumulation of bytes added due to section alignment */
And if the maintainer(s) want to take it as is:
Acked-by: Randy Dunlap <[email protected]>
Thanks.
> 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
Hi Randy,
On Sat, 23 Dec 2023 at 11:03, Randy Dunlap <[email protected]> wrote:
>
> Hi Dipendra,
>
> On 12/22/23 21: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.
> >
> > Signed-off-by: Dipendra Khadka <[email protected]>
>
> This patch is an improvement so it could be merged as is IMO.
> But...
>
> > ---
> > v2:
> > - 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..1dc2085ecd61 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 */
>
> I apologize for not looking at your v1 patch carefully.
> The comment above would be much better as
>
> /** total accumulation of bytes added due to section alignment */
>
So, would you like me to change "cummulative" to accumulation or
should I wait what other maintainer(s) would say?
Thanks.
> And if the maintainer(s) want to take it as is:
> Acked-by: Randy Dunlap <[email protected]>
>
> Thanks.
>
>
> > 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
On 12/22/23 21:29, Dipendra Khadka wrote:
> Hi Randy,
>
> On Sat, 23 Dec 2023 at 11:03, Randy Dunlap <[email protected]> wrote:
>>
>> Hi Dipendra,
>>
>> On 12/22/23 21: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.
>>>
>>> Signed-off-by: Dipendra Khadka <[email protected]>
>>
>> This patch is an improvement so it could be merged as is IMO.
>> But...
>>
>>> ---
>>> v2:
>>> - 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..1dc2085ecd61 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 */
>>
>> I apologize for not looking at your v1 patch carefully.
>> The comment above would be much better as
>>
>> /** total accumulation of bytes added due to section alignment */
>>
>
> So, would you like me to change "cummulative" to accumulation or
> should I wait what other maintainer(s) would say?
>
I think just wait for a few days to see what other review comments there are.
>
>> And if the maintainer(s) want to take it as is:
>> Acked-by: Randy Dunlap <[email protected]>
>>
>> Thanks.
>>
>>
>>> 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
--
#Randy
https://people.kernel.org/tglx/notes-about-netiquette
https://subspace.kernel.org/etiquette.html
Hi,
On 12/23/23 06:18, Randy Dunlap wrote:
> Hi Dipendra,
>
> On 12/22/23 21: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.
>>
>> Signed-off-by: Dipendra Khadka <[email protected]>
>
> This patch is an improvement so it could be merged as is IMO.
> But...
>
>> ---
>> v2:
>> - 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..1dc2085ecd61 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 */
>
> I apologize for not looking at your v1 patch carefully.
> The comment above would be much better as
>
> /** total accumulation of bytes added due to section alignment */
I agree that that is better. Dipendra can you please send a v3
using the new text suggested by Randy ?
Regards,
Hans
>> 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 {
>
Hi,
On Sat, 23 Dec 2023 at 16:10, Hans de Goede <[email protected]> wrote:
>
> Hi,
>
> On 12/23/23 06:18, Randy Dunlap wrote:
> > Hi Dipendra,
> >
> > On 12/22/23 21: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.
> >>
> >> Signed-off-by: Dipendra Khadka <[email protected]>
> >
> > This patch is an improvement so it could be merged as is IMO.
> > But...
> >
> >> ---
> >> v2:
> >> - 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..1dc2085ecd61 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 */
> >
> > I apologize for not looking at your v1 patch carefully.
> > The comment above would be much better as
> >
> > /** total accumulation of bytes added due to section alignment */
>
> I agree that that is better. Dipendra can you please send a v3
> using the new text suggested by Randy ?
Sure,let me send a v3 with the new text suggested by Randy.
Regards,
Dipendra
>
> Regards,
>
> Hans
>
>
>
> >> 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 {
> >
>