Commit a3605300 authored May 12, 2012 by K. Y. Srinivasan Committed by Greg Kroah-Hartman May 14, 2012

Drivers: hv: Get rid of an unnecessary check in vmbus_prep_negotiate_resp()



The vmbus_prep_negotiate_resp() is only invoked when we are negotiating
the version; so the current check in vmbus_prep_negotiate_resp()
is unnecessary. Get rid of it.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Reviewed-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 90394482

drivers/hv/channel_mgmt.c

+21 −22

Original line number	Diff line number	Diff line
		@@ -56,7 +56,6 @@ struct vmbus_channel_message_table_entry {
		void vmbus_prep_negotiate_resp(struct icmsg_hdr *icmsghdrp,
		struct icmsg_negotiate negop, u8 buf)
		{
		if (icmsghdrp->icmsgtype == ICMSGTYPE_NEGOTIATE) {
		icmsghdrp->icmsgsize = 0x10;

		negop = (struct icmsg_negotiate *)&buf[
		@@ -79,7 +78,7 @@ void vmbus_prep_negotiate_resp(struct icmsg_hdr *icmsghdrp,
		negop->icframe_vercnt = 1;
		negop->icmsg_vercnt = 1;
		}
		}

		EXPORT_SYMBOL_GPL(vmbus_prep_negotiate_resp);

		/*