08:33 <aka_[m]> any idea why checkpatch fails like that?:

08:33 <aka_[m]> ModuleNotFoundError: No module named 'git'

08:33 <aka_[m]> ?

08:42 <krzk> aka_[m]: missing git module

08:43 <krzk> sudo apt-get install python-ply python3-git

08:43 <aka_[m]> pip3 install git or something like that?

08:43 <aka_[m]> arch

08:43 <aka_[m]> seems something went off during updates and i lost it

08:43 <krzk> so translate this to arch...

08:44 <krzk> arch by default means user knows how to do everything or find a guide for that, so it is enough to give you ubuntu commands :)

08:44 <aka_[m]> right and i only took it for rolling release

08:47 <aka_[m]> seems it was python-gitpython

09:54 <DylanVanAssche> Is there any userspace stuff for testing SLIMBus things like `qrtr-lookup`? I would like to investigate if I have the SLIMBus controller up if I can reach it etc. ?

09:58 <aka_[m]> krzk: regarding this:

09:58 <aka_[m]> https://lists.sr.ht/~postmarketos/upstreaming/patches/53203#%3C20240609182112.13032-2-a39.skl@gmail.com%3E

09:58 <aka_[m]> i should just follow sm6115.yaml right?

09:59 <aka_[m]> it seems it has it the way you asked for

10:04 <krzk> aka_[m]: depends for what... you have there several comments

10:09 <aka_[m]> krzk: for placement of ifs/ofs/refs and so on

10:09 <aka_[m]> this would be for this one:

10:09 <aka_[m]> https://github.com/Kiciuk/linux/commit/fe9256f0259fd33cd361b4104947466feff38c71

10:10 <aka_[m]> any idea why checkpath complains on Kconfig? does it expect 3 lines of description?

10:16 <krzk> aka_[m]: sorry, I don't understand this change. Why clocks disappeared? I asked about order - first clocks then clock-names. But both are gone!

10:16 <aka_[m]> are those required when i define list of clocks?

10:16 <krzk> I don't understand what this means. device has clocks, right? So yes, clocks are required.

10:17 <aka_[m]> only snoc has them

10:17 <krzk> ah, then other variants should disallow them :false

10:17 <aka_[m]> sm6115 have clocks on top with min as 1 and 4 as max

10:17 <aka_[m]> so if i have 1 i do max=1?

10:18 <krzk> yes

10:18 <krzk> you also moved $ref in the children, even though I did not ask for it.

10:18 <krzk> Please read the comments again

10:23 <aka_[m]> ok managing slowly to understand what you wrote, had terrible night today

10:29 <krzk> for clocks you need https://elixir.bootlin.com/linux/v5.19/source/Documentation/devicetree/bindings/example-schema.yaml#L222

10:30 <aka_[m]> have this:

10:30 <aka_[m]> https://github.com/Kiciuk/linux/commit/e8a4ca6c22c8997a07cbf6b03cad9ddef5d9c54c

10:31 <aka_[m]> on 8937 i need no clocks and entire allOf and stuff seems bloat

10:31 <aka_[m]> qcs404 i bet it was missing regmap.h

10:31 <aka_[m]> no idea how handle that intel build bot response

10:35 <aka_[m]> uh still getting somehow lost with this -ref thing

10:37 <aka_[m]> whatever, time to head job

10:40 <aka_[m]> ah great

10:41 <aka_[m]> https://lists.sr.ht/~postmarketos/upstreaming/patches/53203#%3C20240609182112.13032-4-a39.skl@gmail.com%3E-32

10:41 <aka_[m]> If i drop this one props from ref no longer gets included, should it be left there or moved somewhere?

10:42 <aka_[m]> aa,, Spaghetti keeps pumping out

11:39 <konradybcio> JIaxyga: i would say that if the clock controllers are almost identical, you can smash the same ones into xcc-sm84xx.o, even..

11:39 <konradybcio> iirc 8475 had different plls but was otherwise largely the same there?

11:55 <JIaxyga[m]> <konradybcio> "iirc 8475 had different plls but..." <- https://github.com/cupid-development/android_kernel_xiaomi_sm8450/commit/b051ec6bdc4b4884905002b2e75753e6f1238d20#diff-154087aa14aefea5b10fe3c06884dfa0b45bc96d8f43201e1628e7b742964202

11:55 <JIaxyga[m]> But my series does not include camcc. I think it's better to write a separate driver

11:56 <konradybcio> yeah the camera diff is about the size of the rest of the driver :P