search

PaulStoffregen / USBHost_t36

USB Host Library for Teensy 3.6 and 4.0
165 stars 85 forks source link

Bt multi merge #38

Closed KurtE closed 3 years ago

KurtE commented 4 years ago

@PaulStoffregen @mjs513 (and @defragster):

Paul and Mike and Tim,

I was wondering if it makes sense to go ahead and merge in some USB Host WIP Bluetooth code, that we started adding additional Bluetooth support. We did this work, close to a year ago, that now has some support for a Bluetooth object to be able to support more than one device connected to it. I think I default with a max of 4.

It is probably far from perfect, but back then I had it working with at least two devices at a time. For example a bluetooth Mouse and a bluetooth Keyboard. Likewise with a joystick. I believe there may still be some issues with multiple PS3s, but single one works.

Also over the last year there are others up on forum who have worked off of using my WIP branch.

So recently I logically rebased the changes off of current released stuff (I know that there is probably a real way to get rebase to work, but... I did it by going back to Master branch and then comparing branches and merging the code in...

Anyway I thought I would see what you think.

Note: Long term it would be great to rework some of the Bluetooth code to make things like Pairing not work sort of like the Host Shield code base, where you build the sketch saying you will start off wanting to pair , but instead probably have some API that the code can call (like if someone presses a logical button)... But I am not sure if/when this might get done.

Thanks Kurt

mjs513 commented 4 years ago

@PaulStoffregen @mjs513 (and @Defragster):

Paul and Kurt and Tim,

Think incorporation of the changes we made over the past year or so would be a great. As you said a lot of folks was working off the WIP branch anyway. As you say there is still room for improvements but that will come :)

Defragster commented 4 years ago

Would be cool to have the stuff tested in the T_4.0 timeframe integrated. Tested to work then and good advancement.

As far as the pairing a simple if(Serial.avaialble()) {...} prompt to pair would be better than running- recompiling- uploading. That would provide the example use for any user to add a pin to trigger the same in a single sketch.

PaulStoffregen commented 4 years ago

We're too close to 1.53 release to merge this now.

Shortly after 1.53, I want to merge this and mass storage, and a File base class and other much-needed things.