mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] kdbus: update kernel-doc for kdbus_sync_reply_wakeup()
@ 2015-06-02 15:48 Sergei Zviagintsev
  2015-06-03 10:28 ` David Herrmann
  2015-06-04  6:00 ` Greg Kroah-Hartman
  0 siblings, 2 replies; 3+ messages in thread
From: Sergei Zviagintsev @ 2015-06-02 15:48 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Daniel Mack, David Herrmann, Djalal Harouni
  Cc: linux-kernel, Sergei Zviagintsev

kdbus_sync_reply_wakeup() doesn't remove reply object from connection
reply_list. Update function description.

Signed-off-by: Sergei Zviagintsev <sergei@s15v.net>
---
 ipc/kdbus/reply.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/ipc/kdbus/reply.c b/ipc/kdbus/reply.c
index 008dca801627..89d355b44f63 100644
--- a/ipc/kdbus/reply.c
+++ b/ipc/kdbus/reply.c
@@ -140,8 +140,7 @@ void kdbus_reply_unlink(struct kdbus_reply *r)
  * @reply:	The reply object
  * @err:	Error code to set on the remote side
  *
- * Remove the synchronous reply object from its connection reply_list, and
- * wake up remote peer (method origin) with the appropriate synchronous reply
+ * Wake up remote peer (method origin) with the appropriate synchronous reply
  * code.
  */
 void kdbus_sync_reply_wakeup(struct kdbus_reply *reply, int err)
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] kdbus: update kernel-doc for kdbus_sync_reply_wakeup()
  2015-06-02 15:48 [PATCH] kdbus: update kernel-doc for kdbus_sync_reply_wakeup() Sergei Zviagintsev
@ 2015-06-03 10:28 ` David Herrmann
  2015-06-04  6:00 ` Greg Kroah-Hartman
  1 sibling, 0 replies; 3+ messages in thread
From: David Herrmann @ 2015-06-03 10:28 UTC (permalink / raw)
  To: Sergei Zviagintsev
  Cc: Greg Kroah-Hartman, Daniel Mack, David Herrmann, Djalal Harouni,
	linux-kernel

Hi

On Tue, Jun 2, 2015 at 5:48 PM, Sergei Zviagintsev <sergei@s15v.net> wrote:
> kdbus_sync_reply_wakeup() doesn't remove reply object from connection
> reply_list. Update function description.
>
> Signed-off-by: Sergei Zviagintsev <sergei@s15v.net>
> ---
>  ipc/kdbus/reply.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Reviewed-by: David Herrmann <dh.herrmann@gmail.com>

Thanks!
David

> diff --git a/ipc/kdbus/reply.c b/ipc/kdbus/reply.c
> index 008dca801627..89d355b44f63 100644
> --- a/ipc/kdbus/reply.c
> +++ b/ipc/kdbus/reply.c
> @@ -140,8 +140,7 @@ void kdbus_reply_unlink(struct kdbus_reply *r)
>   * @reply:     The reply object
>   * @err:       Error code to set on the remote side
>   *
> - * Remove the synchronous reply object from its connection reply_list, and
> - * wake up remote peer (method origin) with the appropriate synchronous reply
> + * Wake up remote peer (method origin) with the appropriate synchronous reply
>   * code.
>   */
>  void kdbus_sync_reply_wakeup(struct kdbus_reply *reply, int err)
> --
> 1.8.3.1
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] kdbus: update kernel-doc for kdbus_sync_reply_wakeup()
  2015-06-02 15:48 [PATCH] kdbus: update kernel-doc for kdbus_sync_reply_wakeup() Sergei Zviagintsev
  2015-06-03 10:28 ` David Herrmann
@ 2015-06-04  6:00 ` Greg Kroah-Hartman
  1 sibling, 0 replies; 3+ messages in thread
From: Greg Kroah-Hartman @ 2015-06-04  6:00 UTC (permalink / raw)
  To: Sergei Zviagintsev
  Cc: Daniel Mack, David Herrmann, Djalal Harouni, linux-kernel

On Tue, Jun 02, 2015 at 06:48:47PM +0300, Sergei Zviagintsev wrote:
> kdbus_sync_reply_wakeup() doesn't remove reply object from connection
> reply_list. Update function description.
> 
> Signed-off-by: Sergei Zviagintsev <sergei@s15v.net>
> Reviewed-by: David Herrmann <dh.herrmann@gmail.com>
> ---
>  ipc/kdbus/reply.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Thanks for these, I've applied all 4 of them to the kdbus git repo.

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-06-04  6:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-02 15:48 [PATCH] kdbus: update kernel-doc for kdbus_sync_reply_wakeup() Sergei Zviagintsev
2015-06-03 10:28 ` David Herrmann
2015-06-04  6:00 ` Greg Kroah-Hartman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome