[PATCH 2/2] nvme-apple: return directly instead of else

Eric Curtin ecurtin at redhat.com
Mon Mar 27 12:13:58 PDT 2023


On Mon, 27 Mar 2023 at 06:55, Chaitanya Kulkarni <kch at nvidia.com> wrote:
>
> There is no need for the else when direct return is used at the end of
> the function.
>
> Signed-off-by: Chaitanya Kulkarni <kch at nvidia.com>

Reviewed-by: Eric Curtin <ecurtin at redhat.com>

Is mise le meas/Regards,

Eric Curtin

> ---
>  drivers/nvme/host/apple.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/nvme/host/apple.c b/drivers/nvme/host/apple.c
> index 942a3b71a6d5..33601494ae50 100644
> --- a/drivers/nvme/host/apple.c
> +++ b/drivers/nvme/host/apple.c
> @@ -217,8 +217,8 @@ static unsigned int apple_nvme_queue_depth(struct apple_nvme_queue *q)
>  {
>         if (q->is_adminq)
>                 return APPLE_NVME_AQ_DEPTH;
> -       else
> -               return APPLE_ANS_MAX_QUEUE_DEPTH;
> +
> +       return APPLE_ANS_MAX_QUEUE_DEPTH;
>  }
>
>  static void apple_nvme_rtkit_crashed(void *cookie)
> --
> 2.29.0
>
>




More information about the Linux-nvme mailing list