[PATCH linux dev-4.7] clk: add a ahb clock for aspeed g4

Joel Stanley joel at jms.id.au
Thu Nov 10 11:22:11 AEDT 2016


On Sat, Nov 5, 2016 at 1:05 AM, Cédric Le Goater <clg at kaod.org> wrote:
> and fix g5 ahb clock which has a 3bits ratio.
>

Thanks. I have rewritten the clock drivers as upstream rejected them
in their current form. As I have not submitted them for inclusion in
the dev-4.7 tree yet I will merge this patch.


> Signed-off-by: Cédric Le Goater <clg at kaod.org>

Reviewed-by: Joel Stanley <joel at jms.id.au>

> ---
>  arch/arm/boot/dts/aspeed-g4.dtsi |    7 +++++
>  drivers/clk/aspeed/clk-g4.c      |   48 +++++++++++++++++++++++++++++++++++++++
>  drivers/clk/aspeed/clk-g5.c      |    4 ++-
>  3 files changed, 58 insertions(+), 1 deletion(-)
>
> Index: linux-openbmc-4.7.git/arch/arm/boot/dts/aspeed-g4.dtsi
> ===================================================================
> --- linux-openbmc-4.7.git.orig/arch/arm/boot/dts/aspeed-g4.dtsi
> +++ linux-openbmc-4.7.git/arch/arm/boot/dts/aspeed-g4.dtsi
> @@ -820,6 +820,13 @@
>                                 };
>                         };
>
> +                       clk_ahb: clk_ahb at 1e6e2070 {
> +                               #clock-cells = <0>;
> +                               compatible = "aspeed,g4-ahb-clock";
> +                               reg = <0x1e6e2070 0x4>;
> +                               clocks = <&clk_hpll>;
> +                       };
> +
>                         clk_apb: clk_apb at 1e6e2008 {
>                                 #clock-cells = <0>;
>                                 compatible = "aspeed,g4-apb-clock";
> Index: linux-openbmc-4.7.git/drivers/clk/aspeed/clk-g4.c
> ===================================================================
> --- linux-openbmc-4.7.git.orig/drivers/clk/aspeed/clk-g4.c
> +++ linux-openbmc-4.7.git/drivers/clk/aspeed/clk-g4.c
> @@ -68,6 +68,54 @@ static void __init aspeed_of_hpll_clk_in
>  CLK_OF_DECLARE(aspeed_hpll_clock, "aspeed,g4-hpll-clock",
>                aspeed_of_hpll_clk_init);
>
> +static void __init aspeed_of_ahb_clk_init(struct device_node *node)
> +{
> +       struct clk *clk, *hpll_clk;
> +       void __iomem *base;
> +       int reg, rate, hpll;
> +       const char *name = node->name;
> +       const char *parent_name;
> +
> +       of_property_read_string(node, "clock-output-names", &name);
> +       parent_name = of_clk_get_parent_name(node, 0);
> +
> +       /* Strap register SCU70 */
> +       base = of_iomap(node, 0);
> +       if (!base) {
> +               pr_err("%s: of_iomap failed\n", node->full_name);
> +               return;
> +       }
> +
> +       /* bits 11:10 define the CPU/AHB clock frequency ratio */
> +       reg = (readl(base) >> 10) & 0x03;
> +       iounmap(base);
> +
> +       /* A value of zero is undefined */
> +       WARN_ON(reg == 0);
> +
> +       hpll_clk = of_clk_get(node, 0);
> +       if (IS_ERR(hpll_clk)) {
> +               pr_err("%s: of_clk_get failed\n", node->full_name);
> +               return;
> +       }
> +
> +       hpll = clk_get_rate(hpll_clk);
> +
> +       rate = hpll / (reg + 1);
> +
> +       clk = clk_register_fixed_rate(NULL, name, parent_name, 0, rate);
> +       if (IS_ERR(clk)) {
> +               pr_err("%s: failed to register clock\n", node->full_name);
> +               return;
> +       }
> +
> +       clk_register_clkdev(clk, NULL, name);
> +       of_clk_add_provider(node, of_clk_src_simple_get, clk);
> +}
> +CLK_OF_DECLARE(aspeed_ahb_clock, "aspeed,g4-ahb-clock",
> +              aspeed_of_ahb_clk_init);
> +
> +
>  static void __init aspeed_of_apb_clk_init(struct device_node *node)
>  {
>         struct clk *clk, *hpll_clk;
> Index: linux-openbmc-4.7.git/drivers/clk/aspeed/clk-g5.c
> ===================================================================
> --- linux-openbmc-4.7.git.orig/drivers/clk/aspeed/clk-g5.c
> +++ linux-openbmc-4.7.git/drivers/clk/aspeed/clk-g5.c
> @@ -119,7 +119,9 @@ static void __init aspeed_of_ahb_clk_ini
>                 pr_err("%s: of_iomap failed\n", node->full_name);
>                 return;
>         }
> -       reg = (readl(base) >> 9) & 0x03;
> +
> +       /* bits 11:9 define the AXI/AHB clock frequency ratio */
> +       reg = (readl(base) >> 9) & 0x07;
>         iounmap(base);
>
>         /* A value of zero is undefined */


More information about the openbmc mailing list