On 11/17/2010 05:50 AM, Isaku Yamahata wrote:
+/* Returns 0 to walk children,>  0 to terminate walk,<  0 to skip walk. */

Shouldn't this be the other way round according to the code (< 0 to terminate, > 0 to skip children)?

+/* Returns > 0 if either devfn or busfn terminate walk, 0 otherwise. */

This should be "returns != 0". It can return a value > 0 if the toplevel callback asks to skip the children, < 0 if any callback (including the toplevel) asks to terminate the walk, 0 otherwise.

Paolo


Reply via email to