Skip to content
This repository has been archived by the owner on Feb 16, 2024. It is now read-only.

Add the alternate interface vendor.qti.qcril.am #2

Merged
merged 3 commits into from May 29, 2021

Conversation

Flohack74
Copy link
Member

Thats necessary because some devices use a vendor service called differently to do the job.

@UniversalSuperBox
Copy link
Member

Looks like you've messed up the commit information on these. Could you rewrite the commits to fix that?

@Flohack74 Flohack74 force-pushed the xenial_-_alternate-qti-interface branch from 8f888b0 to 4e66fe0 Compare May 26, 2021 18:33
@Flohack74
Copy link
Member Author

@UniversalSuperBox yeah its not ready to be merged. What do you mean exactly?

@Flohack74 Flohack74 force-pushed the xenial_-_alternate-qti-interface branch from 4e66fe0 to cb18eee Compare May 26, 2021 18:42
@UniversalSuperBox
Copy link
Member

The commits have no authorship information, or the information that's there is incorrect. But if you don't want this reviewed, please mark it as a draft. I'm going through all open PRs right now!

@Flohack74 Flohack74 changed the title Add the alternate interface vendor.qti.qcril.am WIP: Add the alternate interface vendor.qti.qcril.am May 26, 2021
@Flohack74 Flohack74 force-pushed the xenial_-_alternate-qti-interface branch 3 times, most recently from 9fd1dc3 to 6b0c453 Compare May 26, 2021 19:15
@Flohack74 Flohack74 force-pushed the xenial_-_alternate-qti-interface branch from 6b0c453 to 097be97 Compare May 26, 2021 19:24
@Flohack74 Flohack74 changed the title WIP: Add the alternate interface vendor.qti.qcril.am Add the alternate interface vendor.qti.qcril.am May 26, 2021
@Flohack74
Copy link
Member Author

So it matured quickly, ready for review ^^

Flohack74 pushed a commit that referenced this pull request May 26, 2021
Add dummy hw2_0 implementation.
Copy link
Member

@mardy mardy left a comment

Choose a reason for hiding this comment

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

Just a little one :-)

src/impl-qti.c Show resolved Hide resolved
(cherry picked from commit 7e5d70e)
@Flohack74 Flohack74 requested a review from NotKit May 27, 2021 15:24
@erfanoabdi
Copy link

looks good

Copy link
Collaborator

@NotKit NotKit left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@Flohack74 Flohack74 added this to In progress in OTA-18 via automation May 29, 2021
@Flohack74 Flohack74 merged commit 6cc52e1 into xenial May 29, 2021
OTA-18 automation moved this from In progress to QA May 29, 2021
@Flohack74
Copy link
Member Author

Note that I proposed this to upstream as well in mer-hybris#6 - whenever that gets merged I will take care of resolving the conflict and accept the upstream version.

@Flohack74 Flohack74 deleted the xenial_-_alternate-qti-interface branch May 29, 2021 17:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
5 participants