[GitHub] [mynewt-nimble] andrzej-kaczmarek opened a new pull request #700: RFCLK refactor

[GitBox <gi...@apache.org>]

andrzej-kaczmarek opened a new pull request #700: RFCLK refactor
URL: https://github.com/apache/mynewt-nimble/pull/700
 
 
   This is quite an extensive change which addresses recent problems found with ll_rfclk implementation. Basically, as we added new features it was quite difficult to synchronize rfclk between different pieces of code so there were a lot of small fixes addressing concrete problems. I think this is a good opportunity to refactor this code and make it more robust and easier to follow. With new code it should no longer be a problem to figure out "what and where should I do with rfclk in my code?" since the API is not pretty simple and straightforward and the decision to enable/disable clock is in almost all cases done automagically inside rfclk code.
   
   Main change is 319fe8b, however there are some additional changes, mostly cleanups and improvements in scanner which I did when trying to figure out rfclk scenarios. Each change is described in commit message so I will not duplicate this here.
   
   Finally, I decided to change ll_rfclk to ll_rfmgmt since this is meant to manage RF state in general (in case we have more radio drivers in future) and not only XTAL as it does now on Nordic platforms.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services