From: Andrew Morton <akpm@linux-foundation.org>
To: Jean Delvare <khali@linux-fr.org>
Cc: djwong@us.ibm.com, mhoffman@lightlink.com,
linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org,
Roel Kluin <12o3l@tiscali.nl>
Subject: Re: [lm-sensors] [PATCH] i5k_amb: Support Intel 5400 chipset
Date: Mon, 05 May 2008 21:39:18 +0000 [thread overview]
Message-ID: <20080505143918.dcd93b52.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080505232245.6003ac8a@hyperion.delvare>
On Mon, 5 May 2008 23:22:45 +0200
Jean Delvare <khali@linux-fr.org> wrote:
> Hi Andrew,
>
> On Mon, 5 May 2008 13:56:21 -0700, Andrew Morton wrote:
> > On Fri, 2 May 2008 21:55:59 -0700
> > "Darrick J. Wong" <djwong@us.ibm.com> wrote:
> >
> > > Minor rework to support the Intel 5400 chipset.
> >
> > Thanks, I added this to the for-git-hwmon part of the -mm queue.
> >
> > I haven't heard from Mark for quite some time so soon I might declare him
> > absent and I'll go through my hwmon patches with a view to merging them into
> > 2.6.26 directly.
>
> For Mark's status see:
> http://lists.lm-sensors.org/pipermail/lm-sensors/2008-May/023006.html
>
> I guess he'll be happy if you merge a couple patches for him.
hm, OK.
> >
> > That queue currently consists of:
> >
> > hdaps-invert-the-axes-for-hdaps-on-lenovo-r61i-thinkpads.patch
> > adt746x-logical-bitwise-confusion-in-set_max_duty_at_crit.patch
>
> The adt7473:set_max_duty_at_crit() code was originally confusing but
> correct. After this patch the code is clear but broken. Is it
> considered an improvement?
I'll drop it.
> (I would have told you before if that patch had been submitted to the
> lm-sensors list as it should have, given that it modifies a hardware
> monitoring driver... Not to mention that the patch file name doesn't
> match the driver file name, just to add to the confusion.)
Well, y'know, stuff happens. Stuff like you forgetting to cc that
patch's author on this email.
> > adt7473-minor-documentation-update.patch
>
> I reviewed and acked that one, it's ready to go.
OK, thanks.
> > i5k_amb-support-intel-5400-chipset.patch
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
WARNING: multiple messages have this Message-ID (diff)
From: Andrew Morton <akpm@linux-foundation.org>
To: Jean Delvare <khali@linux-fr.org>
Cc: djwong@us.ibm.com, mhoffman@lightlink.com,
linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org,
Roel Kluin <12o3l@tiscali.nl>
Subject: Re: [lm-sensors] [PATCH] i5k_amb: Support Intel 5400 chipset
Date: Mon, 5 May 2008 14:39:18 -0700 [thread overview]
Message-ID: <20080505143918.dcd93b52.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080505232245.6003ac8a@hyperion.delvare>
On Mon, 5 May 2008 23:22:45 +0200
Jean Delvare <khali@linux-fr.org> wrote:
> Hi Andrew,
>
> On Mon, 5 May 2008 13:56:21 -0700, Andrew Morton wrote:
> > On Fri, 2 May 2008 21:55:59 -0700
> > "Darrick J. Wong" <djwong@us.ibm.com> wrote:
> >
> > > Minor rework to support the Intel 5400 chipset.
> >
> > Thanks, I added this to the for-git-hwmon part of the -mm queue.
> >
> > I haven't heard from Mark for quite some time so soon I might declare him
> > absent and I'll go through my hwmon patches with a view to merging them into
> > 2.6.26 directly.
>
> For Mark's status see:
> http://lists.lm-sensors.org/pipermail/lm-sensors/2008-May/023006.html
>
> I guess he'll be happy if you merge a couple patches for him.
hm, OK.
> >
> > That queue currently consists of:
> >
> > hdaps-invert-the-axes-for-hdaps-on-lenovo-r61i-thinkpads.patch
> > adt746x-logical-bitwise-confusion-in-set_max_duty_at_crit.patch
>
> The adt7473:set_max_duty_at_crit() code was originally confusing but
> correct. After this patch the code is clear but broken. Is it
> considered an improvement?
I'll drop it.
> (I would have told you before if that patch had been submitted to the
> lm-sensors list as it should have, given that it modifies a hardware
> monitoring driver... Not to mention that the patch file name doesn't
> match the driver file name, just to add to the confusion.)
Well, y'know, stuff happens. Stuff like you forgetting to cc that
patch's author on this email.
> > adt7473-minor-documentation-update.patch
>
> I reviewed and acked that one, it's ready to go.
OK, thanks.
> > i5k_amb-support-intel-5400-chipset.patch
next prev parent reply other threads:[~2008-05-05 21:39 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-03 4:55 [lm-sensors] [PATCH] i5k_amb: Support Intel 5400 chipset Darrick J. Wong
2008-05-03 4:55 ` Darrick J. Wong
2008-05-05 20:56 ` [lm-sensors] " Andrew Morton
2008-05-05 20:56 ` Andrew Morton
2008-05-05 21:22 ` [lm-sensors] " Jean Delvare
2008-05-05 21:22 ` Jean Delvare
2008-05-05 21:39 ` Andrew Morton [this message]
2008-05-05 21:39 ` Andrew Morton
2008-05-05 23:04 ` [lm-sensors] ibmaem driver status? Darrick J. Wong
2008-05-05 23:04 ` Darrick J. Wong
2008-05-06 21:04 ` [lm-sensors] " Andrew Morton
2008-05-06 21:04 ` Andrew Morton
2008-05-06 22:38 ` [lm-sensors] [resend] [PATCH v2] ibmaem: New driver for Darrick J. Wong
2008-05-06 22:38 ` [resend] [PATCH v2] ibmaem: New driver for power/energy meters in IBM System X hardware Darrick J. Wong
2008-05-07 0:32 ` [lm-sensors] [resend] [PATCH v2] ibmaem: New driver for Andrew Morton
2008-05-07 0:32 ` [resend] [PATCH v2] ibmaem: New driver for power/energy meters in IBM System X hardware Andrew Morton
2008-05-07 0:36 ` [lm-sensors] [resend] [PATCH v2] ibmaem: New driver for Andrew Morton
2008-05-07 0:36 ` [resend] [PATCH v2] ibmaem: New driver for power/energy meters in IBM System X hardware Andrew Morton
2008-05-07 6:01 ` [lm-sensors] [resend] [PATCH v2] ibmaem: New driver for Jean Delvare
2008-05-07 6:01 ` [resend] [PATCH v2] ibmaem: New driver for power/energy meters in IBM System X hardware Jean Delvare
2008-05-07 6:11 ` [lm-sensors] [resend] [PATCH v2] ibmaem: New driver for Andrew Morton
2008-05-07 6:11 ` [resend] [PATCH v2] ibmaem: New driver for power/energy meters in IBM System X hardware Andrew Morton
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=20080505143918.dcd93b52.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=12o3l@tiscali.nl \
--cc=djwong@us.ibm.com \
--cc=khali@linux-fr.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=mhoffman@lightlink.com \
/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.