Re: [PATCH 0/3] clk: meson: fix mpll jitter

From: Jerome Brunet
Date: Mon Apr 01 2019 - 04:40:32 EST


On Sat, 2019-03-30 at 16:58 +0100, Martin Blumenstingl wrote:
> On Fri, Mar 29, 2019 at 4:34 PM Jerome Brunet <jbrunet@xxxxxxxxxxxx> wrote:
> > As reported on this [0] mpll series, We are observing a lot of jitter
> > on the MPLL outputs of the g12a. No such jitter is seen on gx family.
> > On the axg family, only MPLL2 seems affected.
> >
> > The jitter can be as high as +/- 4%.
> >
> > This is a problem for audio application. This may cause distortion on
> > i2s and completely break SPDIF.
> >
> > After exchanging with Amlogic, it seems he have activated (by mistake)
> > the 'spread spectrum' feature.
> >
> > This patchset properly set the bit responsible for the spread spectrum
> > in the mpll driver and add the required correction in the related clock
> > controllers.
> >
> > Jerome Brunet (3):
> > clk: meson: mpll: properly handle spread spectrum
> this depends on a patch from your other series "clk: meson: fixup g12a
> mpll": [0]
> personally I would base the other series on this one because this one
> contains a "Fixes" that (meaning that the patch will get backported
> and that's easier if there aren't any conflicts)

1) The other series [0] had been submitted before we knew about the fix which
is why this one is based on it.
2) Considering the amount of work that went through since the fixed commit, I
doubt it makes much of a difference.

I might do it if both series require a resend.

>
>
> Regards
> Martin
>
>
> [0] https://patchwork.kernel.org/patch/10868837/