Paul Walmsley <paul@xxxxxxxxx> writes: > Hi Kevin, > > a couple of comments, now that i have the chance to look this over > closely. > > Probably we should require every powerdomain to have a voltagedomain. > This will avoid the problems we're having with the clocks, in which not > every clock has a clockdomain pointer. > > So I'd suggest splitting this patch up into two patches -- one to make the > changes the powerdomain structure, and the second to add the code into the > powerdomain.c. Between these two patches, we'd need patches to the > powerdomain data to add the voltagedomain names in. OK, Benoit and I talked earlier today and agreed on this as well. He has already generated the OMAP4 data as well, so split up, and drop the checks below as you suggest. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html