Missing error check of function sensor_sample_fetch() and sensor_channel_get()

[Hxinrong]

Hi,
As described in the API documentation, function sensor_sample_fetch() fetches a sample from the sensor and stores it in an internal driver buffer, returns 0 if successful, and returns negative errno code if failure. And the return value of function sensor_channel_get() has the same format. However, as shown in the following code, two missing error checks take place, sensor_sample_fetch() and sensor_channel_get().

zephyr/samples/sensor/mcp9808/src/main.c

Lines 89 to 91 in 21cac03

	sensor_sample_fetch(dev);
	sensor_channel_get(dev, SENSOR_CHAN_AMBIENT_TEMP, &temp);

And, I see, in the same file, the return values of the two functions are checked. See the following code, for example.

zephyr/samples/sensor/mcp9808/src/main.c

Lines 127 to 137 in 21cac03

	rc = sensor_sample_fetch(dev);
	if (rc != 0) {
	printf("sensor_sample_fetch error: %d\n", rc);
	break;
	}
	rc = sensor_channel_get(dev, SENSOR_CHAN_AMBIENT_TEMP, &temp);
	if (rc != 0) {
	printf("sensor_channel_get error: %d\n", rc);
	break;
	}

[github-actions]

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.