00:20 <apritzel> loki666: for the *CPU* clock it says to switch to a *lower* frequency while relocking the PLL, though there is nothing like this stated for the other PLLs

00:20 <apritzel> what are the GPU OPPs? I think you said they are all the same voltage? Then it might not matter much, but a lower frequency sounds safer

00:29 <apritzel> so I wouldn't go with 600 MHz, but depending on the lowest OPP 400 MHz (/3) sounds safe enough?

06:54 <loki666> apritzel: lowest opp is 420Mhz

06:55 <loki666> Let's got with M=3 then

08:09 <Jookia> apritzel: is it worth me re-sending the patch here https://patchwork.kernel.org/project/linux-arm-kernel/patch/20231217210629.131486-4-bigunclemax@gmail.com/#25642587 with the disabled line removed?

08:28 <Jookia> I don't know how I'd preserve authorship

11:27 <apritzel> Jookia: if you download the mbox file from patchwork (button top right), then import it with "git am v6-3-3-*.patch", you should keep everything (commit author and date)

11:28 <apritzel> then you can edit and "git commit --amend" the patch, and it would keep the authorship

11:29 <apritzel> Jookia: please remember to add your Signed-off-by:

11:31 <Jookia> Do I note I modified it? It feels a little strange to put someone's name on a patch they haven't endorsed, but maybe that's no big deal as it'll still CC them. I'll try and test the patch today and see if it's the only thing that doesn't work.

11:35 <apritzel> if it's just about removing that line, I wouldn't bother. If you do make more significant changes, it's typically noted in square brackets around your S-o-b, check commit a7137cbf6bd5 in the kernel for an example

16:50 <loki666> apritzel: does that looks good ? https://gist.github.com/loki666/9fc85418e20b15ee627c78d7c5c05ff8

17:00 <apritzel> loki666: yes, looks good to me, on a first glance

20:46 <loki666> apritzel_: btw, for CPU reparenting, sometime it will be a *higher* freq, since the lowest cpu opp is 480mhz