diff for duplicates of <200804110218.23912.arnd@arndb.de> diff --git a/a/1.txt b/N1/1.txt index b39c102..6aaf501 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -4,7 +4,7 @@ On Thursday 10 April 2008, Jerone Young wrote: > needed ... though none off the top of my head at the moment)...but it > does allow you to create another wait state for whatever reason a lot > easier. -> +>=20 > So I really don't think this needs to change. Unless everyone really > feels that it just has to be. @@ -14,32 +14,23 @@ not to have code in anticipation of something getting more complicated in the future, you can always add the complexity at the point where you need it. -> > +> >=20 > > static int __init idle_param(char *p) > > { -> > if (!strcmp(modes[i].name, "spin")) -> > ppc_md.power_save = NULL; +> > =A0=A0=A0=A0=A0=A0if (!strcmp(modes[i].name, "spin")) +> > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0ppc_md.power_save =3D NULL; > > } > > early_param("idle", idle_param); -> > -> > if you statically initialize the ppc_md.power_save function to ppc44x_idle +> >=20 +> > if you statically initialize the ppc_md.power_save function to ppc44x_i= +dle > > in the platform setup files? -> +>=20 > The idea is to not statically initialize ppc_md.power_save to > ppc44x_idle in each platform setup file. -> +>=20 Why not? Unlike the platform_initcall, it wouldn't cost anything and your current code has the same effect in the end, but in a less obvious way. Arnd <>< - -------------------------------------------------------------------------- -This SF.net email is sponsored by the 2008 JavaOne(SM) Conference -Don't miss this year's exciting event. There's still time to save $100. -Use priority code J8TL2D2. -http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone -_______________________________________________ -kvm-ppc-devel mailing list -kvm-ppc-devel@lists.sourceforge.net -https://lists.sourceforge.net/lists/listinfo/kvm-ppc-devel diff --git a/a/content_digest b/N1/content_digest index b2b7a82..979cf2b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,8 +2,8 @@ "ref\0200804101544.44457.arnd@arndb.de\0" "ref\01207858131.26428.4.camel@thinkpadL\0" "From\0Arnd Bergmann <arnd@arndb.de>\0" - "Subject\0Re: [kvm-ppc-devel] [PATCH] [v5] Add idle wait support for 44x\0" - "Date\0Fri, 11 Apr 2008 00:18:22 +0000\0" + "Subject\0Re: [PATCH] [v5] Add idle wait support for 44x platforms\0" + "Date\0Fri, 11 Apr 2008 02:18:22 +0200\0" "To\0jyoung5@us.ibm.com\0" "Cc\0kvm-ppc-devel@lists.sourceforge.net" " linuxppc-dev@ozlabs.org\0" @@ -15,7 +15,7 @@ "> needed ... though none off the top of my head at the moment)...but it\n" "> does allow you to create another wait state for whatever reason a lot\n" "> easier.\n" - "> \n" + ">=20\n" "> So I really don't think this needs to change. Unless everyone really\n" "> feels that it just has to be.\n" "\n" @@ -25,34 +25,25 @@ "in the future, you can always add the complexity at the point where you\n" "need it.\n" "\n" - "> > \n" + "> >=20\n" "> > static int __init idle_param(char *p)\n" "> > {\n" - "> > \302\240\302\240\302\240\302\240\302\240\302\240if (!strcmp(modes[i].name, \"spin\"))\n" - "> > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240ppc_md.power_save = NULL;\n" + "> > =A0=A0=A0=A0=A0=A0if (!strcmp(modes[i].name, \"spin\"))\n" + "> > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0ppc_md.power_save =3D NULL;\n" "> > }\n" "> > early_param(\"idle\", idle_param);\n" - "> > \n" - "> > if you statically initialize the ppc_md.power_save function to ppc44x_idle\n" + "> >=20\n" + "> > if you statically initialize the ppc_md.power_save function to ppc44x_i=\n" + "dle\n" "> > in the platform setup files?\n" - "> \n" + ">=20\n" "> The idea is to not statically initialize ppc_md.power_save to\n" "> ppc44x_idle in each platform setup file.\n" - "> \n" + ">=20\n" "\n" "Why not? Unlike the platform_initcall, it wouldn't cost anything and your\n" "current code has the same effect in the end, but in a less obvious way.\n" "\n" - "\tArnd <><\n" - "\n" - "-------------------------------------------------------------------------\n" - "This SF.net email is sponsored by the 2008 JavaOne(SM) Conference \n" - "Don't miss this year's exciting event. There's still time to save $100. \n" - "Use priority code J8TL2D2. \n" - "http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone\n" - "_______________________________________________\n" - "kvm-ppc-devel mailing list\n" - "kvm-ppc-devel@lists.sourceforge.net\n" - https://lists.sourceforge.net/lists/listinfo/kvm-ppc-devel + "\tArnd <><" -2701b073b20b400edb2d31eafbe1022b90dc2d4de5644275365f7bdd99a8a76a +395328df9a2e24b63d9f521926460beff40314b0aa3b0998e64856d5af2eb276
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.