From: Peter Hung <hpeter@gmail.com>
To: Guenter Roeck <linux@roeck-us.net>, jdelvare@suse.de
Cc: lm-sensors@lm-sensors.org, linux-kernel@vger.kernel.org,
tom_tsai@fintek.com.tw, peter_hong@fintek.com.tw,
Peter Hung <hpeter+linux_kernel@gmail.com>
Subject: Re: [lm-sensors] [PATCH V2 2/3] hwmon:f71882fg fix f81866a temp/beep setting
Date: Thu, 02 Jul 2015 03:08:43 +0000 [thread overview]
Message-ID: <5594AB3B.90301@gmail.com> (raw)
In-Reply-To: <559455C7.3030002@roeck-us.net>
SGkgR3VlbnRlciwKCkd1ZW50ZXIgUm9lY2sg5pa8IDIwMTUvNy8yIOS4iuWNiCAwNTowNCDlr6vp
gZM6Cj4gSGkgUGV0ZXIsCj4+ICsgICAgICAgICAgICB9IGVsc2Ugewo+PiArICAgICAgICAgICAg
ICAgIHNpemUgPSBBUlJBWV9TSVpFKGZ4eHh4X3RlbXBfYmVlcF9hdHRyWzBdOwo+Cj4gVHNrVHNr
IC4uLiB0dXJucyBvdXQgeW91IGRpZG4ndCBldmVuIGNvbXBpbGUgdGVzdCB0aGlzIGNvZGUuCj4g
SG93IGFtIEkgc3VwcG9zZWQgdG8ga25vdyB0aGF0IGl0IGlzIHdvcmtpbmcgPwoKU29ycnksIEl0
J3MgbXkgZmF1bHQuCkkgc2hvdWxkIHJlLXRlc3RpbmcgcGF0Y2ggYWZ0ZXIgZml4IGNoZWNrcGF0
Y2guc2ggd2FybmluZ3MuCgpJJ2xsIHJlLWNvbmZpcm0gYWxsIHBhdGNoZXMgYWdhaW4gYmVmb3Jl
IHNlbmQgaXQgdG8gbWFpbnRhaW5lci4KCi0tIApXaXRoIEJlc3QgUmVnYXJkcywKUGV0ZXIgSHVu
ZwoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbG0tc2Vu
c29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1zZW5zb3JzLm9yZwpodHRwOi8vbGlzdHMu
bG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9sbS1zZW5zb3Jz
WARNING: multiple messages have this Message-ID (diff)
From: Peter Hung <hpeter@gmail.com>
To: Guenter Roeck <linux@roeck-us.net>, jdelvare@suse.de
Cc: lm-sensors@lm-sensors.org, linux-kernel@vger.kernel.org,
tom_tsai@fintek.com.tw, peter_hong@fintek.com.tw,
Peter Hung <hpeter+linux_kernel@gmail.com>
Subject: Re: [PATCH V2 2/3] hwmon:f71882fg fix f81866a temp/beep setting
Date: Thu, 2 Jul 2015 11:08:43 +0800 [thread overview]
Message-ID: <5594AB3B.90301@gmail.com> (raw)
In-Reply-To: <559455C7.3030002@roeck-us.net>
Hi Guenter,
Guenter Roeck 於 2015/7/2 上午 05:04 寫道:
> Hi Peter,
>> + } else {
>> + size = ARRAY_SIZE(fxxxx_temp_beep_attr[0];
>
> TskTsk ... turns out you didn't even compile test this code.
> How am I supposed to know that it is working ?
Sorry, It's my fault.
I should re-testing patch after fix checkpatch.sh warnings.
I'll re-confirm all patches again before send it to maintainer.
--
With Best Regards,
Peter Hung
next prev parent reply other threads:[~2015-07-02 3:08 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-30 1:31 [lm-sensors] [PATCH V2 0/3] hwmon:f71882fg add new Fintek SuperIO support Peter Hung
2015-06-30 1:31 ` Peter Hung
2015-06-30 1:31 ` [lm-sensors] [PATCH V2 1/3] hwmon:f71882fg add f81866/f71868 " Peter Hung
2015-06-30 1:31 ` Peter Hung
2015-06-30 1:31 ` [lm-sensors] [PATCH V2 2/3] hwmon:f71882fg fix f81866a temp/beep setting Peter Hung
2015-06-30 1:31 ` Peter Hung
2015-07-01 21:04 ` [lm-sensors] " Guenter Roeck
2015-07-01 21:04 ` Guenter Roeck
2015-07-02 3:08 ` Peter Hung [this message]
2015-07-02 3:08 ` Peter Hung
2015-06-30 1:31 ` [lm-sensors] [PATCH V2 3/3] hwmon:f71882fg fix f81866a voltage protection Peter Hung
2015-06-30 1:31 ` Peter Hung
2015-07-01 17:53 ` [lm-sensors] [PATCH V2 0/3] hwmon:f71882fg add new Fintek SuperIO support Guenter Roeck
2015-07-01 17:53 ` Guenter Roeck
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=5594AB3B.90301@gmail.com \
--to=hpeter@gmail.com \
--cc=hpeter+linux_kernel@gmail.com \
--cc=jdelvare@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=lm-sensors@lm-sensors.org \
--cc=peter_hong@fintek.com.tw \
--cc=tom_tsai@fintek.com.tw \
/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.