On Wed, 2018-11-21 at 00:07 -0800, Stephen Boyd wrote:

Quoting Weiyi Lu (2018-11-19 18:37:34)

quoted

On Tue, 2018-11-13 at 11:31 -0800, Nicolas Boichat wrote:

quoted

On Mon, Nov 5, 2018 at 10:42 PM Weiyi Lu [off-list ref] wrote:

quoted

quoted

quoted

@@ -226,6 +397,7 @@ static int scpsys_power_on(struct generic_pm_domain *genpd)
        if (ret < 0)
                goto err_pwr_ack;

+

Drop this.

Why do we try to drop this? Once the power-on step fail, the following
steps should not be permitted.

I just see a whitespace addition that doesn't do anything. I suspect
it's a style nitpick and common practice to not include spurious diffs
in the patch. So remove this hunk?

Oops! I missed the extra whitespace. I will remove it. Many thanks.