* New pinmux
@ 2010-01-08 15:30 Philip Balister
2010-01-08 18:00 ` [PATCH] omap: Fix cmdline muxing (Re: New pinmux) Tony Lindgren
0 siblings, 1 reply; 5+ messages in thread
From: Philip Balister @ 2010-01-08 15:30 UTC (permalink / raw)
To: linux-omap@vger.kernel.org
I am trying to work out how to use the new pinmux code on my
Overo+Summit system. I have a recent git running with the new pinmux
code built. I am trying to configure the mcspi1_clk pin on the expansion
connector as gpio_171 with the following:
root@overo:~# dmesg |grep mux
Kernel command line: console=ttyS2,115200n8 vram=12M
omapfb.mode=dvi:1024x768MR-16@60 omapfb.debug=y omapdss.def_display=dvi
root=/dev/mmcblk0p2 rw rootfstype=ext3 rootwait
omap_mux=mcspi1_clk.gpio_171=0x0
And this is what happens :(
mux: Could not set signal mcspi1_clk
mux: Setting signal i2c3_scl.i2c3_scl 0x0118 -> 0x0100
Any suggestions?
Philip
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] omap: Fix cmdline muxing (Re: New pinmux)
2010-01-08 15:30 New pinmux Philip Balister
@ 2010-01-08 18:00 ` Tony Lindgren
2010-01-08 18:59 ` Philip Balister
2010-01-08 22:38 ` Philip Balister
0 siblings, 2 replies; 5+ messages in thread
From: Tony Lindgren @ 2010-01-08 18:00 UTC (permalink / raw)
To: Philip Balister; +Cc: linux-omap@vger.kernel.org
[-- Attachment #1: Type: text/plain, Size: 883 bytes --]
* Philip Balister <philip@balister.org> [100108 07:35]:
> I am trying to work out how to use the new pinmux code on my
> Overo+Summit system. I have a recent git running with the new pinmux
> code built. I am trying to configure the mcspi1_clk pin on the
> expansion connector as gpio_171 with the following:
>
> root@overo:~# dmesg |grep mux
>
> Kernel command line: console=ttyS2,115200n8 vram=12M
> omapfb.mode=dvi:1024x768MR-16@60 omapfb.debug=y
> omapdss.def_display=dvi root=/dev/mmcblk0p2 rw rootfstype=ext3
> rootwait omap_mux=mcspi1_clk.gpio_171=0x0
>
> And this is what happens :(
>
> mux: Could not set signal mcspi1_clk
>
> mux: Setting signal i2c3_scl.i2c3_scl 0x0118 -> 0x0100
>
> Any suggestions?
Sounds like the cmdline options got broken by some changes
I did to limit everything to happen as __init..
Can you please try the following patch?
Regards,
Tony
[-- Attachment #2: mux-fix-cmdline.patch --]
[-- Type: text/x-diff, Size: 1188 bytes --]
>From 3b7bbb130c01ec7a2d4337d43bbfbc4e1fe1615b Mon Sep 17 00:00:00 2001
From: Tony Lindgren <tony@atomide.com>
Date: Fri, 8 Jan 2010 09:45:11 -0800
Subject: [PATCH] omap: Fix cmdline muxing
Looks like cmdline muxing got broken at some point when we
decided to limit muxing to __init code. Currently omap_mux_entry
list is not yet initialized when we try to initialize cmdline
muxing.
Fix this by calling omap_mux_init_list() before calling
omap_mux_set_cmdline_signals().
Reported-by: Philip Balister <philip@balister.org>
Signed-off-by: Tony Lindgren <tony@atomide.com>
diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c
index 459ef23..40ea9fd 100644
--- a/arch/arm/mach-omap2/mux.c
+++ b/arch/arm/mach-omap2/mux.c
@@ -998,12 +998,15 @@ int __init omap_mux_init(u32 mux_pbase, u32 mux_size,
omap_mux_package_fixup(package_subset, superset);
if (package_balls)
omap_mux_package_init_balls(package_balls, superset);
- omap_mux_set_cmdline_signals();
- omap_mux_set_board_signals(board_mux);
#endif
omap_mux_init_list(superset);
+#ifdef CONFIG_OMAP_MUX
+ omap_mux_set_cmdline_signals();
+ omap_mux_set_board_signals(board_mux);
+#endif
+
return 0;
}
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] omap: Fix cmdline muxing (Re: New pinmux)
2010-01-08 18:00 ` [PATCH] omap: Fix cmdline muxing (Re: New pinmux) Tony Lindgren
@ 2010-01-08 18:59 ` Philip Balister
2010-01-08 19:35 ` Tony Lindgren
2010-01-08 22:38 ` Philip Balister
1 sibling, 1 reply; 5+ messages in thread
From: Philip Balister @ 2010-01-08 18:59 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap@vger.kernel.org
On 01/08/2010 01:00 PM, Tony Lindgren wrote:
> * Philip Balister<philip@balister.org> [100108 07:35]:
>> I am trying to work out how to use the new pinmux code on my
>> Overo+Summit system. I have a recent git running with the new pinmux
>> code built. I am trying to configure the mcspi1_clk pin on the
>> expansion connector as gpio_171 with the following:
>>
>> root@overo:~# dmesg |grep mux
>>
>> Kernel command line: console=ttyS2,115200n8 vram=12M
>> omapfb.mode=dvi:1024x768MR-16@60 omapfb.debug=y
>> omapdss.def_display=dvi root=/dev/mmcblk0p2 rw rootfstype=ext3
>> rootwait omap_mux=mcspi1_clk.gpio_171=0x0
>>
>> And this is what happens :(
>>
>> mux: Could not set signal mcspi1_clk
>>
>> mux: Setting signal i2c3_scl.i2c3_scl 0x0118 -> 0x0100
>>
>> Any suggestions?
>
> Sounds like the cmdline options got broken by some changes
> I did to limit everything to happen as __init..
>
> Can you please try the following patch?
The patch solved the problem, thanks! Great work guys, this will really
help people working with boards with expansion connectors like the
Beagle and Overo.
Tony, do you need a signoff from me for this? Not to sure what the
procedure is.
Philip
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] omap: Fix cmdline muxing (Re: New pinmux)
2010-01-08 18:59 ` Philip Balister
@ 2010-01-08 19:35 ` Tony Lindgren
0 siblings, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2010-01-08 19:35 UTC (permalink / raw)
To: Philip Balister; +Cc: linux-omap@vger.kernel.org
* Philip Balister <philip@balister.org> [100108 10:57]:
> On 01/08/2010 01:00 PM, Tony Lindgren wrote:
> >* Philip Balister<philip@balister.org> [100108 07:35]:
> >>I am trying to work out how to use the new pinmux code on my
> >>Overo+Summit system. I have a recent git running with the new pinmux
> >>code built. I am trying to configure the mcspi1_clk pin on the
> >>expansion connector as gpio_171 with the following:
> >>
> >>root@overo:~# dmesg |grep mux
> >>
> >>Kernel command line: console=ttyS2,115200n8 vram=12M
> >>omapfb.mode=dvi:1024x768MR-16@60 omapfb.debug=y
> >>omapdss.def_display=dvi root=/dev/mmcblk0p2 rw rootfstype=ext3
> >>rootwait omap_mux=mcspi1_clk.gpio_171=0x0
> >>
> >>And this is what happens :(
> >>
> >>mux: Could not set signal mcspi1_clk
> >>
> >>mux: Setting signal i2c3_scl.i2c3_scl 0x0118 -> 0x0100
> >>
> >>Any suggestions?
> >
> >Sounds like the cmdline options got broken by some changes
> >I did to limit everything to happen as __init..
> >
> >Can you please try the following patch?
>
> The patch solved the problem, thanks! Great work guys, this will
> really help people working with boards with expansion connectors
> like the Beagle and Overo.
Good to hear :)
> Tony, do you need a signoff from me for this? Not to sure what the
> procedure is.
Maybe reply with your Tested-by? See Documentation/SubmittingPatches
for Reported-by/Tested-by etc.
Regards,
Tony
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] omap: Fix cmdline muxing (Re: New pinmux)
2010-01-08 18:00 ` [PATCH] omap: Fix cmdline muxing (Re: New pinmux) Tony Lindgren
2010-01-08 18:59 ` Philip Balister
@ 2010-01-08 22:38 ` Philip Balister
1 sibling, 0 replies; 5+ messages in thread
From: Philip Balister @ 2010-01-08 22:38 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap@vger.kernel.org
On 01/08/2010 01:00 PM, Tony Lindgren wrote:
> * Philip Balister<philip@balister.org> [100108 07:35]:
>> I am trying to work out how to use the new pinmux code on my
>> Overo+Summit system. I have a recent git running with the new pinmux
>> code built. I am trying to configure the mcspi1_clk pin on the
>> expansion connector as gpio_171 with the following:
>>
>> root@overo:~# dmesg |grep mux
>>
>> Kernel command line: console=ttyS2,115200n8 vram=12M
>> omapfb.mode=dvi:1024x768MR-16@60 omapfb.debug=y
>> omapdss.def_display=dvi root=/dev/mmcblk0p2 rw rootfstype=ext3
>> rootwait omap_mux=mcspi1_clk.gpio_171=0x0
>>
>> And this is what happens :(
>>
>> mux: Could not set signal mcspi1_clk
>>
>> mux: Setting signal i2c3_scl.i2c3_scl 0x0118 -> 0x0100
>>
>> Any suggestions?
>
> Sounds like the cmdline options got broken by some changes
> I did to limit everything to happen as __init..
>
> Can you please try the following patch?
>
> Regards,
>
> Tony
Tested-by: Philip Balister <philip@balister.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-01-08 22:38 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-08 15:30 New pinmux Philip Balister
2010-01-08 18:00 ` [PATCH] omap: Fix cmdline muxing (Re: New pinmux) Tony Lindgren
2010-01-08 18:59 ` Philip Balister
2010-01-08 19:35 ` Tony Lindgren
2010-01-08 22:38 ` Philip Balister
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox