diff for duplicates of <56FC3BEB.8030106@elopez.com.ar> diff --git a/a/1.txt b/N1/1.txt index 45f9571..01888dd 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,23 +1,20 @@ [Sorry for the delay, I meant to reply to this post a while back but I forgot] -El 21/03/16 a las 05:25, Jean-Francois Moine escribi=F3: +El 21/03/16 a las 05:25, Jean-Francois Moine escribi?: > On Mon, 21 Mar 2016 08:25:46 +0100 > Maxime Ripard <maxime.ripard@free-electrons.com> wrote: -> = - ->>> - /* find the parent that can help provide the fastest rate <=3D rate */ +> +>>> - /* find the parent that can help provide the fastest rate <= rate */ >>> + /* find the parent that can help provide the fastest rate */ ->>> num_parents =3D clk_hw_get_num_parents(hw); ->>> for (i =3D 0; i < num_parents; i++) { ->>> parent =3D clk_hw_get_parent_by_index(hw, i); ->>> @@ -100,7 +100,7 @@ static int clk_factors_determine_rate(struct clk_hw= - *hw, ->>> child_rate =3D clk_factors_round_rate(hw, req->rate, +>>> num_parents = clk_hw_get_num_parents(hw); +>>> for (i = 0; i < num_parents; i++) { +>>> parent = clk_hw_get_parent_by_index(hw, i); +>>> @@ -100,7 +100,7 @@ static int clk_factors_determine_rate(struct clk_hw *hw, +>>> child_rate = clk_factors_round_rate(hw, req->rate, >>> &parent_rate); ->>> = - ->>> - if (child_rate <=3D req->rate && child_rate > best_child_rate) { +>>> +>>> - if (child_rate <= req->rate && child_rate > best_child_rate) { >>> + if (child_rate > best_child_rate) { >> >> I'm not sure this would work, since you'll end up picking the fastest @@ -29,16 +26,14 @@ El 21/03/16 a las 05:25, Jean-Francois Moine escribi=F3: >> That being said, we had a similar discussion for SPI around a month >> ago where we wanted a rate strictly lower than the requested one. I >> guess it's time to add a flag to tell how you want to round. -> = - +> > You are right, I just removed half of the constraint, but I still wonder > why does this sequence introduced by the commit 862b728387aef3a37 > (clk: sunxi: factors: automatic reparenting support) do -> "provide the fastest rate <=3D rate" +> "provide the fastest rate <= rate" > instead of > "provide the closest rate" ? -> = - +> > Emilio? Overclocking components is usually not a good default in my opinion. I @@ -47,8 +42,3 @@ from playing it safe. Cheers, Emilio - -_______________________________________________ -linux-arm-kernel mailing list -linux-arm-kernel@lists.infradead.org -http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N1/content_digest index 108ea09..2024c34 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,37 +1,29 @@ "ref\020160310081658.B749246B@mail.free-electrons.com\0" "ref\020160321072546.GT30977@lukather\0" "ref\020160321092549.4e6245e4f02839e29aeb86a9@free.fr\0" - "From\0Emilio L\303\263pez <emilio@elopez.com.ar>\0" - "Subject\0Re: [PATCH] clk: sunxi: Accept a greater rate when setting a parent clock\0" + "From\0emilio@elopez.com.ar (Emilio L\303\263pez)\0" + "Subject\0[PATCH] clk: sunxi: Accept a greater rate when setting a parent clock\0" "Date\0Wed, 30 Mar 2016 17:49:47 -0300\0" - "To\0Jean-Francois Moine <moinejf@free.fr>" - " Maxime Ripard <maxime.ripard@free-electrons.com>\0" - "Cc\0Chen-Yu Tsai <wens@csie.org>" - Stephen Boyd <sboyd@codeaurora.org> - linux-clk@vger.kernel.org - " linux-arm-kernel@lists.infradead.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "[Sorry for the delay, I meant to reply to this post a while back but I\n" "forgot]\n" "\n" - "El 21/03/16 a las 05:25, Jean-Francois Moine escribi=F3:\n" + "El 21/03/16 a las 05:25, Jean-Francois Moine escribi?:\n" "> On Mon, 21 Mar 2016 08:25:46 +0100\n" "> Maxime Ripard <maxime.ripard@free-electrons.com> wrote:\n" - "> =\n" - "\n" - ">>> -\t/* find the parent that can help provide the fastest rate <=3D rate */\n" + "> \n" + ">>> -\t/* find the parent that can help provide the fastest rate <= rate */\n" ">>> +\t/* find the parent that can help provide the fastest rate */\n" - ">>> \tnum_parents =3D clk_hw_get_num_parents(hw);\n" - ">>> \tfor (i =3D 0; i < num_parents; i++) {\n" - ">>> \t\tparent =3D clk_hw_get_parent_by_index(hw, i);\n" - ">>> @@ -100,7 +100,7 @@ static int clk_factors_determine_rate(struct clk_hw=\n" - " *hw,\n" - ">>> \t\tchild_rate =3D clk_factors_round_rate(hw, req->rate,\n" + ">>> \tnum_parents = clk_hw_get_num_parents(hw);\n" + ">>> \tfor (i = 0; i < num_parents; i++) {\n" + ">>> \t\tparent = clk_hw_get_parent_by_index(hw, i);\n" + ">>> @@ -100,7 +100,7 @@ static int clk_factors_determine_rate(struct clk_hw *hw,\n" + ">>> \t\tchild_rate = clk_factors_round_rate(hw, req->rate,\n" ">>> \t\t\t\t\t\t &parent_rate);\n" - ">>> =\n" - "\n" - ">>> -\t\tif (child_rate <=3D req->rate && child_rate > best_child_rate) {\n" + ">>> \n" + ">>> -\t\tif (child_rate <= req->rate && child_rate > best_child_rate) {\n" ">>> +\t\tif (child_rate > best_child_rate) {\n" ">>\n" ">> I'm not sure this would work, since you'll end up picking the fastest\n" @@ -43,16 +35,14 @@ ">> That being said, we had a similar discussion for SPI around a month\n" ">> ago where we wanted a rate strictly lower than the requested one. I\n" ">> guess it's time to add a flag to tell how you want to round.\n" - "> =\n" - "\n" + "> \n" "> You are right, I just removed half of the constraint, but I still wonder\n" "> why does this sequence introduced by the commit 862b728387aef3a37\n" "> (clk: sunxi: factors: automatic reparenting support) do\n" - "> \t\"provide the fastest rate <=3D rate\"\n" + "> \t\"provide the fastest rate <= rate\"\n" "> instead of\n" "> \t\"provide the closest rate\" ?\n" - "> =\n" - "\n" + "> \n" "> Emilio?\n" "\n" "Overclocking components is usually not a good default in my opinion. I\n" @@ -60,11 +50,6 @@ "from playing it safe.\n" "\n" "Cheers,\n" - "Emilio\n" - "\n" - "_______________________________________________\n" - "linux-arm-kernel mailing list\n" - "linux-arm-kernel@lists.infradead.org\n" - http://lists.infradead.org/mailman/listinfo/linux-arm-kernel + Emilio -deab558437728372422b3060c455e3d1e2c6a454e4004ddaa176f06b6a449f43 +1750024a0b795524fc8c12ed53b8983f12a00608815c1a4a09c1c298a45f6f59
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.