[GitHub] jnettlet commented on issue #291: nimble/ll: Add API to set controller public address

[GitBox <gi...@apache.org>]

jnettlet commented on issue #291: nimble/ll: Add API to set controller public address
URL: https://github.com/apache/mynewt-nimble/pull/291#issuecomment-454415014
 
 
   It seems backwards to me to have the hal_bsp calling into the nimble stack.  If you look at mynewt_core nothing else is calling any ble_ll functions.  I do think the bsp is the correct place for this initialization code to live.  Perhaps we can export a weak hal_bsp_get_public_addr() function that individual bsp packages can override with custom code?  Then let ble_hw_get_public_addr() call into that?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services