public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Xavi Drudis Ferran <xdrudis@tinet•cat>
To: Fabio Estevam <festevam@gmail•com>
Cc: Xavi Drudis Ferran <xdrudis@tinet•cat>,
	Nicolin Chen <nicoleotsuka@gmail•com>,
	Shengjiu Wang <shengjiu.wang@freescale•com>,
	"alsa-devel@alsa-project•org" <alsa-devel@alsa-project•org>,
	Xiubo Li <Xiubo.Lee@gmail•com>, Timur Tabi <timur@tabi•org>,
	Fabio Estevam <fabio.estevam@nxp•com>,
	"linuxppc-dev@lists•ozlabs.org" <linuxppc-dev@lists•ozlabs.org>
Subject: Re: [alsa-devel] Setting some clocks back to DUMMY fixes spdif output on imx6q wandboard rev B1
Date: Wed, 31 Aug 2016 15:49:25 +0200	[thread overview]
Message-ID: <20160831134925.GB1967@begut> (raw)
In-Reply-To: <CAOMZO5AoCo9Faq7oWgRQUE0Hg+Pg+i6z8eg-qeLFDvwpoEqz_g@mail.gmail.com>

El Wed, Aug 31, 2016 at 10:11:13AM -0300, Fabio Estevam deia:
> 2. SPDIF clock rate not accurate. Probably using PLL4 as SPDIF source
> would help to get more accurate SPDIF clock rates.
> 
> Could you please try the untested change?
> 
> --- a/drivers/clk/imx/clk-imx6q.c
> +++ b/drivers/clk/imx/clk-imx6q.c
> @@ -623,7 +623,7 @@ static void __init imx6q_clocks_init(struct
> device_node *ccm_node)
>                 pr_warn("failed to set up CLKO: %d\n", ret);
> 
>         /* Audio-related clocks configuration */
> -       clk_set_parent(clk[IMX6QDL_CLK_SPDIF_SEL],
> clk[IMX6QDL_CLK_PLL3_PFD3_454M]);
> +       clk_set_parent(clk[IMX6QDL_CLK_SPDIF_SEL],
> clk[IMX6QDL_CLK_PLL4_AUDIO_DIV]);
> 
>         /* All existing boards with PCIe use LVDS1 */
>         if (IS_ENABLED(CONFIG_PCI_IMX6))
>

I'm going to try. I'll take a while. I'll report the result later.

Thank you very much.

  parent reply	other threads:[~2016-08-31 13:49 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20160828160055.GA2122@begut>
2016-08-29 19:28 ` Setting some clocks back to DUMMY fixes spdif output on imx6q wandboard rev B1 Nicolin Chen
2016-08-29 19:54   ` Xavi Drudis Ferran
2016-08-30 11:14     ` Xavi Drudis Ferran
2016-08-31  4:21       ` Nicolin Chen
2016-08-31  9:10         ` Xavi Drudis Ferran
2016-08-31 13:11           ` [alsa-devel] " Fabio Estevam
2016-08-31 13:30             ` Fabio Estevam
2016-08-31 13:47               ` Xavi Drudis Ferran
2016-08-31 14:02                 ` Fabio Estevam
2016-08-31 13:49             ` Xavi Drudis Ferran [this message]
2016-08-31 17:49               ` Xavi Drudis Ferran
2016-08-31 18:02                 ` Fabio Estevam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160831134925.GB1967@begut \
    --to=xdrudis@tinet$(echo .)cat \
    --cc=Xiubo.Lee@gmail$(echo .)com \
    --cc=alsa-devel@alsa-project$(echo .)org \
    --cc=fabio.estevam@nxp$(echo .)com \
    --cc=festevam@gmail$(echo .)com \
    --cc=linuxppc-dev@lists$(echo .)ozlabs.org \
    --cc=nicoleotsuka@gmail$(echo .)com \
    --cc=shengjiu.wang@freescale$(echo .)com \
    --cc=timur@tabi$(echo .)org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox