[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Dirk Behme]

A recent Linux kernel (>= 3.5) has support for the SGTL 5000 sound
on the SabreLite board. To make this work, U-Boot has to configure the
pin mux for PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2 correctly.

Taken from Freescale's ER5 U-Boot for the SabreLite.

Signed-off-by: Dirk Behme <dirk.behme@de.bosch.com>
CC: Troy Kisky <troy.kisky@boundarydevices.com>
CC: Stefano Babic <sbabic@denx.de>
---
 board/freescale/mx6qsabrelite/imximage.cfg |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/board/freescale/mx6qsabrelite/imximage.cfg b/board/freescale/mx6qsabrelite/imximage.cfg
index 62498ab..2d7825a 100644
--- a/board/freescale/mx6qsabrelite/imximage.cfg
+++ b/board/freescale/mx6qsabrelite/imximage.cfg
@@ -168,3 +168,6 @@ DATA 4 0x020e0010 0xF00000CF
 # set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7
 DATA 4 0x020e0018 0x007F007F
 DATA 4 0x020e001c 0x007F007F
+# set PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2
+DATA 4 0x020e0220 0x00000000
+DATA 4 0x020e022c 0x00000004
\ No newline at end of file
-- 
1.7.0.4

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Liu Hui-R64343]

>-----Original Message-----
From: u-boot-bounces@lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
>On Behalf Of Dirk Behme
>Sent: Wednesday, August 01, 2012 7:06 PM
>To: u-boot at lists.denx.de
>Cc: Dirk Behme
>Subject: [U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO
>and GPIO_3__CCM_CLKO2 pin mux
>
>A recent Linux kernel (>= 3.5) has support for the SGTL 5000 sound on the
>SabreLite board. To make this work, U-Boot has to configure the pin mux for
>PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2 correctly.

Why this can't be set in the kernel but relies on u-boot to configure it?

Jason
>
>Taken from Freescale's ER5 U-Boot for the SabreLite.
>
>Signed-off-by: Dirk Behme <dirk.behme@de.bosch.com>
>CC: Troy Kisky <troy.kisky@boundarydevices.com>
>CC: Stefano Babic <sbabic@denx.de>
>---
> board/freescale/mx6qsabrelite/imximage.cfg |    3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
>1.7.0.4
>
>_______________________________________________
>U-Boot mailing list
>U-Boot at lists.denx.de
>http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Dirk Behme]

On 01.08.2012 13:26, Liu Hui-R64343 wrote:
>> -----Original Message-----
>> From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
>> On Behalf Of Dirk Behme
>> Sent: Wednesday, August 01, 2012 7:06 PM
>> To: u-boot at lists.denx.de
>> Cc: Dirk Behme
>> Subject: [U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO
>> and GPIO_3__CCM_CLKO2 pin mux
>>
>> A recent Linux kernel (>= 3.5) has support for the SGTL 5000 sound on the
>> SabreLite board. To make this work, U-Boot has to configure the pin mux for
>> PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2 correctly.
> 
> Why this can't be set in the kernel but relies on u-boot to configure it?

I don't know :(

It took me days to find this U-Boot dependency, thanks to Troy helping 
with this!

I enabled SGTL5000 sound in the kernel and it didn't work. Until I found 
that it works with the ER5 Freescale U-Boot, but not with the recent 
mainline one.

It seems to me that the SGTL5000 kernel feature for the SabreLite was 
developed with a Freescale U-Boot (patching the kernel with DT append) 
and not tested with the mainline U-Boot.

Best regards

Dirk

> Jason
>> Taken from Freescale's ER5 U-Boot for the SabreLite.
>>
>> Signed-off-by: Dirk Behme <dirk.behme@de.bosch.com>
>> CC: Troy Kisky <troy.kisky@boundarydevices.com>
>> CC: Stefano Babic <sbabic@denx.de>
>> ---
>> board/freescale/mx6qsabrelite/imximage.cfg |    3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> 1.7.0.4
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
> 
> 
> 


-- 
======================================================================
Dirk Behme                      Robert Bosch Car Multimedia GmbH
                                 CM-AI/PJ-CF32
Phone: +49 5121 49-3274         Dirk Behme
Fax:   +49 711 811 5053274      PO Box 77 77 77
mailto:dirk.behme at de.bosch.com  D-31132 Hildesheim - Germany

Bosch Group, Car Multimedia (CM)
              Automotive Navigation and Infotainment Systems (AI)
              ProJect - CoreFunctions (PJ-CF)

Robert Bosch Car Multimedia GmbH - Ein Unternehmen der Bosch Gruppe
Sitz: Hildesheim
Registergericht: Amtsgericht Hildesheim HRB 201334
Aufsichtsratsvorsitzender: Volkmar Denner
Gesch?ftsf?hrung: Uwe Thomas, Michael Bolle, Robby Drave, Egbert Hellwig
======================================================================

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Wolfgang Denk]

Dear Dirk Behme,

In message <5019180C.4060109@de.bosch.com> you wrote:
>
> It seems to me that the SGTL5000 kernel feature for the SabreLite was
> developed with a Freescale U-Boot (patching the kernel with DT append)
> and not tested with the mainline U-Boot.

Probably. But the question is still why this should be changed in
U-Boot.  Why doesn't the Linux driver set the pin mux configuration
it needs?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
THIS IS A 100% MATTER  PRODUCT:  In  the  Unlikely  Event  That  This
Merchandise  Should  Contact  Antimatter  in Any Form, a Catastrophic
Explosion Will Result.

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Dirk Behme]

On 01.08.2012 14:55, Wolfgang Denk wrote:
> Dear Dirk Behme,
> 
> In message <5019180C.4060109@de.bosch.com> you wrote:
>> It seems to me that the SGTL5000 kernel feature for the SabreLite was
>> developed with a Freescale U-Boot (patching the kernel with DT append)
>> and not tested with the mainline U-Boot.
> 
> Probably. But the question is still why this should be changed in
> U-Boot.  Why doesn't the Linux driver set the pin mux configuration
> it needs?

Sorry, I don't know. The Linux driver developers told me "there is no 
U-Boot dependency". Maybe they could answer this question? CCed.

Best regards

Dirk

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Thomas Petazzoni]

Le Wed, 1 Aug 2012 15:22:30 +0200,
Dirk Behme <dirk.behme@de.bosch.com> a ?crit :

> > Probably. But the question is still why this should be changed in
> > U-Boot.  Why doesn't the Linux driver set the pin mux configuration
> > it needs?
> 
> Sorry, I don't know. The Linux driver developers told me "there is no 
> U-Boot dependency". Maybe they could answer this question? CCed.

The kernel has a pinctrl driver for i.MX 6, so I would rather suggest
to fix your imx6q-sabrelite.dts Device Tree source file so that the
audio device is properly associated with the correct pinmux
configuration.

Each device can have a pinctrl-0 DT property, which points to one or
more pinmux configurations that are defined in imx6q.dtsi file. You can
add additional pinmux configurations here if needed.

But yeah, definitely, the kernel shouldn't rely too much on U-Boot
having set the right pinmux configuration.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Dirk Behme]

On 01.08.2012 15:33, Thomas Petazzoni wrote:
> Le Wed, 1 Aug 2012 15:22:30 +0200,
> Dirk Behme <dirk.behme@de.bosch.com> a ?crit :
> 
>>> Probably. But the question is still why this should be changed in
>>> U-Boot.  Why doesn't the Linux driver set the pin mux configuration
>>> it needs?
>> Sorry, I don't know. The Linux driver developers told me "there is no 
>> U-Boot dependency". Maybe they could answer this question? CCed.
> 
> The kernel has a pinctrl driver for i.MX 6, so I would rather suggest
> to fix your imx6q-sabrelite.dts Device Tree source file so that the
> audio device is properly associated with the correct pinmux
> configuration.

I'm using

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=b7879fe6dad97ce08e8df0bf8d408942c436d358

> Each device can have a pinctrl-0 DT property, which points to one or
> more pinmux configurations that are defined in imx6q.dtsi file. You can
> add additional pinmux configurations here if needed.

Sounds like the above is incomplete, then?

> But yeah, definitely, the kernel shouldn't rely too much on U-Boot
> having set the right pinmux configuration.

Best regards

Dirk

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Thomas Petazzoni]

Le Wed, 1 Aug 2012 15:43:03 +0200,
Dirk Behme <dirk.behme@de.bosch.com> a ?crit :

> > The kernel has a pinctrl driver for i.MX 6, so I would rather suggest
> > to fix your imx6q-sabrelite.dts Device Tree source file so that the
> > audio device is properly associated with the correct pinmux
> > configuration.
> 
> I'm using
> 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=b7879fe6dad97ce08e8df0bf8d408942c436d358
> 
> > Each device can have a pinctrl-0 DT property, which points to one or
> > more pinmux configurations that are defined in imx6q.dtsi file. You can
> > add additional pinmux configurations here if needed.
> 
> Sounds like the above is incomplete, then?

Yes. If this device needs certain pins to be muxed in a certain
configuration, then it needs:

	pinctrl-names = "default";
	pinctrl-0 = <&pinctrl_some_config_name>;

And then in the imx6q.dtsi, the pinctrl_some_config_name pinctrl
configuration needs to be defined.

But I guess this starts to be a bit off-topic for the U-Boot list. What
about raising the issue with i.MX 6 kernel maintainers on LAKML instead?

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Fabio Estevam]

Hi Dirk,

On Wed, Aug 1, 2012 at 10:43 AM, Dirk Behme <dirk.behme@de.bosch.com> wrote:

> Sounds like the above is incomplete, then?

I will send a kernel patch to setup these two clocks for sabrelite. I
don't have a sabrelite handy now, but if you have a chance to test it,
please let me know.

Regards,

Fabio Estevam

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Dirk Behme]

On 01.08.2012 13:05, Dirk Behme wrote:
> A recent Linux kernel (>= 3.5) has support for the SGTL 5000 sound
> on the SabreLite board. To make this work, U-Boot has to configure the
> pin mux for PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2 correctly.
> 
> Taken from Freescale's ER5 U-Boot for the SabreLite.
> 
> Signed-off-by: Dirk Behme <dirk.behme@de.bosch.com>
> CC: Troy Kisky <troy.kisky@boundarydevices.com>
> CC: Stefano Babic <sbabic@denx.de>
> ---
>  board/freescale/mx6qsabrelite/imximage.cfg |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/board/freescale/mx6qsabrelite/imximage.cfg b/board/freescale/mx6qsabrelite/imximage.cfg
> index 62498ab..2d7825a 100644
> --- a/board/freescale/mx6qsabrelite/imximage.cfg
> +++ b/board/freescale/mx6qsabrelite/imximage.cfg
> @@ -168,3 +168,6 @@ DATA 4 0x020e0010 0xF00000CF
>  # set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7
>  DATA 4 0x020e0018 0x007F007F
>  DATA 4 0x020e001c 0x007F007F
> +# set PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2
> +DATA 4 0x020e0220 0x00000000
> +DATA 4 0x020e022c 0x00000004
> \ No newline at end of file

This is replaced with a kernel patch

http://www.spinics.net/lists/arm-kernel/msg187222.html

and therefore this patch is obsolete now.

Thanks

Dirk

[U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

[Stefano Babic]

On 02/08/2012 09:36, Dirk Behme wrote:
> On 01.08.2012 13:05, Dirk Behme wrote:
>> A recent Linux kernel (>= 3.5) has support for the SGTL 5000 sound
>> on the SabreLite board. To make this work, U-Boot has to configure the
>> pin mux for PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2 correctly.
>>
>> Taken from Freescale's ER5 U-Boot for the SabreLite.
>>
>> Signed-off-by: Dirk Behme <dirk.behme@de.bosch.com>
>> CC: Troy Kisky <troy.kisky@boundarydevices.com>
>> CC: Stefano Babic <sbabic@denx.de>
>> ---
>>  board/freescale/mx6qsabrelite/imximage.cfg |    3 +++
>>  1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/board/freescale/mx6qsabrelite/imximage.cfg
>> b/board/freescale/mx6qsabrelite/imximage.cfg
>> index 62498ab..2d7825a 100644
>> --- a/board/freescale/mx6qsabrelite/imximage.cfg
>> +++ b/board/freescale/mx6qsabrelite/imximage.cfg
>> @@ -168,3 +168,6 @@ DATA 4 0x020e0010 0xF00000CF
>>  # set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7
>>  DATA 4 0x020e0018 0x007F007F
>>  DATA 4 0x020e001c 0x007F007F
>> +# set PAD_GPIO_0__CCM_CLKO and PAD_GPIO_3__CCM_CLKO2
>> +DATA 4 0x020e0220 0x00000000
>> +DATA 4 0x020e022c 0x00000004
>> \ No newline at end of file
> 
> This is replaced with a kernel patch
> 
> http://www.spinics.net/lists/arm-kernel/msg187222.html
> 
> and therefore this patch is obsolete now.

Ok, thanks.

Regards,
Stefano

-- 
=====================================================================
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
=====================================================================