On 09/07/2017 03:42 PM, Gavin Jacobs wrote:
> Josh,
> Thanks for those clarifications. I have worked through the references and
> compared to the osmocom implementation of hackrf, and I agree with your
> assessment that VGA and AMP are in the wrong order. I did verify that
> changing bb_gain at run time would indeed turn the amp on and off, and that
> would be ok for the sink because bb_gain is not used; but for the source it
> is problematic because we need to change both bb_gain and rf_gain
> independently. I am working on the correction.
>
> I have never tried to do a "pull request" for any project, so I read the
> Hello-World tutorial on github. The suggested workflow is to create a
> branch, edit the files, then create a pull request for the project team,
> and then if it is approved the branch gets merged. Some other tutorials
> suggest making a fork/clone on my computer and then edit, etc. Which is the
> way to go? I only ask because I don't see any other branches and it appears
> that the only commits are on the master branch. Also, there has only been
> one other pull request in the project history, so I couldn't quite see how
> it's done.
>
> Thanks for your patience.
Cool np. So in general for a pull request:
1) from the soapyhackrf git hub page click "fork" to get a copy of the
repo on your personal github page
2) clone this project from your personal github page on to your PC
3) make any modifications and branches as desired and push to the "origin"
4) a pull request wizard option basically shows up on your personal
github fork
More info
https://help.github.com/articles/creating-a-pull-request/
-josh
PS Also, if you are interested in direct repo access, I can set that up
too. Either one works, just let me know your github username.