uas: HACK needed for UAS host.
Subsystems (2)
PATCH 5/5 uas: HACK needed for UAS host.
REMOVE LATER ON!
In today's UAS host driver MODE SENSE command isn't handled correctly. Each time after power up there is a unit attention condition of RESET_OCCURRED that needs to be cleared (= acknowledged by the host) by the MODE SENSE command.
Due to the above this patch resets the RESET_OCCURRED unit attention.
drivers/usb/gadget/uasp_cmdiu.c +7 -0
@@ -83,6 +83,13 @@ static __u32 check_cmdiu(struct uasp_dev *udev, curlun->lun->unit_attention_data =SS_NO_SENSE ;return ua_data; } +/* HACK!!! REMOVE!!! */ +if (curlun->lun->unit_attention_data ==SS_RESET_OCCURRED && + cmdiu->cdb[0 ] ==INQUIRY ) { + curlun->lun->unit_attention_data =SS_NO_SENSE ; +DBG (udev->ucommon->common,"check_cmdiu() - " +"HACK!!! RESETTING unit attention condition\n" ); + } }if (curlun && !(curlun->lun->filp) && needs_medium) {
Signature · 5 lines
1.7.0.4 -- Sent by a Consultant for Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum
Quoted 1 line · Tatyana Brokhman
REMOVE LATER ON!
Remove what? The patch or this description?
confused,
greg k-h
This patch is not for submission, it's a HACK and the bug should be fixed in another way. I've uploaded it because it's needed at the moment for working with the UAS gadget driver and the current version of UAS host. I'm working on resolving this issue with Sarah on another email thread. Just thought that if someone wishes to use the driver right now this patch will be useful to him.
Best regards,
Tanya Brokhman
Consultant for Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum
Quoted 16 lines
-----Original Message-----
From: Greg KH [mailto:[email protected]]
Sent: Monday, March 07, 2011 6:52 PM
To: Tatyana Brokhman
Cc: [email protected]; [email protected]; open list:USB GADGET/PERIPH...; open list
Subject: Re: [RFC/PATCH 5/5] uas: HACK needed for UAS host.Quoted 1 line · Tatyana Brokhman
REMOVE LATER ON!
Remove what? The patch or this description?
confused,
greg k-h