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 0C147FD375E for ; Wed, 25 Feb 2026 13:13:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type: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=/2iXmGHfS/NXEAjPWYjEaZZnH6x43O895hkr7rfUvYE=; b=dkrraDDZKOXZa3g/R/5RcyShsj tMUbE95MRZICr1l72u6VbW/7u6x41g+M1GrMjMalfXo7dQFwnfom0eML2ktyi2mPV70XzMTDa0XmV 8x5B9Ps4UnsrqyZvWXjKJclUm6cyzVw3em4KJRF82r0q6zMLvLtx0yDDLJNmGOPlKutX229lZFjAU 9lof3rx7lCzJ3i+W2nDYlh12PBCcw0U+3dOSJb/pD9F7yr9Mek88Vg2qpCFiyTnTh4Bj5V+iYbabr jSW0EZDVXcxkMBDWaJPWuGctVcUirsqCBS2Ir65k1RJr6SCI97Pg9uCJt44b9simIAFE8x/uaFHjC L6UoYYfg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vvEhU-000000043FI-3XNj; Wed, 25 Feb 2026 13:13:12 +0000 Received: from bali.collaboradmins.com ([148.251.105.195]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vvEhO-0000000437U-1KwB; Wed, 25 Feb 2026 13:13:07 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1772025184; bh=9nwd6J7vPnLix9sIrT5JA6n5SKgLQzROMtQ/2zyqkUs=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=im/XYZ3U7IBhKG3hMNpSZWXbSP/UCDRog0YG9LEu7mEp1bG+o+4sWRgyAuLVyCqcI CT+ZeiXosUlmLTmSYDOF8SgWapuK75nNR30kxOX5vO12K7jwqKhguRAY43oV7rcGeK cq7mc4QU50wRSpWWx49+OeavS5BvZnpT2C67V48sD1ngD6dUdEJteEpdx35KNxhx7V vOweyG4ifkrlwyCGW57RvyZ0RP4O5C7wJ34ShQ5Ff4O69TZMczxVZEFy5lyLjkj967 +okSWbtLufruEAsiytKqUcsK61lQv51b918iVvytlxippO+oNLxc+ys3/UbDx7L1+u 7XLGx1wl0eWAQ== Received: from [192.168.1.100] (2-237-20-237.ip236.fastwebnet.it [2.237.20.237]) (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: kholk11) by bali.collaboradmins.com (Postfix) with ESMTPSA id 6A3E117E0D91; Wed, 25 Feb 2026 14:13:03 +0100 (CET) Message-ID: Date: Wed, 25 Feb 2026 14:13:03 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 3/7] PCI: mediatek-gen3: Split out device power helpers To: Chen-Yu Tsai , Matthias Brugger , Ryder Lee , Jianjun Wang , Lorenzo Pieralisi , =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= , Manivannan Sadhasivam , Rob Herring , Bjorn Helgaas Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Bartosz Golaszewski , linux-pci@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org References: <20260225072225.3345307-1-wenst@chromium.org> <20260225072225.3345307-4-wenst@chromium.org> From: AngeloGioacchino Del Regno Content-Language: en-US In-Reply-To: <20260225072225.3345307-4-wenst@chromium.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260225_051306_517539_6DF0A383 X-CRM114-Status: GOOD ( 11.17 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 25/02/26 08:22, Chen-Yu Tsai ha scritto: > Split out existing code that deals with device power sequencing into > into helpers to allow cleaning up the error paths and integrating the > new PCI pwrctrl API, > > This change is purely code movement. > > Signed-off-by: Chen-Yu Tsai I was really about to (and really wrote 80% of it) write a comment saying that this is not really doing power up, but then I looked at the other patches before continuing. Eh, I agree about doing what you're doing in this number of steps, but please write that spoiler in the commit description here :-) Something along the lines of... In preparation for adding full power on/off control with the pwrctrl API, split out the existing code that only partially deals with device power sequencing into separate helper functions. This is purely moving code around, and brings no functional changes. After which: Reviewed-by: AngeloGioacchino Del Regno