> When iio_get_acpi_device_name_and_data() fails, the ddata may be left > uninitialised. Initialise it to NULL. Would a summary phrase like “Assign NULL to ddata in isl29018_probe()” be helpful? How do you think about a null pointer check also with the local variable “name” instead? Regards, Markus