00:57 <digetx> DavidHeidelberg: you may use https://dpaste.com/2CFYGUXKL.txt for v2

01:19 <digetx> omg, why "drm/tegra: dc: rgb: Move PCLK shifter programming to CRTC" in next now? it supposed to be in 5.16

01:20 <digetx> tagr: do you have any other unapplied patches?

01:25 <digetx> ah, I see now that previous PR wasn't pulled

01:25 <digetx> drm PR

01:27 <digetx> the TC bridge patches also missed previous PR

01:30 <digetx> but it's in drm-misc-next, so should be applied this time

05:55 <tagr> digetx: yeah, I have no idea why it wasn't pulled last time... let's hope it gets pulled this time, because it just keeps getting bigger and bigger

12:19 <DavidHeidelberg> <digetx> "David Heidelberg: you may use..." <- Thanks, i'll look into it in evening

15:40 <DavidHeidelberg> digetx: incorporated your part of patch, should I put also your S-off-by on it?

15:42 <digetx> add my reviewed-by

15:42 <DavidHeidelberg> ok

15:43 <DavidHeidelberg> also about that `"^panel@[0-3]$":` I would make it `panel@[0-1]`, since we have only panel@1, but we could assume there would be panel@0 in future?

15:43 <DavidHeidelberg> or do you think it would be better to "hardcode it" to `panel@1`

15:43 <digetx> what panel? you don't need to add it anymore

15:45 <digetx> just use my v2 patch, you don't no need to add anything else there

15:45 <digetx> panel is defined in the dsi-controller.yaml

15:46 <DavidHeidelberg> you didn't removed the patternProperties with panel@ in v2 patch. I did now, I'll just re-run CI

15:47 <digetx> that patch isn't made on top of your v1

15:47 <digetx> if you'll get error from checker, then that's because you need to patch dtschema to get working unevaluatedProperties

15:48 <DavidHeidelberg> I haven't found link to v2 in history, can u reference?

15:49 <digetx> I mean that my v2 diff is the complete change that you should use instead of your v1

15:50 <DavidHeidelberg> nvm, does https://github.com/okias/linux/commit/356d97d3e14e5009d3baf2248090fd6a23a5bc2b look good?

15:51 <digetx> no, you just need to update the commit message and use my diff, add my r-b

15:51 <digetx> remove #address-cells and #size-cells and it will be okay

15:52 <DavidHeidelberg> oh right, I got it. I just had to apply what you sent on clear tree, not the previous patch

15:52 <digetx> yes

15:52 <DavidHeidelberg> sorry, my brain is off today

15:54 <digetx> please also don't add log to the commit message, you should add it to the formatted patch after ---

15:56 <digetx> tagr: how are you going to manage the DPAUX patches?

15:59 <tagr> fwiw, it should be fine to add the changelog to the commit like that, git am will strip it away when applying

15:59 <tagr> although it can sometimes get in the way when you do cherry-picks

16:00 <tagr> digetx: I'm still looking into a proper fix for that DPAUX issue

16:03 <digetx> --- shouldn't cut off the s-o-bs at least

16:04 <digetx> tagr: we don't need proper fix right now, we need a simple and backportable fix, you will be able to change it later on

16:09 <tagr> I'm pretty sure the proper fix will also be simple and backportable and won't require making a mess and then untangling it again

16:09 <tagr> and, honestly, this has now been broken for what, 4 releases? so we're not exactly in a hurry

16:23 <DavidHeidelberg> digetx: I know, I just keep it before sending in commit msg before S-off-by's

16:25 <tagr> oh... heh, didn't realize that it was before the sign-offs

16:26 <tagr> I often keep the log with the scissor line below the sign-offs, that way the sign-offs stay in the right place and git format-patch will still include the changelog (though it will add another scissor line, which then looks slightly off)

16:26 <tagr> odd even