From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9FC3E3D5C3A for ; Tue, 2 Jun 2026 10:16:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780395413; cv=none; b=mA4MB3mmUbhGesqkuWQ91QU4aPWQsiNFmry5NNf7PfW2Zbpnwo9yyTg4DRDmV+BIHm6UTmSi/HO/+JJbOpt4Wu3t1z0r3sTtsAJIyyYyqriDYIyYhbnZ759VY6tKdvZLmI8nUM08450cWTwyQGfXmjoiMajtYTxpuBCpX/dFCuc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780395413; c=relaxed/simple; bh=vhEkZTw4QlzQZ3n04KU8Ww88nfpYBe6Oem2sU6WQQ/g=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Tt4s2u4KJUZ8aazjkSEuCktSJ9oMh7mWcIy7LJhh4lpspHso2hyWh1D6wuoL/euYzzTlcQ1i6VZqtgL1eqXJHDOEi+/iy4EUWy20UQFBjTIoOLOOAiPz8lw4GqJlWf+JHml6P8EtnphoEBIMOVUJpaP8eevZCTAttXxKYqBcumk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=AKUOceib; arc=none smtp.client-ip=209.85.214.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="AKUOceib" Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-2c0c2d8b95bso15209435ad.1 for ; Tue, 02 Jun 2026 03:16:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780395412; x=1781000212; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=rCb8DxBgLNYKl2se7nFRUFkwYJO2PqG9toAr7Dgva9M=; b=AKUOceibMMmAKWnzXZzBgWiyiwi/+OxZoPWK36atW27zLbhPRMVx0xhWqR0yFRTlaM RUjS+/5ukYAmwFeliiiJzgjsYcRhmzJXeAseByAuWKiias3jIWcjHYrWVO77VqA++mfR LVn2n1+C0eN7goUADfuE5eYtk3Od7gXzFGbw7uFz57RUtX9rHQraym7Qi5tqt42kr09e NlUcQ6eqxVNRHg3jdvFmWhBsFhqSJnd23wz3vRjO26Dqp/3zyd8vHxc+aO7/K0tzwxK8 I8n+r2uVICI4UWzDS7/t2O157yRryCQHMTyxQCz236IO6J1HInXp8byUy4abFdCw+ei6 BDCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780395412; x=1781000212; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=rCb8DxBgLNYKl2se7nFRUFkwYJO2PqG9toAr7Dgva9M=; b=XQqF7nGsHSQyHvrwkY4VpCImHxB5zOzCpGLAXNGTiqAPGkBumMtlbct8iJy8GUZeYJ MOwFhkpN4vmAWK4cW2X/XrzTJzg1AHDPk+VmmkYNaVOCdLri0h5NPjVEYHEJxKMdmw7Q 4UPrSqn+wqXLxIr5H8AoZd7Z9erEO4UUvA3VrPy6dMse80yFow67Wu6ETAiDJZQs7KaR txj1+XZLYzFs4CkZ5DB2sdJz9TEotRJYOtqilm0FYS5LPl5iUigMqBR09/cpENzbIaVZ /4Zo2rvuhJZ3jUPyfEjA2RhMNGcaOk+vNBt5vaDY0eR5uKAi+Afy6MNzmT43akHWOup0 SFSQ== X-Forwarded-Encrypted: i=1; AFNElJ9qN5t5RhYO47RFxGwn/NNIiL3sAhSMfK5u+XxQglFXHc8danFwacyObasEG8SCkUpLkumoy2dRHTD1NnY=@vger.kernel.org X-Gm-Message-State: AOJu0Ywmgvmc4K626dO5iddyM0Ic1UMdJ6FmQDUsy47eXToJU1ZpzRLz EtxItOyztBDrkpx/4wVo2Y2bs/freu7YeUK/I27EN7Syw1qvkpGo2VW+ X-Gm-Gg: Acq92OEoKtJ+v3RD6I9+RmYYpdcdPWabsiFVTsGTSmDfNnjOPkY5eKjoUEPXXIQ/9h9 y5hOTs6LgziDoUQxXNFeS2NEJINRSIQojYtvZua9LHYyAQvvWDImpdUleK+zdgfWRid+mmd1zb0 JPV27/qFjmxEELnYCltFxOXRf/vYaJFPvLd2aVP/VJb1+kGp+qyliZPlELoPxfuktJTMFdw/zWw lXL1YVBDg2vPg2rsB980161HMAOIE/frc7E0ZUxVDbndMuuzvR5TJz90D6lumdf/Pb5cGvAV/zN pKRu1sGoIGkk9HiW7solqjMedATOYqtykcGxM410+5u7btfjM1s3cG9qltw4quIEfQqh64ihBjd upsvUHOCb7n0twmP5pncHKWgSmmUQKtDx1nkl4tb/sN0u5Z0ZbIfGIbq510geOKrhr2RY/MKFFS rVyD5hL9gJX7XfYRshoA0vIB/puWJe+fAfVMI/JsRzMLukcHZeWEETYvXsxZpYRt3krs6b X-Received: by 2002:a17:903:46cf:b0:2c0:c625:400d with SMTP id d9443c01a7336-2c0c6254383mr111686365ad.37.1780395411951; Tue, 02 Jun 2026 03:16:51 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2bf239fd84fsm132766305ad.23.2026.06.02.03.16.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 02 Jun 2026 03:16:51 -0700 (PDT) From: phucduc.bui@gmail.com To: Heiko Stuebner , Mark Brown , Liam Girdwood , Krzysztof Kozlowski Cc: Rob Herring , Conor Dooley , Jaroslav Kysela , Takashi Iwai , devicetree@vger.kernel.org, linux-sound@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, bui duc phuc Subject: [PATCH v2 3/5] ASoC: rockchip: rockchip_pdm: Reorder clock enable sequence Date: Tue, 2 Jun 2026 17:16:06 +0700 Message-ID: <20260602101608.45137-4-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260602101608.45137-1-phucduc.bui@gmail.com> References: <20260602101608.45137-1-phucduc.bui@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: bui duc phuc Enable the 'hclk' bus clock before the 'clk' controller clock during runtime resume. The bus clock provides the register access interface, so enable it before the controller clock. This also makes the resume sequence the reverse of the suspend sequence, which keeps the clock ordering consistent. Signed-off-by: bui duc phuc --- NOTE: This patch is compile-tested only. Please help test if you have the relevant Rockchip hardware. Changes in v2: - Clarify in the commit message that the resume sequence becomes the reverse of the suspend sequence. sound/soc/rockchip/rockchip_pdm.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/sound/soc/rockchip/rockchip_pdm.c b/sound/soc/rockchip/rockchip_pdm.c index c69cdd6f2499..8f78f7bc1806 100644 --- a/sound/soc/rockchip/rockchip_pdm.c +++ b/sound/soc/rockchip/rockchip_pdm.c @@ -422,16 +422,16 @@ static int rockchip_pdm_runtime_resume(struct device *dev) struct rk_pdm_dev *pdm = dev_get_drvdata(dev); int ret; - ret = clk_prepare_enable(pdm->clk); + ret = clk_prepare_enable(pdm->hclk); if (ret) { - dev_err(pdm->dev, "clock enable failed %d\n", ret); + dev_err(pdm->dev, "hclock enable failed %d\n", ret); return ret; } - ret = clk_prepare_enable(pdm->hclk); + ret = clk_prepare_enable(pdm->clk); if (ret) { - clk_disable_unprepare(pdm->clk); - dev_err(pdm->dev, "hclock enable failed %d\n", ret); + clk_disable_unprepare(pdm->hclk); + dev_err(pdm->dev, "clock enable failed %d\n", ret); return ret; } -- 2.43.0