14:04 <macromorgan> CounterPillow: What ARM SCMI patches were you referencing the other day?

14:05 <CounterPillow> macromorgan: https://lore.kernel.org/all/20220523171559.472112-1-cristian.marussi@arm.com/

14:06 <CounterPillow> no clue if it's in -rc2 yet

14:06 <macromorgan> thanks

14:06 <macromorgan> I'm still working on the bootloader though, so I haven't got that far yet...

14:38 <diederik> CounterPillow: it's not in -rc2

14:42 <diederik> https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/commit/?id=122839b58a089ff7f231759e2c8f63790724cae2 "Applied to sudeep.holla/linux (for-next/scmi), thanks!" so I guess that'll be for 5.20?

14:48 <hanetzer> nice

14:59 <macromorgan> still haven't approved the battery driver :-(

14:59 <macromorgan> that thing's been pending since at least 5.16

15:27 <CounterPillow> diederik: no, it'll be the next 5.19 rc cycle likely

15:27 <CounterPillow> it's a bug that needs to be fixed before 5.19 release

15:27 <CounterPillow> otherwise shit will be broken

15:27 <CounterPillow> they promised they will fix it before 5.19 so if they mess this up and release a 5.19 without this patch I will go goblin mode

15:28 <CounterPillow> macromorgan: I'll give it another test soon-ish and do a tested-by, I also really want to see it included

15:28 <macromorgan> thank you

15:30 <diederik> CounterPillow: thanks for clarifying :)

15:38 <CounterPillow> just got confirmation that it will be in the next 5.19 fixes PR from the arm side of things to Linus

15:38 <diederik> :)

15:53 <macromorgan> CounterPillow: I think it took so long because when I did the audio driver I got flak for not converting the rk808.txt docs to yaml, and that took a while to get approved (because of some wrong devicetrees which also needed to be fixed). All prerequisites are now upstreamed though.

15:54 <macromorgan> I honestly don't think I've had anyone review the actual battery driver code itself yet to make sure it conforms to mainline standards

15:55 <CounterPillow> your codec work needing like 11 tries to get in inspired me to become a kernel developer myself and start porting the i2s/tdm audio driver, so it's not all bad :D

15:55 <CounterPillow> Yeah driver code rarely gets enough review I've noticed :/

15:55 <CounterPillow> my submission for i2s/tdm that made it in had some pretty bad bugs in it

15:59 <macromorgan> odd that I'm an inspiration since I still honestly don't know 90% of what I'm doing, but I'm glad it got others involved

16:00 <CounterPillow> I was basically like "okay if this guy submitted his driver without even yaml bindings in the first iteration, I'm gonna try as well"

16:01 <hanetzer> heh. the guy who got my clock driver is a helluva nitpicker :P

16:01 <macromorgan> well to be fair clocks and pins are kind of important

16:02 <hanetzer> tru but I'm probably the only person using this driver lmao

16:05 <CounterPillow> still needs to be correct :V

16:05 <CounterPillow> the reason why BSP linux trees usually suck is because nobody is there to nitpick when they shove in changes

16:05 <hanetzer> it was (well, as correct as I can decipher from bad datasheets)

16:05 <hanetzer> it just lacked a dt-bindings txt :P

16:07 <hanetzer> and some bad dts orrdering/conventions but otherwise functional and not too terrible looking :)

16:07 <hanetzer> but, it was mostly an rfc so I guess I literally asked for nitpicks :)