Re: problem w/ USB cdrom on 5.2

Thank you! Your patch fixes the problem for me.

Yuri

sanpei_at_freebsd.org wrote:
>>>Doug White wrote:
>>>
>>>>On Tue, 13 Jan 2004, Yuri Khotyaintsev wrote:
>>>>
>>>>
>>>>
>>>>>I cannot use Dell USB cdrom with 5.2-RELEASE.
>>>>>The same drive works without any problems on 4.9.
>>>>>
>>>>>Any clues? Boot log attached.
>>>>
>>>>
>>>>Not much info here ... uhci, some phase error on initializing umass. What
>>>>messages do you get if you boot up without it attached, then plug it in?
>>>
>>>The messages are:
>>>
>>>umass0: Dell USB Drive A01, rev 2.00/1.04, addr 2
>>>umass0: Get Max Lun not supported (STALLED)
>>>umass0:1:0:-1: Attached to scbus1
>>>umass0: Phase Error, residue = 0
>>>(probe0:umass-sim0:0:0:0): Request completed with CAM_REQ_CMP_ERR
>>>(probe0:umass-sim0:0:0:0): Retrying Command
>>>umass0: Phase Error, residue = 0
>>>(probe0:umass-sim0:0:0:0): Request completed with CAM_REQ_CMP_ERR
>>>(probe0:umass-sim0:0:0:0): Retrying Command
>>>umass0: Phase Error, residue = 0
>>>(probe0:umass-sim0:0:0:0): Request completed with CAM_REQ_CMP_ERR
>>>(probe0:umass-sim0:0:0:0): Retrying Command
>>>umass0: Phase Error, residue = 0
>>>(probe0:umass-sim0:0:0:0): Request completed with CAM_REQ_CMP_ERR
>>>(probe0:umass-sim0:0:0:0): Retrying Command
>>>umass0: Phase Error, residue = 0
>>>(probe0:umass-sim0:0:0:0): Request completed with CAM_REQ_CMP_ERR
>>>(probe0:umass-sim0:0:0:0): error 5
>>>(probe0:umass-sim0:0:0:0): Retries Exausted
> 
> 
>   I have same problem with Panasonic KXL-RW32AN USB CD-RW Drive.
>   with uhci and ohci.
>   (it works on 4-stable...)
> umass0: GENERIC CDRRW09 USB CD-R/RW drive, rev 2.00/11.10, addr 2
> umass0: Phase Error, residue = 95
> umass0: Phase Error, residue = 95
> umass0: Phase Error, residue = 95
> umass0: Phase Error, residue = 95
> umass0: Phase Error, residue = 95
> 
>   I use it on 5-current with below durty patch. Try it.
> 
> Index: src/sys/dev/usb/umass.c
> ===================================================================
> RCS file: /home/ncvs/src/sys/dev/usb/umass.c,v
> retrieving revision 1.98
> diff -u -r1.98 umass.c
> --- src/sys/dev/usb/umass.c	3 Jan 2004 13:52:44 -0000	1.98
> +++ src/sys/dev/usb/umass.c	16 Jan 2004 14:59:37 -0000
> _at__at_ -1570,8 +1570,10 _at__at_
>  			printf("%s: Phase Error, residue = %d\n",
>  				USBDEVNAME(sc->sc_dev), Residue);
>  
> +#if 0 /* XXX */
>  			umass_bbb_reset(sc, STATUS_WIRE_FAILED);
>  			return;
> +#endif
>  
>  		} else if (sc->transfer_actlen > sc->transfer_datalen) {
>  			/* Buffer overrun! Don't let this go by unnoticed */
> 
> 
> ---
> MIHIRA, Sanpei Yoshiro
> Tokyo, Japan.
> 
> 

-- 

Yuri Khotyaintsev, PhD

Swedish Institute of Space Physics,  http://www.cluster.irfu.se/yuri 

Uppsala Division (IRF-U)             http://ovt.irfu.se