public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* USB/Storage - transport.c - Olympus D150Zoom
@ 2002-12-21  3:31 Scorpion
  0 siblings, 0 replies; only message in thread
From: Scorpion @ 2002-12-21  3:31 UTC (permalink / raw)
  To: linux-kernel



Hi,
I was trying to put my digital camera Olympus Brio Zoom D-150Zoom
to work on my RedHat 7.3 (2.4.18-3, redhat) when found this web page:
http://www.gingerbear.org/~esm/olympus/

Clicking on transport.c.diff link and taking a look into
/usr/src/linux-2.4.18-3/drivers/usb/storage/transport.c file
I started to ask my self what is doing the
if (bcs.Signature != cpu_to_le32(US_BULK_CS_SIGN) ||
statement there? Please if anyone could, answer me...
The patch applied to "support" this camera just remove this comparison,
so what it does?

Best regards,
Scorpion.
---------------transport.c.diff----------------
--- drivers/usb/storage/transport.c	2002/08/07 13:14:59	1.1
+++ drivers/usb/storage/transport.c	2002/08/07 13:15:08
@@ -1197,8 +1197,7 @@
 	US_DEBUGP("Bulk status Sig 0x%x T 0x%x R %d Stat 0x%x\n",
 		  le32_to_cpu(bcs.Signature), bcs.Tag, 
 		  bcs.Residue, bcs.Status);
-	if (bcs.Signature != cpu_to_le32(US_BULK_CS_SIGN) || 
-	    bcs.Tag != bcb.Tag || 
+	if (bcs.Tag != bcb.Tag || 
 	    bcs.Status > US_BULK_STAT_PHASE || partial != 13) {
 		US_DEBUGP("Bulk logical error\n");
 		return USB_STOR_TRANSPORT_ERROR;
---------------transport.c.diff----------------



^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2002-12-21  4:17 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-12-21  3:31 USB/Storage - transport.c - Olympus D150Zoom Scorpion

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox