From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 297ECC27C76 for ; Wed, 25 Jan 2023 12:54:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=m4WAkWqZIP0b6s1vC/GI6NoXyHNI5R4/COKJ9peRZ8I=; b=yNskj84l0gW6OF +jR1JB0aU7AElmY1j4UKmgYBDXd1sPXysPl6XG3A6dkS61fOWeP5WZ+NyMf8zqTMdSdmIuMg5DNTa 4Lx2FRVnaXfQn7rNx2T9ZMZlcmhlHawGQEYTdsgM0QxVG3Kk/6+GL1/EnakbAP9lYMB8DFEz5R4I3 OsDhIexrkulwNEzY3nEg2UvvoUo6W5avKr2F6kTZfCn8y3eI99hFp11jKkMaozC5uBfbbnjXJIHV8 cMZUfSqOHMLTs8xollsxI4vV4mvYx0AgMYM8ec6iyW/eYVds1tDGAEvL/kJMDT+r/7ADqZpfCS+Ub UTdWanRREKwi/dn4rC1A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKfH7-007Kjr-Jf; Wed, 25 Jan 2023 12:53:13 +0000 Received: from madras.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e5ab]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKfH3-007Kiu-K4 for linux-arm-kernel@lists.infradead.org; Wed, 25 Jan 2023 12:53:11 +0000 Received: from [192.168.2.197] (109-252-117-89.nat.spd-mgts.ru [109.252.117.89]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: dmitry.osipenko) by madras.collabora.co.uk (Postfix) with ESMTPSA id 015C36602E4D; Wed, 25 Jan 2023 12:53:04 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1674651186; bh=FsmNHA2oh2Kx15JYgMiYPMwo9fJuN5w1csBgs9yBag0=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=RpwnSDTKnV/756GSwFe99d13fNSHYTBDoI1mjxpTJsvNS7ku4HBu1ZgsTst6A0Dji Q776uDOZdQbH5Rqgit4GJWEZ3wL+2EAjVJekWRsEtkNaIKutJjJCaHDtIlHIsqGORy dDa+G53xoAZawv88Hv7ZzQZmLSefHyEMkK9Ll0mHuhhqBpowO9D5U9abGd2itwph3u QGoz6uXnF3HXtEvGqD+Z1ShQWD4IE/5QeLBcxsQLGCcjG3zwDF4z3ZDG6/LXbLpZSz UcfYhS4O90WWIjHzDOTV3TpOmihjVdGROF+1dP2eQhae8k6NKUU4X39mPRQc/DoaVm B+2oAPubBm6VA== 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" Cc: John Ogness , Kai-Heng Feng , Luis Chamberlain , Petr Mladek , YueHaibing , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, tangmeng , Linux PM list , Samuel Holland , Mark Rutland , Lorenzo Pieralisi , Sudeep Holla References: <20230101181715.42199-1-samuel@sholland.org> From: Dmitry Osipenko In-Reply-To: <20230101181715.42199-1-samuel@sholland.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230125_045309_839112_04A6C166 X-CRM114-Status: GOOD ( 23.50 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel