Hi Aditya, Sending to destination with same address as source will result in sending msg to source. It will not go into the air.
Best Łukasz On Thu, 17 May 2018 at 12:21, Aditya Xavier <[email protected]> wrote: > Hi Łukasz, > > Sorry for the late reply. > > I created a test application for BLE + MESH co-existence, to test the > functionality and your patch. > > The following are my observations. > > 1. On enabling any of the following flags, MESH is unable to send / > receive messages don’t work. (Using mesh_model_send) > > BLE_ROLE_BROADCASTER: 1 > BLE_ROLE_PERIPHERAL: 1 > BLE_EXT_ADV: 1 > > 2. Incase using the same Node_Address on two devices, send/ receive > messages don't work. ( This might be as per protocol specifications, am not > aware) > > You can try the application to verify the same. > 1. Do remember the node_address value should be unique. You can > change the same in mesh_init.c > 2. You can enable BLE / mesh by changing the relevant value in > headers.h ( ble_enable and mesh_enable ) > > > Please let me know your findings and in case of any mistake on my part, > please let me know. > > Thanks for looking into it. > > Thanks, > Aditya Xavier. > > > > > > On 15-May-2018, at 7:59 PM, Łukasz Rymanowski < > [email protected]> wrote: > > > > Hi Aditya, > > > > On Sat, 12 May 2018 at 08:23, Aditya Xavier <[email protected]> wrote: > > > >> Hi Łukasz, > >> > >> Thanks for looking into it. > >> > >> As per the documentation, > >> > >> To make use of it application needs to set BLE_EXT_ADV and > >> BLE_MULTI_ADV_INSTANCES at least to 1. Mesh will use last available > >> advertising instance for PB-ADV and second to last for PB-GATT and > >> PROXY > >> > >> What would happen when the Application is not utilizing PB-ADV / > PB-GATT ? > >> i.e. provisioned Mesh Instance co-existing with a BLE instance ? > >> > > > > Ah this is error in the documentation. Instead of PB_ADV/GATT It should > be > > ADV bearer and GATT bearer. Actually also name of #defines > > BLE_ADV_PB_ADV/GATT_IDX is not good. Thanks for pointing it out. Would > you > > mind to send PR for this? > > > > In general, when BLE_EXT_ADV is enabled and application wants to use BLE > > instance and Mesh, then > > a) BLE_MULTI_ADV_INSTANCES shall be set at least to 1 if ADV bearer is > used > > b) BLE_MULTI_ADV_INSTANCES shall be set at least to 2 if ADV and GATT > > bearer is used > > > > > >> Trying to understand whether this would require me to enable PB-ADV / > >> PB-GATT because my application uses neither. > >> > > > > There should be no need for this. > > > > > >> > >> Similarly, please correct me if am wrong but I would need to set > >> BLE_MULTI_ADV_INSTANCES to 2 to have MESH and BLE coexist. > >> > > > > No, 1 is enough. It is because BLE_MULTI_ADV_INSTANCES stays for an > > additional instances. There is always one available. > > Actually I understand that name 'BLE_MULTI_ADV_INSTANCES' might be > > confusing. Hopefully decryption of this flag in nimble/syscfg.yml makes > it > > clear. > > > > > >> > >> Thanks, > >> Aditya Xavier. > >> > >> > > Best > > Łukasz > > > > > >>> On 10-May-2018, at 1:16 PM, Łukasz Rymanowski < > >> [email protected]> wrote: > >>> > >>> Hi, > >>> > >>> We found the issue which was related to latest controller changes. > >>> Basically controller does not allow now to mix legacy HCI with ext adv > >> HCI > >>> which of course is good. The PR > >>> https://github.com/apache/mynewt-nimble/pull/8 is fixed now. > >>> I removed RFC and I plan to merge it soon. > >>> > >>> @Aditya - feedback very welcome. > >>> > >>> \Łukasz > >>> > >>> On Mon, 7 May 2018 at 13:41, Łukasz Rymanowski < > >>> [email protected]> wrote: > >>> > >>>> Hi Aditya, > >>>> > >>>> Sorry for late answer. > >>>> > >>>> Could you please provide btmon logs along with console logs so we can > >> help > >>>> you to analyze what is going on? > >>>> Here is instruction how to get btmon logs: > >>>> https://www.codecoup.pl/blog/support-for-btmon-in-mynewt/ > >>>> > >>>> Thanks and best regards > >>>> Łukasz > >>>> > >>>> > >>>> On Fri, 27 Apr 2018 at 11:48, Aditya Xavier <[email protected]> > >> wrote: > >>>> > >>>>> Hi Łukasz, > >>>>> > >>>>> Any update on it? Were you able to check this ? > >>>>> > >>>>> Bt_mesh_model_send does not work after enabling BLE_EXT_ADV.. > >>>>> > >>>>> Or, can you give me a sample code where it works so that I can check > if > >>>>> there is something am doing wrong. > >>>>> > >>>>> Thanks, > >>>>> Aditya Xavier. > >>>>> > >>>>> > >>>>>> On 20-Apr-2018, at 3:56 PM, Aditya Xavier <[email protected]> > >> wrote: > >>>>>> > >>>>>> Hi Łukasz, > >>>>>> > >>>>>> Thanks, was able to build btshell + blemesh into nrf52832. > >>>>>> > >>>>>> I think I found an issue with regards to bt_mesh_model_send method. > >>>>>> > >>>>>> It seems bt_mesh_model_send is no longer working. > >>>>>> > >>>>>> In my test app, I have remove all ble code, and using only blemesh ( > >>>>> auto provisioning ) and send a message to another device over mesh > on a > >>>>> press of a button. > >>>>>> > >>>>>> When I disable BLE_EXT_ADV and BLE_MULTI_ADV_INSTANCES, it works. > >>>>>> > >>>>>> Can you try and confirm it works ? > >>>>>> > >>>>>> Thanks, > >>>>>> Aditya Xavier > >>>>>> > >>>>>>> On 19-Apr-2018, at 2:00 PM, Łukasz Rymanowski < > >>>>> [email protected] <mailto:[email protected] > >> > >>>>> wrote: > >>>>>>> > >>>>>>> Hi Aditya, > >>>>>>> > >>>>>>> I suggest to hack around flash map instead of removing code. > >>>>>>> > >>>>>>> I would do something like this (hopefully calculations are OK). > >>>>>>> > >>>>>>> +++ b/hw/bsp/nrf52dk/bsp.yml > >>>>>>> @@ -41,11 +41,11 @@ bsp.flash_map: > >>>>>>> FLASH_AREA_IMAGE_0: > >>>>>>> device: 0 > >>>>>>> offset: 0x00008000 > >>>>>>> - size: 232kB > >>>>>>> + size: 462kB > >>>>>>> FLASH_AREA_IMAGE_1: > >>>>>>> device: 0 > >>>>>>> - offset: 0x00042000 > >>>>>>> - size: 232kB > >>>>>>> + offset: 0x0007B800 > >>>>>>> + size: 2kB > >>>>>>> FLASH_AREA_IMAGE_SCRATCH: > >>>>>>> device: 0 > >>>>>>> offset: 0x0007c000 > >>>>>>> diff --git a/hw/bsp/nrf52dk/nrf52xxaa.ld > >> b/hw/bsp/nrf52dk/nrf52xxaa.ld > >>>>>>> index 9433e37fd..55e293da1 100644 > >>>>>>> --- a/hw/bsp/nrf52dk/nrf52xxaa.ld > >>>>>>> +++ b/hw/bsp/nrf52dk/nrf52xxaa.ld > >>>>>>> @@ -17,7 +17,7 @@ > >>>>>>> */ > >>>>>>> MEMORY > >>>>>>> { > >>>>>>> - FLASH (rx) : ORIGIN = 0x00008000, LENGTH = 0x3a000 > >>>>>>> + FLASH (rx) : ORIGIN = 0x00008000, LENGTH = 0x73800 > >>>>>>> RAM (rwx) : ORIGIN = 0x20000000, LENGTH = 0x10000 > >>>>>>> } > >>>>>>> > >>>>>>> > >>>>>>> Make sure to rebuild boot and app and then try. > >>>>>>> > >>>>>>> Best > >>>>>>> Łukasz > >>>>>>> > >>>>>>> On 19 April 2018 at 07:30, Aditya Xavier <[email protected] > >> <mailto: > >>>>> [email protected]> <mailto:[email protected] <mailto: > >>>>> [email protected]>>> wrote: > >>>>>>> > >>>>>>>> Hi Łukasz, > >>>>>>>> > >>>>>>>> PFA the app I used to test Mesh using multi advertising instances. > >>>>>>>> > >>>>>>>> I get the following error. > >>>>>>>> [ts=275343728ssb, mod=4 level=3] adv_send: Advertising failed: > err 3 > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> I had to comment out a lot of code to get it in a flash-able size. > >>>>>>>> > >>>>>>>> Thanks, > >>>>>>>> Aditya Xavier. > >>>>>>>> > >>>>>>>> > >>>>>>>> On 18-Apr-2018, at 1:13 PM, Aditya Xavier <[email protected] > >>>>> <mailto:[email protected]>> wrote: > >>>>>>>> > >>>>>>>> Hi Łukasz, > >>>>>>>> > >>>>>>>> Am using nrf52832, hence the problem of flash overflow. > >>>>>>>> > >>>>>>>> Would create a test app, using btshell + blemesh + the flags which > >> you > >>>>>>>> recommended, and test again. > >>>>>>>> > >>>>>>>> Thanks, > >>>>>>>> Aditya Xavier. > >>>>>>>> > >>>>>>>> On 18-Apr-2018, at 12:29 PM, Łukasz Rymanowski < > >>>>>>>> [email protected] <mailto: > [email protected] > >>>> > >>>>> wrote: > >>>>>>>> > >>>>>>>> Hi Aditya, > >>>>>>>> > >>>>>>>> If there is flash overflow consider removing some features from > the > >>>>>>>> configuration. > >>>>>>>> What HW are you using? We are running on nrf52840 > >>>>>>>> > >>>>>>>> BTW There is no special application. It is btshell plus those 4 > >> flags > >>>>> ( > >>>>>>>> BLE_EXT_ADV, BLE_MULTI_ADV_INSTANCES, BLE_MESH, BLE_MESH_SHELL) . > >> Of > >>>>>>>> course you need my PR. I did not test it personally, but I know > it > >>>>> worked > >>>>>>>> for Michal. > >>>>>>>> > >>>>>>>> Please share your target configuration, > >>>>>>>> > >>>>>>>> Best > >>>>>>>> Łukasz > >>>>>>>> > >>>>>>>> On 17 April 2018 at 12:04, Aditya Xavier <[email protected] > >>>>> <mailto:[email protected]> <mailto: > >>>>>>>> [email protected] <mailto:[email protected]> <mailto: > >>>>> [email protected] <mailto:[email protected]>> < > [email protected] > >>>>> <mailto:[email protected]> <mailto:[email protected] <mailto: > >>>>> [email protected]>>>>> wrote: > >>>>>>>> > >>>>>>>> Hi Łukasz, > >>>>>>>> > >>>>>>>> Been trying to join both blemesh_shell and bt_shell, but there is > a > >>>>>>>> problem of flash overflow. > >>>>>>>> > >>>>>>>> Created another app, which basically is btshell and some portions > of > >>>>>>>> blemesh, but that didn’t work. > >>>>>>>> > >>>>>>>> Is it possible for you to share a sample / test app ? > >>>>>>>> > >>>>>>>> Thanks, > >>>>>>>> Aditya Xavier. > >>>>>>>> > >>>>>>>> On 10-Apr-2018, at 1:09 PM, Łukasz Rymanowski < > >>>>>>>> > >>>>>>>> [email protected] <mailto: > [email protected] > >>> > >>>>> <mailto:[email protected] <mailto: > >>>>> [email protected]>>> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Hi Michał, Aditya, > >>>>>>>> > >>>>>>>> I just upload a new version of PR: > >>>>>>>> https://github.com/apache/mynewt-nimble/pull/8 < > >>>>> https://github.com/apache/mynewt-nimble/pull/8> < > >>>>> https://github.com/apache/mynewt-nimble/pull/8 < > >>>>> https://github.com/apache/mynewt-nimble/pull/8>> > >>>>>>>> It contains fixes for the problem mentioned above, however > solution > >> is > >>>>>>>> > >>>>>>>> bit > >>>>>>>> > >>>>>>>> different from what Michał suggested. > >>>>>>>> @MIchał, could you take a look? > >>>>>>>> > >>>>>>>> @Aditya, Could you be able to test it and give us a feedback on > >> this? > >>>>>>>> > >>>>>>>> Best > >>>>>>>> Łukasz > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> On 6 April 2018 at 14:08, Michał Narajowski < > >>>>>>>> > >>>>>>>> [email protected] <mailto: > [email protected] > >>> > >>>>> <mailto:[email protected] <mailto: > >>>>> [email protected]>>> > >>>>>>>> > >>>>>>>> wrote: > >>>>>>>> > >>>>>>>> Hi Aditya, > >>>>>>>> > >>>>>>>> Mesh is using Adv extensions under the hood if you have Łukasz's > >> patch > >>>>>>>> and enable BLE_EXT_ADV and set BLE_MULTI_ADV_INSTANCES to at least > >> 1. > >>>>>>>> Blemesh_shell has a command "init" which initializes mesh stack > and > >>>>>>>> starts advertising Unprovisioned Mesh Beacon. > >>>>>>>> > >>>>>>>> I tested this now and I noticed a bug. Here is a patch for that > bug: > >>>>>>>> https://pastebin.com/gbyX8H56 <https://pastebin.com/gbyX8H56> < > >>>>> https://pastebin.com/gbyX8H56 <https://pastebin.com/gbyX8H56>> > >>>>>>>> Please apply it on top of Łukasz's branch. > >>>>>>>> > >>>>>>>> Hope that helps. Let us know how it works for you. > >>>>>>>> > >>>>>>>> BR, > >>>>>>>> Michał > >>>>>>>> > >>>>>>>> 2018-04-06 11:09 GMT+02:00 Aditya Xavier <[email protected] > >>>>> <mailto:[email protected]> <mailto:[email protected] <mailto: > >>>>> [email protected]>>>: > >>>>>>>> > >>>>>>>> Hi Michał / Łukasz, > >>>>>>>> > >>>>>>>> I have been trying to understand the blemesh_shell, and I fail to > >>>>>>>> > >>>>>>>> understand how / where it is using the Advertisement extensions. > >>>>>>>> > >>>>>>>> > >>>>>>>> Basically, could you point me towards the difference if I need to > >>>>>>>> > >>>>>>>> implement, in order to use blemesh instead. > >>>>>>>> > >>>>>>>> > >>>>>>>> From what I gathered / understood after going through the code is > >> that > >>>>>>>> > >>>>>>>> blemesh_shell basically allows various functions to be triggered > >>>>> through > >>>>>>>> shell commands. > >>>>>>>> > >>>>>>>> > >>>>>>>> Thanks, > >>>>>>>> Aditya Xavier. > >>>>>>>> > >>>>>>>> > >>>>>>>> On 03-Apr-2018, at 3:54 PM, Michał Narajowski < > >>>>>>>> > >>>>>>>> [email protected] <mailto: > [email protected] > >>> > >>>>> <mailto:[email protected] <mailto: > >>>>> [email protected]>>> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Hi Aditya, > >>>>>>>> > >>>>>>>> Please set BLE_MESH: 1 and BLE_MESH_SHELL: 1 and you should be > able > >> to > >>>>>>>> use both btshell and mesh shell. Let us know how that works for > you. > >>>>>>>> [email protected] <mailto:[email protected]> <mailto:[email protected] > >>>>> <mailto:[email protected]>> <mailto:[email protected] <mailto: > >> [email protected]> > >>>>> <mailto:[email protected] <mailto:[email protected]>> <[email protected] > >>>>> <mailto:[email protected]> <mailto:[email protected] <mailto: > >> [email protected] > >>>>>>>>> > >>>>>>>> Best regards > >>>>>>>> Michał > >>>>>>>> > >>>>>>>> 2018-04-03 7:56 GMT+02:00 Aditya Xavier <[email protected] > >> <mailto: > >>>>> [email protected]> <mailto:[email protected] <mailto: > >>>>> [email protected]>> <mailto: > >>>>>>>> [email protected] <mailto:[email protected]> <mailto: > >>>>> [email protected] <mailto:[email protected]>> < > [email protected] > >>>>> <mailto:[email protected]> <mailto:[email protected] <mailto: > >>>>> [email protected]>>>>>: > >>>>>>>> > >>>>>>>> Hi Łukasz, > >>>>>>>> > >>>>>>>> Any pointers, as to what needs to be implemented from the > >>>>>>>> > >>>>>>>> blemesh_shell app ? > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> Thanks, > >>>>>>>> Aditya Xavier. > >>>>>>>> > >>>>>>>> On 02-Apr-2018, at 11:34 PM, Łukasz Rymanowski < > >>>>>>>> > >>>>>>>> [email protected] <mailto: > [email protected] > >>> > >>>>> <mailto:[email protected] <mailto: > >>>>> [email protected]>> <mailto: > [email protected] > >>>>> <mailto:[email protected]> <mailto: > >>>>> [email protected] <mailto:[email protected] > >> > >>>>>>>> <[email protected] <mailto: > >> [email protected]> > >>>>> <mailto:[email protected] <mailto: > >>>>> [email protected]>>>>> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Second thought > >>>>>>>> [email protected] <mailto:[email protected]> <mailto:[email protected] > >>>>> <mailto:[email protected]>> <mailto:[email protected] <mailto: > >> [email protected]> > >>>>> <mailto:[email protected] <mailto:[email protected]>> <[email protected] > >>>>> <mailto:[email protected]> <mailto:[email protected] <mailto: > >> [email protected] > >>>>>>>>> > >>>>>>>> Aditya, > >>>>>>>> Since I did not test it a lot, would it be possible to give us > >>>>>>>> > >>>>>>>> feedback how > >>>>>>>> > >>>>>>>> it works for you? > >>>>>>>> > >>>>>>>> Best > >>>>>>>> Lukasz > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> On Mon, Apr 2, 2018, 20:01 Łukasz Rymanowski < > >>>>>>>> > >>>>>>>> [email protected]> > >>>>>>>> > >>>>>>>> wrote: > >>>>>>>> > >>>>>>>> Hi Aditya. > >>>>>>>> > >>>>>>>> On Mon, Apr 2, 2018, 19:14 Aditya Xavier <[email protected]> > >>>>>>>> > >>>>>>>> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> Hi Łukasz, > >>>>>>>> > >>>>>>>> Is there anything special required to get this working along with > >>>>>>>> > >>>>>>>> BLE ? > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> For e.g. would the btshell app code for ADV_EXT work along with > >>>>>>>> > >>>>>>>> mesh with > >>>>>>>> > >>>>>>>> the provided patches ? > >>>>>>>> > >>>>>>>> > >>>>>>>> In addition to configuration mentioned in PR commit message, the > >>>>>>>> > >>>>>>>> btshell > >>>>>>>> > >>>>>>>> app would have to enable ble mesh and ble mesh shell (check > >>>>>>>> > >>>>>>>> blemesh_shell > >>>>>>>> > >>>>>>>> app for that) > >>>>>>>> > >>>>>>>> > >>>>>>>> Mesh and BLE seems to compile however, am currently unable to get > >>>>>>>> > >>>>>>>> Mesh > >>>>>>>> > >>>>>>>> working. > >>>>>>>> > >>>>>>>> Also, any reason why this was not accepted yet ? > >>>>>>>> > >>>>>>>> > >>>>>>>> People are busy with other stuff I guess. I think it will be > merged > >>>>>>>> eventually. > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> Thanks, > >>>>>>>> Aditya Xavier. > >>>>>>>> > >>>>>>>> > >>>>>>>> Best > >>>>>>>> Lukasz > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> On 20-Mar-2018, at 12:57 AM, Sterling Hughes < > >>>>>>>> > >>>>>>>> [email protected]> wrote: > >>>>>>>> > >>>>>>>> > >>>>>>>> +1 - this is great, thanks Lukasz! > >>>>>>>> > >>>>>>>> On 19 Mar 2018, at 6:36, Łukasz Rymanowski wrote: > >>>>>>>> > >>>>>>>> Hi All, > >>>>>>>> > >>>>>>>> I saw people asking around about possibility to advertise with > >>>>>>>> > >>>>>>>> non-mesh > >>>>>>>> > >>>>>>>> data while mesh is running on Mynewt. Well this is possible to > >>>>>>>> > >>>>>>>> do but > >>>>>>>> > >>>>>>>> of > >>>>>>>> > >>>>>>>> course it brings a lot of risk for mesh operations and > >>>>>>>> > >>>>>>>> especially > >>>>>>>> > >>>>>>>> for > >>>>>>>> > >>>>>>>> friendship scenario. However I went ahead and added support for > >>>>>>>> > >>>>>>>> this in > >>>>>>>> > >>>>>>>> Mynewt and I'm interested in how it works for you. > >>>>>>>> > >>>>>>>> So here is a PR which makes use of multi instances from BT5 > >>>>>>>> > >>>>>>>> Advertising > >>>>>>>> > >>>>>>>> extensions and basically allows you to create additional > >>>>>>>> > >>>>>>>> instances > >>>>>>>> > >>>>>>>> which > >>>>>>>> > >>>>>>>> contains non-mesh data. > >>>>>>>> > >>>>>>>> https://github.com/apache/mynewt-nimble/pull/8 > >>>>>>>> > >>>>>>>> Instructions on how to enable it you can find in the commit > >>>>>>>> > >>>>>>>> message. > >>>>>>>> > >>>>>>>> > >>>>>>>> Comments are welcome > >>>>>>>> > >>>>>>>> Best > >>>>>>>> Łukasz > >>>>> > >>>>> > >> > >> > >
