i have added a variable to hold (i % 4) in ich_fill_data: unsigned int bite; /* offset of the current byte within a 32b word */
dunno if that makes it more or less readable? should i drop it again? besides that i think there is not much room left for improvement.
and i got rid of the return values (hwseq needs a fixup to work with this).