* [PATCH] RDMA/restrack: update kernel documentation for ib_create_named_qp()
@ 2020-12-07 17:32 Lukas Bulwahn
2020-12-08 6:44 ` Leon Romanovsky
2020-12-10 17:29 ` Jason Gunthorpe
0 siblings, 2 replies; 3+ messages in thread
From: Lukas Bulwahn @ 2020-12-07 17:32 UTC (permalink / raw)
To: Doug Ledford, Jason Gunthorpe, Leon Romanovsky, linux-rdma
Cc: linux-kernel, linux-doc, kernel-janitors, Jonathan Corbet, Lukas Bulwahn
Commit 66f57b871efc ("RDMA/restrack: Support all QP types") extends
ib_create_qp() to a named ib_create_named_qp(), which takes the caller's
name as argument, but it did not add the new argument description to the
function's kerneldoc.
make htmldocs warns:
./drivers/infiniband/core/verbs.c:1206: warning: Function parameter or
member 'caller' not described in 'ib_create_named_qp'
Add a description for this new argument based on the description of the
same argument in other related functions.
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
applies on next-20201207
Doug, Jason, Leon, please pick this minor doc fix on your -next tree.
drivers/infiniband/core/verbs.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/infiniband/core/verbs.c b/drivers/infiniband/core/verbs.c
index 5d4c7c263665..57206fe1bad2 100644
--- a/drivers/infiniband/core/verbs.c
+++ b/drivers/infiniband/core/verbs.c
@@ -1197,6 +1197,7 @@ static struct ib_qp *create_xrc_qp_user(struct ib_qp *qp,
* @qp_init_attr: A list of initial attributes required to create the
* QP. If QP creation succeeds, then the attributes are updated to
* the actual capabilities of the created QP.
+ * @caller: caller's build-time module name
*
* NOTE: for user qp use ib_create_qp_user with valid udata!
*/
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] RDMA/restrack: update kernel documentation for ib_create_named_qp()
2020-12-07 17:32 [PATCH] RDMA/restrack: update kernel documentation for ib_create_named_qp() Lukas Bulwahn
@ 2020-12-08 6:44 ` Leon Romanovsky
2020-12-10 17:29 ` Jason Gunthorpe
1 sibling, 0 replies; 3+ messages in thread
From: Leon Romanovsky @ 2020-12-08 6:44 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Doug Ledford, Jason Gunthorpe, linux-rdma, linux-kernel,
linux-doc, kernel-janitors, Jonathan Corbet
On Mon, Dec 07, 2020 at 06:32:55PM +0100, Lukas Bulwahn wrote:
> Commit 66f57b871efc ("RDMA/restrack: Support all QP types") extends
> ib_create_qp() to a named ib_create_named_qp(), which takes the caller's
> name as argument, but it did not add the new argument description to the
> function's kerneldoc.
>
> make htmldocs warns:
>
> ./drivers/infiniband/core/verbs.c:1206: warning: Function parameter or
> member 'caller' not described in 'ib_create_named_qp'
>
> Add a description for this new argument based on the description of the
> same argument in other related functions.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
Thanks,
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] RDMA/restrack: update kernel documentation for ib_create_named_qp()
2020-12-07 17:32 [PATCH] RDMA/restrack: update kernel documentation for ib_create_named_qp() Lukas Bulwahn
2020-12-08 6:44 ` Leon Romanovsky
@ 2020-12-10 17:29 ` Jason Gunthorpe
1 sibling, 0 replies; 3+ messages in thread
From: Jason Gunthorpe @ 2020-12-10 17:29 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Doug Ledford, Leon Romanovsky, linux-rdma, linux-kernel,
linux-doc, kernel-janitors, Jonathan Corbet
On Mon, Dec 07, 2020 at 06:32:55PM +0100, Lukas Bulwahn wrote:
> Commit 66f57b871efc ("RDMA/restrack: Support all QP types") extends
> ib_create_qp() to a named ib_create_named_qp(), which takes the caller's
> name as argument, but it did not add the new argument description to the
> function's kerneldoc.
>
> make htmldocs warns:
>
> ./drivers/infiniband/core/verbs.c:1206: warning: Function parameter or
> member 'caller' not described in 'ib_create_named_qp'
>
> Add a description for this new argument based on the description of the
> same argument in other related functions.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
> ---
> drivers/infiniband/core/verbs.c | 1 +
> 1 file changed, 1 insertion(+)
Applies to for-next, thanks
Jason
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-12-10 17:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-07 17:32 [PATCH] RDMA/restrack: update kernel documentation for ib_create_named_qp() Lukas Bulwahn
2020-12-08 6:44 ` Leon Romanovsky
2020-12-10 17:29 ` Jason Gunthorpe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).