Hi a question On Mon, 12 Dec 2011, Tero Kristo wrote: So the patch description says: > From: R, Govindraj <govindraj.raja@xxxxxx> > > Add API to determine IO-PAD wakeup event status for a given > hwmod dynamic_mux pad. But the code does: > + for (i = 0; i < hmux->nr_pads; i++) { > + struct omap_device_pad *pad = &hmux->pads[i]; which is going to check all of the pads, not just the dynamic ones. So it seems to me that we need to decide whether this code should be testing all the pads, or just the dynamically remuxed ones. The same thing should be decided for the code in patch 1. Naïvely it seems to me that we want to test all of the pads in both patches 1 and 2, not just the dynamically remuxable ones. Comments? - Paul