lkml.org 
[lkml]   [2014]   [Nov]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3/7] ARM: shmobile: R-Mobile: Generalize adding/looking up special PM domains
    Date
    Make adding special PM domains to an array, and looking them up
    later, more generic, so it can be used for all special hardware blocks.
    The type of PM domain is also stored, so rmobile_setup_pm_domain() can
    use a switch() statement instead of a chain of if/else statements.

    Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
    ---
    arch/arm/mach-shmobile/pm-rmobile.c | 107 ++++++++++++++++++++++--------------
    1 file changed, 67 insertions(+), 40 deletions(-)

    diff --git a/arch/arm/mach-shmobile/pm-rmobile.c b/arch/arm/mach-shmobile/pm-rmobile.c
    index 3d383d3dcd4ec640..774a8e276049a202 100644
    --- a/arch/arm/mach-shmobile/pm-rmobile.c
    +++ b/arch/arm/mach-shmobile/pm-rmobile.c
    @@ -226,71 +226,89 @@ static int rmobile_pd_suspend_console(void)
    return console_suspend_enabled ? 0 : -EBUSY;
    }

    -#define MAX_NUM_CPU_PDS 8
    +enum pd_types {
    + PD_NORMAL,
    + PD_CPU,
    + PD_CONSOLE,
    + PD_DEBUG,
    +};

    -static unsigned int num_cpu_pds __initdata;
    -static struct device_node *cpu_pds[MAX_NUM_CPU_PDS] __initdata;
    -static struct device_node *console_pd __initdata;
    -static struct device_node *debug_pd __initdata;
    +#define MAX_NUM_SPECIAL_PDS 16

    -static void __init get_special_pds(void)
    +static struct special_pd {
    + struct device_node *pd;
    + enum pd_types type;
    +} special_pds[MAX_NUM_SPECIAL_PDS] __initdata;
    +
    +static unsigned int num_special_pds __initdata;
    +
    +static void __init add_special_pd(struct device_node *np, enum pd_types type)
    {
    - struct device_node *np, *pd;
    unsigned int i;
    + struct device_node *pd;

    - /* PM domains containing CPUs */
    - for_each_node_by_type(np, "cpu") {
    - pd = of_parse_phandle(np, "power-domains", 0);
    - if (!pd)
    - continue;
    + pd = of_parse_phandle(np, "power-domains", 0);
    + if (!pd)
    + return;

    - for (i = 0; i < num_cpu_pds; i++)
    - if (pd == cpu_pds[i])
    - break;
    -
    - if (i < num_cpu_pds) {
    - of_node_put(pd);
    - continue;
    - }
    + for (i = 0; i < num_special_pds; i++)
    + if (pd == special_pds[i].pd && type == special_pds[i].type)
    + break;

    - if (num_cpu_pds == MAX_NUM_CPU_PDS) {
    - pr_warn("Too many CPU PM domains\n");
    - of_node_put(pd);
    - continue;
    - }
    + if (i < num_special_pds) {
    + of_node_put(pd);
    + return;
    + }

    - cpu_pds[num_cpu_pds++] = pd;
    + if (num_special_pds == ARRAY_SIZE(special_pds)) {
    + pr_warn("Too many special PM domains\n");
    + of_node_put(pd);
    + return;
    }

    + pr_debug("Special PM domain %s type %d for %s\n", pd->name, type,
    + np->full_name);
    +
    + special_pds[num_special_pds].pd = pd;
    + special_pds[num_special_pds].type = type;
    + num_special_pds++;
    +}
    +
    +static void __init get_special_pds(void)
    +{
    + struct device_node *np;
    +
    + /* PM domains containing CPUs */
    + for_each_node_by_type(np, "cpu")
    + add_special_pd(np, PD_CPU);
    +
    /* PM domain containing console */
    if (of_stdout)
    - console_pd = of_parse_phandle(of_stdout, "power-domains", 0);
    + add_special_pd(of_stdout, PD_CONSOLE);

    /* PM domain containing Coresight-ETM */
    np = of_find_compatible_node(NULL, NULL, "arm,coresight-etm3x");
    if (np)
    - debug_pd = of_parse_phandle(np, "power-domains", 0);
    + add_special_pd(np, PD_DEBUG);
    }

    static void __init put_special_pds(void)
    {
    unsigned int i;

    - for (i = 0; i < num_cpu_pds; i++)
    - of_node_put(cpu_pds[i]);
    - of_node_put(console_pd);
    - of_node_put(debug_pd);
    + for (i = 0; i < num_special_pds; i++)
    + of_node_put(special_pds[i].pd);
    }

    -static bool __init pd_contains_cpu(const struct device_node *pd)
    +static enum pd_types __init pd_type(const struct device_node *pd)
    {
    unsigned int i;

    - for (i = 0; i < num_cpu_pds; i++)
    - if (pd == cpu_pds[i])
    - return true;
    + for (i = 0; i < num_special_pds; i++)
    + if (pd == special_pds[i].pd)
    + return special_pds[i].type;

    - return false;
    + return PD_NORMAL;
    }

    static void __init rmobile_setup_pm_domain(struct device_node *np,
    @@ -298,7 +316,8 @@ static void __init rmobile_setup_pm_domain(struct device_node *np,
    {
    const char *name = pd->genpd.name;

    - if (pd_contains_cpu(np)) {
    + switch (pd_type(np)) {
    + case PD_CPU:
    /*
    * This domain contains the CPU core and therefore it should
    * only be turned off if the CPU is not in use.
    @@ -306,11 +325,15 @@ static void __init rmobile_setup_pm_domain(struct device_node *np,
    pr_debug("PM domain %s contains CPU\n", name);
    pd->gov = &pm_domain_always_on_gov;
    pd->suspend = rmobile_pd_suspend_busy;
    - } else if (np == console_pd) {
    + break;
    +
    + case PD_CONSOLE:
    pr_debug("PM domain %s contains serial console\n", name);
    pd->gov = &pm_domain_always_on_gov;
    pd->suspend = rmobile_pd_suspend_console;
    - } else if (np == debug_pd) {
    + break;
    +
    + case PD_DEBUG:
    /*
    * This domain contains the Coresight-ETM hardware block and
    * therefore it should only be turned off if the debug module is
    @@ -319,6 +342,10 @@ static void __init rmobile_setup_pm_domain(struct device_node *np,
    pr_debug("PM domain %s contains Coresight-ETM\n", name);
    pd->gov = &pm_domain_always_on_gov;
    pd->suspend = rmobile_pd_suspend_busy;
    + break;
    +
    + case PD_NORMAL:
    + break;
    }

    rmobile_init_pm_domain(pd);
    --
    1.9.1


    \
     
     \ /
      Last update: 2014-11-19 17:41    [W:3.109 / U:0.500 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site