Commit 968a3bed authored by Marek Vasut's avatar Marek Vasut Committed by Jonathan Cameron
Browse files

iio: light: noa1305: Use static table lookup of scale values 



Move scale values into a static table, perform look up of those
scale values in noa1305_scale() simply by using the integration
time register content as an index, because the integration time
register content directly maps to the table values.

Signed-off-by: default avatarMarek Vasut <marex@denx.de>
Link: https://patch.msgid.link/20240715183120.143417-3-marex@denx.de


Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent dde56b6d

drivers/iio/light/noa1305.c

+15 −36
Original line number Diff line number Diff line
@@ -29,6 +29,7 @@ 
#define   NOA1305_INTEGR_TIME_25MS	0x05

#define   NOA1305_INTEGR_TIME_12_5MS	0x06

#define   NOA1305_INTEGR_TIME_6_25MS	0x07

#define   NOA1305_INTEGR_TIME_MASK	0x07

#define NOA1305_REG_INT_SELECT		0x3

#define   NOA1305_INT_SEL_ACTIVE_HIGH	0x01

#define   NOA1305_INT_SEL_ACTIVE_LOW	0x02
@@ -43,6 +44,17 @@ 
#define NOA1305_DEVICE_ID	0x0519

#define NOA1305_DRIVER_NAME	"noa1305"



static int noa1305_scale_available[] = {

	100, 8 * 77,		/* 800 ms */

	100, 4 * 77,		/* 400 ms */

	100, 2 * 77,		/* 200 ms */

	100, 1 * 77,		/* 100 ms */

	1000, 5 * 77,		/* 50 ms */

	10000, 25 * 77,		/* 25 ms */

	100000, 125 * 77,	/* 12.5 ms */

	1000000, 625 * 77,	/* 6.25 ms */

};



struct noa1305_priv {

	struct i2c_client *client;

	struct regmap *regmap;
@@ -78,42 +90,9 @@ static int noa1305_scale(struct noa1305_priv *priv, int *val, int *val2) 
	 * Integration Constant = 7.7

	 * Integration Time in Seconds

	 */

	switch (data) {

	case NOA1305_INTEGR_TIME_800MS:

		*val = 100;

		*val2 = 77 * 8;

		break;

	case NOA1305_INTEGR_TIME_400MS:

		*val = 100;

		*val2 = 77 * 4;

		break;

	case NOA1305_INTEGR_TIME_200MS:

		*val = 100;

		*val2 = 77 * 2;

		break;

	case NOA1305_INTEGR_TIME_100MS:

		*val = 100;

		*val2 = 77;

		break;

	case NOA1305_INTEGR_TIME_50MS:

		*val = 1000;

		*val2 = 77 * 5;

		break;

	case NOA1305_INTEGR_TIME_25MS:

		*val = 10000;

		*val2 = 77 * 25;

		break;

	case NOA1305_INTEGR_TIME_12_5MS:

		*val = 100000;

		*val2 = 77 * 125;

		break;

	case NOA1305_INTEGR_TIME_6_25MS:

		*val = 1000000;

		*val2 = 77 * 625;

		break;

	default:

		return -EINVAL;

	}

	data &= NOA1305_INTEGR_TIME_MASK;

	*val = noa1305_scale_available[2 * data + 0];

	*val2 = noa1305_scale_available[2 * data + 1];



	return IIO_VAL_FRACTIONAL;

}