drivers: modem: add Quectel EG25-G

[ldenefle]

No description provided.

[ycsin]

Commit should have message and sign-off

[bbilas]

Would be not better if we put all of those definitions into separate headers? I mean something like

...snip...
#if DT_HAS_COMPAT_STATUS_OKAY(quectel_eg25_g)
#include "quectel_eg25_g.h"
#endif

#if DT_HAS_COMPAT_STATUS_OKAY(quectel_egXX)
#include "quectel_egXX.h"
#endif
...snip...

(or use some macrology)
where quectel_eg25_g.h would contain:

MODEM_CHAT_SCRIPT_CMDS_DEFINE(quectel_eg25_g_init_chat_script_cmds,
			      MODEM_CHAT_SCRIPT_CMD_RESP("ATE0", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CFUN=4", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CMEE=1", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CREG=1", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGREG=1", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CEREG=1", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CREG?", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CEREG?", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGREG?", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGSN", imei_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGMM", cgmm_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP_NONE("AT+CMUX=0,0,5,127,10,3,30,10,2",
							      0));

MODEM_CHAT_SCRIPT_DEFINE(quectel_eg25_g_init_chat_script, quectel_eg25_g_init_chat_script_cmds,
			 abort_matches, modem_cellular_chat_callback_handler, 10);

MODEM_CHAT_SCRIPT_CMDS_DEFINE(quectel_eg25_g_dial_chat_script_cmds,
			      MODEM_CHAT_SCRIPT_CMD_RESP_MULT("AT+CGACT=0,1", allow_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CGDCONT=1,\"IP\","
							 "\""CONFIG_MODEM_CELLULAR_APN"\"",
							 ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP("AT+CFUN=1", ok_match),
			      MODEM_CHAT_SCRIPT_CMD_RESP_NONE("ATD*99***1#", 0),);

MODEM_CHAT_SCRIPT_DEFINE(quectel_eg25_g_dial_chat_script, quectel_eg25_g_dial_chat_script_cmds,
			 dial_abort_matches, modem_cellular_chat_callback_handler, 10);

[bjarki-trackunit]

@bbilas We definitely need to separate it out somehow, I still don't know exactly what the best solution is though, and it should be its own PR I think :)

[ldenefle]

rebased and added periodic chat scripts

[ycsin]

Is it possible to generalize Quectel chat scripts? They look pretty similar, EC21 and its variants would be using the same init process as well

[ldenefle]

They all look very similar tbf, I think there's a potential for a refacto there but should it be done in a separate PR ?
I also don't have other quectel modems with me so won't be able to test any changes.

[ldenefle]

Hello, what's the update on the status of this PR ?
I could potentially merge the quectel init scripts but I could also do this in a separate PR, I'd like to get this one merged :)

[bjarki-trackunit]

Sorry, forgot about it :) @rerickson1 @ycsin