On 10/10/18 08:54, Giacinto Cifelli wrote:
---
  src/modem.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/modem.c b/src/modem.c
index 9e254482..74dbe7ad 100644
--- a/src/modem.c
+++ b/src/modem.c
@@ -729,6 +729,8 @@ static void sim_state_watch(enum ofono_sim_state new_state, 
void *user)
        case OFONO_SIM_STATE_READY:
                modem_change_state(modem, MODEM_STATE_OFFLINE);
+ ofono_lte_set_reg_info(modem);
+

modem_change_state(...OFFLINE) results in the post_sim() implementation being called.  The drivers that implement the lte atom all call ofono_lte_create() there.

Your .._set_reg_info function checks for the existence of the atom and does some work.  As such, why not just merge the contents of ofono_lte_set_reg_info() into the at_lte_probe function instead... from there, it's more obvious what's going on.

/Jonas


                /* Modem is always online, proceed to online state. */
                if (modem_is_always_online(modem) == TRUE)
                        set_online(modem, TRUE);

_______________________________________________
ofono mailing list
[email protected]
https://lists.ofono.org/mailman/listinfo/ofono

Reply via email to