From: Wei Liu <wei.liu@kernel.org>
To: Michael Kelley <mhklinux@outlook.com>
Cc: Roman Kisel <romank@linux.microsoft.com>,
"kys@microsoft.com" <kys@microsoft.com>,
"haiyangz@microsoft.com" <haiyangz@microsoft.com>,
"wei.liu@kernel.org" <wei.liu@kernel.org>,
"decui@microsoft.com" <decui@microsoft.com>,
"sashal@kernel.org" <sashal@kernel.org>,
"linux-hyperv@vger.kernel.org" <linux-hyperv@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"apais@microsoft.com" <apais@microsoft.com>,
"benhill@microsoft.com" <benhill@microsoft.com>,
"ssengar@microsoft.com" <ssengar@microsoft.com>,
"sunilmut@microsoft.com" <sunilmut@microsoft.com>,
"vdso@hexbites.dev" <vdso@hexbites.dev>
Subject: Re: [PATCH] Drivers: hv: vmbus: Fix the misplaced function description
Date: Sat, 3 Aug 2024 00:13:20 +0000 [thread overview]
Message-ID: <Zq12ILqoxEzjLOq2@liuwe-devbox-debian-v2> (raw)
In-Reply-To: <SN6PR02MB41578B75E804B59A2B8A8E24D4B32@SN6PR02MB4157.namprd02.prod.outlook.com>
On Fri, Aug 02, 2024 at 03:51:37PM +0000, Michael Kelley wrote:
> From: Roman Kisel <romank@linux.microsoft.com> Sent: Thursday, August 1, 2024 2:23 PM
> >
> > When hv_synic_disable_regs was introduced, it received the description
> > of hv_synic_cleanup. Fix that.
> >
> > Fixes: dba61cda3046 ("Drivers: hv: vmbus: Break out synic enable and disable
> > operations")
> >
> > Signed-off-by: Roman Kisel <romank@linux.microsoft.com>
> > ---
> > drivers/hv/hv.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c
> > index e0d676c74f14..36d9ba097ff5 100644
> > --- a/drivers/hv/hv.c
> > +++ b/drivers/hv/hv.c
> > @@ -342,9 +342,6 @@ int hv_synic_init(unsigned int cpu)
> > return 0;
> > }
> >
> > -/*
> > - * hv_synic_cleanup - Cleanup routine for hv_synic_init().
> > - */
> > void hv_synic_disable_regs(unsigned int cpu)
> > {
> > struct hv_per_cpu_context *hv_cpu =
> > @@ -436,6 +433,9 @@ static bool hv_synic_event_pending(void)
> > return pending;
> > }
> >
> > +/*
> > + * hv_synic_cleanup - Cleanup routine for hv_synic_init().
> > + */
> > int hv_synic_cleanup(unsigned int cpu)
> > {
> > struct vmbus_channel *channel, *sc;
> >
> > base-commit: 831bcbcead6668ebf20b64fdb27518f1362ace3a
> > --
> > 2.34.1
> >
>
> Reviewed-by: Michael Kelley <mhkelley@outlook.com>
>
Applied to hyperv-fixes. Thanks.
prev parent reply other threads:[~2024-08-03 0:13 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-01 21:22 [PATCH] Drivers: hv: vmbus: Fix the misplaced function description Roman Kisel
2024-08-02 15:51 ` Michael Kelley
2024-08-03 0:13 ` Wei Liu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Zq12ILqoxEzjLOq2@liuwe-devbox-debian-v2 \
--to=wei.liu@kernel.org \
--cc=apais@microsoft.com \
--cc=benhill@microsoft.com \
--cc=decui@microsoft.com \
--cc=haiyangz@microsoft.com \
--cc=kys@microsoft.com \
--cc=linux-hyperv@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mhklinux@outlook.com \
--cc=romank@linux.microsoft.com \
--cc=sashal@kernel.org \
--cc=ssengar@microsoft.com \
--cc=sunilmut@microsoft.com \
--cc=vdso@hexbites.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox