-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 12/12/12 11:28 AM, Guillaume Gardet wrote:
Le 12/12/2012 17:19, Jeff Mahoney a écrit : On 12/12/12 11:17 AM, Jeff Mahoney wrote:
On 12/12/12 11:10 AM, Greg Kroah-Hartman wrote:
On 12/12/12 10:07 AM, Greg KH wrote: > On Wed, Dec 12, 2012 at 11:39:33AM +0100, Guillaume > Gardet wrote: >> Hi, >> >> please find in attachment a patch to fix armv7 >> kernel-default config. >> >> CONFIG_USB_EZUSB_FX2 must be a module instead of >> built-in to fix build > What build problem? This should be resolved properly > upstream if it is still an issue. I thought I fixed it > a while ago, if not, please let me know upstream. Hi Greg -
Here's what I'm seeing on x86_64 w/ 3.7-final: ERROR: "ezusb_fx1_ihex_firmware_download" [drivers/usb/serial/whiteheat.ko] undefined! ERROR: "ezusb_fx1_ihex_firmware_download" [drivers/usb/serial/keyspan_pda.ko] undefined! ERROR: "ezusb_fx1_set_reset" [drivers/usb/serial/keyspan_pda.ko] undefined! ERROR: "ezusb_fx1_ihex_firmware_download" [drivers/usb/serial/keyspan.ko] undefined!
Somehow the exported symbols aren't being made available to modules when ezusb is a built-in. Is USB being built as a module? If so, can you add commit 9db72fe6c943852032d9ed863c87e8c02d3cb9da in Linus's kernel
On Wed, Dec 12, 2012 at 10:59:21AM -0500, Jeff Mahoney wrote: tree to your kernel and see if that fixes the issue? It's being queued up right now for the 3.7.1 release and should solve this. If not, it would be great to know. Ah. Perfect. That's the patch I came up with as well. It does fix it. Guillaume -
Your patch is still valid since this will force CONFIG_USB_EZUSB_FX2=m.
I'll commit both and we'll bet set here.
Jeff,
My patch was just a workaround to fix the build, then no need to force CONFIG_USB_EZUSB_FX2 to build as a module instead of built-in if not needed. But if it is better (desired/mandatory) to have it as a module, we should fix all the kernel configs using CONFIG_USB_EZUSB_FX2, for consistency.
All the configs other than ARM (and vanilla, strange) had =m already to avoid this issue. I just didn't have time to look into it that day. This patch forces all of them to =m. I didn't use your patch directly as I just ran scripts/run_oldconfig.sh after applying the patch to update the configs. - -Jeff - -- Jeff Mahoney SUSE Labs -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.18 (Darwin) Comment: GPGTools - http://gpgtools.org Comment: Using GnuPG with undefined - http://www.enigmail.net/ iQIcBAEBAgAGBQJQyLHvAAoJEB57S2MheeWyjsMP/1qIz0jyrJE9LM2ORoEuy2Bn +A1joy871rKy52lQKbCaQ640A2r1L2vMNLxnSqGI3g5ly+BIuPLn25L6lq4GPHY9 8D1mYe0RS4aFEBb7PM2tfmbku2/y0KUXGWjxmThceIpAGRqKy6XNDtCD9sAOJ6PG lfZHqyFu2WEeVI7Zr9XZLWUk2oYsk1s2J3ySaJX6fmCYVahmZy0lJPcXaJHHDl+1 3QuSOWt4beAGFLOfIIwKKXco5IAy55wPXMDo6zFsXjeNhY+LZcalLmL0M6Pl1zoQ d+lTJJVlxaXueeDYHPorzbbcJYFzb2NolnwSaovE/WwtrcX4FQt+TOTieoNePF84 4twssEre+2N9BFScHQNNqwTPazYZOSr3z+R8V8XW906I+GC2+6gA8sEUr4EMtTzg ZsPNWb5sVL6s3z/IDOUDEVGP0KDJpdjcbOgkEG9/pbInUdufbwxJaVgMnV/Lf0qe eCdHSpdPNngnJXZAhmoCLq8qRxHca2i9/et1rAQH4Q7Ntd1Z234/c6/mdLRb/k5h 86Uw13cKNg/3TU2JpraZhxHKY8v/pZOwUopwvPh6HAWO7Y/5srQSwB6mZO8aGSOD LtLT3SzaPW97bKtTZqDk/eeOhIQvVLa+HJY+cwaQCW3AsRqcW+uoL0A9cH0QjJW0 Hc01A2Qrbyg/CW1Cz1ef =1/Wx -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-kernel+owner@opensuse.org