samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

btfmcodec: Update with list_for_each_entry_safe API

Browse Source 
This changes updates list_for_each_entry API with
list_for_each_entry_safe API.

Change-Id: Ifeb684ba1407123aef5b5b922e843a8089a6236f
Signed-off-by: Balakrishna Godavarthi <quic_bgodavar@quicinc.com>
This commit is contained in:
Balakrishna Godavarthi
2023-08-29 15:31:19 +05:30
parent 7fb1282936
commit d1e8e16be6
1 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
btfmcodec/btfm_codec_interface.c
View File

@@ -273,7 +273,7 @@ void btfmcodec_wq_hwep_shutdown(struct work_struct *work)
wq_hwep_shutdown); wq_hwep_shutdown);
struct btfmcodec_data *btfmcodec = (struct btfmcodec_data *)btfmcodec_dev->btfmcodec; struct btfmcodec_data *btfmcodec = (struct btfmcodec_data *)btfmcodec_dev->btfmcodec;
struct list_head *head = &btfmcodec->config_head; struct list_head *head = &btfmcodec->config_head;
struct hwep_configurations *hwep_configs = NULL; struct hwep_configurations *hwep_configs = NULL, *tmp;
int ret = -1; int ret = -1;
int idx = BTM_PKT_TYPE_HWEP_SHUTDOWN; int idx = BTM_PKT_TYPE_HWEP_SHUTDOWN;
@@ -281,7 +281,7 @@ void btfmcodec_wq_hwep_shutdown(struct work_struct *work)
/* Just check if first Rx has to be closed first or /* Just check if first Rx has to be closed first or
* any order should be ok. * any order should be ok.
*/ */
list_for_each_entry(hwep_configs, head, dai_list) { list_for_each_entry_safe(hwep_configs, tmp, head, dai_list) {
BTFMCODEC_INFO("shuting down dai id:%d", hwep_configs->stream_id); BTFMCODEC_INFO("shuting down dai id:%d", hwep_configs->stream_id);
ret = btfmcodec_hwep_shutdown(btfmcodec, hwep_configs->stream_id, true); ret = btfmcodec_hwep_shutdown(btfmcodec, hwep_configs->stream_id, true);
if (ret < 0) { if (ret < 0) {
@@ -301,10 +301,10 @@ void btfmcodec_wq_hwep_shutdown(struct work_struct *work)
static int btfmcodec_delete_configs(struct btfmcodec_data *btfmcodec, uint8_t id) static int btfmcodec_delete_configs(struct btfmcodec_data *btfmcodec, uint8_t id)
{ {
struct list_head *head = &btfmcodec->config_head; struct list_head *head = &btfmcodec->config_head;
struct hwep_configurations *hwep_configs; struct hwep_configurations *hwep_configs, *tmp;
int ret = -1; int ret = -1;
list_for_each_entry(hwep_configs, head, dai_list) { list_for_each_entry_safe(hwep_configs, tmp, head, dai_list) {
if (hwep_configs->stream_id == id) { if (hwep_configs->stream_id == id) {
BTFMCODEC_INFO("deleting configs with id %d", id); BTFMCODEC_INFO("deleting configs with id %d", id);
list_del(&hwep_configs->dai_list); list_del(&hwep_configs->dai_list);
@@ -389,10 +389,10 @@ static int btfmcodec_dai_hw_params(struct snd_pcm_substream *substream,
bool btfmcodec_is_valid_cache_avb(struct btfmcodec_data *btfmcodec) bool btfmcodec_is_valid_cache_avb(struct btfmcodec_data *btfmcodec)
{ {
struct list_head *head = &btfmcodec->config_head; struct list_head *head = &btfmcodec->config_head;
struct hwep_configurations *hwep_configs; struct hwep_configurations *hwep_configs, *tmp;
bool cache_avb = false; bool cache_avb = false;
list_for_each_entry(hwep_configs, head, dai_list) { list_for_each_entry_safe(hwep_configs, tmp, head, dai_list) {
cache_avb = true; cache_avb = true;
break; break;
} }
@@ -405,9 +405,9 @@ static int btfmcodec_check_and_cache_configs(struct btfmcodec_data *btfmcodec,
int id, uint8_t codectype) int id, uint8_t codectype)
{ {
struct list_head *head = &btfmcodec->config_head; struct list_head *head = &btfmcodec->config_head;
struct hwep_configurations *hwep_configs; struct hwep_configurations *hwep_configs, *tmp;
list_for_each_entry(hwep_configs, head, dai_list) { list_for_each_entry_safe(hwep_configs, tmp, head, dai_list) {
if (hwep_configs->stream_id == id) { if (hwep_configs->stream_id == id) {
BTFMCODEC_WARN("previous entry for %d is already available", BTFMCODEC_WARN("previous entry for %d is already available",
id); id);
@@ -641,13 +641,13 @@ void btfmcodec_wq_hwep_configure(struct work_struct *work)
wq_hwep_configure); wq_hwep_configure);
struct btfmcodec_data *btfmcodec = (struct btfmcodec_data *)btfmcodec_dev->btfmcodec; struct btfmcodec_data *btfmcodec = (struct btfmcodec_data *)btfmcodec_dev->btfmcodec;
struct list_head *head = &btfmcodec->config_head; struct list_head *head = &btfmcodec->config_head;
struct hwep_configurations *hwep_configs = NULL; struct hwep_configurations *hwep_configs = NULL, *tmp;
int ret; int ret;
int idx = BTM_PKT_TYPE_HWEP_CONFIG; int idx = BTM_PKT_TYPE_HWEP_CONFIG;
uint32_t sample_rate, direction; uint32_t sample_rate, direction;
uint8_t id, bit_width, codectype, num_channels; uint8_t id, bit_width, codectype, num_channels;
list_for_each_entry(hwep_configs, head, dai_list) { list_for_each_entry_safe(hwep_configs, tmp, head, dai_list) {
id = hwep_configs->stream_id; id = hwep_configs->stream_id;
sample_rate = hwep_configs->sample_rate; sample_rate = hwep_configs->sample_rate;
bit_width = hwep_configs->bit_width; bit_width = hwep_configs->bit_width;