Re: [PATCH 1/2] crypto: ccp: Fix sparse, use plain integer as NULL pointer

["Hook, Gary" <ghook@xxxxxxx>]

On 2/15/2018 2:34 PM, Brijesh Singh wrote:
> Fix sparse warning: Using plain integer as NULL pointer. Replaces
> assignment of 0 to pointer with NULL assignment.
>
> Fixes: 200664d5237f (Add Secure Encrypted Virtualization ...)
> Cc: Borislav Petkov <bp@xxxxxxx>
> Cc: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> Cc: Gary Hook <gary.hook@xxxxxxx>
> Cc: Tom Lendacky <thomas.lendacky@xxxxxxx>
> Signed-off-by: Brijesh Singh <brijesh.singh@xxxxxxx>

Acked-by: Gary R Hook <gary.hook@xxxxxxx>

> ---
>   drivers/crypto/ccp/psp-dev.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/crypto/ccp/psp-dev.c b/drivers/crypto/ccp/psp-dev.c
> index fcfa5b1eae61..b3afb6cc9d72 100644
> --- a/drivers/crypto/ccp/psp-dev.c
> +++ b/drivers/crypto/ccp/psp-dev.c
> @@ -211,7 +211,7 @@ static int __sev_platform_shutdown_locked(int *error)
>   {
>   	int ret;
>   
> -	ret = __sev_do_cmd_locked(SEV_CMD_SHUTDOWN, 0, error);
> +	ret = __sev_do_cmd_locked(SEV_CMD_SHUTDOWN, NULL, error);
>   	if (ret)
>   		return ret;
>   
> @@ -271,7 +271,7 @@ static int sev_ioctl_do_reset(struct sev_issue_cmd *argp)
>   			return rc;
>   	}
>   
> -	return __sev_do_cmd_locked(SEV_CMD_FACTORY_RESET, 0, &argp->error);
> +	return __sev_do_cmd_locked(SEV_CMD_FACTORY_RESET, NULL, &argp->error);
>   }
>   
>   static int sev_ioctl_do_platform_status(struct sev_issue_cmd *argp)
> @@ -299,7 +299,7 @@ static int sev_ioctl_do_pek_pdh_gen(int cmd, struct sev_issue_cmd *argp)
>   			return rc;
>   	}
>   
> -	return __sev_do_cmd_locked(cmd, 0, &argp->error);
> +	return __sev_do_cmd_locked(cmd, NULL, &argp->error);
>   }
>   
>   static int sev_ioctl_do_pek_csr(struct sev_issue_cmd *argp)
> @@ -624,7 +624,7 @@ EXPORT_SYMBOL_GPL(sev_guest_decommission);
>   
>   int sev_guest_df_flush(int *error)
>   {
> -	return sev_do_cmd(SEV_CMD_DF_FLUSH, 0, error);
> +	return sev_do_cmd(SEV_CMD_DF_FLUSH, NULL, error);
>   }
>   EXPORT_SYMBOL_GPL(sev_guest_df_flush);
>