search

StephenBlackWasAlreadyTaken / xDrip

Android Application that collects dex signals, allows calibrations, and uploads
GNU General Public License v3.0
209 stars 176 forks source link

bluetooth_ble: make HM10 with CC41 work #81

Closed raduiliescu closed 9 years ago

raduiliescu commented 9 years ago

set characteristic notify only on what we are interested of; just set notify, no need to modify descriptor

this is working fine with all my Hm10 modules that were previously not working; working fine with my HM11 module;

this is how the official android example app was implementing the read notification; most likely the HM10 had a strange firmeware and was not allowing to modify a characteristic.

christerjensen commented 9 years ago

Built xDrip with a HM-10 CC41. The StephenBlackWasAlreadyTaken code didn't work, but then I tried your code and it worked! Thanks.

StephenBlackWasAlreadyTaken commented 9 years ago

Yes, so it looks like one of my cc41s has extra discriptors and xdrip assumed it would be the same as the hm10 discriptors, its not, this does indeed fix it for that one. Can you point this at the experimental branch so I can get it merged in please (Or with your permission Ill pull it and point it there?)

StephenBlackWasAlreadyTaken commented 9 years ago

These changes are in Master now and are working great!! Sorry I take forever to do these!!