16:59 <abhinav__> krzk bamse konradybcio Hi, I had a question about adding some additional clocks to dp-controller.yaml for msm. For adding MST feature , we will need to add additional clocks for the stream1 (pixel clock). These clocks would be needed on chipsets where we will add MST support. Is it okay to extend current clock array to add the additional clocks?

17:03 <abhinav__> lumag too

17:06 <konradybcio> abhinav__: so long as you stick them at the end, totally fine

17:08 <abhinav__> konradybcio ack thanks

17:18 <abhinav__> konradybcio one follow up question, since these clocks will be on chipsets only which have MST, do we need to add any exceptions under allOf to indicate that these clocks apply for which chipsets?

17:19 <abhinav__> so for example sc7180 does not have MST support

17:23 <abhinav__> or perhaps some minitems in the clocks to indicate that current set of clocks are always needed (for single stream) but MST clocks are needed only on chipsets which support MST

17:32 <konradybcio> you can just add them to allowed clocks

17:32 <konradybcio> and then keep minItems as NUM_CLOCKS_WITHOUT_MST

17:33 <abhinav__> konradybcio ack makes sense to me

17:33 <abhinav__> thanks so much

20:39 <lumag> abhinav__, add the minItems to the array, then add if/then which either uses maxItems to specify SST-only clocks for platforms with no MST support

22:29 <abhinav__> lumag ack, thanks!