Hey, On Mon, Feb 24, 2014 at 05:02:52PM -0800, Loc Ho wrote: > The completion of the RAM removal from shutdown is quite fast. As per > spec, the max time is 1ms but from the run-time code, it only take one > (1us) or two (2us) read for this to completed. An 1 ms hard delay is more > than 100 time slower. The code isn't that complex. If you strongly advise > that I should remove the while loop and use 1 ms, then okay. Otherwise, I > would prefer to keep the original code. The code is rather silly because as it's written the delay may be upto 100ms. If it's sitting in the hot path, using things like usleep_range() makes sense. In probe / shutdown paths, it's just an unnecessary distraction. If the timeout is as short as 1ms, it doesn't really matter whether the hardware can do it faster or not. It's just irrelevant. Please just write simple code. Thanks. -- tejun -- 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
