Re: [PATCH 3/6] libnvdimm: Add device-tree based driver

On Fri, Mar 23, 2018 at 1:12 AM, Oliver O'Halloran [off-list ref] wrote:

This patch adds peliminary device-tree bindings for the NVDIMM driver.

*preliminary

Currently this only supports one bus (created at probe time) which all
regions are added to with individual regions being created by a platform
device driver.

Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
---
I suspect the platform driver should be holding a reference to the
created region. I left that out here since previously Dan has said
he'd rather keep the struct device internal to libnvdimm and the only
other way a region device can disappear is when the bus is unregistered.

Hmm, but this still seems broken if bus teardown races region
teardown. I think the more natural model, given the way libnvdimm is
structured, to have each of these of_nd_region instances create their
own nvdimm bus. Thoughts?