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
> >>>
> >>>
>
>

Reply via email to