RE: [PATCH] Drivers: hv: Change hv_free_hyperv_page() to take void * argument

From: Dexuan Cui
Date: Thu Jun 22 2023 - 23:10:00 EST


> From: Kameron Carr <kameroncarr@xxxxxxxxxxxxxxxxxxx>
> Sent: Thursday, June 22, 2023 1:07 PM
> ...
> 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.
> ---

Hi Kameron, you forgot to add your Signed-off-by :-)
Otherwise, it looks good to me.

Reviewed-by: Dexuan Cui <decui@xxxxxxxxxxxxx>