use tBDAddr in all places?

[gilbertjuly]

I noticed there is a data type called "tBDAddr" to describe Bluetooth device address.

Some API uses this data type to define its variable.

void ble_host_connect(tBDAddr bdaddr);

However, not all of the places use this, some of API still use uint8_t*, for example:

void ble_host_set_scan_param(uint8_t* own_address, uint8_t tx_power_level, uint16_t scan_interval);

Apparently, tBDAddr is a better choice compared with unit8_t * when referring to a specific data type. agree?

[junfenggeng]

we should name it as bdaddr_t to follow the conventions. beside, beaddr_t is better because BLE address has a type and a 6 bytes mac. So byte array is not complete.

[gilbertjuly]

same as tBleStatus, it is a low level data type, should not be directly used in app level.