You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's a problem using the AT command APDU with quectel modems. It seems to be related to the method of writing to the serial device.
@steely-glint has a fork which solves the problem. In driver/apdu/at.c, it replaces fprintf(fuart...) with possibly smaller sized write(uart_fd...). After merging this fork with the current upstream and reducing the maximum segment size ctx->es10x_mss, it works well on a Quectel modem. I haven't tested on another modem. Hopefully, this method of communicating through the serial device would also be compatible with other modems. (Or maybe an environment variable could be used to select the method.) It looks like the fork on its own does not work since it doesn't have the ability to reduce ctx->ex10x_mss from upstream.
There's a problem using the AT command APDU with quectel modems. It seems to be related to the method of writing to the serial device.
@steely-glint has a fork which solves the problem. In driver/apdu/at.c, it replaces fprintf(fuart...) with possibly smaller sized write(uart_fd...). After merging this fork with the current upstream and reducing the maximum segment size ctx->es10x_mss, it works well on a Quectel modem. I haven't tested on another modem. Hopefully, this method of communicating through the serial device would also be compatible with other modems. (Or maybe an environment variable could be used to select the method.) It looks like the fork on its own does not work since it doesn't have the ability to reduce ctx->ex10x_mss from upstream.
Would it be feasible to merge this upstream? If needed, I could provide a patch or PR with the merge conflicts resolved.
main...pipe:lpac-quectel:main#diff-d85497acc8400d3fa4f69b00a762a91dc21f1ef20bdacc9e37cf46218c529287R99
The text was updated successfully, but these errors were encountered: