On Wed, 30 Oct 2024 at 19:59, Taniya Das <quic_tdas@xxxxxxxxxxx> wrote: > > > > On 10/19/2024 1:55 AM, Dmitry Baryshkov wrote: > >> #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 <dmitry.baryshkov@xxxxxxxxxx> -- With best wishes Dmitry