~mil/sxmo-tickets#616: 
sxmo_deviceprofile_xiaomi,beryllium.sh is broken!

the /usr/bin/sxmo_deviceprofile_xiaomi,beryllium.sh is broken, to get it working on both panel variants we need to replace:

export SXMO_MONITOR="0:0:Novatek_NT36XXX_Touchscreen"

with:

export SXMO_MONITOR="DSI-1"
Status
RESOLVED CLOSED
Submitter
~magdesign
Assigned to
No-one
Submitted
2 months ago
Updated
2 months ago
Labels
No labels applied.

~proycon 2 months ago

On Tue Feb 4, 2025 at 5:28 PM CET, ~magdesign wrote:

the /usr/bin/sxmo_deviceprofile_xiaomi,beryllium.sh is broken, to get it working on both panel variants we need to replace:

export SXMO_MONITOR="0:0:Novatek_NT36XXX_Touchscreen"

with:

export SXMO_MONITOR="DSI-1"

Thanks. Do we have users of both panel types to confirm this works? Which one are you on? I know @anjan has one too (I don't).

~sicelo 2 months ago

On Wed, Feb 05, 2025 at 07:48:01PM +0000, ~proycon wrote:

On Tue Feb 4, 2025 at 5:28 PM CET, ~magdesign wrote:

the /usr/bin/sxmo_deviceprofile_xiaomi,beryllium.sh is broken, to get it working on both panel variants we need to replace:

export SXMO_MONITOR="0:0:Novatek_NT36XXX_Touchscreen"

with:

export SXMO_MONITOR="DSI-1"

Thanks. Do we have users of both panel types to confirm this works? Which one are you on? I know @anjan has one too (I don't).

While I do not have these devices, it appears that magdesign is right here, since the variable is actually set to a touchscreen node, which is unlikely to be correct.

Looks like a simple typo.

~magdesign 2 months ago

i have the Tianma Panel. we can also remove the line completely and it still seems to work. when you compare with another sdm845 deviceprofile, like the OP6, the export SXMO_MONITOR="DSI-1" line works also there..

~proycon 2 months ago

On Wed Feb 5, 2025 at 9:12 PM CET, ~magdesign wrote:

i have the Tianma Panel. we can also remove the line completely and it still seems to work. when you compare with another sdm845 deviceprofile, like the OP6, the export SXMO_MONITOR="DSI-1" line works also there..

Yeah, I think you're right. I've prepared and submitted a patch with your fix.

~magdesign 2 months ago

thanks 😘

~anjan REPORTED CLOSED 2 months ago

magdesign referenced this ticket in commit b9821b9.

Register here or Log in to comment, or comment via email.