ChanServ changed the topic of #msm8937-mainline to: Boot Linux on your MSM8917/37/40 and QM215 mobile! | GitHub: https://github.com/msm89x7-mainline | Logs: https://oftc.irclog.whitequark.org/msm8937-mainline | Bridged to #msm8937-mainline:kde.org on Matrix
<NekoCWD[m]> Audio on agassi works :D
<NekoCWD[m]> anyone here has qm215-nokia-weeknd ?
<NekoCWD[m]> <NekoCWD[m]> "Audio on agassi works :D" <- but mic doesnt
<barni2000[m]> <NekoCWD[m]> "anyone here has qm215-nokia-..." <- ig no
<NekoCWD[m]> It has audio works, but it uses another pm (without wcd). I think i need to move sound to device dt or make 89x7-sound.dtsi
<NekoCWD[m]> Because nokia dt wont build
<NekoCWD[m]> Now i can watch movies via nfs on my tablet:D
<barni2000[m]> wait let me check your PR
<barni2000[m]> so the issue is msm8937.dtsi cannot contain the dais
<NekoCWD[m]> 8917
<barni2000[m]> both
<NekoCWD[m]> Ok
<barni2000[m]> because it is related to pm codecs
<barni2000[m]> and pm can be different
<barni2000[m]> so we should put dais into the device dts
<barni2000[m]> voicemmode also would be an issue
<NekoCWD[m]> Ok, i will update my pr.
<NekoCWD[m]> Can't we make a msm89x7-audio.dtsi? As downstream
<barni2000[m]> maybe some common bases can be made for devices
<barni2000[m]> NekoCWD[m]: almost every device have different architecture
<NekoCWD[m]> But there are a lot of common parts
<NekoCWD[m]> like dais
<barni2000[m]> mm1-mm4 can be added to msm8917
<barni2000[m]> maybe only the codec part
<NekoCWD[m]> barni2000do you know, is there any way to increase charge current on smb1360?
<barni2000[m]> idk
<barni2000[m]> <NekoCWD[m]> "Now i can watch movies via nfs..." <- btw hw decode also can be enabled relative easily
<barni2000[m]> 1 struct should be filled
<barni2000[m]> +dt binding
<NekoCWD[m]> v4l2 venus?
<barni2000[m]> yep
<barni2000[m]> but i am not sure it worth it
<barni2000[m]> s/it//
<barni2000[m]> there are serious frame drops on msm8953 and freq scaling also not the best
<NekoCWD[m]> First for 8952
<NekoCWD[m]> for 8917 reg presets seems to be same as 8953
<NekoCWD[m]> but max hwload 352800 /* 1080p@30 + 720p@30 */
<NekoCWD[m]> seems that in 8953 wrong value. In downstream ther are <1044480>; /* 4096 x 2176 @ 30fps */
<NekoCWD[m]> Ahh. How pepoples calculate freq_table for venus?
<NekoCWD[m]> I was tried to start race in super tux kart and tablet just rebooted :D
<Luna[m]> has msm89x7 been merged to upstream? im gonna try on my santoni
<NekoCWD[m]> <Luna[m]> "has msm89x7 been merged to..." <- It was merged, but there are also [update draft](https://gitlab.postmarketos.org/postmarketOS/pmaports/-/merge_requests/6005) pending
ungeskriptet has joined #msm8937-mainline
<Luna[m]> Turns out I own ugg
<barni2000[m]> <NekoCWD[m]> "I was tried to start race in..." <- don't try it on 8917 it has same bug like 8916
<barni2000[m]> <Luna[m]> "Turns out I own ugg" <- also supported
<Luna[m]> bt audio works
<Luna[m]> that's neat
<barni2000[m]> ofc because bluetooth works :D
<barni2000[m]> earpiece, headphones could be enable easily on ugg/santoni
<barni2000[m]> s/enable/enabled/
<barni2000[m]> iirc santoni have smart amp for speaker
<barni2000[m]> i have not check ugg
<Luna[m]> is the main speaker the only problem
<barni2000[m]> idk i have not checked yet
<barni2000[m]> as i see speaker can be enabled with simple amp on ugg
<Luna[m]> <barni2000[m]> "earpiece, headphones could be..." <- how do you enable headphones
<barni2000[m]> as i have checked it will be very similar to prada
<NekoCWD[m]> Yea, seems to be very similar. Only gpio changes. Speaker amp: gpio 1
<NekoCWD[m]> * gpio 1, hedphones in: gpio 16, headphones switch gpio 25
<NekoCWD[m]> Maybe. I think when barni end with 8917 upstreaming i will send. But idk what to do with panel
<barni2000[m]> 8917 is upstreamed
<Andr[m]> panel can be skipped, but everything working with upstream kernel should be sent upstream
<NekoCWD[m]> Andr[m]: Do you have Huawei MediaPad T3 10? Look at my agassi/6.12-develop branch. There are working audio and quircky touchscreen
<barni2000[m]> Andr[m]: base should be upstreamed device dts not necessary
<barni2000[m]> i have upstreamed riva because 1 device was needed to be applied
<NekoCWD[m]> Andr[m]: Hmm. Than I will send it upstream today
<barni2000[m]> remove non upstreamed driver bindings from it and check the dt style again
<barni2000[m]> you can send panel to upstream btw
<NekoCWD[m]> Panel gets temprorary damage without dsi video. Effect like on old photos. I think it's not ready to upstream
<NekoCWD[m]> AndréDo you have huawei agassi? It will be nice, if you will test my changes
<Andr[m]> No, with msm8917 I only have a Motorola Moto G5 (cedric).
<barni2000[m]> <NekoCWD[m]> "Panel gets temprorary damage..." <- what about simple drm?
<barni2000[m]> i have upstreamed riva with simple drm
<NekoCWD[m]> simplefb works fine
<barni2000[m]> that is enough for upstreaming
<barni2000[m]> usb also will not work at upstream btw
<barni2000[m]> only if you can make a proper extcon
<barni2000[m]> * proper extcon config
<barni2000[m]> * unless if you can make a proper extcon config
<barni2000[m]> msm8916 have msm8916-modem-q6dsp.dtsi
<barni2000[m]> s/q6dsp/qdsp6/
<barni2000[m]> it contains the dais
<barni2000[m]> sound works fine on Moto E5 Plus except speaker
<barni2000[m]> it uses external codec for speaker
<veynamer[m]> barni2000: 917D
<barni2000[m]> ig that is the difference but is it a replacement display or original?
<barni2000[m]> <barni2000[m]> "it uses external codec for..." <- hm i am not sure anymore
<barni2000[m]> s/hm//
<barni2000[m]> it is the simplest architecture only using pm8937 codec
<veynamer[m]> I have a completely original redmi 5a
<veynamer[m]> Honestly, even the battery is a little swollen, but I haven't changed it yet.
<barni2000[m]> you could try reenable pinctrl for goodix
<barni2000[m]> it caused issues with 915L so if it is helping we should find out something
<veynamer[m]> I looked at the stock kernel 3.18 and it mentions my touchscreen
<barni2000[m]> btw in my rolex i have 917d and it is working fine
<barni2000[m]> goodix driver can handle more chip
<barni2000[m]> does not matter if the config says 917D
<barni2000[m]> so i also have 917D
<barni2000[m]> but both works fine for me
<barni2000[m]> maybe your fw is corrupted i had that with focaltech variant once
<barni2000[m]> solution was reinstalling the factory rom with flash_all.sh
<barni2000[m]> after first boot it was recvored
<barni2000[m]> I am struggling with pmi8950 charger i have replaced extcon logic to pretened usb always connected like what dummy-extcon do but it is still not working, usb gadget is falling
<barni2000[m]> if put back dummy extcon as an extcon for vbus what is doing the same usb working again
<barni2000[m]> i think i have find out
<barni2000[m]> it is working
<barni2000[m]> charger and fg should be built as a built-in
<barni2000[m]> maybe this caused the power supply issues with bq and cw
<barni2000[m]> otg works fine on moto :D
<barni2000[m]> so how about finalize audio stuff and merge it?
<NekoCWD[m]> <barni2000[m]> "so how about finalize audio..." <- Ok, than i need to make msm89x7-audio.dtsi and update my PR?
<barni2000[m]> pm8937-audio.dtsi
<barni2000[m]> or idk but msm89x7 cannot be upstreamd later i think
<NekoCWD[m]> Ah, ok
<barni2000[m]> msm8937-pm8937-audio.dtsi or ms8917- also would be fine
<NekoCWD[m]> ms?
<barni2000[m]> or msm8937-qdsp6.dtsi
<barni2000[m]> and it can be included for 8917 devices also
<NekoCWD[m]> barni2000[m]: Oh, I like that. Will update my pr, thanks
<barni2000[m]> btw i think you should remove your nick from your sign-off and please use status as a last property
<NekoCWD[m]> barni2000 Should I leave wcd codec as part of pm8937.dtsi?
<barni2000[m]> yes
<barni2000[m]> that is part of pm8937
<NekoCWD[m]> barni2000[m]: Also I should leave q6asmdai and q6afedai dais in 8937?
<barni2000[m]> idk i should check
<barni2000[m]> 8916 dtsi contains them
<barni2000[m]> so yes lets do the same way
<NekoCWD[m]> audio-routing should in device dts, I think
<NekoCWD[m]> barni2000what about tlmm cdc_pdm_lines*?
<NekoCWD[m]> nokia-weekend define it in dt, but it's the same ones as our
<NekoCWD[m]> s/weekend/weeknd/
<barni2000[m]> ig all 3 soc have the same
<barni2000[m]> but 8937 have wsa
<barni2000[m]> and 8917 dont
<barni2000[m]> these should be handled per device ig but you can put where you like
<barni2000[m]> we can clean up that later
<NekoCWD[m]> will put it in msm8937-qdsp6
<NekoCWD[m]> Should i include pm8937 in msm8937-qdsp6.dtsi?
<barni2000[m]> ig no but that means we should care about the order
<barni2000[m]> if you include it all msm8917 device should use pm8937 when as far as i know there can be come with pm8916 also
<barni2000[m]> btw haptic also can be enabled there is a driver in msm8953 repo what was come from sdm845 rep :D only small modifications are in msm8953 repo but they are needed
<NekoCWD[m]> Hehe, agassi uses gpio-vibrator for haptic :D
<barni2000[m]> i mean pmi8950
<barni2000[m]> none of the 8917 devices using that
<barni2000[m]> or very rare
<NekoCWD[m]> barni2000 updated PR, take a look please https://github.com/msm89x7-mainline/linux/pull/8
<NekoCWD[m]> Ah, commit author issue, sorry
<barni2000[m]> checkpatch reviewed it
<barni2000[m]> you don't have sources for agassi?
<NekoCWD[m]> I have
<barni2000[m]> how about fixed the thresholds
<NekoCWD[m]> barni2000which license i should use for -qdsp6.dtsi?
<barni2000[m]> NekoCWD[m]: BSD3
<NekoCWD[m]> barni2000/*do not support button 4, so modify value*/ in it. Do i need to also switch values?
<barni2000[m]> ig that is why it was set so high
<barni2000[m]> so yes
<NekoCWD[m]> Ok, both of my device have working audio with that PR
<NekoCWD[m]> Tested
<NekoCWD[m]> Is there any way to make auto-switching to headphones on connect?
<NekoCWD[m]> will test `load-module module-switch-on-connect`
<barni2000[m]> it was working before but it seems with pipewire migration or newer pa it is broken
<barni2000[m]> i should update mono/stereo also in the configs
<barni2000[m]> but it should be done at msm8953 and rebase our config repo also
<NekoCWD[m]> pactl load-module module-switch-on-port-available
<NekoCWD[m]> It works
<barni2000[m]> so pa config was modified
<NekoCWD[m]> barni2000Is output-low in tlmm means that it is low by default?
<barni2000[m]> idk, i have always copying tlmm from downstream source but some times they are bad there
<barni2000[m]> s/have/am/
<NekoCWD[m]> `Definition: The specified pins are configured in output mode, driven low.`
<NekoCWD[m]> I dont understand `driven low`
<NekoCWD[m]> If i add output-low to pradas speaker amp tlmm, it won't noise during boot.
<NekoCWD[m]> Yep, that works
<barni2000[m]> I have reviewed the PR