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.
