> +++ b/arch/arm/mach-hisi/hotplug.c
> @@ -173,11 +173,15 @@  static bool hix5hd2_hotplug_init(void)
…
> +     ctrl_base = of_iomap(np, 0);
> +     of_node_put(np);
> +     if (!ctrl_base)
> +             return false;
> +
> +     return true;
>  }
>
>  void hix5hd2_set_cpu(int cpu, bool enable)

Would the use of a ternary operator be more succinct here?

+       return ctrl_base ? true : false;

Regards,
Markus

Reply via email to