Hi Bart,

On Wednesday 24 October 2007 00:33:08 Bartlomiej Zolnierkiewicz wrote:
quoted text
> Hi,
>
> > > hdparm --Istdout /dev/hdd
>
> Thanks, the identify block looks quite "interesting".
[...]
quoted text
> word 93 is 0x2000
>
> bit 0x4000 is not set despite the fact that ATA spec (>= ATA-5) requires
> it to be set (the device claims ATA/ATAPI-3/4/5/6/7 compatiblity, a bit too
> optimistic since it looks like the firmware was based on ATA/ATAPI-4 spec)
>
> bit 0x2000 is set which would indicate that the 80-wires cable is
> correctly detected by the device
>
> => the device/firmware pair is a good candidate for ivb_list[]

Interesting, I fully understand.

quoted text
> There seems to be a new firmware (SB01) for this device:
> http://www.samsungodd.com/Lib/popup/Download.asp?path=FW_FWDownload&fname=2
>00710011656260232_SH-S202J_%20SB01.exe
> It would be useful to know whether it has the same problem...

I cannot use this - I haven't used windows at home for a few years, and have 
no way to flash the device up.  It would be interesting though if this does 
make it conform.

quoted text
> Could you try this patch?
>
> [PATCH] ide: add SH-S202J to ivb_list[]

Thank you!  This works very well!


hdd: ATAPI 48X DVD-ROM DVD-R-RAM CD-R/RW drive, 2048kB Cache<7>hdd: skipping 
word 93 validity check
, UDMA(66)


Many thanks indeed!

Nick

quoted text
> From the report by Nick Warne.
>
> Cc: Nick Warne <nick@ukfsn.org>
> Cc: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>
> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
> ---
>  drivers/ide/ide-iops.c |    3 +++
>  1 file changed, 3 insertions(+)
>
> Index: b/drivers/ide/ide-iops.c
> ===================================================================
> --- a/drivers/ide/ide-iops.c
> +++ b/drivers/ide/ide-iops.c
> @@ -582,9 +582,12 @@ EXPORT_SYMBOL_GPL(ide_in_drive_list);
>  /*
>   * Early UDMA66 devices don't set bit14 to 1, only bit13 is valid.
>   * We list them here and depend on the device side cable detection for
> them. + *
> + * Some optical devices with the buggy firmwares have the same problem.
>   */
>  static const struct drive_list_entry ivb_list[] = {
>  	{ "QUANTUM FIREBALLlct10 05"	, "A03.0900"	},
> +	{ "TSSTcorp CDDVDW SH-S202J"	, "SB00"	},
>  	{ NULL				, NULL		}
>  };



-- 
Free Software Foundation Associate Member 5508
-
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/