From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Tue, 13 Dec 2011 20:02:53 +0000 Subject: Re: [lm-sensors] [PATCH] hwmon: (f75375s) Added F75387SG/RG support Message-Id: <20111213200253.GA24345@ericsson.com> List-Id: References: <2D36B16B-9487-4A4D-B4EB-60FA8766BFFB@bgerhart.de> In-Reply-To: <2D36B16B-9487-4A4D-B4EB-60FA8766BFFB@bgerhart.de> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: lm-sensors@vger.kernel.org On Tue, Dec 13, 2011 at 02:34:18PM -0500, Bj=F6rn Gerhart wrote: > Hi Guenter, >=20 > 2011/12/13 Guenter Roeck : > > On Tue, Dec 13, 2011 at 01:19:58PM -0500, Bj=F6rn Gerhart wrote: > >> Added support for Fintek F75387SG/RG. > >>=20 > >> Cc: Riku Voipio > >> Signed-off-by: Bjoern Gerhart > >> --- > >>=20 > >> Tested with kernel 2.6.39, but without fans so far. > >>=20 > >> v2: > >> - suitable ontop of the latest set of f75375s related patches > >> - simplified name of a local char variable > >>=20 > >> v3: > >> - detected reason for corrupted patch, should be solved now > >> - fixed F75387 fan handling: detect/select pwm_enable[0,1]=20 > >>=20 > > Yes, this time it worked. > >=20 > > There was one checkpatch warning (line too long) which I fixed myself. > > Applied to -next. > >=20 > That's fine, I'm really happy about that. Thanks for your helpful guidanc= e! > However, I'll look up the patch file related rules to avoid such checkpat= ch warnings in future ;-) >=20 Hi Bjoern, All you need to do is to run scripts/checkpatch.pl on your patch. Guenter _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors