* [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register
@ 2013-04-17 18:33 Fabio Estevam
2013-04-17 18:57 ` Eric Nelson
2013-04-25 19:15 ` Stefano Babic
0 siblings, 2 replies; 6+ messages in thread
From: Fabio Estevam @ 2013-04-17 18:33 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
Setup CCM_CCOSR register to provide a CKO1 clock frequency of 16.5 MHz.
CKO1 drives sgtl5000 codec clock on mx6qsabrelite and doing this setup in the
bootloader will allow us to remove a lot of code in arch/arm/mach-imx/mach-imx6q.c
from the mainline kernel.
mx6q_4x_mt41j128.cfg is also used by mx6qsabresd, and it is safe to use it for
this board as well.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg b/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg
index f4cae5e..4661775 100644
--- a/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg
+++ b/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg
@@ -172,3 +172,14 @@ DATA 4 0x020e0010 0xF00000CF
/* set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7 */
DATA 4 0x020e0018 0x007F007F
DATA 4 0x020e001c 0x007F007F
+
+/*
+ * Setup CCM_CCOSR register as follows:
+ *
+ * cko1_en = 1 --> CKO1 enabled
+ * cko1_div = 111 --> divide by 8
+ * cko1_sel = 1011 --> ahb_clk_root
+ *
+ * This sets CKO1 at ahb_clk_root/8 = 132/8 = 16.5 MHz
+ */
+DATA 4 0x020c4060 0x000000fb
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register
2013-04-17 18:33 [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register Fabio Estevam
@ 2013-04-17 18:57 ` Eric Nelson
2013-04-17 19:22 ` Fabio Estevam
2013-04-25 19:15 ` Stefano Babic
1 sibling, 1 reply; 6+ messages in thread
From: Eric Nelson @ 2013-04-17 18:57 UTC (permalink / raw)
To: u-boot
Thanks Fabio,
On 04/17/2013 11:33 AM, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Setup CCM_CCOSR register to provide a CKO1 clock frequency of 16.5 MHz.
>
> CKO1 drives sgtl5000 codec clock on mx6qsabrelite and doing this setup in the
> bootloader will allow us to remove a lot of code in arch/arm/mach-imx/mach-imx6q.c
> from the mainline kernel.
>
More importantly than eliminating code in the mainline kernel,
enabling the clock will remove squeal after an ungraceful
reboot (watchdog) if hooked up to speakers. I believe you
can see this under Android by issuing a 'reboot' command.
We've been meaning to push a patch for this for a while.
We did it in two parts.
1.) Add routine to enable the clock:
https://github.com/boundarydevices/u-boot-imx6/commit/7087d645ea1ad476825ea96a1b6f3747f5980028
2.) Enable it on each of SABRE Lite, Nitrogen6X and one of our custom
boards.
https://github.com/boundarydevices/u-boot-imx6/commit/44b1015f69b52c0a7d7b930b0607af7a724e29bb
> mx6q_4x_mt41j128.cfg is also used by mx6qsabresd, and it is safe to use it for
> this board as well.
>
Does wandboard also use SGTL5000 connected up to CKO1?
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg b/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg
> index f4cae5e..4661775 100644
> --- a/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg
> +++ b/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg
> @@ -172,3 +172,14 @@ DATA 4 0x020e0010 0xF00000CF
> /* set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7 */
> DATA 4 0x020e0018 0x007F007F
> DATA 4 0x020e001c 0x007F007F
> +
> +/*
> + * Setup CCM_CCOSR register as follows:
> + *
> + * cko1_en = 1 --> CKO1 enabled
> + * cko1_div = 111 --> divide by 8
> + * cko1_sel = 1011 --> ahb_clk_root
> + *
> + * This sets CKO1 at ahb_clk_root/8 = 132/8 = 16.5 MHz
> + */
> +DATA 4 0x020c4060 0x000000fb
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register
2013-04-17 18:57 ` Eric Nelson
@ 2013-04-17 19:22 ` Fabio Estevam
2013-04-18 5:27 ` prem s
0 siblings, 1 reply; 6+ messages in thread
From: Fabio Estevam @ 2013-04-17 19:22 UTC (permalink / raw)
To: u-boot
Hi Eric,
On Wed, Apr 17, 2013 at 3:57 PM, Eric Nelson
<eric.nelson@boundarydevices.com> wrote:
> More importantly than eliminating code in the mainline kernel,
> enabling the clock will remove squeal after an ungraceful
> reboot (watchdog) if hooked up to speakers. I believe you
> can see this under Android by issuing a 'reboot' command.
Ok, great. I was not aware of this problem. Glad to know that it also fixes it.
I can send this same change to nitrogen and wandboard.
> Does wandboard also use SGTL5000 connected up to CKO1?
Yes, it does.
I started to adding audio support to wandboard in the mainline kernel,
then I saw all the code that mx6qsabrelite uses to configure the cko1,
and I preferred not to repeat it again.
It is much simpler to let the bootloader take care of it, as it is a
simple register write.
Regards,
Fabio Estevam
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register
2013-04-17 19:22 ` Fabio Estevam
@ 2013-04-18 5:27 ` prem s
2013-04-18 15:56 ` Fabio Estevam
0 siblings, 1 reply; 6+ messages in thread
From: prem s @ 2013-04-18 5:27 UTC (permalink / raw)
To: u-boot
Hi all,
I am working on IMX6Q-SabreLite Board.in that board we have sgtl5000
codec.so I would like to add (sgtl5000) codec support in *u-boot level*.There
is no code available in u-boot level for this codec.Can you please help me
How to initialize this audio codec and how to test this one in UBOOT level.
It will be very helpful if you send any u-boot level patch for this
codec(sgtl5000).
Thanks
Prem.
On Thu, Apr 18, 2013 at 12:52 AM, Fabio Estevam <festevam@gmail.com> wrote:
> Hi Eric,
>
> On Wed, Apr 17, 2013 at 3:57 PM, Eric Nelson
> <eric.nelson@boundarydevices.com> wrote:
>
> > More importantly than eliminating code in the mainline kernel,
> > enabling the clock will remove squeal after an ungraceful
> > reboot (watchdog) if hooked up to speakers. I believe you
> > can see this under Android by issuing a 'reboot' command.
>
> Ok, great. I was not aware of this problem. Glad to know that it also
> fixes it.
>
> I can send this same change to nitrogen and wandboard.
>
> > Does wandboard also use SGTL5000 connected up to CKO1?
>
> Yes, it does.
>
> I started to adding audio support to wandboard in the mainline kernel,
> then I saw all the code that mx6qsabrelite uses to configure the cko1,
> and I preferred not to repeat it again.
>
> It is much simpler to let the bootloader take care of it, as it is a
> simple register write.
>
> Regards,
>
> Fabio Estevam
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register
2013-04-18 5:27 ` prem s
@ 2013-04-18 15:56 ` Fabio Estevam
0 siblings, 0 replies; 6+ messages in thread
From: Fabio Estevam @ 2013-04-18 15:56 UTC (permalink / raw)
To: u-boot
On Thu, Apr 18, 2013 at 2:27 AM, prem s <prem441@gmail.com> wrote:
> Hi all,
>
> I am working on IMX6Q-SabreLite Board.in that board we have sgtl5000
> codec.so I would like to add (sgtl5000) codec support in u-boot level.There
> is no code available in u-boot level for this codec.Can you please help me
> How to initialize this audio codec and how to test this one in UBOOT level.
>
> It will be very helpful if you send any u-boot level patch for this
> codec(sgtl5000).
I don't have plans to adding audio support into U-boot for imx.
Maybe you could try to use the OBDS code from Freescale, which
provides standalone code for setting up various peripherals and then
you can use it as a reference for adding audio support in U-boot.
Just curious: what is your requirement in terms of providing the user
with an audio response?
Would booting the kernel fast and providing the audio response in the
kernel meet your goal?
Regards,
Fabio Estevam
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register
2013-04-17 18:33 [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register Fabio Estevam
2013-04-17 18:57 ` Eric Nelson
@ 2013-04-25 19:15 ` Stefano Babic
1 sibling, 0 replies; 6+ messages in thread
From: Stefano Babic @ 2013-04-25 19:15 UTC (permalink / raw)
To: u-boot
On 17/04/2013 20:33, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Setup CCM_CCOSR register to provide a CKO1 clock frequency of 16.5 MHz.
>
> CKO1 drives sgtl5000 codec clock on mx6qsabrelite and doing this setup in the
> bootloader will allow us to remove a lot of code in arch/arm/mach-imx/mach-imx6q.c
> from the mainline kernel.
>
> mx6q_4x_mt41j128.cfg is also used by mx6qsabresd, and it is safe to use it for
> this board as well.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
Applied to u-boot-imx, thanks.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-04-25 19:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-17 18:33 [U-Boot] [PATCH] imx: mx6q_4x_mt41j128.cfg: Setup CCM_CCOSR register Fabio Estevam
2013-04-17 18:57 ` Eric Nelson
2013-04-17 19:22 ` Fabio Estevam
2013-04-18 5:27 ` prem s
2013-04-18 15:56 ` Fabio Estevam
2013-04-25 19:15 ` Stefano Babic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox