diff mbox series

[iwl-next] i40e: correct i40e_addr_to_hkey() name in kdoc

Message ID 20240705-i40e-kdoc-v1-1-529d0808a1ef@kernel.org (mailing list archive)
State Awaiting Upstream
Delegated to: Netdev Maintainers
Headers show
Series [iwl-next] i40e: correct i40e_addr_to_hkey() name in kdoc | expand

Checks

Context Check Description
netdev/series_format warning Single patches do not need cover letters; Target tree name not specified in the subject
netdev/tree_selection success Guessed tree name to be net-next
netdev/ynl success Generated files up to date; no warnings/errors; no diff in generated;
netdev/fixes_present success Fixes tag not required for -next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 816 this patch: 816
netdev/build_tools success No tools touched, skip
netdev/cc_maintainers warning 3 maintainers not CCed: kuba@kernel.org edumazet@google.com pabeni@redhat.com
netdev/build_clang success Errors and warnings before: 821 this patch: 821
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 821 this patch: 821
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 8 lines checked
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 11 this patch: 11
netdev/source_inline success Was 0 now: 0

Commit Message

Simon Horman July 5, 2024, 1:15 p.m. UTC
Correct name of i40e_addr_to_hkey() in it's kdoc.

kernel-doc -none reports:

 drivers/net/ethernet/intel/i40e/i40e.h:739: warning: expecting prototype for i40e_mac_to_hkey(). Prototype was for i40e_addr_to_hkey() instead

Signed-off-by: Simon Horman <horms@kernel.org>
---
 drivers/net/ethernet/intel/i40e/i40e.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Przemek Kitszel July 5, 2024, 1:24 p.m. UTC | #1
On 7/5/24 15:15, Simon Horman wrote:
> Correct name of i40e_addr_to_hkey() in it's kdoc.
> 
> kernel-doc -none reports:
> 
>   drivers/net/ethernet/intel/i40e/i40e.h:739: warning: expecting prototype for i40e_mac_to_hkey(). Prototype was for i40e_addr_to_hkey() instead
> 
> Signed-off-by: Simon Horman <horms@kernel.org>

thank you,
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>

(unrelated digression below)

> ---
>   drivers/net/ethernet/intel/i40e/i40e.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/intel/i40e/i40e.h b/drivers/net/ethernet/intel/i40e/i40e.h
> index bca2084cc54b..d546567e0286 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e.h
> +++ b/drivers/net/ethernet/intel/i40e/i40e.h
> @@ -735,7 +735,7 @@ __i40e_pf_next_veb(struct i40e_pf *pf, int *idx)
>   	     _i++, _veb = __i40e_pf_next_veb(_pf, &_i))
>   
>   /**
> - * i40e_mac_to_hkey - Convert a 6-byte MAC Address to a u64 hash key
> + * i40e_addr_to_hkey - Convert a 6-byte MAC Address to a u64 hash key
>    * @macaddr: the MAC Address as the base key
>    *
>    * Simply copies the address and returns it as a u64 for hashing
> 

I really look forward to the day that sender will be able to add
metadata to the patch displaying options - here changing context to 5
would make review an instant thing, as now there is no signature line:
static inline u64 i40e_addr_to_hkey(const u8 *macaddr).
But if Simon will send like that, this will be lost when applied to 
Tony's tree.

I'm picking such trivial patch to bring the topic to don't derail any
which will have it's own on-topic discussion, but this is for the
benefit of reviewers of bigger changes of course.
diff mbox series

Patch

diff --git a/drivers/net/ethernet/intel/i40e/i40e.h b/drivers/net/ethernet/intel/i40e/i40e.h
index bca2084cc54b..d546567e0286 100644
--- a/drivers/net/ethernet/intel/i40e/i40e.h
+++ b/drivers/net/ethernet/intel/i40e/i40e.h
@@ -735,7 +735,7 @@  __i40e_pf_next_veb(struct i40e_pf *pf, int *idx)
 	     _i++, _veb = __i40e_pf_next_veb(_pf, &_i))
 
 /**
- * i40e_mac_to_hkey - Convert a 6-byte MAC Address to a u64 hash key
+ * i40e_addr_to_hkey - Convert a 6-byte MAC Address to a u64 hash key
  * @macaddr: the MAC Address as the base key
  *
  * Simply copies the address and returns it as a u64 for hashing