Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752743AbbBAKL5 (ORCPT ); Sun, 1 Feb 2015 05:11:57 -0500 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:35269 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751961AbbBAKLy (ORCPT ); Sun, 1 Feb 2015 05:11:54 -0500 Message-ID: <54CDFBE8.7060609@kernel.org> Date: Sun, 01 Feb 2015 10:11:52 +0000 From: Jonathan Cameron User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Hartmut Knaack , Daniel Baluta CC: lars@metafoo.de, pmeerw@pmeerw.net, srinivas.pandruvada@linux.intel.com, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] iio: imu: kmx61: Drop odr_bits from kmx61_samp_freq_table References: <1420449702-2085-1-git-send-email-daniel.baluta@intel.com> <54AB179F.8020206@gmx.de> In-Reply-To: <54AB179F.8020206@gmx.de> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4337 Lines: 138 On 05/01/15 23:00, Hartmut Knaack wrote: > Daniel Baluta schrieb am 05.01.2015 um 10:21: >> odr_bits values are between 0 and 11, so we can use the index >> in kmx61_samp_freq_table instead of odr_bits structure member. >> >> Signed-off-by: Daniel Baluta > Reviewed-by: Hartmut Knaack Applied to the togreg branch of iio.git - pushed out as testing. Sorry, I marked this to apply when there was a messup over my first pull request then failed to actually pick it up once that got resolved. Now unfortunately 3.21 material. Ah well, not a critical change anyway, more of a cleanup ;) Jonathan >> --- >> Changes since v1: >> * use ARRAY_SIZE instead of an hardcoded value >> * review is at: https://lkml.org/lkml/2015/1/1/46 >> >> drivers/iio/imu/kmx61.c | 64 ++++++++++++++++++++----------------------------- >> 1 file changed, 26 insertions(+), 38 deletions(-) >> >> diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c >> index b60b22d..a32ddbb 100644 >> --- a/drivers/iio/imu/kmx61.c >> +++ b/drivers/iio/imu/kmx61.c >> @@ -169,19 +169,18 @@ static const u16 kmx61_uscale_table[] = {9582, 19163, 38326}; >> static const struct { >> int val; >> int val2; >> - u8 odr_bits; >> -} kmx61_samp_freq_table[] = { {12, 500000, 0x00}, >> - {25, 0, 0x01}, >> - {50, 0, 0x02}, >> - {100, 0, 0x03}, >> - {200, 0, 0x04}, >> - {400, 0, 0x05}, >> - {800, 0, 0x06}, >> - {1600, 0, 0x07}, >> - {0, 781000, 0x08}, >> - {1, 563000, 0x09}, >> - {3, 125000, 0x0A}, >> - {6, 250000, 0x0B} }; >> +} kmx61_samp_freq_table[] = { {12, 500000}, >> + {25, 0}, >> + {50, 0}, >> + {100, 0}, >> + {200, 0}, >> + {400, 0}, >> + {800, 0}, >> + {1600, 0}, >> + {0, 781000}, >> + {1, 563000}, >> + {3, 125000}, >> + {6, 250000} }; >> >> static const struct { >> int val; >> @@ -302,24 +301,10 @@ static int kmx61_convert_freq_to_bit(int val, int val2) >> for (i = 0; i < ARRAY_SIZE(kmx61_samp_freq_table); i++) >> if (val == kmx61_samp_freq_table[i].val && >> val2 == kmx61_samp_freq_table[i].val2) >> - return kmx61_samp_freq_table[i].odr_bits; >> - return -EINVAL; >> -} >> - >> -static int kmx61_convert_bit_to_freq(u8 odr_bits, int *val, int *val2) >> -{ >> - int i; >> - >> - for (i = 0; i < ARRAY_SIZE(kmx61_samp_freq_table); i++) >> - if (odr_bits == kmx61_samp_freq_table[i].odr_bits) { >> - *val = kmx61_samp_freq_table[i].val; >> - *val2 = kmx61_samp_freq_table[i].val2; >> - return 0; >> - } >> + return i; >> return -EINVAL; >> } >> >> - >> static int kmx61_convert_wake_up_odr_to_bit(int val, int val2) >> { >> int i; >> @@ -478,7 +463,7 @@ static int kmx61_set_odr(struct kmx61_data *data, int val, int val2, u8 device) >> >> static int kmx61_get_odr(struct kmx61_data *data, int *val, int *val2, >> u8 device) >> -{ int i; >> +{ >> u8 lodr_bits; >> >> if (device & KMX61_ACC) >> @@ -490,13 +475,13 @@ static int kmx61_get_odr(struct kmx61_data *data, int *val, int *val2, >> else >> return -EINVAL; >> >> - for (i = 0; i < ARRAY_SIZE(kmx61_samp_freq_table); i++) >> - if (lodr_bits == kmx61_samp_freq_table[i].odr_bits) { >> - *val = kmx61_samp_freq_table[i].val; >> - *val2 = kmx61_samp_freq_table[i].val2; >> - return 0; >> - } >> - return -EINVAL; >> + if (lodr_bits >= ARRAY_SIZE(kmx61_samp_freq_table)) >> + return -EINVAL; >> + >> + *val = kmx61_samp_freq_table[lodr_bits].val; >> + *val2 = kmx61_samp_freq_table[lodr_bits].val2; >> + >> + return 0; >> } >> >> static int kmx61_set_range(struct kmx61_data *data, u8 range) >> @@ -580,8 +565,11 @@ static int kmx61_chip_init(struct kmx61_data *data) >> } >> data->odr_bits = ret; >> >> - /* set output data rate for wake up (motion detection) function */ >> - ret = kmx61_convert_bit_to_freq(data->odr_bits, &val, &val2); >> + /* >> + * set output data rate for wake up (motion detection) function >> + * to match data rate for accelerometer sampling >> + */ >> + ret = kmx61_get_odr(data, &val, &val2, KMX61_ACC); >> if (ret < 0) >> return ret; >> >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/