Messages in this thread | | | Date | Tue, 31 Oct 2023 06:58:48 -0700 | Subject | Re: [PATCH 2/2] drivers: rtc: max31335: initial commit | From | Guenter Roeck <> |
| |
On 10/31/23 04:23, Miclaus, Antoniu wrote: [ ... ] >>> + if (status != MAX31335_STATUS1_DEFAULT) >>> + dev_err_probe(&client->dev, -EINVAL, >>> + "Unable to read from device.\n"); >>> + >> >> That is misleading. The device returned an unexpected status. >> I don't know if this really reflects a problem, but it is not >> "Unable to read from device". >> > > Since the device lacks an ID register, I found this as a suitable > replacement for checking that the communication with the > device actually works before the probe function finishes > successfully. >
But the register read was successful. Also, why would you want to not instantiate the driver if the chip runs on VBAT, if the oscillator is running, if VBAT is low, or if some interrupt is pending for some reason ?
Guenter
| |