Re: [PATCH 3/7] drivers: firmware: psci: Split psci_dt_cpu_init_idle()

From: Mark Rutland
Date: Fri Mar 01 2019 - 12:07:29 EST


On Thu, Feb 28, 2019 at 02:59:15PM +0100, Ulf Hansson wrote:
> Let's split the psci_dt_cpu_init_idle() function into two functions. This
> makes the code clearer and provides better re-usability.
>
> Cc: Lina Iyer <ilina@xxxxxxxxxxxxxx>
> Co-developed-by: Lina Iyer <lina.iyer@xxxxxxxxxx>
> Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>

Acked-by: Mark Rutland <mark.rutland@xxxxxxx>

Mark.

> ---
> drivers/firmware/psci/psci.c | 42 ++++++++++++++++++++----------------
> 1 file changed, 23 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/firmware/psci/psci.c b/drivers/firmware/psci/psci.c
> index c80ec1d03274..9788bfc1cf8b 100644
> --- a/drivers/firmware/psci/psci.c
> +++ b/drivers/firmware/psci/psci.c
> @@ -270,9 +270,26 @@ static int __init psci_features(u32 psci_func_id)
> #ifdef CONFIG_CPU_IDLE
> static DEFINE_PER_CPU_READ_MOSTLY(u32 *, psci_power_state);
>
> +static int psci_dt_parse_state_node(struct device_node *np, u32 *state)
> +{
> + int err = of_property_read_u32(np, "arm,psci-suspend-param", state);
> +
> + if (err) {
> + pr_warn("%pOF missing arm,psci-suspend-param property\n", np);
> + return err;
> + }
> +
> + if (!psci_power_state_is_valid(*state)) {
> + pr_warn("Invalid PSCI power state %#x\n", *state);
> + return -EINVAL;
> + }
> +
> + return 0;
> +}
> +
> static int psci_dt_cpu_init_idle(struct device_node *cpu_node, int cpu)
> {
> - int i, ret, count = 0;
> + int i, ret = 0, count = 0;
> u32 *psci_states;
> struct device_node *state_node;
>
> @@ -291,29 +308,16 @@ static int psci_dt_cpu_init_idle(struct device_node *cpu_node, int cpu)
> return -ENOMEM;
>
> for (i = 0; i < count; i++) {
> - u32 state;
> -
> state_node = of_parse_phandle(cpu_node, "cpu-idle-states", i);
> + ret = psci_dt_parse_state_node(state_node, &psci_states[i]);
> + of_node_put(state_node);
>
> - ret = of_property_read_u32(state_node,
> - "arm,psci-suspend-param",
> - &state);
> - if (ret) {
> - pr_warn(" * %pOF missing arm,psci-suspend-param property\n",
> - state_node);
> - of_node_put(state_node);
> + if (ret)
> goto free_mem;
> - }
>
> - of_node_put(state_node);
> - pr_debug("psci-power-state %#x index %d\n", state, i);
> - if (!psci_power_state_is_valid(state)) {
> - pr_warn("Invalid PSCI power state %#x\n", state);
> - ret = -EINVAL;
> - goto free_mem;
> - }
> - psci_states[i] = state;
> + pr_debug("psci-power-state %#x index %d\n", psci_states[i], i);
> }
> +
> /* Idle states parsed correctly, initialize per-cpu pointer */
> per_cpu(psci_power_state, cpu) = psci_states;
> return 0;
> --
> 2.17.1
>