Skip to content

Conversation

@Smankusors
Copy link

I can only validate the tm5p2_r63350_1080p_video, because it is the display my phone currently uses.

Please let me know if there's anything else I need to do regarding this PR.


OPTIONS=(-r vsn -r vsp)
PANELS=(
[ctc5p5_ili7807b_1080p_video]="asus,ctc5p5_ili7807b"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[ctc5p5_ili7807b_1080p_video]="asus,ctc5p5_ili7807b"
[ctc5p5_ili7807b_1080p_video]="asus,zs520kl-ili7807b-cts5p5"

Copy link
Author

@Smankusors Smankusors Aug 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wait, what is this for?

btw "5p2" is for ze520kl, while "5p5" is for ze552kl

and the lk2nd PR for these panels was merged a month ago, do I need to open another PR to them? 😵‍💫
msm8916-mainline/lk2nd#392

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can update lk2nd, than use ze552kl and ze520kl prefix and remove the post fixes

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or you can used postfixes without 5p5 and 5p2

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm... wdyt about this commit? looks good?
Smankusors/lk2nd@8f33f76

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes it seems fine

[ctc5p5_ili7807b_1080p_video]="asus,ctc5p5_ili7807b"
[tm5p5_r63350_1080p_video]="asus,tm5p5_r63350"
[tm5p2_r63350_1080p_video]="asus,tm5p2_r63350"
[boe5p2_ili7807b_1080p_video]="asus,boe5p2_ili7807b"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[boe5p2_ili7807b_1080p_video]="asus,boe5p2_ili7807b"
[boe5p2_ili7807b_1080p_video]="asus,zs520kl-ili7807b-boe5p2"

[tm5p5_r63350_1080p_video]="asus,tm5p5_r63350"
[tm5p2_r63350_1080p_video]="asus,tm5p2_r63350"
[boe5p2_ili7807b_1080p_video]="asus,boe5p2_ili7807b"
[txd5p5_nt35596_1080p_video]="asus,txd5p5_nt35596"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[txd5p5_nt35596_1080p_video]="asus,txd5p5_nt35596"
[txd5p5_nt35596_1080p_video]="asus,zs520kl-nt35596-txd5p5"

[tm5p2_r63350_1080p_video]="asus,tm5p2_r63350"
[boe5p2_ili7807b_1080p_video]="asus,boe5p2_ili7807b"
[txd5p5_nt35596_1080p_video]="asus,txd5p5_nt35596"
[lce5p5_otm1901a_1080p_video]="asus,lce5p5_otm1901a"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[lce5p5_otm1901a_1080p_video]="asus,lce5p5_otm1901a"
[lce5p5_otm1901a_1080p_video]="asus,zs520kl-otm1901a-lce5p5"

OPTIONS=(-r vsn -r vsp)
PANELS=(
[ctc5p5_ili7807b_1080p_video]="asus,ctc5p5_ili7807b"
[tm5p5_r63350_1080p_video]="asus,tm5p5_r63350"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[tm5p5_r63350_1080p_video]="asus,tm5p5_r63350"
[tm5p5_r63350_1080p_video]="asus,zs520kl-r63350-tm5p5"

PANELS=(
[ctc5p5_ili7807b_1080p_video]="asus,ctc5p5_ili7807b"
[tm5p5_r63350_1080p_video]="asus,tm5p5_r63350"
[tm5p2_r63350_1080p_video]="asus,tm5p2_r63350"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[tm5p2_r63350_1080p_video]="asus,tm5p2_r63350"
[tm5p2_r63350_1080p_video]="asus,zs520kl-r63350-tm5p2"

@barni2000 barni2000 merged commit f65d2b8 into msm8953-mainline:master Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants