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
_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono
