Hi.
Takashi Iwai wrote:
I applied the patch now as is, as I suppose you tested it well :)
Well, no! :) I only tested it on 2 machines yet. Please give it more testing if you can.
I don't remember exactly the changes for pcsp driver right now, but the change of the initial delay was due to the change of start logic. At least, at the time HRTIMER_CB_IRQSAFE was removed, starting with zero didn't work at all on my machine. Maybe something got fixed in the core side.
OK, it would be nice if you can check whether or not the problem is there on that particular machine...
But, I still don't understand why it has to be zero. The first bit-flip is done inside the trigger-start, so the next wakeup should be after the calculated ns. Isn't it?
That was the logic you invented, but initially, and with my patch, there is no bitflip on start trigger. Or, at least, not supposed to be - there was only a timer mode setup, or if not - that was a bug. :) Also, you added some caching variables:
+ unsigned int fmt_size; + unsigned int is_signed;
What was the intention? Is this a speed-up? If not - reverting that too?