From: khali@linux-fr.org (Jean Delvare)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv1 1/3] hwmon: Add support for GMT G762/G763 PWM fan controller
Date: Sat, 27 Apr 2013 16:12:04 +0200 [thread overview]
Message-ID: <20130427161204.3957686c@endymion.delvare> (raw)
In-Reply-To: <20130427140331.GL17771@kw.sim.vm.gnt>
On Sat, 27 Apr 2013 16:03:31 +0200, Simon Guinot wrote:
> On Wed, Apr 24, 2013 at 12:05:56AM +0200, Arnaud Ebalard wrote:
> > + /*
> > + * Set default configuration values before passing the structure
> > + * to OF helpers to overload them using those provided by .dts
> > + * file (if any). Final config is then commited.
> > + */
> > + g762_config_init(&conf);
> > + g762_config_of_overload(client, &conf);
> > + err = g762_config_commit(client, &conf);
> > + if (err)
> > + goto out;
>
> One more comment... Sorry for the multiple replies :)
>
> I am not sure that applying a configuration anyway is a good idea.
> I think it could be best to stick with the bootloader configuration if
> no platform data nor device tree data are given.
FWIW, the common policy for hwmon drivers is actually to not change the
device configuration unless they really have to.
--
Jean Delvare
next prev parent reply other threads:[~2013-04-27 14:12 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-18 22:27 [RFC,PATCHv0 0/3] Add support for GMT G762/G763 PWM fan controller Arnaud Ebalard
2013-04-18 22:28 ` [PATCH 1/3] Add support for GMT G72/G763 " Arnaud Ebalard
2013-04-19 4:35 ` Guenter Roeck
2013-04-19 5:34 ` Arnaud Ebalard
2013-04-23 22:05 ` [PATCHv1 0/3] hwmon: " Arnaud Ebalard
2013-04-23 22:05 ` [PATCHv1 1/3] hwmon: Add support for GMT G762/G763 " Arnaud Ebalard
2013-04-24 5:37 ` Andrew Lunn
2013-04-24 9:06 ` Arnaud Ebalard
2013-04-24 10:04 ` Simon Guinot
2013-04-24 10:50 ` Arnaud Ebalard
2013-04-24 13:38 ` Guenter Roeck
2013-04-24 20:28 ` Arnaud Ebalard
2013-04-24 22:47 ` Guenter Roeck
2013-04-25 10:14 ` Simon Guinot
2013-04-24 17:06 ` Simon Guinot
2013-04-24 23:37 ` Guenter Roeck
2013-04-25 9:58 ` Simon Guinot
2013-04-27 14:03 ` Simon Guinot
2013-04-27 14:12 ` Jean Delvare [this message]
2013-04-27 16:56 ` Guenter Roeck
2013-04-27 18:55 ` Arnaud Ebalard
2013-04-23 22:06 ` [PATCHv1 2/3] hwmon: Add documentation for g762 driver Arnaud Ebalard
2013-04-24 17:32 ` Guenter Roeck
2013-04-24 20:33 ` Arnaud Ebalard
2013-04-23 22:06 ` [PATCHv1 3/3] hwmon: Add DT " Arnaud Ebalard
2013-04-23 22:23 ` Jason Cooper
2013-04-24 5:43 ` Arnaud Ebalard
2013-04-19 5:50 ` [PATCH 1/3] Add support for GMT G72/G763 PWM fan controller Andrew Lunn
2013-04-19 11:30 ` Arnaud Ebalard
2013-04-19 13:37 ` Guenter Roeck
2013-04-19 6:05 ` Jean Delvare
2013-04-19 11:31 ` Arnaud Ebalard
2013-04-18 22:28 ` [RFC,PATCHv0 2/3] Add DT documentation for G762 " Arnaud Ebalard
2013-04-18 22:28 ` [PATCH 3/3] Add documentation for g762 driver Arnaud Ebalard
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=20130427161204.3957686c@endymion.delvare \
--to=khali@linux-fr.org \
--cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox