From: greg@kroah.com (Greg KH)
To: Martin Schlemmer <azarah@gentoo.org>
Cc: Andrey Borzenkov <arvidjaar@mail.ru>,
LKML <linux-kernel@vger.kernel.org>,
Sensors <sensors@stimpy.netroedge.com>
Subject: [PATCH 2.6] Fix conversion from milli volts in store_in_reg() for
Date: Thu, 19 May 2005 06:24:15 +0000 [thread overview]
Message-ID: <20030904184035.GA11160@kroah.com> (raw)
In-Reply-To: <1062622452.5275.38.camel@nosferatu.lan>
On Wed, Sep 03, 2003 at 10:54:12PM +0200, Martin Schlemmer wrote:
> Hi
>
> I am not sure if it was a later patch from me that fixed in_* to display
> milli volts in sysfs, or if it was a patch from Jan Dittmer, but the
> conversion in the store_in_*() functions is wrong, and cause something
> like:
>
> ----------------------------
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 3632
> nosferatu patch # echo '3700' > /sys/bus/i2c/devices/1-0290/in_max2
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 400
> nosferatu patch # echo '3640' > /sys/bus/i2c/devices/1-0290/in_max2
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 400
> nosferatu patch # echo '3632' > /sys/bus/i2c/devices/1-0290/in_max2
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 400
> nosferatu patch #
> -----------------------------
>
> I think Andrey also noticed this (if I did not smoke too much Weedbix
> this morning ;) - if so, please verify that this fixes it ... it does
> here at least.
>
> Anyhow, patch is attached and should apply to 2.6.0-test4-bk5.
Applied, thanks.
greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: Greg KH <greg@kroah.com>
To: Martin Schlemmer <azarah@gentoo.org>
Cc: Andrey Borzenkov <arvidjaar@mail.ru>,
LKML <linux-kernel@vger.kernel.org>,
Sensors <sensors@stimpy.netroedge.com>
Subject: Re: [PATCH 2.6] Fix conversion from milli volts in store_in_reg() for w83781d.c
Date: Thu, 4 Sep 2003 11:40:35 -0700 [thread overview]
Message-ID: <20030904184035.GA11160@kroah.com> (raw)
In-Reply-To: <1062622452.5275.38.camel@nosferatu.lan>
On Wed, Sep 03, 2003 at 10:54:12PM +0200, Martin Schlemmer wrote:
> Hi
>
> I am not sure if it was a later patch from me that fixed in_* to display
> milli volts in sysfs, or if it was a patch from Jan Dittmer, but the
> conversion in the store_in_*() functions is wrong, and cause something
> like:
>
> ----------------------------
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 3632
> nosferatu patch # echo '3700' > /sys/bus/i2c/devices/1-0290/in_max2
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 400
> nosferatu patch # echo '3640' > /sys/bus/i2c/devices/1-0290/in_max2
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 400
> nosferatu patch # echo '3632' > /sys/bus/i2c/devices/1-0290/in_max2
> nosferatu patch # cat /sys/bus/i2c/devices/1-0290/in_max2
> 400
> nosferatu patch #
> -----------------------------
>
> I think Andrey also noticed this (if I did not smoke too much Weedbix
> this morning ;) - if so, please verify that this fixes it ... it does
> here at least.
>
> Anyhow, patch is attached and should apply to 2.6.0-test4-bk5.
Applied, thanks.
greg k-h
next prev parent reply other threads:[~2005-05-19 6:24 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-05-24 18:37 [OOPS] w83781d during rmmod (2.5.69-bk17) Mark M. Hoffman
2005-05-19 6:23 ` Mark M. Hoffman
2005-05-19 6:23 ` Mark D. Studebaker
2003-06-01 14:38 ` [RFC PATCH] " Mark M. Hoffman
2005-05-19 6:23 ` Mark M. Hoffman
2003-06-02 17:20 ` Greg KH
2005-05-19 6:23 ` Greg KH
2003-06-03 5:22 ` Martin Schlemmer
2005-05-19 6:23 ` Martin Schlemmer
2003-06-03 19:43 ` Philip Pokorny
2005-05-19 6:23 ` Philip Pokorny
2003-06-04 5:57 ` Martin Schlemmer
2005-05-19 6:23 ` Martin Schlemmer
2003-06-05 2:39 ` Mark M. Hoffman
2005-05-19 6:23 ` Mark M. Hoffman
2003-06-05 19:47 ` Greg KH
2005-05-19 6:23 ` Greg KH
2003-06-09 5:34 ` Martin Schlemmer
2005-05-19 6:23 ` Martin Schlemmer
2003-06-10 5:38 ` Martin Schlemmer
2005-05-19 6:23 ` Martin Schlemmer
2003-06-10 5:41 ` OOPS w83781d during rmmod (2.5.70-bk1[1234]) Mark M. Hoffman
2005-05-19 6:23 ` Mark M. Hoffman
2003-06-10 5:51 ` Martin Schlemmer
2005-05-19 6:23 ` Martin Schlemmer
2003-06-11 5:44 ` Martin Schlemmer
2005-05-19 6:23 ` Martin Schlemmer
2003-06-12 6:57 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d during rmmod (2.5.70-bk1[1234])) Martin Schlemmer
2005-05-19 6:23 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d Martin Schlemmer
2003-06-13 2:36 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d during rmmod (2.5.70-bk1[1234])) Mark M. Hoffman
2005-05-19 6:23 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d Mark M. Hoffman
2003-06-13 6:08 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d during rmmod (2.5.70-bk1[1234])) Martin Schlemmer
2005-05-19 6:23 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS Martin Schlemmer
2003-06-14 6:26 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d during rmmod (2.5.70-bk1[1234])) Martin Schlemmer
2005-05-19 6:24 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS Martin Schlemmer
2003-06-16 18:41 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d during rmmod (2.5.70-bk1[1234])) Greg KH
2005-05-19 6:24 ` [RFC][2.5] list_for_each_safe not so safe (was Re: OOPS w83781d Greg KH
2003-09-03 20:54 ` [PATCH 2.6] Fix conversion from milli volts in store_in_reg() for w83781d.c Martin Schlemmer
2005-05-19 6:24 ` [PATCH 2.6] Fix conversion from milli volts in store_in_reg() for Martin Schlemmer
2003-09-04 18:40 ` Greg KH [this message]
2005-05-19 6:24 ` Greg KH
2003-09-07 15:41 ` [PATCH 2.6] Fix conversion from milli volts in store_in_reg() for w83781d.c Andrey Borzenkov
2005-05-19 6:24 ` [PATCH 2.6] Fix conversion from milli volts in store_in_reg() for Andrey Borzenkov
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=20030904184035.GA11160@kroah.com \
--to=greg@kroah.com \
--cc=arvidjaar@mail.ru \
--cc=azarah@gentoo.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sensors@stimpy.netroedge.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.