drivers: bluetooth: Rename Silabs HCI driver #80155
Conversation
zephyrbot
added
platform: Silabs
zephyrbot
requested review from
hermabe,
HoZHel,
jerome-pouiller,
sjanc,
Thalley and
theob-pro
|
Added the DNM flag, since this actually depends on an update to CMakeLists.txt in the HAL tree. I think it's better to wait for #80148 to be merged first since then it's the main tree |
There was a problem hiding this comment.
The functions in this file are prefixed with slz_. Maybe we could rename them?
There was a problem hiding this comment.
The functions in this file are prefixed with
slz_. Maybe we could rename them?
Those functions are private to the driver, and I was trying to keep this PR as minimal as possible. The names can be changed at any point without affecting anything outside of the driver. What would you like the names to be? (I considered doing the rename but then dropped the idea since I couldn't decide on a nice and short prefix)
There was a problem hiding this comment.
You're right. I foresee a long discussion. Let's postpone that change.
Rename the Silabs HCI driver to hci_silabs_efr32.c to better indicate what hardware it supports. Also rename the associated devicetree binding and Kconfig options to be consistent with the new driver name. Signed-off-by: Johan Hedberg <johan.hedberg@silabs.com>
jhedberg
dismissed stale reviews from asmellby and jerome-pouiller
via
501e175
Rebased now and DNM label removed. |
Rename the Silabs HCI driver to hci_silabs_efr32.c to better indicate what hardware it supports. Also rename the associated devicetree binding and Kconfig options to be consistent with the new driver name.