08:02 <minecrell> bryanodonoghue: Is there something connected to pm8916_l11 and/or pm8916_l12 on apq8039-t2? This is normally used for SD card but I guess it doesn't have that?

09:59 <bryanodonoghue> minecrell yes I'm OOO atm will verify what the thing is on Monday

10:00 <minecrell> bryanodonoghue: thanks

10:00 <bryanodonoghue> anyway whatever it is - is not in the upstream dts so is probably safe to zap for the moment

10:02 <minecrell> bryanodonoghue: The reason I ask is that it would kind of go against my idea of having the common regulator voltages defined in msm8939-pm8916.dtsi if the standard regulators are reused for other purposes when unused

10:02 <minecrell> I've never seen this though

10:03 <bryanodonoghue> one or two values in the regulator part are a mystery to me

10:03 <bryanodonoghue> the downstream dts is not illuminating

10:03 <bryanodonoghue> I suspect the RPM firmware is just doing the right thing

10:04 <minecrell> bryanodonoghue: yeah the RPM firmware is really magic

10:06 <minecrell> bryanodonoghue: Basically, I would expect that pm8916_l11 and l12 are not connected to anything in the apq8039-t2 schematic (if you have it). If that is the case it should be fine to configure those with the typical SD card voltage and then just keep them unused so they get disabled

10:14 <bryanodonoghue> Sorry I think - l11 and l12 are used for the display

10:14 <bryanodonoghue> I don't have the schematic with me ATM

10:14 <bryanodonoghue> Let me check the wip dts for the display

10:14 <bryanodonoghue> ....

10:20 <minecrell> bryanodonoghue: thanks, no need to rush. Next week is fine as well :)

10:24 <bryanodonoghue> ah I see - no I don't think we need this for the t2a we use an eMMC

10:39 <minecrell> bryanodonoghue: so l11 and l12 are not used for the display or anything else?

10:45 <bryanodonoghue> Can't be sure until I check the schematic

10:47 <minecrell> bryanodonoghue: ok :)

10:47 <minecrell> bryanodonoghue: related question, is there a special reason you made pm8916_l6 regulator-always-on on sony-tulip? I don't see that in Sony's vendor kernel and I think it shouldn't be needed when setting up the -supplys properly everywhere

10:48 <minecrell> I can either drop it during the refactoring or keep it as an explicit override, whatever you prefer

10:57 <bryanodonoghue> No I think you can drop it

10:57 <bryanodonoghue> https://github.com/bartcubbins/kernel_kanuti_mainline/blob/master/arch/arm64/boot/dts/qcom/msm8939-sony-xperia-kanuti-tulip.dts

10:57 <bryanodonoghue> this is a good reference

10:59 <minecrell> ack, thanks

16:30 <aka_[m]> djakov: https://github.com/msm8953-mainline/linux/commit/6da50691cc6da2e9708d344e9ac10db1bd7d2244

16:35 <konradybcio> aka_: how is that related to Georgi who maintains icc? :P

16:36 <aka_[m]> Wasn't he author of clk-regmap-mux-div?

16:37 <aka_[m]> Author: Georgi Djakov