On Fri, Mar 23, 2018 at 1:12 AM, Oliver O'Halloran <oohall@xxxxxxxxx> 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@xxxxxxxxx> > --- > 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? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
