4 times Thread.Sleep(1) in Nrf24l01 Send ... why?

[jblahos]

Hello,

I was looking at this library (Iot.Device.Bindings), considering to use it with my raspberry pi project. I want to use the Nrf24l01 wrapper. But when I checked it out internally, I noticed that its Send() method has 4x Thread.Sleep(1) inside it. This would massively slow down my sending rate. I want my nrf to only send data, never receive anything, and I want to send data rapidly. So why is the Send implemented like this, switching back to receive after every send and having those sleeps?

Thread.Sleep(1);

I would suggest that in Send there should be simple write of the data and nothing else.

[krwq]

[Triage] This code was there from the beginning externally contributed - I'd recommend to try out if you can remove it or shorten significantly (ideally according to documentation). You can use https://github.com/dotnet/iot/blob/main/src/devices/Common/System/Device/DelayHelper.cs#L12 for that (for small delays consider also passing false to allowThreadYield argument)