Allt som har med skatter är per definition inte logiskt!



On Wed, 23 Apr 2008, Peter Korsgaard wrote:

"Aaron" == Aaron Sells <[EMAIL PROTECTED]> writes:

Hi,

Aaron> [EMAIL PROTECTED]:/# 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
_______________________________________________
Linuxppc-embedded mailing list
Linuxppc-embedded@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-embedded

_______________________________________________
Linuxppc-embedded mailing list
Linuxppc-embedded@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-embedded

Reply via email to