diff for duplicates of <2653032.av3sFfN860@phil> diff --git a/a/1.txt b/N1/1.txt index 86cec17..d02b4e6 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,36 +1,36 @@ Hi, Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng: -> On 2015=E5=B9=B409=E6=9C=8824=E6=97=A5 11:04, Xing Zheng wrote: +> On 2015年09月24日 11:04, Xing Zheng wrote: > >>> #define RK3066_PLL_RATE(_rate, _nr, _nf, _no) \ -> >>>=20 +> >>> > >>> @@ -95,12 +106,31 @@ enum rockchip_pll_type { -> >>>=20 -> >>> .nb =3D _nb, \ -> >>> =20 +> >>> +> >>> .nb = _nb, \ +> >>> > >>> } -> >>>=20 +> >>> > >>> +#define RK3036_PLL_RATE(_rate, _refdiv, _fbdiv, _postdiv1, \ > >>> + _postdiv2, _dsmpd, _frac) \ > >>> +{ \ -> >>> + .rate =3D _rate##U, \ -> >>> + .fbdiv =3D _fbdiv, \ -> >>> + .postdiv1 =3D _postdiv1, \ -> >>> + .refdiv =3D _refdiv, \ -> >>> + .postdiv2 =3D _postdiv2, \ -> >>> + .dsmpd =3D _dsmpd, \ -> >>> + .frac =3D _frac, \ +> >>> + .rate = _rate##U, \ +> >>> + .fbdiv = _fbdiv, \ +> >>> + .postdiv1 = _postdiv1, \ +> >>> + .refdiv = _refdiv, \ +> >>> + .postdiv2 = _postdiv2, \ +> >>> + .dsmpd = _dsmpd, \ +> >>> + .frac = _frac, \ > >>> +} > >>> + -> >>>=20 +> >>> > >>> struct rockchip_pll_rate_table { -> >>> =20 +> >>> > >>> unsigned long rate; > >>> unsigned int nr; > >>> unsigned int nf; > >>> unsigned int no; > >>> unsigned int nb; -> >>>=20 +> >>> > >>> + /* for RK3036 */ > >>> + unsigned int fbdiv; > >>> + unsigned int postdiv1; @@ -38,16 +38,15 @@ Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng: > >>> + unsigned int postdiv2; > >>> + unsigned int dsmpd; > >>> + unsigned int frac; -> >>=20 +> >> > >> same for these 2 ... should be part of the pll addition itself -> >>=20 +> >> > > }; -> >=20 +> > > > Done. ->=20 +> > Sorry, I have one question: -> The "struct rockchip_pll_rate_table" is called in "rockchip/clk-pll.c= -" +> The "struct rockchip_pll_rate_table" is called in "rockchip/clk-pll.c" > on many functions, I think I could add a struct like: > struct rk3036_pll_rate_table { > unsigned int fbdiv; @@ -57,18 +56,14 @@ Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng: > unsigned int dsmpd; > unsigned int frac; > }; -> but, it will add many redundancy codes in "rockchip/clk-pll.c" just f= -or +> but, it will add many redundancy codes in "rockchip/clk-pll.c" just for > call "struct rk3036_pll_rate_table". -One possible solution may be to cast to void* in the general functions,= - so=20 -have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have=20 -rockchip_clk_register_pll and friends handle it as void* and then only = -have=20 -the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use the= -m as=20 +One possible solution may be to cast to void* in the general functions, so +have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have +rockchip_clk_register_pll and friends handle it as void* and then only have +the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use them as their actual type, as they know which they need. diff --git a/a/content_digest b/N1/content_digest index 7817910..9a1fbec 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -16,36 +16,36 @@ "Hi,\n" "\n" "Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng:\n" - "> On 2015=E5=B9=B409=E6=9C=8824=E6=97=A5 11:04, Xing Zheng wrote:\n" + "> On 2015\345\271\26409\346\234\21024\346\227\245 11:04, Xing Zheng wrote:\n" "> >>> #define RK3066_PLL_RATE(_rate, _nr, _nf, _no) \\\n" - "> >>>=20\n" + "> >>> \n" "> >>> @@ -95,12 +106,31 @@ enum rockchip_pll_type {\n" - "> >>>=20\n" - "> >>> .nb =3D _nb, \\\n" - "> >>> =20\n" + "> >>> \n" + "> >>> .nb = _nb, \\\n" + "> >>> \n" "> >>> }\n" - "> >>>=20\n" + "> >>> \n" "> >>> +#define RK3036_PLL_RATE(_rate, _refdiv, _fbdiv, _postdiv1, \\\n" "> >>> + _postdiv2, _dsmpd, _frac) \\\n" "> >>> +{ \\\n" - "> >>> + .rate =3D _rate##U, \\\n" - "> >>> + .fbdiv =3D _fbdiv, \\\n" - "> >>> + .postdiv1 =3D _postdiv1, \\\n" - "> >>> + .refdiv =3D _refdiv, \\\n" - "> >>> + .postdiv2 =3D _postdiv2, \\\n" - "> >>> + .dsmpd =3D _dsmpd, \\\n" - "> >>> + .frac =3D _frac, \\\n" + "> >>> + .rate = _rate##U, \\\n" + "> >>> + .fbdiv = _fbdiv, \\\n" + "> >>> + .postdiv1 = _postdiv1, \\\n" + "> >>> + .refdiv = _refdiv, \\\n" + "> >>> + .postdiv2 = _postdiv2, \\\n" + "> >>> + .dsmpd = _dsmpd, \\\n" + "> >>> + .frac = _frac, \\\n" "> >>> +}\n" "> >>> +\n" - "> >>>=20\n" + "> >>> \n" "> >>> struct rockchip_pll_rate_table {\n" - "> >>> =20\n" + "> >>> \n" "> >>> unsigned long rate;\n" "> >>> unsigned int nr;\n" "> >>> unsigned int nf;\n" "> >>> unsigned int no;\n" "> >>> unsigned int nb;\n" - "> >>>=20\n" + "> >>> \n" "> >>> + /* for RK3036 */\n" "> >>> + unsigned int fbdiv;\n" "> >>> + unsigned int postdiv1;\n" @@ -53,16 +53,15 @@ "> >>> + unsigned int postdiv2;\n" "> >>> + unsigned int dsmpd;\n" "> >>> + unsigned int frac;\n" - "> >>=20\n" + "> >> \n" "> >> same for these 2 ... should be part of the pll addition itself\n" - "> >>=20\n" + "> >> \n" "> > };\n" - "> >=20\n" + "> > \n" "> > Done.\n" - ">=20\n" + "> \n" "> Sorry, I have one question:\n" - "> The \"struct rockchip_pll_rate_table\" is called in \"rockchip/clk-pll.c=\n" - "\"\n" + "> The \"struct rockchip_pll_rate_table\" is called in \"rockchip/clk-pll.c\"\n" "> on many functions, I think I could add a struct like:\n" "> struct rk3036_pll_rate_table {\n" "> unsigned int fbdiv;\n" @@ -72,21 +71,17 @@ "> unsigned int dsmpd;\n" "> unsigned int frac;\n" "> };\n" - "> but, it will add many redundancy codes in \"rockchip/clk-pll.c\" just f=\n" - "or\n" + "> but, it will add many redundancy codes in \"rockchip/clk-pll.c\" just for\n" "> call \"struct rk3036_pll_rate_table\".\n" "\n" "\n" - "One possible solution may be to cast to void* in the general functions,=\n" - " so=20\n" - "have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have=20\n" - "rockchip_clk_register_pll and friends handle it as void* and then only =\n" - "have=20\n" - "the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use the=\n" - "m as=20\n" + "One possible solution may be to cast to void* in the general functions, so \n" + "have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have \n" + "rockchip_clk_register_pll and friends handle it as void* and then only have \n" + "the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use them as \n" "their actual type, as they know which they need.\n" "\n" "\n" Heiko -691e722c3bd5d1999d68289641a6b09500831d4350cd2742736d087bcaee1be7 +d55bf0f40fd06c7388bb395300451d04bbd70abb80456e4deae127f614cefeaf
diff --git a/a/1.txt b/N2/1.txt index 86cec17..20de302 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,36 +1,36 @@ Hi, Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng: -> On 2015=E5=B9=B409=E6=9C=8824=E6=97=A5 11:04, Xing Zheng wrote: +> On 2015?09?24? 11:04, Xing Zheng wrote: > >>> #define RK3066_PLL_RATE(_rate, _nr, _nf, _no) \ -> >>>=20 +> >>> > >>> @@ -95,12 +106,31 @@ enum rockchip_pll_type { -> >>>=20 -> >>> .nb =3D _nb, \ -> >>> =20 +> >>> +> >>> .nb = _nb, \ +> >>> > >>> } -> >>>=20 +> >>> > >>> +#define RK3036_PLL_RATE(_rate, _refdiv, _fbdiv, _postdiv1, \ > >>> + _postdiv2, _dsmpd, _frac) \ > >>> +{ \ -> >>> + .rate =3D _rate##U, \ -> >>> + .fbdiv =3D _fbdiv, \ -> >>> + .postdiv1 =3D _postdiv1, \ -> >>> + .refdiv =3D _refdiv, \ -> >>> + .postdiv2 =3D _postdiv2, \ -> >>> + .dsmpd =3D _dsmpd, \ -> >>> + .frac =3D _frac, \ +> >>> + .rate = _rate##U, \ +> >>> + .fbdiv = _fbdiv, \ +> >>> + .postdiv1 = _postdiv1, \ +> >>> + .refdiv = _refdiv, \ +> >>> + .postdiv2 = _postdiv2, \ +> >>> + .dsmpd = _dsmpd, \ +> >>> + .frac = _frac, \ > >>> +} > >>> + -> >>>=20 +> >>> > >>> struct rockchip_pll_rate_table { -> >>> =20 +> >>> > >>> unsigned long rate; > >>> unsigned int nr; > >>> unsigned int nf; > >>> unsigned int no; > >>> unsigned int nb; -> >>>=20 +> >>> > >>> + /* for RK3036 */ > >>> + unsigned int fbdiv; > >>> + unsigned int postdiv1; @@ -38,16 +38,15 @@ Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng: > >>> + unsigned int postdiv2; > >>> + unsigned int dsmpd; > >>> + unsigned int frac; -> >>=20 +> >> > >> same for these 2 ... should be part of the pll addition itself -> >>=20 +> >> > > }; -> >=20 +> > > > Done. ->=20 +> > Sorry, I have one question: -> The "struct rockchip_pll_rate_table" is called in "rockchip/clk-pll.c= -" +> The "struct rockchip_pll_rate_table" is called in "rockchip/clk-pll.c" > on many functions, I think I could add a struct like: > struct rk3036_pll_rate_table { > unsigned int fbdiv; @@ -57,18 +56,14 @@ Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng: > unsigned int dsmpd; > unsigned int frac; > }; -> but, it will add many redundancy codes in "rockchip/clk-pll.c" just f= -or +> but, it will add many redundancy codes in "rockchip/clk-pll.c" just for > call "struct rk3036_pll_rate_table". -One possible solution may be to cast to void* in the general functions,= - so=20 -have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have=20 -rockchip_clk_register_pll and friends handle it as void* and then only = -have=20 -the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use the= -m as=20 +One possible solution may be to cast to void* in the general functions, so +have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have +rockchip_clk_register_pll and friends handle it as void* and then only have +the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use them as their actual type, as they know which they need. diff --git a/a/content_digest b/N2/content_digest index 7817910..610cdf9 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,51 +1,45 @@ "ref\01442478540-15068-1-git-send-email-zhengxing@rock-chips.com\0" "ref\05603683F.4020302@rock-chips.com\0" "ref\056036EAE.9040201@rock-chips.com\0" - "From\0Heiko Stuebner <heiko@sntech.de>\0" - "Subject\0Re: [PATCH v2 3/9] clk: rockchip: add clock controller for rk3036\0" + "From\0heiko@sntech.de (Heiko Stuebner)\0" + "Subject\0[PATCH v2 3/9] clk: rockchip: add clock controller for rk3036\0" "Date\0Wed, 07 Oct 2015 12:24:32 +0200\0" - "To\0Xing Zheng <zhengxing@rock-chips.com>\0" - "Cc\0linux-rockchip@lists.infradead.org" - Michael Turquette <mturquette@baylibre.com> - Stephen Boyd <sboyd@codeaurora.org> - linux-kernel@vger.kernel.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" "Hi,\n" "\n" "Am Donnerstag, 24. September 2015, 11:31:58 schrieb Xing Zheng:\n" - "> On 2015=E5=B9=B409=E6=9C=8824=E6=97=A5 11:04, Xing Zheng wrote:\n" + "> On 2015?09?24? 11:04, Xing Zheng wrote:\n" "> >>> #define RK3066_PLL_RATE(_rate, _nr, _nf, _no) \\\n" - "> >>>=20\n" + "> >>> \n" "> >>> @@ -95,12 +106,31 @@ enum rockchip_pll_type {\n" - "> >>>=20\n" - "> >>> .nb =3D _nb, \\\n" - "> >>> =20\n" + "> >>> \n" + "> >>> .nb = _nb, \\\n" + "> >>> \n" "> >>> }\n" - "> >>>=20\n" + "> >>> \n" "> >>> +#define RK3036_PLL_RATE(_rate, _refdiv, _fbdiv, _postdiv1, \\\n" "> >>> + _postdiv2, _dsmpd, _frac) \\\n" "> >>> +{ \\\n" - "> >>> + .rate =3D _rate##U, \\\n" - "> >>> + .fbdiv =3D _fbdiv, \\\n" - "> >>> + .postdiv1 =3D _postdiv1, \\\n" - "> >>> + .refdiv =3D _refdiv, \\\n" - "> >>> + .postdiv2 =3D _postdiv2, \\\n" - "> >>> + .dsmpd =3D _dsmpd, \\\n" - "> >>> + .frac =3D _frac, \\\n" + "> >>> + .rate = _rate##U, \\\n" + "> >>> + .fbdiv = _fbdiv, \\\n" + "> >>> + .postdiv1 = _postdiv1, \\\n" + "> >>> + .refdiv = _refdiv, \\\n" + "> >>> + .postdiv2 = _postdiv2, \\\n" + "> >>> + .dsmpd = _dsmpd, \\\n" + "> >>> + .frac = _frac, \\\n" "> >>> +}\n" "> >>> +\n" - "> >>>=20\n" + "> >>> \n" "> >>> struct rockchip_pll_rate_table {\n" - "> >>> =20\n" + "> >>> \n" "> >>> unsigned long rate;\n" "> >>> unsigned int nr;\n" "> >>> unsigned int nf;\n" "> >>> unsigned int no;\n" "> >>> unsigned int nb;\n" - "> >>>=20\n" + "> >>> \n" "> >>> + /* for RK3036 */\n" "> >>> + unsigned int fbdiv;\n" "> >>> + unsigned int postdiv1;\n" @@ -53,16 +47,15 @@ "> >>> + unsigned int postdiv2;\n" "> >>> + unsigned int dsmpd;\n" "> >>> + unsigned int frac;\n" - "> >>=20\n" + "> >> \n" "> >> same for these 2 ... should be part of the pll addition itself\n" - "> >>=20\n" + "> >> \n" "> > };\n" - "> >=20\n" + "> > \n" "> > Done.\n" - ">=20\n" + "> \n" "> Sorry, I have one question:\n" - "> The \"struct rockchip_pll_rate_table\" is called in \"rockchip/clk-pll.c=\n" - "\"\n" + "> The \"struct rockchip_pll_rate_table\" is called in \"rockchip/clk-pll.c\"\n" "> on many functions, I think I could add a struct like:\n" "> struct rk3036_pll_rate_table {\n" "> unsigned int fbdiv;\n" @@ -72,21 +65,17 @@ "> unsigned int dsmpd;\n" "> unsigned int frac;\n" "> };\n" - "> but, it will add many redundancy codes in \"rockchip/clk-pll.c\" just f=\n" - "or\n" + "> but, it will add many redundancy codes in \"rockchip/clk-pll.c\" just for\n" "> call \"struct rk3036_pll_rate_table\".\n" "\n" "\n" - "One possible solution may be to cast to void* in the general functions,=\n" - " so=20\n" - "have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have=20\n" - "rockchip_clk_register_pll and friends handle it as void* and then only =\n" - "have=20\n" - "the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use the=\n" - "m as=20\n" + "One possible solution may be to cast to void* in the general functions, so \n" + "have sturct rk3066_pll_rate_table, rk3036_pll_rate_table, have \n" + "rockchip_clk_register_pll and friends handle it as void* and then only have \n" + "the rockchip_rk3066_pll_* functions as well as the clk-rkxxxx.c use them as \n" "their actual type, as they know which they need.\n" "\n" "\n" Heiko -691e722c3bd5d1999d68289641a6b09500831d4350cd2742736d087bcaee1be7 +b8d04dd0378efbd1531a4f1743d8b860a183390e605a3bcfd12dd3e9541a9f32
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.