libqmi-glib: use ((guint64) 1) << 31 for QMI_*_LTE_BAND_*_EUTRAN_32 (!61) · Merge requests · Mobile broadband connectivity / libqmi · GitLab

Snippets Groups Projects

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

The migration is almost done, at least the rest should happen in the background. There are still a few technical difference between the old cluster and the new ones, and they are summarized in this issue. Please pay attention to the TL:DR at the end of the comment.

Merged Dan Williams requested to merge dcbw/libqmi:band-32-shift into master 5 years ago

The current definitions of QMI_LTE_BAND_EUTRAN_32 get converted to a negative value (0xffffffff80000000) which causes side effects like the incorrect ModemManager to QMI band mapping in https://lists.freedesktop.org/archives/modemmanager-devel/2019-August/007371.html .

Replace 1 << 31 with ((guint64) 1) << 31 for QMI_LTE_BAND_EUTRAN_32 to avoid this.

Reported-by: Nick mips171@icloud.com Signed-off-by: Reinhard Speyerer rspmn@arcor.de

Activity

Dan Williams merged 5 years ago

merged

Please register or sign in to reply