steadfasterX / SALT

SALT - [S]teadfasterX [A]ll-in-one [L]G [T]ool
https://forum.xda-developers.com/t/tool-locked-unlocked-salt-the-lg-up-revolution-begins.3717864/
GNU Lesser General Public License v2.1
58 stars 20 forks source link

LM-G710 Connection and multiple errors #38

Open phodina opened 1 year ago

phodina commented 1 year ago

Hi, I've connected the LM-G710 (LG G7 ThinQ) SDM845 based device under the download mode and it somehow connected but there were multiple errors and in the end the feature for full backup did not work and threw more errors.

SALT_Full_Backup SALT_LM-G710TM salt.log

phodina commented 1 year ago

There is no requirement that mentions that the lsb_release should be installed, therefore the command fails.

F_LOG "OS (lsb): $(lsb_release -a)"

I'll prepare PR with check if the tool is installed.

phodina commented 1 year ago

This issue is more for lglaf but the exception should be catched if there's no USB connected.

Traceback (most recent call last):
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 738, in <module>
    main()
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 694, in main
    comm = autodetect_device(args.cr)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 581, in autodetect_device
    return USBCommunication(cr)
           ^^^^^^^^^^^^^^^^^^^^
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 386, in __init__
    raise RuntimeError("USB device not found")
RuntimeError: USB device not found

Will also prepare PR to fix this issue.

phodina commented 1 year ago

@steadfasterX Is this the normal behavior to get this response to any command? I saw this also on other LG phone with Mediatek chipset.

Hello, I am LAF. Nice to meet you.
phodina commented 1 year ago

Seems most of the issues are connected to the lglafng. Should we discuss it here or in the repo for the tool?

phodina commented 1 year ago

Here's the dmesg from the LAF as all other commands see to be disabled.

lm-g710-laf.log

@steadfasterX Is this the normal behavior to get this response to any command? I saw this also on other LG phone with Mediatek chipset.

Hello, I am LAF. Nice to meet you.
steadfasterX commented 1 year ago

@steadfasterX Is this the normal behavior to get this response to any command? I saw this also on other LG phone with Mediatek chipset.

Hello, I am LAF. Nice to meet you.

see: https://github.com/steadfasterX/lglaf/issues/14#issuecomment-1671725947

steadfasterX commented 1 year ago

This issue is more for lglaf but the exception should be catched if there's no USB connected.

Traceback (most recent call last):
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 738, in <module>
    main()
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 694, in main
    comm = autodetect_device(args.cr)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 581, in autodetect_device
    return USBCommunication(cr)
           ^^^^^^^^^^^^^^^^^^^^
  File "/home/cylon2p0/programs/lglafng/lglaf.py", line 386, in __init__
    raise RuntimeError("USB device not found")
RuntimeError: USB device not found

Will also prepare PR to fix this issue.

that must be handled in the lglaf repo not here