If we are in a situation where there are no prints from
libsamsung-ril at all with 'logcat -b radio', having it
print a log in all situations helps debugging why this
could be hapenning.

This way even if there is nothing in the logs, we at
least know it was supposed to print a message at startup.

This already happened with Android 11, where before the
commit 34fdca14f7fb1623f733b64954c781b7dcf520f5 ("Fix
logging on Android 11") we had nothing in the logs.

Knowing that it should print something already takes
out of the equasion silent failures without having to
read lot of code or make error prone jugements.

It is also more reliable than having to go through
all the error paths like it is done with the
commit daa706393f381002205e35ebaaa42a586ad91e52 ("RIL_Init:
Make sure there are prints for each error paths").

Signed-off-by: Denis 'GNUtoo' Carikli <gnu...@cyberdimension.org>
---
 samsung-ril.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/samsung-ril.c b/samsung-ril.c
index fe7a703..aaeedeb 100644
--- a/samsung-ril.c
+++ b/samsung-ril.c
@@ -1545,6 +1545,8 @@ const RIL_RadioFunctions *RIL_Init(const struct RIL_Env 
*env,
        unsigned int i;
        int rc;
 
+       RIL_LOGD("libsamsung-ril: Started from %s\n", __func__);
+
        if (env == NULL) {
                RIL_LOGE("%s: Aborting: Invalid RIL_Env (RIL_Env is NULL)",
                         __func__);
-- 
2.30.1

_______________________________________________
Replicant mailing list
Replicant@osuosl.org
https://lists.osuosl.org/mailman/listinfo/replicant

Reply via email to