Shivaprasad G Bhat <sb...@linux.ibm.com> writes:
> + /* Create DT entries for cold plugged NVDIMM devices */
> + dimms = nvdimm_get_device_list();
> + for (; dimms; dimms = dimms->next) {
> + NVDIMMDevice *nvdimm = dimms->data;
> +
> + spapr_populate_nvdimm_node(fdt, offset, nvdimm);
> + }
> + g_slist_free(dimms);To free the whole list you'll need another variable in the loop above, right? Cheers
