On Thu, Oct 01, 2015 at 03:37:53PM -0500, Andrew F. Davis wrote:

+static const struct of_device_id tps65912_regulator_of_match_table[] = {
+	{ .compatible = "ti,tps65912-regulator", },
+	{ /* sentinel */ },
+};
+MODULE_DEVICE_TABLE(of, tps65912_regulator_of_match_table);

Does this IP block exist outside of the tps65912?  If not why is it
directly represented in DT?  It seems like this is describing how Linux
loads drivers not how the hardware is constructed but DT should describe
the hardware.