public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@sunsite•dk>
To: Aaron Sells <aaron.sells@zin-tech•com>
Cc: linuxppc-embedded@ozlabs•org
Subject: Re: SecretLab 2.6.24 with USB
Date: Wed, 23 Apr 2008 17:12:59 +0200	[thread overview]
Message-ID: <87iqy8haf8.fsf@macbook.be.48ers.dk> (raw)
In-Reply-To: <480F4E8E.4010506@zin-tech.com> (Aaron Sells's message of "Wed\, 23 Apr 2008 10\:58\:22 -0400")

>>>>> "Aaron" == Aaron Sells <aaron.sells@zin-tech•com> writes:

Hi,

 Aaron> root@xilinx-ml403:/# dmesg | grep -i usb
 Aaron> [    0.175963] usbcore: registered new interface driver usbfs
 Aaron> [    0.178207] usbcore: registered new interface driver hub
 Aaron> [    0.179810] usbcore: registered new device driver usb
 Aaron> [    3.426016] usbcore: registered new interface driver ub
 Aaron> [    4.156086] usbmon: debugfs is not available
 Aaron> [    4.209160] Initializing USB Mass Storage driver...
 Aaron> [    4.268302] usbcore: registered new interface driver usb-storage
 Aaron> [    4.336109] USB Mass Storage support registered.
 Aaron> [    4.958294] usbcore: registered new interface driver usbhid
 Aaron> [    5.020120] drivers/hid/usbhid/hid-core.c: v2.6:USB HID core driver

Yes, the USB core and drivers gets loaded, but because you haven't
provided a struct platform_device the c67x00 driver doesn't get
probed.

 Aaron> It still doesn't look like the ML403 Cypress host device is being
 Aaron> recognized.  No messages appear upon insertion and removal of USB
 Aaron> devices.

 Aaron> Regards,
 Aaron> Aaron Sells
 Aaron> diff linux-2.6-xlnx_arch-ppc-syslib-virtex_devices.c secretlab_arch-ppc-syslib-virtex_devices.c
 Aaron> 77,92d76
 Aaron> <  * ML300/ML403 Video Device: shortcut macro for single instance

Please use diff -u.

I don't see anything with platform devices for the c67x00 driver. You
need something like:

#include <linux/usb/c67x00.h>

static struct resource c67x00_resources[] = {
	[0] = {
		.start	= 0x84000000,
		.end	= 0x8400000f,
		.flags	= IORESOURCE_MEM,
	},
	[1] = {
		.start	= 3,
		.end	= 3,
		.flags	= IORESOURCE_IRQ,
	},
};

static struct c67x00_platform_data thinlite_c67x00_data = {
	.sie_config		= C67X00_SIE1_HOST | C67X00_SIE2_PERIPHERAL_B,
	.hpi_regstep		= 0x02, /* A0 not connected on 16bit bus */
};

static struct platform_device thinlite_c67x00 = {
	.name			= "c67x00",
	.id			= 0,
	.num_resources		= ARRAY_SIZE(c67x00_resources),
	.resource		= c67x00_resources,
	.dev.platform_data	= &thinlite_c67x00_data,
};

Which you then register with the platform bus with
platform_add_devices or similar. Either you do this by hand or you
create XPAR_* macros which take the needed info out of xparameter.h
(sorry, I don't have an xparameters.h with those defines at hand).

 Aaron> diff -Naur c67x00.old/c67x00-drv.c c67x00/c67x00-drv.c
 Aaron> --- c67x00.old/c67x00-drv.c	2008-04-23 10:36:16.000000000 -0400
 Aaron> +++ c67x00/c67x00-drv.c	2008-04-23 10:39:20.000000000 -0400
 Aaron> @@ -57,8 +57,7 @@
 Aaron>  		c67x00_hcd_probe(sie);
 Aaron>  		break;
 
 Aaron> -	case C67X00_SIE_PERIPHERAL_A:
 Aaron> -	case C67X00_SIE_PERIPHERAL_B:
 Aaron> +	case C67X00_SIE_PERIPHERAL:

This seems to be from an earlier version of my driver.

-- 
Bye, Peter Korsgaard

  reply	other threads:[~2008-04-23 15:13 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-15  2:27 SecretLab 2.6.24 with USB Sam Karp
2008-04-21 18:09 ` Grant Likely
2008-04-22 19:23   ` aaronsells
2008-04-22 20:09     ` Grant Likely
2008-04-22 20:46       ` Aaron Sells
2008-04-22 20:55         ` Grant Likely
2008-04-23 14:58           ` Aaron Sells
2008-04-23 15:12             ` Peter Korsgaard [this message]
2008-04-23 15:41               ` Henrik Larson
     [not found]                 ` <800687fe0804231007l5fdf664fn1d94a1e40dd7fe74@mail.gmail.com>
2008-04-23 17:47                   ` Fwd: " Sam Karp
2008-04-23 18:27                     ` Peter Korsgaard
2008-04-24 16:04               ` Aaron Sells
2008-04-24 18:25                 ` Peter Korsgaard
2008-04-23  6:43     ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87iqy8haf8.fsf@macbook.be.48ers.dk \
    --to=jacmet@sunsite$(echo .)dk \
    --cc=aaron.sells@zin-tech$(echo .)com \
    --cc=linuxppc-embedded@ozlabs$(echo .)org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox