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 59021C98304 for ; Wed, 23 Sep 2026 20:38:19 +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=/KLcoXD2VbiRh8mC0TjXM3oaWpGetLBX1IZdCZJ6pOU=; b=HT5xK+CeEabq6c Wcj7H2FvaygRi70t1r/nSuKNwDxZLWQjccMIEnW2WmDZucqzEe5VSzw9ELHZJlGyNhlqvHAn7JCU/ QdqOfa1L3VN0pcHFxQd6KiUvbaDSFGO+InXm1/VqIOnUY+MXBwmFeZ9sLh/WGiHkxg6sHIBykVl8b 6BQLk/i97pNa0+QUygYvWCTEgs1+k7Oi1BhjyhcX+PWWgvkLZkNas1bSrHC4FFjUdfuzLEhkEAUvj S8mo8kpoAuHK92bATbMLh+jzklbg27ofh2FDo1x7oriwMbUdyl2lcDZB1gmtC5+3LaSselaLjyJM1 7Cf2aSmNNXhAMbSFwCrA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x9TjL-00000009UEy-2pkZ; Wed, 23 Sep 2026 20:38:15 +0000 Received: from [95.181.193.9] (helo=mail.auroraos.dev) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x9TjI-00000009UD1-0YAl; Wed, 23 Sep 2026 20:38:14 +0000 Received: from [192.168.2.104] (91.78.46.39) by exch16.corp.auroraos.dev (10.189.209.38) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1847.3; Wed, 23 Sep 2026 23:38:02 +0300 Message-ID: Date: Wed, 23 Sep 2026 23:38:02 +0300 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] pmdomain: rockchip: fix domain check in rockchip_pm_add_one_domain() To: Ulf Hansson CC: Ulf Hansson , Heiko Stuebner , , , Caesar Wang , Kevin Hilman , References: <20260920175928.42566-1-s.shtylyov@auroraos.dev> Content-Language: en-US From: Sergey Shtylyov In-Reply-To: X-Originating-IP: [91.78.46.39] X-ClientProxiedBy: exch16.corp.auroraos.dev (10.189.209.38) To exch16.corp.auroraos.dev (10.189.209.38) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260923_133812_192542_7DE55E36 X-CRM114-Status: GOOD ( 12.89 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On 9/23/26 3:14 PM, Ulf Hansson wrote: [...] >> In rockchip_pm_add_one_domain(), there's the check (most probably for >> the case where the "reg" prop has an unexpected value?) which doesn't >> make much sense as the variable pd_info will be NULL iff pmu->info-> >> domain_info is NULL and the variable id is 0. What does make sense >> there is checking pd_info->pwr_mask and pd_info->req_mask which (as >> seems to follow from the code) can't both be 0 for a valid domain... >> >> Found by Linux Verification Center (linuxtesting.org) with the Svace >> static analysis tool. >> >> Fixes: 7c696693a4f5 ("soc: rockchip: power-domain: Add power domain driver") >> Signed-off-by: Sergey Shtylyov > > Applied for next, but without the fixes tag, thanks! > > Note, that I am dropping the fixes tag because if there was a real > problem, we would require yet another fix on top to get the > corresponding domain declaration to be correct. Yet, the patch is > useful as is! I'm not sure I understand you. The patch should hopefully be enough to detect the undefined entries (implicitly init'ed with all 0s). What fix for the domain definitions do you have in mind, turning the arrays into lookup tables by adding a domain ID as a field to *struct* rockchip_domain_info? > Kind regards > Uffe MBR, Sergey _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip