Hi Phillip, On 5/21/22 23:56, Phillip Potter wrote:
Hi Pavel, I agree with you totally - we should change these semantics to reflect how the rest of the kernel generally does things. That said, that is a bigger patch set and I noticed the driver was broken before I read this thread, so I've submitted a patch already just to fix the breakage for now. Changing these semantics is a bigger patch/patchset and I wanted to get this out in the meantime - if you are looking at doing this conversion I will by all means leave that alone as no desire to tread on anyones toes :-)
I think, redoing more then 500 place will take quite a long time, but driver should be fixed ASAP.
So let's apply your patch and then someone (maybe me) will cook a patch. With regards, Pavel Skripkin
