> Move the caller's pointer back to match the change in the region's > start, > rather than alter a byte of the device tree's content. Maybe you could change the variable names too, so it would be harder to make such obvious coding mistakes? > - *p -= nextra; > + *pp -= nextra; Segher