MLK-18435-2 power: imx8: remove the resource owned check before poewr off
authorPeng Fan <peng.fan@nxp.com>
Wed, 30 May 2018 01:48:34 +0000 (09:48 +0800)
committerYe Li <ye.li@nxp.com>
Fri, 24 May 2019 11:28:14 +0000 (04:28 -0700)
For all the devices used and set ACTIVE in U-Boot, U-Boot needs
to power off all of them without the check of resource owner.

When we create software partition before booting Linux, the resource
own checkw will return false, and cause the power domain not powered
off. If without the check of resource owner, the power domain in
the other software partition could be powered off with parent
partition could access child partition resources.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
(cherry picked from commit 3e29e8adace18035850be9d56cc277c64a221e85)
(cherry picked from commit 81e64d882c2904f5aa38121d020f6e21e142aaf8)

drivers/power/domain/imx8-power-domain.c

index d51dbaa..bce38d5 100644 (file)
@@ -108,13 +108,13 @@ static int imx8_power_domain_off_node(struct power_domain *power_domain)
        }
 
        if (pdata->resource_id != SC_R_LAST) {
-               if (!sc_rm_is_resource_owned(-1, pdata->resource_id)) {
-                       printf("%s not owned by curr partition\n", dev->name);
-                       return 0;
-               }
                ret = sc_pm_set_resource_power_mode(-1, pdata->resource_id,
                                                    SC_PM_PW_MODE_OFF);
                if (ret) {
+                       if (!sc_rm_is_resource_owned(-1, pdata->resource_id)) {
+                               printf("%s not owned by curr partition %d\n", dev->name, pdata->resource_id);
+                               return 0;
+                       }
                        printf("Error: %s Power off failed! (error = %d)\n",
                               dev->name, ret);
                        return -EIO;