| Message ID | 20210112051342.26064-1-unixbhaskar@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [V2] drivers: net: marvell: Fix two spellings, controling to controlling and oen to one | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Guessed tree name to be net-next |
| netdev/subject_prefix | warning | Target tree name not specified in the subject |
| netdev/cc_maintainers | success | CCed 5 of 5 maintainers |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 5 this patch: 5 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 5 this patch: 5 |
| netdev/header_inline | success | Link |
| netdev/stable | success | Stable not CCed |
On 23:20 Mon 11 Jan 2021, Gustavo A. R. Silva wrote: > > >On 1/11/21 23:13, Bhaskar Chowdhury wrote: >> s/controling/controlling/ >> >> s/oen/one/ >> >> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> >> --- >> As Gustavo mentioned in reply, so included that missed one before > >This is not how you version patches for maintainers and reviewers to >know you made changes to the patch. > Ahhh... Recollects my faint memory about it ...let me try again with that >I encourage you to take a look at this to see examples on how to properly >version your patches and other good practices: > >https://kernelnewbies.org/Outreachyfirstpatch > >-- >Gustavo > >> >> drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h b/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h >> index 8867f25afab4..663157dc8062 100644 >> --- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h >> +++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h >> @@ -143,7 +143,7 @@ struct mvpp2_cls_c2_entry { >> /* Number of per-port dedicated entries in the C2 TCAM */ >> #define MVPP22_CLS_C2_PORT_N_FLOWS MVPP2_N_RFS_ENTRIES_PER_FLOW >> >> -/* Each port has oen range per flow type + one entry controling the global RSS >> +/* Each port has one range per flow type + one entry controlling the global RSS >> * setting and the default rx queue >> */ >> #define MVPP22_CLS_C2_PORT_RANGE (MVPP22_CLS_C2_PORT_N_FLOWS + 1) >> -- >> 2.26.2 >>
diff --git a/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h b/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h index 8867f25afab4..663157dc8062 100644 --- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h +++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h @@ -143,7 +143,7 @@ struct mvpp2_cls_c2_entry { /* Number of per-port dedicated entries in the C2 TCAM */ #define MVPP22_CLS_C2_PORT_N_FLOWS MVPP2_N_RFS_ENTRIES_PER_FLOW -/* Each port has oen range per flow type + one entry controling the global RSS +/* Each port has one range per flow type + one entry controlling the global RSS * setting and the default rx queue */ #define MVPP22_CLS_C2_PORT_RANGE (MVPP22_CLS_C2_PORT_N_FLOWS + 1)
s/controling/controlling/ s/oen/one/ Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com> --- As Gustavo mentioned in reply, so included that missed one before drivers/net/ethernet/marvell/mvpp2/mvpp2_cls.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.26.2