mer-hybris/ofono-binder-plugin
1
0
Fork 0
mirror of https://github.com/mer-hybris/ofono-binder-plugin synced 2025-11-19 06:55:34 +08:00
Code Issues Packages Projects Releases Wiki Activity

[util] Proper handling of the NR extension enum. JB#62994

Browse Source 
It is not ideal to try to handle two separate enums within one switch.
Handle those separately to avoid:

src/binder_util.c:282:5: warning: case value ‘23’ not in enumerated type ‘RADIO_PREF_NET_TYPE’ {aka ‘enum radio_pref_net_type’} [-Wswitch]
  282 |     case RADIO_PREF_NET_NR_ONLY:
      |     ^~~~
src/binder_util.c:285:5: warning: case value ‘24’ not in enumerated type ‘RADIO_PREF_NET_TYPE’ {aka ‘enum radio_pref_net_type’} [-Wswitch]
  285 |     case RADIO_PREF_NET_NR_LTE:
This commit is contained in:
Jussi Laakkonen
2025-08-07 12:21:30 +03:00
parent be1f94671d
commit 42f60e6709
1 changed files with 38 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

57
src/binder_util.c
View File

@@ -256,6 +256,40 @@ binder_pref_from_raf(
} }
} }
static
int
binder_pref_mask_internal(
RADIO_PREF_NET_TYPE_INTERNAL pref,
int none,
int gsm_mask,
int umts_mask,
int lte_mask,
int nr_mask)
{
switch (pref) {
case RADIO_PREF_NET_NR_ONLY:
return nr_mask;
case RADIO_PREF_NET_NR_LTE:
return lte_mask | nr_mask;
case RADIO_PREF_NET_NR_LTE_CDMA_EVDO:
case RADIO_PREF_NET_NR_LTE_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_WCDMA:
return umts_mask | lte_mask | nr_mask;
case RADIO_PREF_NET_NR_LTE_GSM_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_GSM:
case RADIO_PREF_NET_NR_LTE_CDMA_EVDO_GSM_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_GSM_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_CDMA_EVDO_GSM_WCDMA:
return gsm_mask | umts_mask | lte_mask | nr_mask;
}
return none;
}
static static
int int
binder_pref_mask( binder_pref_mask(
@@ -279,12 +313,6 @@ binder_pref_mask(
case RADIO_PREF_NET_LTE_CDMA_EVDO: case RADIO_PREF_NET_LTE_CDMA_EVDO:
return lte_mask; return lte_mask;
case RADIO_PREF_NET_NR_ONLY:
return nr_mask;
case RADIO_PREF_NET_NR_LTE:
return lte_mask | nr_mask;
case RADIO_PREF_NET_TD_SCDMA_GSM: case RADIO_PREF_NET_TD_SCDMA_GSM:
case RADIO_PREF_NET_GSM_WCDMA: case RADIO_PREF_NET_GSM_WCDMA:
case RADIO_PREF_NET_GSM_WCDMA_AUTO: case RADIO_PREF_NET_GSM_WCDMA_AUTO:
@@ -305,24 +333,15 @@ binder_pref_mask(
case RADIO_PREF_NET_TD_SCDMA_LTE_CDMA_EVDO_GSM_WCDMA: case RADIO_PREF_NET_TD_SCDMA_LTE_CDMA_EVDO_GSM_WCDMA:
return gsm_mask | umts_mask | lte_mask; return gsm_mask | umts_mask | lte_mask;
case RADIO_PREF_NET_NR_LTE_CDMA_EVDO:
case RADIO_PREF_NET_NR_LTE_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_WCDMA:
return umts_mask | lte_mask | nr_mask;
case RADIO_PREF_NET_NR_LTE_GSM_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_GSM:
case RADIO_PREF_NET_NR_LTE_CDMA_EVDO_GSM_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_GSM_WCDMA:
case RADIO_PREF_NET_NR_LTE_TD_SCDMA_CDMA_EVDO_GSM_WCDMA:
return gsm_mask | umts_mask | lte_mask | nr_mask;
case RADIO_PREF_NET_CDMA_ONLY: case RADIO_PREF_NET_CDMA_ONLY:
case RADIO_PREF_NET_EVDO_ONLY: case RADIO_PREF_NET_EVDO_ONLY:
case RADIO_PREF_NET_CDMA_EVDO_AUTO: case RADIO_PREF_NET_CDMA_EVDO_AUTO:
case RADIO_PREF_NET_INVALID: case RADIO_PREF_NET_INVALID:
return none; return none;
default:
return binder_pref_mask_internal((RADIO_PREF_NET_TYPE_INTERNAL)pref,
none, gsm_mask, umts_mask, lte_mask, nr_mask);
} }
DBG("unexpected pref mode %d", pref); DBG("unexpected pref mode %d", pref);