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 Bandit Nano as RX #52

Merged
merged 4 commits into from
Jun 8, 2024
Merged

Add Bandit Nano as RX #52

merged 4 commits into from
Jun 8, 2024

Conversation

IamPete1
Copy link
Contributor

@IamPete1 IamPete1 commented Mar 10, 2024

This adds a target for Bandit Nano as a receiver. Based on in instructions from @JyeSmith here https://discord.com/channels/596350022191415318/1165186741716656230/1167958708249186314

It uses the TX pad and the "SP" pin on the header for the bay adapter.

20240310_025152

I was not able to get it working using the RX pad, any idea why that might be?

@IamPete1
Copy link
Contributor Author

Updated to RX 3 and TX 1 following advice from V-22 (originally 13 and 17), https://discord.com/channels/596350022191415318/1165186741716656230/1216238191795900477

Now works on the provided pads.

20240310_134135

@IamPete1 IamPete1 marked this pull request as ready for review March 10, 2024 13:43
@CapnBry
Copy link
Member

CapnBry commented May 3, 2024

I'm not sure we want to add "TX as RX" targets for every piece of hardware. The chance of a user seeing ".. Bandit Nano ... TX" on the list and flashing their hardware with it, but it being not what they wanted and now their module doesn't even work any more! is too much of an issue. Almost every piece of hardware is capable of being the other thing and would double the already overwhelming list size.

Maybe we can just have some sort of user wiki for posting hardware.json for RX as TX and TX as RX mods?

@IamPete1
Copy link
Contributor Author

IamPete1 commented May 3, 2024

I understand the concern. It did take me a fair bit of messing about to get this working so it would be a same to loose it. Maybe a new "developer" or "advanced" field in targets.json that would hide it for most.

JyeSmith added 2 commits June 8, 2024 13:34
Middle button of the joystick acts as a button.  BUT IS NOT A BOOT BUTTON!
@JyeSmith
Copy link
Member

JyeSmith commented Jun 8, 2024

I pushed a couple of commits to correct the LED and added a working button (middle press down on the joystick).

The Micro Bandit is also supported. If a user has issues with the Micro and receiving TLM from the FC, try moving the Rx wire to the smaller pad on the rear. The inline resistor (next to the pad) value appears to be too high on early versions. I think this was corrected on newer production runs.

image
image
image

@JyeSmith JyeSmith requested review from pkendall64 and wvarty June 8, 2024 04:07
@JyeSmith
Copy link
Member

JyeSmith commented Jun 8, 2024

I'm not sure we want to add "TX as RX" targets for every piece of hardware. The chance of a user seeing ".. Bandit Nano ... TX" on the list and flashing their hardware with it, but it being not what they wanted and now their module doesn't even work any more! is too much of an issue. Almost every piece of hardware is capable of being the other thing and would double the already overwhelming list size.

Maybe we can just have some sort of user wiki for posting hardware.json for RX as TX and TX as RX mods?

These Tx modules were pushed for by the Dev to act as high power receivers. And a number of users have actually been doing this recently. We should support these properly.

Copy link
Contributor

@pkendall64 pkendall64 left a comment

Choose a reason for hiding this comment

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

I'd like to find a better way to do this though.

i.e. have TX-as-RX settings in the RX Json file and perhaps have a "TX as RX" section in configurator/web-flasher in an advanced section.

Copy link
Contributor

@wvarty wvarty left a comment

Choose a reason for hiding this comment

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

What Jye said.

This is where we have been directing our "power users" (pardon the pun) for high power TLM receivers. LGTM for now, and look at the improvements that PK suggested as future enhancements to support the cataloguing in configurator.

@wvarty wvarty merged commit ea71de8 into ExpressLRS:master Jun 8, 2024
1 check passed
@CapnBry
Copy link
Member

CapnBry commented Jun 8, 2024

These Tx modules were pushed for by the Dev to act as high power receivers. And a number of users have actually been doing this recently.

Sounds good to me too then oop it already happened in the night 😂

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.

5 participants