Message-ID: <0a180849-ba1b-2a82-ab06-ed1b8155d5ca@collabora.com>
Date:   Wed, 25 Jan 2023 15:53:01 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [PATCH v2 0/2] firmware/psci: Switch to the sys-off handler API
Content-Language: en-US
To:     "Rafael J . Wysocki" <rafael.j.wysocki@intel.com>
Cc:     John Ogness <john.ogness@linutronix.de>,
        Kai-Heng Feng <kai.heng.feng@canonical.com>,
        Luis Chamberlain <mcgrof@kernel.org>,
        Petr Mladek <pmladek@suse.com>,
        YueHaibing <yuehaibing@huawei.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        tangmeng <tangmeng@uniontech.com>,
        Linux PM list <linux-pm@vger.kernel.org>,
        Samuel Holland <samuel@sholland.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Lorenzo Pieralisi <lpieralisi@kernel.org>,
        Sudeep Holla <sudeep.holla@arm.com>
References: <20230101181715.42199-1-samuel@sholland.org>
From:   Dmitry Osipenko <dmitry.osipenko@collabora.com>
In-Reply-To: <20230101181715.42199-1-samuel@sholland.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 1/1/23 21:17, Samuel Holland wrote:
> I want to convert the axp20x PMIC poweroff handler to use the sys-off
> API, so it can be used as a fallback for the SBI poweroff handler on
> RISC-V. But the PSCI poweroff handler still uses pm_power_off, so done
> alone, this conversion would cause the axp20x callback to be called
> first, before the PSCI poweroff handler.
> 
> In order to prevent this change in behavior, the PSCI poweroff handler
> needs to be converted to the sys-off API first, at a higher priority.
> 
> This series performs the conversion, after accounting for the fact that
> the PSCI poweroff handler is registered quite early during boot.
> 
> The first patch is a dependency for both this series and the SBI
> series[1], so I would like to get at least patch 1 merged soon.
> 
> [1]: https://lore.kernel.org/lkml/20221228161915.13194-1-samuel@sholland.org/
> 
> Changes in v2:
>  - Update commit messages
> 
> Samuel Holland (2):
>   kernel/reboot: Use the static sys-off handler for any priority
>   firmware/psci: Switch to the sys-off handler API
> 
>  drivers/firmware/psci/psci.c |  9 ++++++---
>  kernel/reboot.c              | 10 ++++------
>  2 files changed, 10 insertions(+), 9 deletions(-)
> 

Hello Rafael,

Do you think you will be able to pick up this series for 6.3? I'm going
to continue removing the pm_power_off from kernel, the new power-off API
feels stable now to me. I think the Samuel's improvement for the early
boot memory allocation will be good to have to avoid similar problem for
other drivers.

Ideally, the PSCI patch should get an ack, though the code change is
about the PM stuff, so perhaps will be fine to take it via PM tree if FW
maintainers will show no interest in the nearest time.

Thanks!

-- 
Best regards,
Dmitry