diff for duplicates of <51E39114.505@nvidia.com> diff --git a/a/1.txt b/N1/1.txt index ce308e5..aa036b6 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -55,7 +55,7 @@ Anyway, if you want, I can send this patch as a separate one. :) > device, is that correct? If so we will be presenting the same values > twice to libsensors, which would be confusing. > ->> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> +>> Signed-off-by: Wei Ni <wni@nvidia.com> >> --- >> drivers/hwmon/lm90.c | 112 +++++++++++++++++++++++++++++++------------------- >> 1 file changed, 69 insertions(+), 43 deletions(-) @@ -91,15 +91,15 @@ Anyway, if you want, I can send this patch as a separate one. :) >> - return err; >> >> /* +16 degrees offset for temp2 for the LM99 */ ->> - if (data->kind == lm99 && attr->index == 3) ->> + if (data->kind == lm99 && index == 3) +>> - if (data->kind = lm99 && attr->index = 3) +>> + if (data->kind = lm99 && index = 3) >> val -= 16000; >> >> mutex_lock(&data->update_lock); ->> if (data->kind == adt7461) +>> if (data->kind = adt7461) >> - data->temp8[nr] = temp_to_u8_adt7461(data, val); >> + data->temp8[index] = temp_to_u8_adt7461(data, val); ->> else if (data->kind == max6646) +>> else if (data->kind = max6646) >> - data->temp8[nr] = temp_to_u8(val); >> + data->temp8[index] = temp_to_u8(val); >> else @@ -156,10 +156,10 @@ Ok, I will add error handler in my next version. >> struct lm90_data *data = lm90_update_device(dev); >> int temp; >> ->> if (data->kind == adt7461) +>> if (data->kind = adt7461) >> - temp = temp_from_u16_adt7461(data, data->temp11[attr->index]); >> + temp = temp_from_u16_adt7461(data, data->temp11[index]); ->> else if (data->kind == max6646) +>> else if (data->kind = max6646) >> - temp = temp_from_u16(data->temp11[attr->index]); >> + temp = temp_from_u16(data->temp11[index]); >> else @@ -167,8 +167,8 @@ Ok, I will add error handler in my next version. >> + temp = temp_from_s16(data->temp11[index]); >> >> /* +16 degrees offset for temp2 for the LM99 */ ->> - if (data->kind == lm99 && attr->index <= 2) ->> + if (data->kind == lm99 && index <= 2) +>> - if (data->kind = lm99 && attr->index <= 2) +>> + if (data->kind = lm99 && index <= 2) > > There's a doubled space on this line. It isn't added by your patch, it > was already there before, but please fix it while you're here. @@ -210,7 +210,7 @@ Oh, you are so carefully, I will fix it :) >> - return err; >> >> /* +16 degrees offset for temp2 for the LM99 */ ->> if (data->kind == lm99 && index <= 2) +>> if (data->kind = lm99 && index <= 2) >> @@ -839,6 +848,23 @@ static ssize_t set_temp11(struct device *dev, struct device_attribute *devattr, >> lm90_select_remote_channel(client, data, 0); >> @@ -235,4 +235,10 @@ Oh, you are so carefully, I will fix it :) >> return count; >> } >> -> +> + + +_______________________________________________ +lm-sensors mailing list +lm-sensors@lm-sensors.org +http://lists.lm-sensors.org/mailman/listinfo/lm-sensors diff --git a/a/content_digest b/N1/content_digest index 47ba4fe..e750b1f 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,9 +2,9 @@ "ref\01373615287-18502-2-git-send-email-wni@nvidia.com\0" "ref\020130712152615.23464a6b@endymion.delvare\0" "ref\020130712152615.23464a6b-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org\0" - "From\0Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>\0" - "Subject\0Re: [PATCH v3 1/4] hwmon: (lm90) split set&show temp as common codes\0" - "Date\0Mon, 15 Jul 2013 14:05:08 +0800\0" + "From\0Wei Ni <wni@nvidia.com>\0" + "Subject\0Re: [lm-sensors] [PATCH v3 1/4] hwmon: (lm90) split set&show temp as common codes\0" + "Date\0Mon, 15 Jul 2013 06:05:08 +0000\0" "To\0Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>\0" "Cc\0linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>" thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> @@ -70,7 +70,7 @@ "> device, is that correct? If so we will be presenting the same values\n" "> twice to libsensors, which would be confusing.\n" "> \n" - ">> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>\n" + ">> Signed-off-by: Wei Ni <wni@nvidia.com>\n" ">> ---\n" ">> drivers/hwmon/lm90.c | 112 +++++++++++++++++++++++++++++++-------------------\n" ">> 1 file changed, 69 insertions(+), 43 deletions(-)\n" @@ -106,15 +106,15 @@ ">> -\t\treturn err;\n" ">> \n" ">> \t/* +16 degrees offset for temp2 for the LM99 */\n" - ">> -\tif (data->kind == lm99 && attr->index == 3)\n" - ">> +\tif (data->kind == lm99 && index == 3)\n" + ">> -\tif (data->kind = lm99 && attr->index = 3)\n" + ">> +\tif (data->kind = lm99 && index = 3)\n" ">> \t\tval -= 16000;\n" ">> \n" ">> \tmutex_lock(&data->update_lock);\n" - ">> \tif (data->kind == adt7461)\n" + ">> \tif (data->kind = adt7461)\n" ">> -\t\tdata->temp8[nr] = temp_to_u8_adt7461(data, val);\n" ">> +\t\tdata->temp8[index] = temp_to_u8_adt7461(data, val);\n" - ">> \telse if (data->kind == max6646)\n" + ">> \telse if (data->kind = max6646)\n" ">> -\t\tdata->temp8[nr] = temp_to_u8(val);\n" ">> +\t\tdata->temp8[index] = temp_to_u8(val);\n" ">> \telse\n" @@ -171,10 +171,10 @@ ">> \tstruct lm90_data *data = lm90_update_device(dev);\n" ">> \tint temp;\n" ">> \n" - ">> \tif (data->kind == adt7461)\n" + ">> \tif (data->kind = adt7461)\n" ">> -\t\ttemp = temp_from_u16_adt7461(data, data->temp11[attr->index]);\n" ">> +\t\ttemp = temp_from_u16_adt7461(data, data->temp11[index]);\n" - ">> \telse if (data->kind == max6646)\n" + ">> \telse if (data->kind = max6646)\n" ">> -\t\ttemp = temp_from_u16(data->temp11[attr->index]);\n" ">> +\t\ttemp = temp_from_u16(data->temp11[index]);\n" ">> \telse\n" @@ -182,8 +182,8 @@ ">> +\t\ttemp = temp_from_s16(data->temp11[index]);\n" ">> \n" ">> \t/* +16 degrees offset for temp2 for the LM99 */\n" - ">> -\tif (data->kind == lm99 && attr->index <= 2)\n" - ">> +\tif (data->kind == lm99 && index <= 2)\n" + ">> -\tif (data->kind = lm99 && attr->index <= 2)\n" + ">> +\tif (data->kind = lm99 && index <= 2)\n" "> \n" "> There's a doubled space on this line. It isn't added by your patch, it\n" "> was already there before, but please fix it while you're here.\n" @@ -225,7 +225,7 @@ ">> -\t\treturn err;\n" ">> \n" ">> \t/* +16 degrees offset for temp2 for the LM99 */\n" - ">> \tif (data->kind == lm99 && index <= 2)\n" + ">> \tif (data->kind = lm99 && index <= 2)\n" ">> @@ -839,6 +848,23 @@ static ssize_t set_temp11(struct device *dev, struct device_attribute *devattr,\n" ">> \tlm90_select_remote_channel(client, data, 0);\n" ">> \n" @@ -250,6 +250,12 @@ ">> \treturn count;\n" ">> }\n" ">> \n" - > + "> \n" + "\n" + "\n" + "_______________________________________________\n" + "lm-sensors mailing list\n" + "lm-sensors@lm-sensors.org\n" + http://lists.lm-sensors.org/mailman/listinfo/lm-sensors -2a453cdb482fb39fe0437eb56275209fded28986013b77a8c9bd920592316c24 +f5b6af61db944ff3174f5473c086ca49a1ff4cd81971ad1191e3a2a90f8dab76
diff --git a/a/1.txt b/N2/1.txt index ce308e5..bd45f0d 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -55,7 +55,7 @@ Anyway, if you want, I can send this patch as a separate one. :) > device, is that correct? If so we will be presenting the same values > twice to libsensors, which would be confusing. > ->> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> +>> Signed-off-by: Wei Ni <wni@nvidia.com> >> --- >> drivers/hwmon/lm90.c | 112 +++++++++++++++++++++++++++++++------------------- >> 1 file changed, 69 insertions(+), 43 deletions(-) diff --git a/a/content_digest b/N2/content_digest index 47ba4fe..f6c86e7 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,16 +1,15 @@ "ref\01373615287-18502-1-git-send-email-wni@nvidia.com\0" "ref\01373615287-18502-2-git-send-email-wni@nvidia.com\0" "ref\020130712152615.23464a6b@endymion.delvare\0" - "ref\020130712152615.23464a6b-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org\0" - "From\0Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>\0" + "From\0Wei Ni <wni@nvidia.com>\0" "Subject\0Re: [PATCH v3 1/4] hwmon: (lm90) split set&show temp as common codes\0" "Date\0Mon, 15 Jul 2013 14:05:08 +0800\0" - "To\0Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>\0" - "Cc\0linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>" - thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> - lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org <lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org> - linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> - " linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>\0" + "To\0Jean Delvare <khali@linux-fr.org>\0" + "Cc\0linux@roeck-us.net <linux@roeck-us.net>" + thierry.reding@gmail.com <thierry.reding@gmail.com> + lm-sensors@lm-sensors.org <lm-sensors@lm-sensors.org> + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + " linux-tegra@vger.kernel.org <linux-tegra@vger.kernel.org>\0" "\00:1\0" "b\0" "On 07/12/2013 09:26 PM, Jean Delvare wrote:\n" @@ -70,7 +69,7 @@ "> device, is that correct? If so we will be presenting the same values\n" "> twice to libsensors, which would be confusing.\n" "> \n" - ">> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>\n" + ">> Signed-off-by: Wei Ni <wni@nvidia.com>\n" ">> ---\n" ">> drivers/hwmon/lm90.c | 112 +++++++++++++++++++++++++++++++-------------------\n" ">> 1 file changed, 69 insertions(+), 43 deletions(-)\n" @@ -252,4 +251,4 @@ ">> \n" > -2a453cdb482fb39fe0437eb56275209fded28986013b77a8c9bd920592316c24 +c1e035c51fefec9478b4fd55d21ca41fa2df0022312eccc3ce91ce00ad5c4e93
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.