Re: [PATCH 1/2] arm64: renesas: salvator-x: add 12288000 for sound ADG

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert

> >      clk_multiplier                        1            1    24576000
> >         0 0
> > +       audio_clkout3                      0            0    11289600
> >         0 0
> > +       audio_clkout2                      0            0    11289600
> >         0 0
> > +       audio_clkout1                      0            0    11289600
(snip)
> >   audio_clk_a                              1            1    22579200
> >         0 0
> > -    audio_clkout3                         0            0    11289600
> >         0 0
> > -    audio_clkout2                         0            0    11289600
> >         0 0
> > -    audio_clkout1                         0            0    11289600
(snip)
> > Does this depend on a recent driver change?
> > My tree is based on last renesas-drivers release.

Thank you for pointing it.
This patch (= "arm64: renesas: salvator-x: add 12288000 for sound ADG") was fine,
but driver side had bug.
I will post this fixup patch to ALSA side

About audio_clkoutX, all these output same clock rate which is
11289600 or 12288000 base clock. Basically this is fixed rate.
But in Salvator-X case, because of clock master/slave relation,
codec spec, and clock generator connection, we need to
switch audio_clkoutX in runtime, otherwise, sound gets noise.
For me, this is exception case, and this patch enables it

Best regards
---
Kuninori Morimoto



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux