Commit de0da6ae authored by Guenter Roeck's avatar Guenter Roeck
Browse files

hwmon: (ina2xx) Add support for has_alerts configuration flag 



Add configuration flag indicating if the chip supports alerts and limits
to prepare for adding INA260 support.

Reviewed-by: default avatarTzung-Bi Shih <tzungbi@kernel.org>
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
parent a7e03f96

drivers/hwmon/ina2xx.c

+11 −7
Original line number Diff line number Diff line
@@ -129,6 +129,7 @@ enum ina2xx_ids { ina219, ina226 }; 


struct ina2xx_config {

	u16 config_default;

	bool has_alerts;	/* chip supports alerts and limits */

	int calibration_value;

	int shunt_div;

	int bus_voltage_shift;
@@ -155,6 +156,7 @@ static const struct ina2xx_config ina2xx_config[] = { 
		.bus_voltage_shift = 3,

		.bus_voltage_lsb = 4000,

		.power_lsb_factor = 20,

		.has_alerts = false,

	},

	[ina226] = {

		.config_default = INA226_CONFIG_DEFAULT,
@@ -163,6 +165,7 @@ static const struct ina2xx_config ina2xx_config[] = { 
		.bus_voltage_shift = 0,

		.bus_voltage_lsb = 1250,

		.power_lsb_factor = 25,

		.has_alerts = true,

	},

};
@@ -624,6 +627,7 @@ static umode_t ina2xx_is_visible(const void *_data, enum hwmon_sensor_types type 
				 u32 attr, int channel)

{

	const struct ina2xx_data *data = _data;

	bool has_alerts = data->config->has_alerts;

	enum ina2xx_ids chip = data->chip;



	switch (type) {
@@ -633,12 +637,12 @@ static umode_t ina2xx_is_visible(const void *_data, enum hwmon_sensor_types type 
			return 0444;

		case hwmon_in_lcrit:

		case hwmon_in_crit:

			if (chip == ina226)

			if (has_alerts)

				return 0644;

			break;

		case hwmon_in_lcrit_alarm:

		case hwmon_in_crit_alarm:

			if (chip == ina226)

			if (has_alerts)

				return 0444;

			break;

		default:
@@ -651,12 +655,12 @@ static umode_t ina2xx_is_visible(const void *_data, enum hwmon_sensor_types type 
			return 0444;

		case hwmon_curr_lcrit:

		case hwmon_curr_crit:

			if (chip == ina226)

			if (has_alerts)

				return 0644;

			break;

		case hwmon_curr_lcrit_alarm:

		case hwmon_curr_crit_alarm:

			if (chip == ina226)

			if (has_alerts)

				return 0444;

			break;

		default:
@@ -668,11 +672,11 @@ static umode_t ina2xx_is_visible(const void *_data, enum hwmon_sensor_types type 
		case hwmon_power_input:

			return 0444;

		case hwmon_power_crit:

			if (chip == ina226)

			if (has_alerts)

				return 0644;

			break;

		case hwmon_power_crit_alarm:

			if (chip == ina226)

			if (has_alerts)

				return 0444;

			break;

		default:
@@ -802,7 +806,7 @@ static int ina2xx_init(struct device *dev, struct ina2xx_data *data) 
	if (ret < 0)

		return ret;



	if (data->chip == ina226) {

	if (data->config->has_alerts) {

		bool active_high = device_property_read_bool(dev, "ti,alert-polarity-active-high");



		regmap_update_bits(regmap, INA226_MASK_ENABLE,