patch for "failed to set xfermode (err_mask=0x4)" with PIONEER DVD-RW DVR-212D

patch for "failed to set xfermode (err_mask=0x4)" with PIONEER DVD-RW DVR-212D

[Francesco Antonacci]

[-- Attachment #1: Type: text/plain, Size: 64 bytes --]

Signed-off-by https://bugzilla.kernel.org/show_bug.cgi?id=27502

[-- Attachment #2: libata-fix.patch --]
[-- Type: text/plain, Size: 391 bytes --]

--- linux-2.6.36/drivers/ata/libata-core.c	2010-10-20 16:30:22.000000000 -0400
+++ linux-2.6.36/drivers/ata/libata-core.c.modified	2010-11-04 15:08:35.845000045 -0400
@@ -4258,6 +4258,8 @@
 	 * device and controller are SATA.
 	 */
 	{ "PIONEER DVD-RW  DVRTD08",	"1.00",	ATA_HORKAGE_NOSETXFER },
+	{ "PIONEER DVD-RW  DVR-212D",	"1.28",	ATA_HORKAGE_NOSETXFER },
+	
 
 	/* End Marker */
 	{ }

[PATCH #upstream-fixes] libata: DVR-212D can't do SETXFER DVD-RW DVR-212D

[Tejun Heo]

From: Francesco Antonacci <fraanto@gmail.com>

PIONEER DVR-212D can't to SETXFER like its sibling DVRTD08.  Add
ATA_HORKAGE_NOSETXFER for it.  Reported in bko#27502.

  https://bugzilla.kernel.org/show_bug.cgi?id=27502

Signed-off-by: Francesco Antonacci <fraanto@gmail.com>
Acked-by: Tejun Heo <tj@kernel.org>
---
Heh, that was formatted rather strangely.  Here's properly reformatted
version.

Thanks.

 drivers/ata/libata-core.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index a31fe96..d4e52e2 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -4138,6 +4138,7 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = {
 	 * device and controller are SATA.
 	 */
 	{ "PIONEER DVD-RW  DVRTD08",	"1.00",	ATA_HORKAGE_NOSETXFER },
+	{ "PIONEER DVD-RW  DVR-212D",	"1.28", ATA_HORKAGE_NOSETXFER },
 
 	/* End Marker */
 	{ }

Re: patch for "failed to set xfermode (err_mask=0x4)" with PIONEER DVD-RW DVR-212D

[Sergei Shtylyov]

Hello.

Francesco Antonacci wrote:

> Signed-off-by https://bugzilla.kernel.org/show_bug.cgi?id=27502

    You should sign off with your name and should place the signoff within the 
patch file. However, the patch should preferably be inserted into the mail in-line.

WBR, Sergei

Re: [PATCH #upstream-fixes] libata: DVR-212D can't do SETXFER DVD-RW DVR-212D

[Sergei Shtylyov]

Hello.

Tejun Heo wrote:

> From: Francesco Antonacci <fraanto@gmail.com>

> PIONEER DVR-212D can't to SETXFER like its sibling DVRTD08.  Add

    s/to/do/?

> ATA_HORKAGE_NOSETXFER for it.  Reported in bko#27502.

>   https://bugzilla.kernel.org/show_bug.cgi?id=27502

> Signed-off-by: Francesco Antonacci <fraanto@gmail.com>
> Acked-by: Tejun Heo <tj@kernel.org>

WBR, Sergei

[PATCH UPDATED #upstream-fixes] libata: DVR-212D can't do SETXFER DVD-RW DVR-212D

[Tejun Heo]

From: Francesco Antonacci <fraanto@gmail.com>

PIONEER DVR-212D can't do SETXFER like its sibling DVRTD08.  Add
ATA_HORKAGE_NOSETXFER for it.  Reported in bko#27502.

  https://bugzilla.kernel.org/show_bug.cgi?id=27502

Signed-off-by: Francesco Antonacci <fraanto@gmail.com>
Acked-by: Tejun Heo <tj@kernel.org>
---
s/to/do/ as pointed out by Sergei.  :-)

Thanks.

 drivers/ata/libata-core.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index a31fe96..d4e52e2 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -4138,6 +4138,7 @@ static const struct ata_blacklist_entry ata_device_blacklist [] = {
 	 * device and controller are SATA.
 	 */
 	{ "PIONEER DVD-RW  DVRTD08",	"1.00",	ATA_HORKAGE_NOSETXFER },
+	{ "PIONEER DVD-RW  DVR-212D",	"1.28", ATA_HORKAGE_NOSETXFER },
 
 	/* End Marker */
 	{ }