03:28 <niej> Does anyone know how to dump all display registers in downstream Android linux? I see there is function _sde_dump_array() and lots of interface in debugfs: /sys/kernel/debug/dri/0/debug.

12:59 <bryanodonoghue> vknecht[m] unless I'm misreading your code has_vbif should be set in the resources structure and should then be enough to map regs and subsequently set bandwidth

13:00 <bryanodonoghue> I don't see myself why we need an additional switch to verify CAMSS silicon type

13:00 <bryanodonoghue> plus - really want to remove all of this switching for silicon version

13:05 <vknecht[m]> The thing is, that the regs and values I set are specific to 8939. For example, 8974 and 8952 need other regs and values. Granted, those have no camss support for now. But shouldn't the code allow for easy addition later ?

13:07 <vknecht[m]> Or should vbif regs and values be added to VFE resources too ? Guess that then the vbif code wouldn't need switches

13:08 <z3ntu> idk if it's relevant, but msm8974 had some camss patches on the mailing list at some point: https://lore.kernel.org/linux-arm-msm/20220522162802.208275-1-luca@z3ntu.xyz/ , if it's useful for reference