From patchwork Tue Mar 7 00:24:20 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sam Protsenko X-Patchwork-Id: 13162549 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 4D18CC64EC4 for ; Tue, 7 Mar 2023 00:25:33 +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:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=JCqY3eLtnEwBmuSZP3tTz/L/aA15nqhV4EdXcMK8Iqk=; b=A2MAICqXOHWION oAkI2Vbn8/aUVZQA5DIs80TH+9bxVFdEptKUbxFLCOgB6sIvGcguZq7dTxxeH4gtq1Z5zLrBO9Et8 s7uLedLpDoz2TJVZopcopEbmPjwznGFEItOgTMULX01PjVDQ9DjTotfHlOygyOPFFRF4wPGjKHrab I3bzQNX21D3Y/kfM2VBWiTA5Jdo6GN8x849mc3b2G1UhdifUYRUb+Cdjhk2dW/RC6AFlvrhEcAD5G PjV08qb9B5qqcwJ3xTLvcE0jH2e8ISkT1sY2MPr2WPz6sb65ZsxmRS4PcmRXhVDBOtNFS1LFEBdSz NpyXcHn6FVMNz1iYqt2g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pZL83-00FawA-Mr; Tue, 07 Mar 2023 00:24:31 +0000 Received: from mail-oa1-x2c.google.com ([2001:4860:4864:20::2c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pZL7z-00Fatd-2v for linux-arm-kernel@lists.infradead.org; Tue, 07 Mar 2023 00:24:28 +0000 Received: by mail-oa1-x2c.google.com with SMTP id 586e51a60fabf-17671fb717cso11963164fac.8 for ; Mon, 06 Mar 2023 16:24:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1678148665; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=D4ip3sLglqlQ7E7789pVNb0z5WZUEnYuDewtkqp1id4=; b=GdTih1HDlNXVA9H6OR/RW6m37uzRP3VKQX+ND1QSHcoLL9A1O2uIErkClZ3vIEDsGk 9j77xJmF9E00zrLSJGQbtuUIPCIctKSCY+4KgPgxagf8YCVevcRO5QbpPqox2pySrkNu GCnSZ1re7/cUK98AwTGe7VPQbfFK6bWPwMn2vmxbxLn8UYy2seSyvU5YlGakbOZrwzxW aotTntUdU6hMoVhvrSZyrEdx2kQqF3bKmjRocS25yXENF4hTCAIEEJz8j8qdKVgI61Ag +G4XtTxLjSX3nRDWBfHATW95RTYv/n148choNo7Esl/CKLE16QkAqCEwOGz+s0ooxg2E rkFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678148665; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=D4ip3sLglqlQ7E7789pVNb0z5WZUEnYuDewtkqp1id4=; b=AUrtQYiutFbrnZSKIWKFNNxRV4b3LmHH0YqmB58VpTDuB2ZXy9Z6cyIzKMlVhOunIg /39Xc8RjxexOJWeX8E1PEgZdhlvtUdlRgYtlYlY8mGmid/tVf4cVFGoQOU20FvJsP+yC s8g0+dPN98j8Iv5YPgqcLc9Y7OBooJSMfZx+W1dMFt3NduxkpVTgKeoIvEWML5YpG5w6 HgE+wVIamcUDv/k+GKx+SldZcYPgpF6ar6rTu2zbLUx5U9awQMUAqGHLQ8eq5R4pH93B vcNv471N9kj839D81kM3PMPSDDC6FWMerwa+6WAtNAV2fXuDgFo2dow4DrjQGoKBjvzS ZbHg== X-Gm-Message-State: AO0yUKVEjQSMtE8u6hCeHxPYYdSVqpE5jTBxOWaqZYi/rCLoKZHmQ5K2 q776jGwt/KtSvQtlaKqYbFK3Uw== X-Google-Smtp-Source: AK7set/55XNiVHEyI+5t+FJNuGNj6pwtNyPc2xIbVC1QIFUa9KMNTgq5WIEDx6T+IyxjBFyelih5kQ== X-Received: by 2002:a05:6870:c10c:b0:163:51eb:b577 with SMTP id f12-20020a056870c10c00b0016351ebb577mr9102897oad.46.1678148664749; Mon, 06 Mar 2023 16:24:24 -0800 (PST) Received: from localhost ([136.49.140.41]) by smtp.gmail.com with ESMTPSA id x7-20020a9d3787000000b00690dd5e7345sm4783041otb.26.2023.03.06.16.24.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 06 Mar 2023 16:24:24 -0800 (PST) From: Sam Protsenko To: Marek Szyprowski , Krzysztof Kozlowski , Sylwester Nawrocki , Chanwoo Choi Cc: Tomasz Figa , Alim Akhtar , Chanho Park , David Virag , Sumit Semwal , Michael Turquette , Stephen Boyd , linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH v4 0/3] clk: samsung: Add PM support for ARM64 Exynos chips Date: Mon, 6 Mar 2023 18:24:20 -0600 Message-Id: <20230307002423.24454-1-semen.protsenko@linaro.org> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230306_162427_303357_5DF15EC2 X-CRM114-Status: GOOD ( 14.04 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org In order to prepare for PM enablement in clk-exynos850, common PM code was extracted from clk-exynos5433 to clk-exynos-arm64. Also some related cleanups were done prior to that. More specifically: - patches #1..2: cleanups - patch #3: PM code extraction During the extraction of the exynos5433_cmu_probe() content to exynos_arm64_register_cmu_pm() some code was reworked a bit, and also split into smaller functions. In particular: - cmu_data structure now contains a pointer to ctx, which is now allocated in samsung_clk_init() - cmu_data structure initialization was moved into separate function - code for configuring gate clocks was added (optional) Which in turn resulted in somehow modified code of probe function: Original -------- ... devm_platform_ioremap_resource(...); samsung_clk_init(...); exynos_arm64_cmu_prepare_pm(...); exynos_arm64_enable_bus_clk(...); platform_set_drvdata(...); ... Modified -------- ... platform_set_drvdata(...); exynos_arm64_cmu_prepare_pm(...); exynos_arm64_enable_bus_clk(...); exynos_arm64_init_clocks(...); devm_platform_ioremap_resource(...); samsung_clk_init(...); ... That shouldn't really change the logic or mode of operation. It was preliminary tested on Exynos850 based board, with some extra patches on top of this series (will be submitted later). Changes in v4: - Rebased on top of krzk/linux.git/for-next tree - Reworked exynos_arm64_enable_bus_clk() users to only print an error message if that function fails - Removed already applied patches from this series Changes in v3: - Rebased on the most recent soc/for-next tree - Added A-b tags from Marek - Added Marek's authorship to clk-exynos-arm64.c Changes in v2: - Rebased all patches on top of the latest soc/for-next tree - Added Tested-by tag from Marek - Addressed comments from the review Sam Protsenko (3): clk: samsung: Extract clocks registration to common function clk: samsung: Extract parent clock enabling to common function clk: samsung: exynos5433: Extract PM support to common ARM64 layer drivers/clk/samsung/clk-exynos-arm64.c | 229 +++++++++++++++++++++++-- drivers/clk/samsung/clk-exynos-arm64.h | 3 + drivers/clk/samsung/clk-exynos5433.c | 157 +---------------- drivers/clk/samsung/clk.c | 46 +++-- drivers/clk/samsung/clk.h | 2 + 5 files changed, 249 insertions(+), 188 deletions(-)