OnePlusOSS / android_kernel_oneplus_mt6893

Other
43 stars 30 forks source link

USB make device panic on OnePlus Nord 2 5G OSS kernel #2

Closed Mashopy closed 2 years ago

Mashopy commented 3 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Also, I request for MediaTek kernel modules and cust.dtsi and dts for oplus6893_20817 oplus6893_20615 oplus6893_20827 and oplus6893_20831 to be generated and added at their respective directories (Need to be generated from Nord 2 5G BSP since MediaTek removed to possibility to do so on kernel itself since Android 11)

@OnePlusTomCheng

techyminati commented 3 years ago

Yeah @OnePlusOSSAdmin @OnePlusTomCheng , Please push Mtk kernel modules ( vendor/mediatek/kernel_modules ) They're part of kernel code too & are under GPL ( PS: realme oss released them too ) . Also push cust.dtsi for oplus6893_20817 as we can't generate them from kernel tree itself as @TheMalachite Elaborated. Thanks !

8vasa8 commented 3 years ago

@OnePlusOSSAdmin, @OnePlusTomCheng Please help us with this.

akshay7407 commented 3 years ago

@OnePlusOSSAdmin @OnePlusTomcheng please help us with this

heraldjose10 commented 3 years ago

@OnePlusOSSAdmin Please release the Mtk kernel modules

100rbhthakur commented 3 years ago

Please check. @OnePlusOSSAdmin @oneplusTomCheng

HyperX2mate commented 3 years ago

Help us with this

risabhsharma71 commented 3 years ago

@OnePlusOSSAdmin please look into it.

MeTT1S commented 3 years ago

Please check. @OnePlusOSSAdmin

Yatinmanjunath8 commented 3 years ago

Please look into this @OnePlusOSSAdmin

mp3399 commented 3 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Also, I request for MediaTek kernel modules and cust.dtsi for oplus6893_20817 to be generated and added at arch/am64/boot/dts/oplus6893_20817 (Need to be generated from Nord 2 5G BSP since MediaTek removed to possibility to do so on kernel itself since Android 11)

@OnePlusTomCheng

Please Fix This, Community is oneplus core audience, so don't ignore us

Mashopy commented 3 years ago

@oneplustomcheng It's already been 1 full month and not even an answer

Voldemort7273 commented 2 years ago

@OnePlusTomCheng why ignoring us? Being a reputated brand we do expect atleast a reply from you ...

techyminati commented 2 years ago

@OnePlusOSSAdmin @OnePlusTomCheng , Sorry to say but realme-kernel-opensource does better job than you guys, atleast they listen to users and fix sources if they're broken. While you guys dont.

sidxF commented 2 years ago

Please help us with this @OnePlusOSSAdmin , it's been more than a month, how can y'all be so careless , fix the sources please

8vasa8 commented 2 years ago

Hello @OnePlusOSSAdmin and @OnePlusTomCheng This is my first OP device that I bought and main reason was that kernel source was release. But I can see now how you don't care about your customers. I still hope you will help us but if not this will be my last OP device ever...

InfnteLights commented 2 years ago

bump! @OnePlusOSSAdmin

ayushrhll commented 2 years ago

@OnePlusOSSAdmin Please update the kernel sources for this device, it's been a month, why are you guys not updating the kernel source?

sidxF commented 2 years ago

When are you gonna fix the kernel please fix it, it's been two months

kitsune31 commented 2 years ago

@OnePlusOSSAdmin @OnePlusTomCheng Please update the kernel source for this device. It's been 2 months company is ignoring this issue. I hope OnePlus stays OnePlus.

karthikeyann commented 2 years ago

Requesting again @OnePlusOSSAdmin @OnePlusTomCheng @OnePlusOSS

Suru-web commented 2 years ago

We need custom rom support soon, please fix all the issues @OnePlusOSSAdmin

100rbhthakur commented 2 years ago

@OnePlusOSSAdmin please listen to us are we mad requesting aur you peoples are not even replying a word. Shame on OnePlus.

eastw1ng commented 2 years ago

How is it possible that OnePlus fails at holding up their standards. This problem should have been fixed long ago and the proper standard ROM should have been available on their website since the launch.

If they fail to support their phone like they usually do, it will be the end of their brand for the majority of the users. I truly hope they will fix these issues soon and bring the state of the Nord 2 to the standard we were used to!

aizen9876 commented 2 years ago

@OnePlusOSSAdmin listen to us, other brand's devices launched even after nord 2 are all getting good development and you can't even release it so the devs could move furthur now.

Suru-web commented 2 years ago

@OnePlusOSSAdmin guys please listen to us..

Suru-web commented 2 years ago

@OnePlusOSSAdmin guys we want good developement support.. first of all you are using color os code base now.. then u are not even fixing the kernal issue.. do something, we want custom rom support

aizen9876 commented 2 years ago

@OnePlusOSSAdmin please fix the issue by taking it some time , that would be really helpful for us.

techyminati commented 2 years ago

@OnePlusOSSAdmin you guys are rarted, Not fixing kernel sources even after tonns of spams, @realme-kernel-opensource & @oppo-source is much better than you guys, atleast oppo-oss and realme-oss teams listen to custom developer feedback and respond to it unlike oneplus. Oneplus is now what it was not mean to be, non developer friendly OEM, Shame!

sarthakroy2002 commented 2 years ago

@OnePlusOSSAdmin you guys are rarted, Not fixing kernel sources even after tonns of spams, @realme-kernel-opensource & @oppo-source is much better than you guys, atleast oppo-oss and realme-oss teams listen to custom developer feedback and respond to it unlike oneplus. Oneplus is now what it was not mean to be, non developer friendly OEM, Shame!

Exactly @realme-kernel-opensource published my secondary device kernel source. It was perfect without any issues. Also gave kernel modules which itself is a plus point. Oneplus lost its legacy of OSS support to developers. Shameful @OnePlusOSSAdmin .

aizen9876 commented 2 years ago

Tbh i was happy to choose one plus device over mi 11x or realme, they both have tons of roms Now and this doesn't even have one, such a Worst experience, i must say. Be Friendly to developers like mi and Learn from it @OnePlusOSSAdmin

sidxF commented 2 years ago

Please fix this kernel , we really need a custom rom , where's the OnePlus that was once developer friendly, you guys don't give a shit about customers anymore even realme devices have custom roms now and we don't and that's just bullshit, I hope you still care about your brand value and reputation, save it before it's long gone

kitsune31 commented 2 years ago

Shame Shame @OnePlusOSSAdmin @OneplusOSS @OnePlusTomCheng

karthikeyann commented 2 years ago

Taking this thread to twitter would get more eyes.

sidxF commented 2 years ago

Taking this thread to twitter would get more eyes.

True that we should take this to Twitter, pretty sure we'll get atleast some form of response there

100rbhthakur commented 2 years ago

Taking this thread to twitter would get more eyes.

Yes I think so...

aizen9876 commented 2 years ago

Guys, please Mention them here and Please tweet them as well, tweeting them would help us fix it faster rather than Posting it here.

wkcs commented 2 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Can you help me explain in detail your recurrence steps?

Thanks

Mashopy commented 2 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Can you help me explain in detail your recurrence steps?

Thanks

Just flashing the kernel builded from this repo and try to connect the device to PC with an USB cable make it panic, tried from recovery.

It should also happen by just connecting the device to a charger since dmesg reported kernel panic with power delivery (PD) drivers and MediaTek typec one

Mashopy commented 2 years ago

Also, the missing stuff I asked didn't get added in the latest commit from today

wkcs commented 2 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Can you help me explain in detail your recurrence steps?

Thanks

Just flashing the kernel builded from this repo and try to connect the device to PC with an USB cable make it panic, tried from recovery.

It should also happen by just connecting the device to a charger since dmesg reported kernel panic with power delivery (PD) drivers and MediaTek typec one

Can you help provide a dmesg log.

Thank you so much

Mashopy commented 2 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Can you help me explain in detail your recurrence steps? Thanks

Just flashing the kernel builded from this repo and try to connect the device to PC with an USB cable make it panic, tried from recovery. It should also happen by just connecting the device to a charger since dmesg reported kernel panic with power delivery (PD) drivers and MediaTek typec one

Can you help provide a dmesg log.

Thank you so much

dmesg.log It's from a TWRP build when I was building kernel from this repo, USB didn't panic with stock kernel however

Mashopy commented 2 years ago

Hello, I noticied that USB were making kernel panic with the current source code released by OnePlus, it would be really usefull if that's fixed soon.

Can you help me explain in detail your recurrence steps? Thanks

Just flashing the kernel builded from this repo and try to connect the device to PC with an USB cable make it panic, tried from recovery. It should also happen by just connecting the device to a charger since dmesg reported kernel panic with power delivery (PD) drivers and MediaTek typec one

Can you help provide a dmesg log. Thank you so much

dmesg.log It's from a TWRP build when I was building kernel from this repo, USB didn't panic with stock kernel however

console-ramoops.log Here the console-ramoops as well

techyminati commented 2 years ago

@wkcs you are from Oneplus Opensource Team? If yes please push MediaTek kernel modules & cust.dtsi .

karthikeyann commented 2 years ago

Submitted a bug https://forums.oneplus.com/threads/open-source-mediatek-kernel-modules-cust-dtsi.1529077/

NotKit commented 2 years ago

@TheMalachite I had to look into this for Ubuntu Touch port. The direct cause for kernel panic is uninitialized charger manager in mtk_charger driver. When checking further, battery controller did not get detected.

It seems OnePlus did s/vooc/wrap/g during preparing kernel source release for Nord 2. Compare https://github.com/OnePlusOSS/android_kernel_oneplus_mt6893/tree/oneplus/MT6893_R_11.0/drivers/power/oplus/wrap_ic with say https://github.com/realme-kernel-opensource/realmeGT_AndroidR-kernel-source/tree/master/drivers/power/oplus/vooc_ic. One side-effect is that it also breaks DT parsing, so it does not work with stock dtb/dtbo.

Did this as a workaround for now so it can work with both prebuilt and source-compiled DT: https://gitlab.com/ubports/community-ports/android11/oneplus-nord-2/kernel-oneplus-mt6893/-/commit/d14fa08727ed5f8f46c2f315825f04f7713089a9