Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add bias tee check box on device settings #854

Merged
merged 8 commits into from
May 16, 2021

Conversation

sophiekovalevsky
Copy link
Contributor

@sophiekovalevsky sophiekovalevsky commented Mar 25, 2021

This tool is insanely amazing =) congrats on all the effort to pull all this off like a pro.

This PR implements a feature that is was requested way back (#703) however, sadly at my lab I only have bladeRF v1 generation. The implementation should work setting up both bias tee on rx and tx side.

Being said that, @jopohl there is any chance that you could have both bias tee module to test this locally at your lab?

fix #703

@CLAassistant
Copy link

CLAassistant commented Mar 25, 2021

CLA assistant check
All committers have signed the CLA.

@jopohl
Copy link
Owner

jopohl commented Mar 27, 2021

Thanks Kiara and great job getting started with that PR!

Unfortunately, I have no BladeRF v2 to test here, this is the main thing that always hold me back from implementing this myself. Maybe someone from the community can jump it? Would be great if we could find someone to test this.

@sophiekovalevsky
Copy link
Contributor Author

@jopohl thanks for taking a closer look at this. Let me announce this into the community to see if someone can help me to achieve this. Otherwise, I'm guessing this wouldn't be able to be merged, right?

@kaipyroami
Copy link

I have a v2 xA9, BT-200, and BT-100. I'm wiling to help if I can.

src/urh/dev/native/lib/cbladerf.pxd Outdated Show resolved Hide resolved
@sophiekovalevsky
Copy link
Contributor Author

I have a v2 xA9, BT-200, and BT-100. I'm wiling to help if I can.

That's awesome @kaipyroami. Would you mind to test the latest commit on this branch? And verify the following,

With BT-200

  1. Go to File -> Sniff Protocol -> Tick Bias Tee
  2. Hit Start button

Does the green led on BT-200 is on?

With BT-100

  1. Go to File -> Open -> Select your the waveform file
  2. Under interpretation -> Hit "Play" button within Complex Signal.
  3. Tick Bias Tee

Does the blue led on BT-100 is on?

@kaipyroami
Copy link

Ok, so all tests passed

I have a v2 xA9, BT-200, and BT-100. I'm wiling to help if I can.

That's awesome @kaipyroami. Would you mind to test the latest commit on this branch? And verify the following,

With BT-200

  1. Go to File -> Sniff Protocol -> Tick Bias Tee
  2. Hit Start button

Does the green led on BT-200 is on?

With BT-100

  1. Go to File -> Open -> Select your the waveform file
  2. Under interpretation -> Hit "Play" button within Complex Signal.
  3. Tick Bias Tee

Does the blue led on BT-100 is on?

All tests passed per this procedure. Green and blue LEDs function as expected with the exception of needing to toggle the check boxes sometimes. Might just be a GUI issue but I'm not entirely sure.

When recording and leaving the bias-tee checkbox checked it will appear checked when opening the play window. The checkbox must be unchecked and then rechecked for it to take effect.

@sophiekovalevsky
Copy link
Contributor Author

@kaipyroami I was able to reproduce the issue you mentioned, and it was a GUI behavior indeed. Would you mind to test again with the new changes that I've pushed?

@jopohl let me know if I'm wrong but I would expect to maintain the same value when choosing bias tee for the entire session, i.e. if I open up Spectrum Analyzer, and then Record Signal for example. The fallback when another session is open would be False.

I've set an update in checking the bias tee value when signals are triggered so that this can be propagate to the device.

@kaipyroami
Copy link

Bias tee issue fixed.

@sophiekovalevsky
Copy link
Contributor Author

Bias tee issue fixed.

Amazing, thanks you again to test this out.

@kaipyroami
Copy link

Bias tee issue fixed.

Amazing, thanks you again to test this out.

Thank you for adding features to my SDR! Cheers!

src/urh/dev/native/lib/cbladerf.pxd Show resolved Hide resolved
src/urh/dev/native/lib/cbladerf.pxd Show resolved Hide resolved
src/urh/dev/native/lib/cbladerf.pxd Show resolved Hide resolved
src/urh/dev/native/lib/cbladerf.pxd Outdated Show resolved Hide resolved
src/urh/dev/native/lib/bladerf.pyx Outdated Show resolved Hide resolved
src/urh/dev/native/lib/bladerf.pyx Show resolved Hide resolved
src/urh/dev/native/lib/bladerf.pyx Show resolved Hide resolved
Copy link
Owner

@jopohl jopohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, great job! Did you test this again with your BladeRF after addressing the review?

@kaipyroami
Copy link

Looks good, great job! Did you test this again with your BladeRF after addressing the review?

I'll test it again with my device and report back.

@kaipyroami
Copy link

Looks good. No problems that I can see.

@sophiekovalevsky
Copy link
Contributor Author

Looks good. No problems that I can see.

Awesome. Thanks to test again @kaipyroami

@jopohl I tested again with my bladeRF x40 and I didn't find any issues.

@jopohl jopohl merged commit 3aa70db into jopohl:master May 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable/disable external LNAs on bladeRF
4 participants