diff mbox series

[03/35] bnxt_en: Reorganize kerneldoc parameter names

Message ID 20240930112121.95324-4-Julia.Lawall@inria.fr (mailing list archive)
State Changes Requested
Delegated to: Netdev Maintainers
Headers show
Series Reorganize kerneldoc parameter names | expand

Commit Message

Julia Lawall Sept. 30, 2024, 11:20 a.m. UTC
Reorganize kerneldoc parameter names to match the parameter
order in the function header.

Problems identified using Coccinelle.

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>

---
 drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Paolo Abeni Oct. 1, 2024, 9:30 a.m. UTC | #1
On 9/30/24 13:20, Julia Lawall wrote:
> Reorganize kerneldoc parameter names to match the parameter
> order in the function header.
> 
> Problems identified using Coccinelle.
> 
> Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>

As mentioned elsewhere, please split-up the series by sub-system and 
driver, it will make maintainers life easier,

Thanks!

Paolo
diff mbox series

Patch

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c
index d2fd2d04ed47..3ba7a51a8afa 100644
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c
@@ -171,8 +171,8 @@  void hwrm_req_alloc_flags(struct bnxt *bp, void *req, gfp_t gfp)
  *	such as hwrm_req_send(), should thus use req and not new_req (in fact,
  *	calls to HWRM API functions will fail if non-managed request objects
  *	are passed).
- * @len: The length of new_req.
  * @new_req: The pre-built request to copy or reference.
+ * @len: The length of new_req.
  *
  * Replaces the request data in req with that of new_req. This is useful in
  * scenarios where a request object has already been constructed by a third