From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754140AbYBPJGL (ORCPT ); Sat, 16 Feb 2008 04:06:11 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752114AbYBPJFt (ORCPT ); Sat, 16 Feb 2008 04:05:49 -0500 Received: from fk-out-0910.google.com ([209.85.128.188]:16260 "EHLO fk-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752058AbYBPJFp (ORCPT ); Sat, 16 Feb 2008 04:05:45 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=date:to:cc:subject:message-id:reply-to:mail-followup-to:references:mime-version:content-type:content-disposition:content-transfer-encoding:in-reply-to:user-agent:from; b=bPmU4aHxFV1gChS/lvmUaJcHeyQWOzuWdiSadQnP1NaMJTwvdRPueQdVuLMKNTV+2lM2XdZJV8evQ9tX7nkp41N5SQEB7wKO4i/Yw/WdF2Dz7W8NYKjXtZwh65aTEXcrw8PM38SwKiDFX/qW7NZrCcgSUeVGWoxiWvZdC/gRJnM= Date: Sat, 16 Feb 2008 10:05:32 +0100 To: Stefan Bader Cc: bzolnier@gmail.com, linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org Subject: Re: Optiarc DVD RW AD-5200A audio playing Message-ID: <20080216090532.GK13446@gollum.tnic> Reply-To: petkovbb@gmail.com Mail-Followup-To: petkovbb@gmail.com, Stefan Bader , bzolnier@gmail.com, linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org References: <47B5EDB7.5050709@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <47B5EDB7.5050709@canonical.com> User-Agent: Mutt/1.5.17 (2007-11-01) From: Borislav Petkov Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Feb 15, 2008 at 02:53:27PM -0500, Stefan Bader wrote: > Hello Borislav, > > I worked on a problem with an DVD driver (model=Optiarc DVD RW AD-5200A) > which obviously has the same problem as some Matshita drives. The > following patch was reported to enabled audio playing on this drive. > Would this approach be suitable for upstream or are there other > solutions to this problem? > > Regards, > Stefan > > --- a/drivers/ide/ide-cd.c > +++ b/drivers/ide/ide-cd.c > @@ -2988,7 +2988,8 @@ int ide_cdrom_probe_capabilities (ide_drive_t *drive) > if (strcmp(drive->id->model, "MATSHITADVD-ROM SR-8187") == 0 || > strcmp(drive->id->model, "MATSHITADVD-ROM SR-8186") == 0 || > strcmp(drive->id->model, "MATSHITADVD-ROM SR-8176") == 0 || > - strcmp(drive->id->model, "MATSHITADVD-ROM SR-8174") == 0) > + strcmp(drive->id->model, "MATSHITADVD-ROM SR-8174") == 0 || > + strcmp(drive->id->model, "Optiarc DVD RW AD-5200A") == 0) > CDROM_CONFIG_FLAGS(drive)->audio_play = 1; > > #if ! STANDARD_ATAPI Hi Stefan, just to make sure that the audioplay bit is not set in the capabilities page, can you please try the following patch applied against 2.6.25-rc2 and send me the output. Thanks! @Bart: by the way, this cdi->mask thingy is kinda unintuitive doing double negation to check whether a feature is supported or not. Yeah, this comes from "above," i.e. uniform cdrom layer. But still, shouldn't we use a cdi->caps_flags or something similar instead, which mirrors the caps page bits setting? commit 435f0f4496a1b32af2d542f43b2370a890fe2f83 Author: Borislav Petkov Date: Sat Feb 16 09:56:36 2008 +0100 ide-cd: Enable audio play quirk for Optiarc DVD RW AD-5200A drive Reported-by: Stefan Bader Signed-off-by: Borislav Petkov diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c index f77db6b..2c9d06e 100644 --- a/drivers/ide/ide-cd.c +++ b/drivers/ide/ide-cd.c @@ -1750,6 +1750,10 @@ int ide_cdrom_probe_capabilities (ide_drive_t *drive) cdi->mask &= ~(CDC_DVD_RAM | CDC_RAM); if (buf[8 + 3] & 0x10) cdi->mask &= ~CDC_DVD_R; + if (!(buf[8 + 4] & 0x01)) { + printk(KERN_INFO "ide-cd: audio play not advertised in caps page," + " enabling quirk.\n"); + } if ((buf[8 + 4] & 0x01) || (cd->cd_flags & IDE_CD_FLAG_PLAY_AUDIO_OK)) cdi->mask &= ~CDC_PLAY_AUDIO; @@ -1929,6 +1933,7 @@ static const struct cd_list_entry ide_cd_quirks_list[] = { { "MATSHITADVD-ROM SR-8186", NULL, IDE_CD_FLAG_PLAY_AUDIO_OK }, { "MATSHITADVD-ROM SR-8176", NULL, IDE_CD_FLAG_PLAY_AUDIO_OK }, { "MATSHITADVD-ROM SR-8174", NULL, IDE_CD_FLAG_PLAY_AUDIO_OK }, + { "Optiarc DVD RW AD-5200A", NULL, IDE_CD_FLAG_PLAY_AUDIO_OK }, { NULL, NULL, 0 } }; -- Regards/Gruß, Boris.