From patchwork Sat Feb 5 10:36:09 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Jonathan_Neusch=C3=A4fer?= X-Patchwork-Id: 12736039 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 CBC24C433FE for ; Sat, 5 Feb 2022 10:38:13 +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:References:In-Reply-To: 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: List-Owner; bh=Un0SX6kY+ZnuwGAZAby5yqbJWpHZJ0JWSwrWdGAnARI=; b=U4Ke6pq7F5yBdf boWCSCXmVYq0p0x/Z1vB6BAI0CSiPiOdzeq7GPhaDtMk3xQwY952A/0NoEybYAnVwDfuEXlcefZLz unUn26k6zay6zq4+zNkH2KY+XWQmpfXG4uAIDp2fyoety/9bW68ZL/PD9NhDqPiAZHnI2Kr9xeGoU 0CX3Xv08boRMGKuFkByVeRB9xDu+35WsZdgnRwheYJIs1TsmfbCqfJfPkyrJrjoIh8K/iW7fuY7bZ PWxiKssdY5cLewijFqJLuX2WCpcdpLi1JNWwn5zxA9bYO2Hp1Sh6DREm4bEp9h9kB/8sq3Af3aIOk g1mXw6KTkL7mg1beHnEQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nGIQz-006LYH-8r; Sat, 05 Feb 2022 10:36:49 +0000 Received: from mout.gmx.net ([212.227.15.15]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nGIQp-006LVD-Nq for linux-arm-kernel@lists.infradead.org; Sat, 05 Feb 2022 10:36:41 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1644057397; bh=7Jnhrvq7jRctQBKS0gpYrwPxRFguQ8hr+Yc62pb/vc4=; h=X-UI-Sender-Class:From:To:Cc:Subject:Date:In-Reply-To:References; b=h5xsDfehoCDPNAR4CqXfY7qEdmJ+UiwSXXUx4GhwL/DsiVmXrRDbmAwvSbgArXBgA Lx3+uXP2PK98ZLschL1H92KDwhsQ7K4YK6WsSiINYhlZktk+Xdfjv2mz0a2XTgyXpj 5oO8qnU+huyXj16752LWYpwk1LTwxNaQFweEec4g= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from longitude ([5.146.194.160]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1ML9uK-1my4pD2x1e-00IGtB; Sat, 05 Feb 2022 11:36:37 +0100 From: =?utf-8?q?Jonathan_Neusch=C3=A4fer?= To: linux-clk@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, =?utf-8?q?Jonathan_Neusch=C3=A4fer?= , Michael Turquette , Stephen Boyd Subject: [PATCH v2 3/7] clk: mux: Declare u32 *table parameter as const Date: Sat, 5 Feb 2022 11:36:09 +0100 Message-Id: <20220205103613.1216218-4-j.neuschaefer@gmx.net> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220205103613.1216218-1-j.neuschaefer@gmx.net> References: <20220205103613.1216218-1-j.neuschaefer@gmx.net> MIME-Version: 1.0 X-Provags-ID: V03:K1:2D604PBdLT4aQ0NrLOoct9Yj4ORp9f+pTiavGBxkK+l4ExfPbNu 6omMJ+9sB+rUfJ/HjP5GrPQVPKRcQ+BCWK1L5lnUH/NTA9HMkwuWPORUYZpkoMV9NQPVyAo S6dnuR05xlN3R5TzXmqtB6U/oSLCESJwsY7U7NdozfjZp7UUIaBfegp9ByyVzaQqInwX7/l ESA651GAKBGOsmUWU3B5w== X-UI-Out-Filterresults: notjunk:1;V03:K0:zeIonvdIsVs=:9ahvsbG6OrJxncXAPnYrh7 D9c5I8BD9T1tpxsmmgL1EgMpuH+JudYHN+HJp0GlN01aL3thwzAULR/uqv/TR8J2sukh7zXJx k/Uqil5oR2ETLEIepha2Fb7AfVse7xIuce4WzRCCtqb3c0fjrUTjVsyfqZaKinwQ+j/Q6DYJW dlIk3NxxLsQ5K9WqhRyzLcmyKThgm6kTA/hjf5rqSnFsGQG3U2oZzcQrdVrqSqskQYW+0ymot leSy7eXTOZPgK6mfSpmTbTb5+sm+7N/7yE/0S93jsns2hv+chhGkCly3sN4EkTKUh1UlMg8GI Kn5ziB+NojK80bAxfd2y99XvBzhRz67kiqiDR7LVX3IdZjTvC4U7eZFr6dop2ih7JJZZxhoYC O/KEolzZeWFHQBL5n2sPGPN28Tw4MQIZ6XCD5IbA5MKJJxshit5sRGHWpss53uvEhD8j0r82m QA1ejjFOsCr2uwpwLHi2iKVp2Z99m8ZfZI2lsL89wrN356o6UL2qk98eR+FHdiV2f5jI6gJqs mf3NkKZ3v+iiMnTBDf5XK8EIra4NIKt7D6EIyx3Ed/oi/ZqWQBR9jBZprVPIph91g0u51UKs1 ZpIb1hZqRLkBEEa4b1Bz9n+jxYMOWr8fjkFNr15aoJwhOVRzmUyIKbRT58Jl3srizrOSl3ReG wHws+3YzbO+y8QqgVqcf0POHIt1wHu5ulZ4dLIb9dJaPO+6m3U/VzzyKH847IsTFv558UuGLR L2+KelCiHSQyHkHaSqu6npjOa+JPsF/h7EqmnLOYTAR29Ytb2NVPHtCeKNC9pXkCftvldjUSe 9ko3b1MPZkujO7qk+6XU+s3YKPwyQGkHKZkcCZStmkxgL7qU7DieeaSKaDyKq4PhUkOQBhO8O mUTaC+VPwM/L66kDm8M3/81j6CKf782kV4/zlrI+D64WZY9flJMIUgZ5AlJK5E78GGlPawghc J8o684Li36dVmpt72mP6rOHeY4CEg0jPNelYGbcvQz1/MA/VH2JEsc6e0tEj1i5I2orwmhKam v0QnkULmV0bOPSP+t4K2v4lliqxuatQHrRfySKwDG4DBC+GMuDwQobyP3VFt48cqZWJyYTsMr za9lWFA9wX0n6A= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220205_023640_103602_E66460B7 X-CRM114-Status: GOOD ( 12.46 ) 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 The elements of the table are never modified in clk-mux.c. To make this clear to clock drivers, declare the parameter as const u32 *table. Signed-off-by: Jonathan Neuschäfer --- v2: - no changes --- drivers/clk/clk-mux.c | 10 +++++----- include/linux/clk-provider.h | 12 ++++++------ 2 files changed, 11 insertions(+), 11 deletions(-) -- 2.34.1 diff --git a/drivers/clk/clk-mux.c b/drivers/clk/clk-mux.c index 20582aae7a35f..214045f6e9895 100644 --- a/drivers/clk/clk-mux.c +++ b/drivers/clk/clk-mux.c @@ -40,7 +40,7 @@ static inline void clk_mux_writel(struct clk_mux *mux, u32 val) writel(val, mux->reg); } -int clk_mux_val_to_index(struct clk_hw *hw, u32 *table, unsigned int flags, +int clk_mux_val_to_index(struct clk_hw *hw, const u32 *table, unsigned int flags, unsigned int val) { int num_parents = clk_hw_get_num_parents(hw); @@ -67,7 +67,7 @@ int clk_mux_val_to_index(struct clk_hw *hw, u32 *table, unsigned int flags, } EXPORT_SYMBOL_GPL(clk_mux_val_to_index); -unsigned int clk_mux_index_to_val(u32 *table, unsigned int flags, u8 index) +unsigned int clk_mux_index_to_val(const u32 *table, unsigned int flags, u8 index) { unsigned int val = index; @@ -152,7 +152,7 @@ struct clk_hw *__clk_hw_register_mux(struct device *dev, struct device_node *np, const struct clk_hw **parent_hws, const struct clk_parent_data *parent_data, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock) + u8 clk_mux_flags, const u32 *table, spinlock_t *lock) { struct clk_mux *mux; struct clk_hw *hw; @@ -218,7 +218,7 @@ struct clk_hw *__devm_clk_hw_register_mux(struct device *dev, struct device_node const struct clk_hw **parent_hws, const struct clk_parent_data *parent_data, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock) + u8 clk_mux_flags, const u32 *table, spinlock_t *lock) { struct clk_hw **ptr, *hw; @@ -244,7 +244,7 @@ EXPORT_SYMBOL_GPL(__devm_clk_hw_register_mux); struct clk *clk_register_mux_table(struct device *dev, const char *name, const char * const *parent_names, u8 num_parents, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock) + u8 clk_mux_flags, const u32 *table, spinlock_t *lock) { struct clk_hw *hw; diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h index 2faa6f7aa8a87..27be575288747 100644 --- a/include/linux/clk-provider.h +++ b/include/linux/clk-provider.h @@ -888,7 +888,7 @@ void clk_hw_unregister_divider(struct clk_hw *hw); struct clk_mux { struct clk_hw hw; void __iomem *reg; - u32 *table; + const u32 *table; u32 mask; u8 shift; u8 flags; @@ -913,18 +913,18 @@ struct clk_hw *__clk_hw_register_mux(struct device *dev, struct device_node *np, const struct clk_hw **parent_hws, const struct clk_parent_data *parent_data, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock); + u8 clk_mux_flags, const u32 *table, spinlock_t *lock); struct clk_hw *__devm_clk_hw_register_mux(struct device *dev, struct device_node *np, const char *name, u8 num_parents, const char * const *parent_names, const struct clk_hw **parent_hws, const struct clk_parent_data *parent_data, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock); + u8 clk_mux_flags, const u32 *table, spinlock_t *lock); struct clk *clk_register_mux_table(struct device *dev, const char *name, const char * const *parent_names, u8 num_parents, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock); + u8 clk_mux_flags, const u32 *table, spinlock_t *lock); #define clk_register_mux(dev, name, parent_names, num_parents, flags, reg, \ shift, width, clk_mux_flags, lock) \ @@ -962,9 +962,9 @@ struct clk *clk_register_mux_table(struct device *dev, const char *name, (shift), BIT((width)) - 1, (clk_mux_flags), \ NULL, (lock)) -int clk_mux_val_to_index(struct clk_hw *hw, u32 *table, unsigned int flags, +int clk_mux_val_to_index(struct clk_hw *hw, const u32 *table, unsigned int flags, unsigned int val); -unsigned int clk_mux_index_to_val(u32 *table, unsigned int flags, u8 index); +unsigned int clk_mux_index_to_val(const u32 *table, unsigned int flags, u8 index); void clk_unregister_mux(struct clk *clk); void clk_hw_unregister_mux(struct clk_hw *hw);