linux-kernel - Re: [PATCH v2] staging: r8188eu: Fix misspelling in comment

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] [day] [month] [year] [list]

Message-ID: <7fc8382e-629c-b271-d20d-e668d0f9690e@gmail.com>
Date:   Mon, 11 Oct 2021 22:54:48 +0100
From:   "Siou-Jhih, Guo" <hallblazzar@...il.com>
To:     Joe Perches <joe@...ches.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Larry Finger <Larry.Finger@...inger.net>,
        Phillip Potter <phil@...lpotter.co.uk>,
        linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] staging: r8188eu: Fix misspelling in comment


On 06/10/2021 02:02, Joe Perches wrote:
> On Tue, 2021-10-05 at 22:16 +0100, Siou-Jhih, Guo wrote:
>> If anything should I additionally aware, I'd like to fix them.
>
> Maybe fix these too?
>
> $ git grep -w -i -P -n 'issuign|fittign|settign|enablign|followign|chargign|beign|startign|accordign|runnign'
> arch/arm/mach-s3c/sleep-s3c64xx.S:33:    * requires to get the kernel runnign again. This code expects to be
> drivers/crypto/ccree/cc_driver.c:278:   /* AXI interrupt config are obsoleted startign at cc7x3 */
> drivers/hid/hid-thrustmaster.c:80: * beign conducted yet. The position and meaning of fields are a
> drivers/net/ethernet/chelsio/cxgb4/t4fw_api.h:809: * our own HELLO may get lost if it happens right as the MASTER is issuign a
> drivers/net/ethernet/qlogic/qede/qede_filter.c:2029:    /* Translate the flow specification into something fittign our DB */
> drivers/net/wireless/realtek/rtlwifi/rtl8821ae/dm.c:1158: * Overview:   88E change all channel tx power accordign to flag.
> drivers/net/wireless/realtek/rtlwifi/rtl8821ae/dm.c:1837: * Overview:   88E change all channel tx power accordign to flag.
> drivers/power/supply/ab8500_charger.c:1325: * ab8500_charger_led_en() - turn on/off chargign led
> drivers/power/supply/ab8500_charger.c:1327: * @on:              flag to turn on/off the chargign led
> drivers/spi/spi-tegra20-slink.c:543:            /* HW need small delay after settign Packed mode */
> drivers/staging/media/atomisp/pci/runtime/bufq/src/bufq.c:412:   * by some test apps. Enablign logging here floods the log
> drivers/staging/media/atomisp/pci/runtime/bufq/src/bufq.c:434:   * by some test apps. Enablign logging here floods the log
> drivers/staging/r8188eu/core/rtw_ap.c:747:Set to 0 (HT pure) under the followign conditions
> drivers/staging/r8188eu/hal/HalPhyRf_8188e.c:72: * Overview:    88E change all channel tx power accordign to flag.
> drivers/thermal/tegra/soctherm.c:844:    * programmed on a different one. This beign a LEVEL interrupt won't
> drivers/usb/gadget/function/f_hid.c:312:         * Remove this from list to protect it from beign free()
> net/mac80211/mesh_plink.c:1077:                  * B's peer link establishment or B beign
>
>

Hello Joe,

Thanks for your advice. It sounds really great. But this is my very
first patch, and I'm not sure adding so many modifications is proper or
not ... Maybe I could contribute after I could successfully pass my
first patch?

Regards,
Bill G.