Bugfix/fix bluetooth bt tool gatt server moudle cb change to const

[huangyulong3]

Memory Optimization: Change the bluetooth bt_tool gatt server module to const for reduce the ram size.

bug:v/84971

Rootcause: To reduce RAM resource consumption, the s_dis_attr_db,
s_bas_attr_db, s_iot_attr_db, g_gatts_tables
in the bt_tools gatt server module must be marked with the const modifier.
This ensures it is compiled and linked into Flash memory instead of RAM.

[Copilot]

Pull request overview

This PR optimizes RAM usage in the Bluetooth stack by converting static data structures to const, which causes them to be stored in Flash memory instead of RAM. The PR addresses bug v/84971 by marking GATT attribute databases, command tables, callback structures, and other static configuration data as const across the Bluetooth tools and service stack.

Changes:

• Convert static GATT attribute databases, service databases, and command tables to const across tools modules
• Convert static callback structures to const in service stacks, profiles, IPC layers, and framework components
• Add necessary type casts when passing const data to APIs that don't accept const pointers

Reviewed changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tools/gatt_server.c	Made GATT attribute databases (s_dis_attr_db, s_bas_attr_db, s_iot_attr_db) and command table (g_gatts_tables) const
tools/gatt_client.c	Made GATT client command table and callback structure const
tools/bt_tools.c	Made command tables (g_cmd_tables, g_set_cmd_tables, g_get_cmd_tables, g_pair_cmd_tables) and option structures const
tools/adv.c	Made advertising command table, callback structure, and option structures const
tools/a2dp_source.c	Made A2DP command table const
tools/async/*.c	Applied same pattern to async versions of scan, log, gatt_client, gap, and adv modules
service/stacks/zephyr/sal_*.c	Made callback structures const in SPP, HID, HFP, AVRCP, adapter, and A2DP interface files
service/profiles/*.c	Made profile interface structures const in SPP and HID device services
service/ipc/socket/src/*.c	Made callback structures const in socket IPC layer
framework/btwrap/async/bt_gatt_feature.c	Made GATT client callback structure const
feature/feature_async/src/bluetooth_ble_impl.c	Made advertiser and GATT client callback structures const

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Casting away const to work around API signature mismatch is a code smell. While functionally safe if the API only reads the data, this violates const correctness principles. The proper fix would be to update gatt_srv_db_t.attr_db to accept 'const gatt_attr_db_t*', but that would require broader API changes. Consider filing a technical debt issue to properly fix the API signatures to accept const pointers.

[Copilot]

These forward declarations for le_conn_add and le_conn_find are unused. They have no implementations in this file and are not called anywhere. These should be removed as they are dead code.

Suggested change

	static le_conn_info_t* le_conn_add(const bt_address_t* addr);
	static le_conn_info_t* le_conn_find(const bt_address_t* addr);