Hi, Alan.
Thank you very much for your prompt reply.
On Jul 08 2004, Alan Stern wrote:
> On Thu, 8 Jul 2004, Rogério Brito wrote:
> > Dear Alan Stern,
> >
> > Ealier this year, we exchanged some messages regarding a problem that I had
> > with my USB Drive, which didn't work correctly with my VIA southbridge. In
> > the message quoted below, you advised me to use a patch for my future
> > kernels and that's precisely what I'm doing right now.
>
> I think there wouldn't be any problem having the standard kernel change
> the size of that array to 8 bytes, rather than 64 as in the patch you're
> using. However it would be a good idea to make sure that your device
> will work okay with that change. Use the patch below (in addition to the
> one you've been using all along) and post your system log for when you
> plug in the USB drive.
Well, the patch doesn't apply here cleanly with kernel 2.6.7. Here is what
I have around the line where the patch you just sent me should be applied:
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
default: /* presumably an error */
/* Cause a hub reset after 10 consecutive errors */
dev_dbg (&hub->intf->dev, "transfer --> %d\n", urb->status);
if ((++hub->nerrors < 10) || hub->error)
goto resubmit;
hub->error = urb->status;
/* FALL THROUGH */
/* let khubd handle things */
case 0: /* we got data: port status changed */
break;
}
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
That's the final part of a switch statement. I don't have the for that you
had in your patch. Do you want me to upgrade my kernel tree to yours so
that we can stay in sync?
I will try anyway to put that if right after the "case 0:" clause.
> > 2 - I still have those SCSI layer problems listed below. Can I send you
> > debbuging messages of the kernels that I compile so we can try to track
> > what the reason of the problem is? Or should I take this with someone
> > else?
>
> Go ahead and post the debugging messages here. Be sure to turn on the
> usb-storage debugging option in the kernel configuration.
Ok. Should I do that after the issue above is resolved?
> Alan Stern
>
>
> +++ 2.6.7/drivers/usb/core/hub.c Thu Jul 8 11:07:24 2004
> @@ -257,6 +257,9 @@
>
> /* let khubd handle things */
> case 0: /* we got data: port status changed */
> + if (urb->actual_length > 1)
> + printk(KERN_INFO "hub_irq: actual_length = %d\n",
> + urb->actual_length);
> bits = 0;
> for (i = 0; i < urb->actual_length; ++i)
> bits |= ((unsigned long) ((*hub->buffer)[i]))
>
Thanks, Rogério.
--
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Rogério Brito - [EMAIL PROTECTED] - http://www.ime.usp.br/~rbrito
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 -
digital self defense, top technical experts, no vendor pitches,
unmatched networking opportunities. Visit www.blackhat.com
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
