From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: writable mmc profiles actually are writable Date: Tue, 7 Oct 2003 08:48:29 +0200 Sender: linux-scsi-owner@vger.kernel.org Message-ID: <20031007064829.GQ972@suse.de> References: <1065463946.5185.8.camel@patehci2> <20031006182223.GF972@suse.de> <20031006182510.GG972@suse.de> <1065469831.5185.44.camel@patehci2> <20031006203816.GJ972@suse.de> <1065473899.6835.41.camel@patehci2> <1065478448.2361.1.camel@patehci2> <1065484607.2899.7.camel@patehci2> <20031007053858.GL972@suse.de> <20031006234543.A11937@one-eyed-alien.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from ns.virtualhost.dk ([195.184.98.160]:16861 "EHLO virtualhost.dk") by vger.kernel.org with ESMTP id S261806AbTJGGse (ORCPT ); Tue, 7 Oct 2003 02:48:34 -0400 Content-Disposition: inline In-Reply-To: <20031006234543.A11937@one-eyed-alien.net> List-Id: linux-scsi@vger.kernel.org To: Pat LaVarre , linux-scsi@vger.kernel.org Cc: mdharm-scsi@one-eyed-alien.net On Mon, Oct 06 2003, Matthew Dharm wrote: > On Tue, Oct 07, 2003 at 07:38:58AM +0200, Jens Axboe wrote: > > On Mon, Oct 06 2003, Pat LaVarre wrote: > > > Yep, still happy, same in -test6 as in -test5 e.g. > > > > > > $ dmesg | grep ^sr > > > sr0: scsi3-mmc drive: 125x/125x caddy > > > sr0: scsi3-mmc maybe not writeable > > > sr0: scsi3-mmc writable profile: 0x0002 > > > $ > > > $ grep MMC /proc/sys/dev/cdrom/info > > > Can write other MMC-RW: 1 ... > > > > I don't quite agree with this patch. Please add the GET_CONFIGURATION > > stuff to cdrom.c instead so it works with ide-cd as well (ide-scsi is > > basically dead, and the number of scsi drives out there isn't exactly > > big and increasing). GPCMD_GET_CONFIGURATION already exists. Also, make > > DVD-RAM drives set CDC_MMC_RW as well and only check for that at open. > > While the number of SCSI CD-ROM drives may be shrinking, the ATAPI drives > behind a USB/ATAPI bridge is skyrocketing. Doesn't change the picture, if you put it in cdrom.c it will work for both. -- Jens Axboe