On Wed, 30 Oct 2024 at 19:59, Taniya Das [off-list ref] wrote:

On 10/19/2024 1:55 AM, Dmitry Baryshkov wrote:

quoted

quoted

#include "common.h"
+#include "gdsc.h"
+#include "reset.h"
+
+enum {
+    DT_BI_TCXO,
+    DT_GPLL0,
+    DT_DSI0_PHY_PLL_OUT_BYTECLK,
+    DT_DSI0_PHY_PLL_OUT_DSICLK,
+    DT_DSI1_PHY_PLL_OUT_DSICLK,

Is there a DSI 1 PLL on this platform?

As per the design of the clock controller it has a DSI1 port.

It's just surprising to have the DSI1 DSICLK and not the BYTECLK. But
if you say so, it's fine.

Reviewed-by: Dmitry Baryshkov <redacted>

-- 
With best wishes
Dmitry