From: Guenter Roeck <guenter.roeck@ericsson.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] [PATCH:hwmon:v4:]Merging_pkgtemp_with_coretemp
Date: Mon, 07 Mar 2011 14:54:29 +0000 [thread overview]
Message-ID: <20110307145429.GA13343@ericsson.com> (raw)
In-Reply-To: <1299101865-10797-1-git-send-email-durgadoss.r@intel.com>
On Mon, Mar 07, 2011 at 03:14:20AM -0500, R, Durgadoss wrote:
> Hi Guenter,
> Ping..Any comments on this version of the patch ?
>
> > -----Original Message-----
> > From: R, Durgadoss
> > Sent: Thursday, March 03, 2011 3:08 AM
> > To: guenter.roeck@ericsson.com; khali@linux-fr.org; Yu, Fenghua
> > Cc: lm-sensors@lm-sensors.org; R, Durgadoss
> > Subject: [PATCH:hwmon:v4:]Merging_pkgtemp_with_coretemp
> >
> > This patch merges the pkgtemp driver with the coretemp driver.
> > This merged driver creates one hwmon device per physical CPU.
> >
> > Changes from v3 of this patch:
> > Added appropriate support for CONFIG_HOTPLUG_CPU.
> >
Problem is that you don't provide a complete changelog ... meaning I have no idea
if you addressed the crash problem I reported earlier, much less how you addressed
it if you did. So I'll have to load the latest kernel on a machine and test your
patch with it. Then I'll have to compare this version of the patch with the previous
version to see if I find out what you may have changed to fix the crash condition.
And I'll have to do all that before I can actually start reviewing the patch itself.
In other words, you are loading a lot of work on me. I don't have the time for all this
right now, so you'll have to be patient.
Guenter
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2011-03-07 14:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-02 21:49 [lm-sensors] [PATCH:hwmon:v4:]Merging_pkgtemp_with_coretemp Durgadoss R
2011-03-07 14:54 ` Guenter Roeck [this message]
2011-03-08 6:23 ` R, Durgadoss
2011-04-04 18:16 ` Yu, Fenghua
2011-04-04 19:25 ` Guenter Roeck
2011-04-05 5:30 ` R, Durgadoss
2011-04-05 14:43 ` R, Durgadoss
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110307145429.GA13343@ericsson.com \
--to=guenter.roeck@ericsson.com \
--cc=lm-sensors@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.