Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 28 Jun 2023 17:51:56 +0000
From:   Wei Liu <wei.liu@kernel.org>
To:     Kameron Carr <kameroncarr@linux.microsoft.com>
Cc:     arnd@arndb.de, decui@microsoft.com, haiyangz@microsoft.com,
        kys@microsoft.com, linux-arch@vger.kernel.org,
        linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org,
        wei.liu@kernel.org
Subject: Re: [PATCH v2] Drivers: hv: Change hv_free_hyperv_page() to take
 void * argument
Message-ID: <ZJxzPBE5mDVBUKCE@liuwe-devbox-debian-v2>
References: <1687558189-19734-1-git-send-email-kameroncarr@linux.microsoft.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1687558189-19734-1-git-send-email-kameroncarr@linux.microsoft.com>
Precedence: bulk

On Fri, Jun 23, 2023 at 03:09:49PM -0700, Kameron Carr wrote:
> Currently hv_free_hyperv_page() takes an unsigned long argument, which
> is inconsistent with the void * return value from the corresponding
> hv_alloc_hyperv_page() function and variants. This creates unnecessary
> extra casting.
> 
> Change the hv_free_hyperv_page() argument type to void *.
> Also remove redundant casts from invocations of
> hv_alloc_hyperv_page() and variants.
> 
> Signed-off-by: Kameron Carr <kameroncarr@linux.microsoft.com>

Applied to hyperv-fixes. Thanks!