Yes, thank you.Plain 0 would be enoughAgreed.
checkpatch should not be an excuse for crazy code... and this one isIndeed a sleep is needed but checkpatch complains about it so usleep_range() was added.+val |= ASTINTX;This is crazy. Either you need exact timing or you don't, but I don't
Sleep has nowhere near microsecond accuracy. If you can tolerate
bigger delay, feel free to specify reasonable range. But in this case
you probably want "as close as possible to 1msec", and no,
usleep_range is not right primitive for that.
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany