16:20 <calebccff> I'm preparing a series to add the RRADC DTS nodes [1], dtbs_check is giving me this error which I don't understand: "pmic@2: adc@4500:compatible: 'oneOf' conditional failed, one must be fixed ['qcom,pmi8998-rradc'] is too short 'qcom,pms405-adc' was expected 'qcom,pmi8998-rradc' is not one of ['qcom,spmi-vadc', 'qcom,spmi-adc5', 'qcom,spmi-adc-rev2', 'qcom,spmi-adc7']". [1]: https://gitlab.com/sdm845-mainline/linux/-/commits/upstreaming/

16:20 <calebccff> dts-rradc

16:21 <calebccff> https://gitlab.com/sdm845-mainline/linux/-/commits/upstreaming/dts-rradc

16:28 <flamingradian[m]> I think you need to add a ref here: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml#n101

16:29 <flamingradian[m]> and rename the node to rradc@4500 so it can tell rradc and vadc apart

16:30 <calebccff> ah, awesome. thanks!

16:31 <z3ntu> calebccff: no need to rename the node, you should be able to do something like this: https://lore.kernel.org/linux-arm-msm/20220925211744.133947-2-luca@z3ntu.xyz/

16:32 <calebccff> sweet, will do that

17:18 <aka_[m]> sboyd: Any idea where vals for hfpll were taken from?... (full message at <https://matrix.org/_matrix/media/r0/download/matrix.org/bgYUgfiYonTsCCWGWwXwutrH>)

17:44 <aka_[m]> turns out difference is hfpll write 0011 for... (full message at <https://matrix.org/_matrix/media/r0/download/matrix.org/MzEJPcIbjcaAoiPupdAgxcqT>)

17:45 <aka_[m]> could be great if anyone can tell how problematic this can be

18:42 <aka_[m]> interesting that enabling EARLY output is done in fashion of magic code which toggle BIT(3) (user_val) later VCO is enabled

18:49 <aka_[m]> now what does affect setting post divide ratio?

18:49 <aka_[m]> freq at main output?

18:50 <aka_[m]> if we are using early output on apcs mux then it shouldnt bother us at all