On Thu, Mar 21, 2024 at 02:55:07AM +0530, Ayush Tiwari wrote: > Address different kinds of checkpatch complains for the rtl8712 module > to ensure adherence to coding style guidelines. Thanks for sending v3 as a new email, rather than threaded with prior revisions. I think you repeat the changelog in the patches, but I'll go ahead and comment right here. It applies in the patch too. > > Changes in v3: Fixed issues about backupPMKIDList and verified with > CONFIG_WERROR set and built the kernel. > > Changes in v2: Checked any possible reuse of backup_PMKID_list > manually and rebuilt, rebooted the kernel and loaded the driver > with modprobe. The change log is to inform reviewers of what actually changed in the patch since previous version. So let's be specific in these v2, v3, and next a v4 changelog. I think this is right: v4: Update the changelog to match revision history v3: Update memcpy() param to match new name backup_PMKID_list v2: No changes. Just a patch sending error. There's a section in the first patch tutorial about replying to patch feedback and you can read any of the lore lists to see examples. Folks will expect you to reply to their feedback. Even if it's just a simple "Thanks, got it!" Then, they will look to see if their feedback affected the next revision. --Alison > > Ayush Tiwari (3): > staging: rtl8712: rename backupPMKIDList to backup_PMKID_list > staging: rtl8712: rename backupPMKIDIndex to backup_PMKID_index > staging: rtl8712: rename backupTKIPCountermeasure to > backup_TKIP_countermeasure > > drivers/staging/rtl8712/mlme_linux.c | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) > > -- > 2.40.1 > >
