Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753636AbbBMQnJ (ORCPT ); Fri, 13 Feb 2015 11:43:09 -0500 Received: from down.free-electrons.com ([37.187.137.238]:49340 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752415AbbBMQnF (ORCPT ); Fri, 13 Feb 2015 11:43:05 -0500 From: Antoine Tenart To: sebastian.hesselbarth@gmail.com, mturquette@linaro.org, sboyd@codeaurora.org Cc: Antoine Tenart , zmxu@marvell.com, jszhang@marvell.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/7] clk: convert clock mux to accept regmap Date: Fri, 13 Feb 2015 17:42:55 +0100 Message-Id: <1423845781-7480-2-git-send-email-antoine.tenart@free-electrons.com> X-Mailer: git-send-email 2.3.0 In-Reply-To: <1423845781-7480-1-git-send-email-antoine.tenart@free-electrons.com> References: <1423845781-7480-1-git-send-email-antoine.tenart@free-electrons.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7107 Lines: 222 Rework the clk_mux helpers to either use an iomem base address or a regmap. Signed-off-by: Antoine Tenart --- drivers/clk/clk-mux.c | 72 +++++++++++++++++++++++++++++++++++++------- include/linux/clk-provider.h | 39 +++++++++++++++++++----- 2 files changed, 93 insertions(+), 18 deletions(-) diff --git a/drivers/clk/clk-mux.c b/drivers/clk/clk-mux.c index 6e1ecf94bf58..1cd0ecd35ce3 100644 --- a/drivers/clk/clk-mux.c +++ b/drivers/clk/clk-mux.c @@ -42,7 +42,11 @@ static u8 clk_mux_get_parent(struct clk_hw *hw) * OTOH, pmd_trace_clk_mux_ck uses a separate bit for each clock, so * val = 0x4 really means "bit 2, index starts at bit 0" */ - val = clk_readl(mux->reg) >> mux->shift; + if (mux->reg_type == CLK_REG_TYPE_IOMEM) + val = clk_readl(mux->reg.iomem) >> mux->shift; + else + regmap_read(mux->reg.regmap, mux->reg.offset, &val); + val &= mux->mask; if (mux->table) { @@ -83,20 +87,24 @@ static int clk_mux_set_parent(struct clk_hw *hw, u8 index) index++; } - if (mux->lock) - spin_lock_irqsave(mux->lock, flags); + if (mux->reg_type == CLK_REG_TYPE_IOMEM && mux->reg.lock) + spin_lock_irqsave(mux->reg.lock, flags); if (mux->flags & CLK_MUX_HIWORD_MASK) { val = mux->mask << (mux->shift + 16); } else { - val = clk_readl(mux->reg); + val = clk_readl(mux->reg.iomem); val &= ~(mux->mask << mux->shift); } val |= index << mux->shift; - clk_writel(val, mux->reg); - if (mux->lock) - spin_unlock_irqrestore(mux->lock, flags); + if (mux->reg_type == CLK_REG_TYPE_IOMEM) + clk_writel(val, mux->reg.iomem); + else + regmap_write(mux->reg.regmap, mux->reg.offset, val); + + if (mux->reg_type == CLK_REG_TYPE_IOMEM && mux->reg.lock) + spin_unlock_irqrestore(mux->reg.lock, flags); return 0; } @@ -113,10 +121,10 @@ const struct clk_ops clk_mux_ro_ops = { }; EXPORT_SYMBOL_GPL(clk_mux_ro_ops); -struct clk *clk_register_mux_table(struct device *dev, const char *name, +struct clk *__clk_register_mux_table(struct device *dev, const char *name, const char **parent_names, u8 num_parents, unsigned long flags, - void __iomem *reg, u8 shift, u32 mask, - u8 clk_mux_flags, u32 *table, spinlock_t *lock) + union clk_reg reg, enum clk_reg_type reg_type, u8 shift, + u32 mask, u8 clk_mux_flags, u32 *table) { struct clk_mux *mux; struct clk *clk; @@ -149,10 +157,10 @@ struct clk *clk_register_mux_table(struct device *dev, const char *name, /* struct clk_mux assignments */ mux->reg = reg; + mux->reg_type = reg_type; mux->shift = shift; mux->mask = mask; mux->flags = clk_mux_flags; - mux->lock = lock; mux->table = table; mux->hw.init = &init; @@ -163,8 +171,37 @@ struct clk *clk_register_mux_table(struct device *dev, const char *name, return clk; } + +struct clk *clk_register_mux_table(struct device *dev, const char *name, + const char **parent_names, u8 num_parents, unsigned long flags, + void __iomem *reg, u8 shift, u32 mask, + u8 clk_mux_flags, u32 *table, spinlock_t *lock) +{ + union clk_reg clk_reg; + + clk_reg.iomem = reg; + clk_reg.lock = lock; + return __clk_register_mux_table(dev, name, parent_names, num_parents, + flags, clk_reg, CLK_REG_TYPE_IOMEM, + shift, mask, clk_mux_flags, table); +} EXPORT_SYMBOL_GPL(clk_register_mux_table); +struct clk *clk_register_mux_table_regmap(struct device *dev, const char *name, + const char **parent_names, u8 num_parents, unsigned long flags, + struct regmap *regmap, unsigned int offset, u8 shift, u32 mask, + u8 clk_mux_flags, u32 *table) +{ + union clk_reg clk_reg; + + clk_reg.regmap = regmap; + clk_reg.offset = offset; + return __clk_register_mux_table(dev, name, parent_names, num_parents, + flags, clk_reg, CLK_REG_TYPE_REGMAP, + shift, mask, clk_mux_flags, table); +} +EXPORT_SYMBOL_GPL(clk_register_mux_table_regmap); + struct clk *clk_register_mux(struct device *dev, const char *name, const char **parent_names, u8 num_parents, unsigned long flags, void __iomem *reg, u8 shift, u8 width, @@ -177,3 +214,16 @@ struct clk *clk_register_mux(struct device *dev, const char *name, NULL, lock); } EXPORT_SYMBOL_GPL(clk_register_mux); + +struct clk *clk_register_mux_regmap(struct device *dev, const char *name, + const char **parent_names, u8 num_parents, unsigned long flags, + struct regmap *regmap, unsigned int offset, u8 shift, u8 width, + u8 clk_mux_flags) +{ + u32 mask = BIT(width) - 1; + + return clk_register_mux_table_regmap(dev, name, parent_names, + num_parents, flags, regmap, offset, shift, + mask, clk_mux_flags, NULL); +} +EXPORT_SYMBOL_GPL(clk_register_mux_regmap); diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h index d936409520f8..3d2b9ab2130d 100644 --- a/include/linux/clk-provider.h +++ b/include/linux/clk-provider.h @@ -14,6 +14,7 @@ #include #include #include +#include #ifdef CONFIG_COMMON_CLK @@ -258,6 +259,22 @@ struct clk *clk_register_fixed_rate_with_accuracy(struct device *dev, void of_fixed_clk_setup(struct device_node *np); +union clk_reg { + struct { + void __iomem *iomem; + spinlock_t *lock; + }; + struct { + struct regmap *regmap; + unsigned int offset; + }; +}; + +enum clk_reg_type { + CLK_REG_TYPE_IOMEM, + CLK_REG_TYPE_REGMAP, +}; + /** * struct clk_gate - gating clock * @@ -384,13 +401,13 @@ struct clk *clk_register_divider_table(struct device *dev, const char *name, * indicate changing mux bits. */ struct clk_mux { - struct clk_hw hw; - void __iomem *reg; - u32 *table; - u32 mask; - u8 shift; - u8 flags; - spinlock_t *lock; + struct clk_hw hw; + union clk_reg reg; + enum clk_reg_type reg_type; + u32 *table; + u32 mask; + u8 shift; + u8 flags; }; #define CLK_MUX_INDEX_ONE BIT(0) @@ -405,11 +422,19 @@ struct clk *clk_register_mux(struct device *dev, const char *name, const char **parent_names, u8 num_parents, unsigned long flags, void __iomem *reg, u8 shift, u8 width, u8 clk_mux_flags, spinlock_t *lock); +struct clk *clk_register_mux_regmap(struct device *dev, const char *name, + const char **parent_names, u8 num_parents, unsigned long flags, + struct regmap *regmap, unsigned int offset, u8 shift, u8 width, + u8 clk_mux_flags); struct clk *clk_register_mux_table(struct device *dev, const char *name, const char **parent_names, u8 num_parents, unsigned long flags, void __iomem *reg, u8 shift, u32 mask, u8 clk_mux_flags, u32 *table, spinlock_t *lock); +struct clk *clk_register_mux_table_regmap(struct device *dev, const char *name, + const char **parent_names, u8 num_parents, unsigned long flags, + struct regmap *regmap, unsigned int offset, u8 shift, u32 mask, + u8 clk_mux_flags, u32 *table); void of_fixed_factor_clk_setup(struct device_node *node); -- 2.3.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/