On 08/02/17 06:26 PM, Monk Liu wrote: > > + WREG32(temp + i, > + unique_indices[i] & 0x3FFFF); > + WREG32(data + i, > + unique_indices[i] >> 20); Use a single line in both cases (or fix the indentation of the second lines to align with the opening parens). -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer