Re: [PATCH v3 2/2] drm: rcar-du: calculate DPLLCR to be more small jitter
From: Kuninori Morimoto
Date: Sun Dec 17 2017 - 18:46:47 EST
Hi Geert
> >> > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> >> > In general, PLL has VCO (= Voltage controlled oscillator),
> >> > one of the very important electronic feature called as "jitter"
> >> > is related to this VCO.
> >> > In academic generalism, VCO should be maximum to be more small jitter.
> >> > In high frequency clock, jitter will be large impact.
> >> > Thus, selecting Hi VCO is general theory.
> >>
> >> Thanks for your patch!
> >>
> >> > One note here is that it should be 2000 < fvco < 4096MHz
> >>
> >> 2000 Hz? (else it could be misinterpreted that MHz applies to both values).
> >
> > Laurent had asked same question ;)
> > But, yes, it is 2000 Hz
>
> I've seen his question, that's why I think you should make it unambiguous.
Ahh... OK yes indeed
will fix in v4
Best regards
---
Kuninori Morimoto