From: Guenter Roeck <linux@roeck-us.net>
To: Iker Perez <iker.perez@codethink.co.uk>
Cc: linux-hwmon@vger.kernel.org, jdelvare@suse.com,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 3/4] hwmon: (lm75) Add new fields into lm75_params_
Date: Thu, 8 Aug 2019 07:11:54 -0700 [thread overview]
Message-ID: <20190808141154.GA9369@roeck-us.net> (raw)
In-Reply-To: <20190808080246.8371-4-iker.perez@codethink.co.uk>
On Thu, Aug 08, 2019 at 09:02:45AM +0100, Iker Perez wrote:
> From: Iker Perez del Palomar Sustatxa <iker.perez@codethink.co.uk>
>
> The new fields are included to prepare the driver for next patch. The
> fields are:
>
> * *resolutions: Stores all the supported resolutions by the device.
> * num_sample_times: Stores the number of possible sample times.
> * *sample_times: Stores all the possible sample times to be set.
> * sample_set_masks: The set_masks for the possible sample times
> * sample_clr_mask: Clear mask to set the default sample time.
>
> Signed-off-by: Iker Perez del Palomar Sustatxa <iker.perez@codethink.co.uk>
> ---
>
> Changes since v1:
> - In the lm75_params structure documentation there have been the next changes:
> - @num_sample_times description has been extended.
> - @sample_times description has been extended.
> - @sample_set_masks description has been extended.
> - @resolutions description has been included.
>
> drivers/hwmon/lm75.c | 36 +++++++++++++++++++++++++++++++-----
> 1 file changed, 31 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
> index a32d7952d799..ed72455bcfa3 100644
> --- a/drivers/hwmon/lm75.c
> +++ b/drivers/hwmon/lm75.c
> @@ -61,15 +61,34 @@ enum lm75_type { /* keep sorted in alphabetical order */
> * @resolution_limits: Limit register resolution. Optional. Should be set if
> * the resolution of limit registers does not match the
> * resolution of the temperature register.
> + * @resolutions List of resolutions associated with sample times.
@resolutions:
> + * Optional. Should be set if num_sample_times is larger
> + * than 1, and if the resolution changes with sample times.
> + * If set, number of entries must match num_sample_times.
> * default_sample_time: Sample time to be set by default.
@default_sample_time:
No need to resend; I'll fix that up when applying.
> + * @num_sample_times: Number of possible sample times to be set. Optional.
> + * Should be set if the number of sample times is larger
> + * than one.
> + * @sample_times: All the possible sample times to be set. Mandatory if
> + * num_sample_times is larger than 1. If set, number of
> + * entries must match num_sample_times.
> + * @sample_set_masks: All the set_masks for the possible sample times.
> + * Mandatory if num_sample_times is larger than 1.
> + * If set, number of entries must match num_sample_times.
> + * @sample_clr_mask: Clear mask to set the default sample time.
> */
>
> struct lm75_params {
> - u8 set_mask;
> - u8 clr_mask;
> - u8 default_resolution;
> - u8 resolution_limits;
> - unsigned int default_sample_time;
> + u8 set_mask;
> + u8 clr_mask;
> + u8 default_resolution;
> + u8 resolution_limits;
> + const u8 *resolutions;
> + unsigned int default_sample_time;
> + u8 num_sample_times;
> + const unsigned int *sample_times;
> + const u8 *sample_set_masks;
> + u8 sample_clr_mask;
> };
>
> /* Addresses scanned */
> @@ -221,7 +240,14 @@ static const struct lm75_params device_params[] = {
> [tmp75b] = { /* not one-shot mode, Conversion rate 37Hz */
> .clr_mask = 1 << 7 | 3 << 5,
> .default_resolution = 12,
> + .sample_set_masks = (u8 []){ 0 << 5, 1 << 5, 2 << 5,
> + 3 << 5 },
> + .sample_clr_mask = 3 << 5,
> .default_sample_time = MSEC_PER_SEC / 37,
> + .sample_times = (unsigned int []){ MSEC_PER_SEC / 37,
> + MSEC_PER_SEC / 18,
> + MSEC_PER_SEC / 9, MSEC_PER_SEC / 4 },
> + .num_sample_times = 4,
> },
> [tmp75c] = {
> .clr_mask = 1 << 5, /*not one-shot mode*/
next prev parent reply other threads:[~2019-08-08 14:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-08 8:02 [PATCH v2 0/4] Add support for variable sample time in lm75 driver Iker Perez
2019-08-08 8:02 ` [PATCH v2 1/4] hwmon: (lm75) Create structure to save all the configuration parameters Iker Perez
2019-08-08 8:02 ` [PATCH v2 2/4] hwmon: (lm75) Create function from code to write into registers Iker Perez
2019-08-08 8:02 ` [PATCH v2 3/4] hwmon: (lm75) Add new fields into lm75_params_ Iker Perez
2019-08-08 14:11 ` Guenter Roeck [this message]
2019-08-08 8:02 ` [PATCH v2 4/4] hwmon: (lm75) Modularize lm75_write and make hwmon_chip writable Iker Perez
2019-08-08 14:14 ` [PATCH v2 0/4] Add support for variable sample time in lm75 driver 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=20190808141154.GA9369@roeck-us.net \
--to=linux@roeck-us.net \
--cc=iker.perez@codethink.co.uk \
--cc=jdelvare@suse.com \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).