From: Cedric Le Goater <clg@fr.ibm.com>
To: Thomas Haschka <haschka@gmail.com>,
stable@vger.kernel.org, benh@kernel.crashing.org,
linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH 1/1]: thermal driver therm_adt746.c
Date: Mon, 23 Feb 2015 16:23:13 +0100 [thread overview]
Message-ID: <54EB45E1.7040805@fr.ibm.com> (raw)
In-Reply-To: <CADAoK=UH4q9x-OMyOpqCmGpNwDo3vey41xajBw-PDdupmEHnAA@mail.gmail.com>
On 02/23/2015 12:58 PM, Thomas Haschka wrote:
> Hi everyone,
>
> The current driver linux/drivers/macintosh/therm_adt746x.c does not take the HDD BUTTOMSIDE sensor into account. It actually should as the 12" Powerbooks and IBooks are build in a way that the airflow cools the harddrive and components around it. Actually there are air intake openings just beneath the Harddrive. If you experiance hot enviromental temperatures as I did in summer on certain occations, you will find out that in MacOSX the fan spins up while it doesn't in linux. As this probably causes harddrives, and maybe other components to fail early I think this should be regarded as a fix to a severe bug.
>
> Hence, I created this patch for single fan 12" Albooks, Ibooks etc.
I would happy to give it a try on the "brand new" iBook G4 I just
got this week-end but I would need you to reformat your patch.
Could you please send it again to the mailing using :
git send-email --to linuxppc-dev@lists.ozlabs.org my.patch
You might want to run the script ./scripts/checkpatch.pl on the
patch file before sending.
Cheers,
C.
next prev parent reply other threads:[~2015-02-23 15:23 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-23 11:58 [PATCH 1/1]: thermal driver therm_adt746.c Thomas Haschka
2015-02-23 15:23 ` Cedric Le Goater [this message]
2015-02-23 21:18 ` Benjamin Herrenschmidt
2015-02-25 11:24 ` Thomas Haschka
2015-02-25 21:12 ` Benjamin Herrenschmidt
2015-02-26 9:48 ` Thomas Haschka
2015-03-25 5:36 ` Benjamin Herrenschmidt
2015-03-26 16:23 ` Thomas Haschka
2015-03-26 21:31 ` Benjamin Herrenschmidt
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=54EB45E1.7040805@fr.ibm.com \
--to=clg@fr.ibm.com \
--cc=benh@kernel.crashing.org \
--cc=haschka@gmail.com \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=stable@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.