Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fixup] Fix Header Paths of oplus mtk Power Drivers #1

Open
wants to merge 1 commit into
base: oneplus/MT6893_R_11.0
Choose a base branch
from

Conversation

techyminati
Copy link

  • The kernel path for mtk power drivers should be ../drivers/misc/mediatek/ , but for some reasons, the path here is ../../../../kernel-4.14/drivers/misc/mediatek/ , this is probably due to oplus's build system ,
    it works fine in their enviroment, but for third party developer it throws error, , so lets just define its correct path & fix the error

Test: Error is fixed & kernel code compiles successfully.

* The kernel path for mtk power drivers should be ../drivers/misc/mediatek/ , but for some reasons, the path here is ../../../../kernel-4.14/drivers/misc/mediatek/ , this is probably due to oplus's build system ,
 it works fine in their enviroment, but for third party developer it throws error, this error is common seen in realme-kernel-oss kernels too, so lets just define its correct path

Test: Error is fixed & kernel code compiles successfully.

Change-Id: If9a2919b96fddc6892da50a5ed62bc8ed63971d7
Signed-off-by: techyminati <[email protected]>
@CyberJalagam
Copy link

Looks good

@sarthakroy2002
Copy link

Looks fine

@Maitreya29
Copy link

Built just now and it seems to be building fine, gg.

@neel0210
Copy link

neel0210 commented Aug 7, 2021

Noice!!

@risabhsharma71
Copy link

looks nice

@aaryanporwal
Copy link

Thank you!

@Beanow
Copy link

Beanow commented Oct 10, 2022

"OnePlus" years later. This is still / also a problem in the new kernel oneplus/mt6893_s_12.1_oneplus_nord2t.
I've cherry picked your commit as-is, just to make a point. 😿
Works fantastic, well to move on to the next compile error that is.

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.

8 participants