power: imx8: remove the resource owned check before power off
authorPeng Fan <peng.fan@nxp.com>
Mon, 4 May 2020 13:16:52 +0000 (21:16 +0800)
committerStefano Babic <sbabic@denx.de>
Sun, 10 May 2020 18:55:20 +0000 (20:55 +0200)
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>
drivers/power/domain/imx8-power-domain-legacy.c

index e6e619358c358beea002741d444bf6a8face0daa..b68be40766de5363252ecaff7546556a33666710 100644 (file)
@@ -137,13 +137,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;