Re-initialization of the sensor causing heap memory growth

[gmanrc]

Re-initialization of the sensor causing heap memory growth

[ladyada]

are you deleting the object? allocated memory is freed there
https://github.com/adafruit/Adafruit_ICM20X/blob/master/Adafruit_ICM20X.cpp#L54

[caternuson]

see:
adafruit/Adafruit_MS8607#4 (comment)

[gmanrc]

@caternuson & @ladyada thanks for your help on that for the MS8607. Unfortunately, for the icm20948 does not appear to currently have an init() function for re-initializing the sensors. There is reset(); but calling begin() after reset; is still taking up the free memory.

[caternuson]

calling begin() after reset; is still taking up the free memory.

Call begin only once, in setup. Call reset in the loop.

Nothing in reset should be eating memory:

Adafruit_ICM20X/Adafruit_ICM20X.cpp

Lines 150 to 166 in 6ec93fe

	void Adafruit_ICM20X::reset(void) {
	_setBank(0);
	Adafruit_BusIO_Register pwr_mgmt1 = Adafruit_BusIO_Register(
	i2c_dev, spi_dev, ADDRBIT8_HIGH_TOREAD, ICM20X_B0_PWR_MGMT_1, 1);
	Adafruit_BusIO_RegisterBits reset_bit =
	Adafruit_BusIO_RegisterBits(&pwr_mgmt1, 1, 7);
	reset_bit.write(1);
	delay(20);
	while (reset_bit.read()) {
	delay(10);
	};
	delay(50);
	}

[gmanrc]

calling begin() after reset; is still taking up the free memory.

Call begin only once, in setup. Call reset in the loop.

Nothing in reset should be eating memory:

Adafruit_ICM20X/Adafruit_ICM20X.cpp

Lines 150 to 166 in 6ec93fe

	void Adafruit_ICM20X::reset(void) {
	_setBank(0);
	Adafruit_BusIO_Register pwr_mgmt1 = Adafruit_BusIO_Register(
	i2c_dev, spi_dev, ADDRBIT8_HIGH_TOREAD, ICM20X_B0_PWR_MGMT_1, 1);
	Adafruit_BusIO_RegisterBits reset_bit =
	Adafruit_BusIO_RegisterBits(&pwr_mgmt1, 1, 7);
	reset_bit.write(1);
	delay(20);
	while (reset_bit.read()) {
	delay(10);
	};
	delay(50);
	}

Thanks for the quick reply @caternuson. Unfortunately, when I call reset() in the loop without begin(), the the data output of the sensors is only zeroes. Again, as with the other sensor, I am turning this sensor on and off each time to economize battery life.

[gmanrc]

I tried both the reset embedded with icm20x.h and the reset in icm20948.h. The first causes the device to hang. The latter causes the sensors to output zeroes.

[caternuson]

The code in _init is likely needed to be called also, to re-establish the basic sensor config. But that's private, so not accessible in user sketch. It's also where the unchecked allocations are occurring:

Adafruit_ICM20X/Adafruit_ICM20X.cpp

Lines 207 to 210 in 6ec93fe

	temp_sensor = new Adafruit_ICM20X_Temp(this);
	accel_sensor = new Adafruit_ICM20X_Accelerometer(this);
	gyro_sensor = new Adafruit_ICM20X_Gyro(this);
	mag_sensor = new Adafruit_ICM20X_Magnetometer(this);

So seems like like maybe this is not possible with the current state of the library.

[gmanrc]

Thanks for identifying this @caternuson. I'll monitor this space for updates to the library, as having this all in sensor is great.
But I will need to adjust my hardware for now to multiple sensors to capture the same data and be compliant with my battery saving measures.