Linaro U-Boot

omap4: u-boot overrides pin-muxing settings from x-loader

Bug #645167 reported by Sebastien JAN
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro U-Boot
Fix Released
Undecided
Unassigned
u-boot-linaro (Ubuntu)
Fix Released
High
Unassigned
Ubuntu ubuntu-10.10
Maverick
Fix Released
High
Unassigned
Ubuntu ubuntu-10.10

Bug Description

u-boot is able to define a complete pin-muxing setting.
On OMAP4 today, we rely on x-loader for this purpose.

So we shall either:
1) deactivate pin-muxing setting for omap4, or
2) or use it and ensure we have the latest pin-mux updates integrated

Option 1 was used here: http://dev.omapzoom.org/?p=bootloader/u-boot.git;a=commit;h=cc49b2f4915bb550294b8c74f292e81cfdb11f4d

Today for example, the pin-mux for GPMC_A22 is not properly set in u-boot, resulting in non-functional BT feature.
For this specific BT issue, the correct pin-mux is described in this x-loader patch (included into x-loader-omap4 package):
http://gitorious.org/pandaboard/x-loader/commit/5d05eafab768f4b88aae4f890fc26fd5eb3002ce

Tags: armel
Oliver Grawert (ogra)
Changed in u-boot-linaro (Ubuntu Maverick):
importance: Undecided → High
milestone: none → ubuntu-10.10
Revision history for this message
Loïc Minier (lool) wrote :

(If we ever want to get rid of x-loader as a separate body of code, I'd rather we arrange for the pin settings to be matching in the two source trees.)

Revision history for this message
Nicolas Dechesne (ndec) wrote :

@ogra: this bug is very critical since our WLAN and BT chips on pandaboard don't work without the proper pin muxing.

@lool: agreed. uboot should do the proper pin mux

Revision history for this message
Steve Sakoman (steve-sakoman) wrote : Re: [Bug 645167] Re: omap4: u-boot overrides pin-muxing settings from x-loader

On Wed, 2010-09-22 at 13:15 +0000, Nicolas Dechesne wrote:
> @ogra: this bug is very critical since our WLAN and BT chips on
> pandaboard don't work without the proper pin muxing.
>
> @lool: agreed. uboot should do the proper pin mux

I'll prepare a patch to correct the Panda pinmux.

Steve

Revision history for this message
Nicolas Dechesne (ndec) wrote :

@steeve: thx for quick answer. feel free to contact Xavier Boudet if you need and for the testing.

Revision history for this message
Ricardo Salveti (rsalveti) wrote :

The proposed fix: http://www.sakoman.com/cgi-bin/gitweb.cgi?p=u-boot.git;a=commitdiff;h=56b18c5248a1478b6d6263e428ada02af174df77

(if gets rebased, look for "ARMV7: OMAP4: Fix Panda pinmux setting to enable Wifi/BT Module" at omap4-exp branch)

Revision history for this message
Sebastien JAN (sebjan) wrote :

Tested http://www.sakoman.com/cgi-bin/gitweb.cgi?p=u-boot.git;a=commit;h=56b18c5248a1478b6d6263e428ada02af174df77 on top of u-boot-linaro_2010.09~rc1.1 package => solves the issue with the BT controller on panda!

Steve Langasek (vorlon)
Changed in u-boot-linaro (Ubuntu Maverick):
status: New → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

The source package was built with a non-Ubuntu dpkg-dev toolchain, closing manually:

 u-boot-linaro (2010.09~rc1.1-0ubuntu3) maverick; urgency=low
 .
   [ Steve Sakoman ]
   * ARMV7: OMAP4: Fix Panda pinmux setting to enable Wifi/BT Module
     - LP: #645167

Changed in u-boot-linaro (Ubuntu Maverick):
status: Fix Committed → Fix Released
Revision history for this message
John Rigby (jcrigby) wrote :

changing to fix released to match other two instances

Changed in u-boot-linaro:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.