08:06 <gamiee_> Hi apritzel, I think you was not there when I sent you message, so I will send it again

08:06 <gamiee_> apritzel : the DTS you made for Banana Pi M4 Berry seems to be done according to PCB V00. V10 (which I have) have different RAM (the same one as on Orange Pi Zero 3, but 2GiB version instead of 4GiB), and WiFi is now connected through USB hub. BTW do you have downloaded the V00 schematics? The Google Drive link is dead.

08:39 <apritzel> gamiee: good thing I am bit of a hoarder: check the wiki ;-)

08:40 <gamiee> apritzel : oh sweet, thank you! I will add V10 schematics later. About DTS, it will be okay if it will target only V10 version? I have no idea how many V00 were manufactured at all, so not sure if it is worth to have DTS for both.

08:41 <apritzel> yeah, was thinking the same. It's not like many people asked for support so far ;-)

08:45 <gamiee> yeah. Also, you mentioned "And we need the DRAM parameters from a vendor image, for U-Boot". With this I guess you meant u-boot's `CONFIG_DRAM_SUNXI_XXXXX` config params. I found those params in BPI's BSP sys_config.fex file, and they are 100% identical with what's in OPI Zero 3 u-boot config file, what makes sense, since it's identical RAM chip, just different 2GiB instead of 4GiB.

12:52 <jernej> apritzel: do you remember if you left dram type out of this change for a reason? https://source.denx.de/u-boot/custodians/u-boot-sunxi/-/commit/78aa00c38e86ea06e625c9c7e0365fc87353d5a5

12:54 <apritzel> jernej: this refactoring was to separate the dynamic (changing/autodetected) parameters like rows and columns from the ones fixed at build time, like DRAM type, frequency and timing parameters

12:55 <jernej> no, there is no DRAM type :)

12:55 <apritzel> are you planning on autodetecting the DRAM type, or does this otherwise change?

12:55 <jernej> my point is that you wanted to separate constant parameters out, but you forgot to move "type"

12:56 <jernej> I'm just wondering if you forget "type" or is there any reason?

12:56 <jernej> eh

12:56 <jernej> sorry, I mixed constant and dynamic

12:57 <jernej> it's all ok

12:57 <apritzel> ah, good, just started to wonder .. ;-)

12:58 <apritzel> admittedly para and config are not very telling names, just not sure renaming them to something with fixed or dynamic is worth the churn

13:17 <jernej> apritzel: due to code sharing, all DRAM drivers now require that split

13:17 <jernej> so, I guess, we're slowly moving to common infrastructure :)

15:33 <jernej> apritzel: here is H6/H616 DRAM code consolidation: https://github.com/jernejsk/u-boot/commits/h6-dram/

15:33 <jernej> any comments before I post it?

15:46 <apritzel> looks good - as usual ;-) - so please go ahead

15:48 <apritzel> jernej: can you just rebase this on latest upstream master? That should now contain all the patches from sunxi/next

15:57 <jernej> sure

17:14 <apritzel> thanks, will test on the weekend, and see if I can plug this into the A133 driver as well

17:16 <jernej> that would be a good litmus test