linux-kernel - Re: [PATCH] staging: media: atomisp: pci: Fix spelling mistake in ia_css_acc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Fri, 22 Dec 2023 12:31:17 -0800
From: Randy Dunlap <rdunlap@...radead.org>
To: Dipendra Khadka <kdipendra88@...il.com>
Cc: hdegoede@...hat.com, mchehab@...nel.org, sakari.ailus@...ux.intel.com,
 gregkh@...uxfoundation.org, hpa@...hat.com, linux-media@...r.kernel.org,
 linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: media: atomisp: pci: Fix spelling mistake in
 ia_css_acc_types.h



On 12/22/23 12:29, Dipendra Khadka wrote:
> On Sat, 23 Dec 2023 at 01:57, Randy Dunlap <rdunlap@...radead.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@...il.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