RE: [RFC PATCH 1/3] radio-settings: add frequency band selection methods
Hi Lucas,
+enum ofono_radio_band_gsm {
+ OFONO_RADIO_BAND_GSM_ANY = 0,
+ OFONO_RADIO_BAND_GSM_850 = 1,
+ OFONO_RADIO_BAND_GSM_900P = 2,
+ OFONO_RADIO_BAND_GSM_900E = 3,
+ OFONO_RADIO_BAND_GSM_1800 = 4,
+ OFONO_RADIO_BAND_GSM_1900 = 5,
+};
+
+enum ofono_radio_band_umts {
+ OFONO_RADIO_BAND_UMTS_ANY = 0,
+ OFONO_RADIO_BAND_UMTS_850 = 1,
+ OFONO_RADIO_BAND_UMTS_900 = 2,
+ OFONO_RADIO_BAND_UMTS_1700AWS = 3,
+ OFONO_RADIO_BAND_UMTS_1900 = 4,
+ OFONO_RADIO_BAND_UMTS_2100 = 5,
+};
A minor coding style comment. Accodring to code style M11: Naming of
enums,
If the enum contents have values (e.g. from specification) then the
above
formatting you have used is OK. But as these values are not defined as
such
in the specification as such, then it should be,
enum ofono_radio_band_gsm {
OFONO_RADIO_BAND_GSM_ANY,
OFONO_RADIO_BAND_GSM_850,
OFONO_RADIO_BAND_GSM_900P,
OFONO_RADIO_BAND_GSM_900E,
OFONO_RADIO_BAND_GSM_1800,
OFONO_RADIO_BAND_GSM_1900,
};
enum ofono_radio_band_umts {
OFONO_RADIO_BAND_UMTS_ANY,
OFONO_RADIO_BAND_UMTS_850,
OFONO_RADIO_BAND_UMTS_900,
OFONO_RADIO_BAND_UMTS_1700AWS,
OFONO_RADIO_BAND_UMTS_19004,
OFONO_RADIO_BAND_UMTS_21005,
};
BR,
Rajesh