Re: [PATCH 0/4] Get/Set forbidden operator list via SimManager

Hi Sean,

On 7/1/25 4:29 AM, Sean Nyekjaer wrote:
> IoT providers allow for enabling and disabling of operators.
> If the modem have detected an forbiddel operator it will block
> connections to that network.
> This will allow controlling software to clear that list.
> (Our IoT provider recommends to clear the list everytime we power off
> the modem).
> 
> Get:
> $ busctl call org.ofono /sim76xx_0 org.ofono.SimManager GetProperties
> [...]
> "PreferredLanguages" as 4 "da" "en" "de" "fr" "PinRequired" s "none"
> "Retries" a{sy} 4 "pin" 3 "pin2" 3 "puk"
>   10 "puk2" 10 "CardSlotCount" u 1 "ActiveCardSlot" u 1 "ForbiddenPLMN"
>   as 4 "23802" "23902" "32431" "331022"
> 
> $ busctl call org.ofono /sim76xx_0 org.ofono.SimManager SetProperty sv
> ForbiddenPLMN as 1 "23802"
> 
> Todo:
> I'm still missing to do continious updates of the list.
> Any idea to update it periodically?
> 
> Would also really like some feedback on the dbus interface
> 

FYI:

Fedora (glibc) clang debug+sanitizers
=====================================
Configure: PASS
Build: FAIL
     src/sim.c:881:24: error: format specifies type 'int' but the argument has 
type 'unsigned long' [-Werror,-Wformat]
       881 |                         DBG("%s %d", value, strlen(value));
           |                                 ~~          ^~~~~~~~~~~~~
           |                                 %lu
     ./include/ofono/log.h:53:34: note: expanded from macro 'DBG'
        52 |                 ofono_debug("%s:%s() " fmt, \
           |                                        ~~~
        53 |                                         __FILE__, __FUNCTION__ , ## 
arg); \
           |                                                                     ^~~
     1 error generated.
     make[1]: *** [Makefile:4090: src/sim.o] Error 1
     make[1]: Target 'all-am' not remade because of errors.
     make: *** [Makefile:2405: all] Error 2

Checkpatch Output
=================
0001-simutil-Add-Forbidden-operatator-list-PLMN-fileid.patch:4: WARNING: 
'operatator' may be misspelled - perhaps 'operator'?
0001-simutil-Add-Forbidden-operatator-list-PLMN-fileid.patch:7: WARNING: Missing 
commit description - Add an appropriate one
total: 0 errors, 2 warnings, 7 lines checked
0002-sim-Show-forbidden-operator-list-PLMN-with-GetProper.patch:8: WARNING: 
Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 109 lines checked
0004-doc-sim-api-document-ForbiddenPLMN.patch:7: WARNING: Missing commit 
description - Add an appropriate one
total: 0 errors, 1 warnings, 10 lines checked

Regards,
-Denis