Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262029AbTERMgF (ORCPT ); Sun, 18 May 2003 08:36:05 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262031AbTERMgE (ORCPT ); Sun, 18 May 2003 08:36:04 -0400 Received: from hermes.fachschaften.tu-muenchen.de ([129.187.202.12]:39624 "HELO hermes.fachschaften.tu-muenchen.de") by vger.kernel.org with SMTP id S262029AbTERMgA (ORCPT ); Sun, 18 May 2003 08:36:00 -0400 Date: Sun, 18 May 2003 14:48:51 +0200 From: Adrian Bunk To: Marcelo Tosatti , alan@redhat.com Cc: linux-kernel , Erik Andersen , trivial@rustcorp.com.au Subject: [2.4 patch] 2.4.21-rc1 pointless IDE noise reduction Message-ID: <20030518124851.GB12766@fs.tum.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3129 Lines: 100 The patch forwarded below still applies against 2.4.21-rc2. Please apply or explain why you reject it. TIA Adrian ----- Forwarded message from Adrian Bunk ----- Date: Sun, 27 Apr 2003 14:21:04 +0200 From: Adrian Bunk To: Erik Andersen , Marcelo Tosatti , linux-kernel , alan@redhat.com Subject: Re: [PATCH] 2.4.21-rc1 pointless IDE noise reduction On Thu, Apr 24, 2003 at 03:34:43AM -0600, Erik Andersen wrote: > The ide driver does not list whether drives support things like > write cache, SMART, SECURITY ERASE UNIT. But for some silly > reason it tells us at boot whether each drive is capable of > supporting the Host Protected Area feature set. If people want > to know the capabilites of their drive, they can run 'hdparm' > and find out. > > This patch removes this pointless noise. Please apply, > > > --- linux/drivers/ide/ide-disk.c.orig 2003-04-24 03:23:53.000000000 -0600 > +++ linux/drivers/ide/ide-disk.c 2003-04-24 03:24:54.000000000 -0600 > @@ -1133,10 +1133,7 @@ > */ > static inline int idedisk_supports_host_protected_area(ide_drive_t *drive) > { > - int flag = (drive->id->cfs_enable_1 & 0x0400) ? 1 : 0; > - if (flag) > - printk("%s: host protected area => %d\n", drive->name, flag); > - return flag; > + return((drive->id->cfs_enable_1 & 0x0400) ? 1 : 0); > } > > /* Looking at the only user of this function it seems we can completely remove it (patch below). Alan: Is the patch below OK or are there any future plans for more uses of idedisk_supports_host_protected_area? > -Erik cu Adrian --- linux-2.4.21-rc1-full/drivers/ide/ide-disk.c.old 2003-04-27 13:26:17.000000000 +0200 +++ linux-2.4.21-rc1-full/drivers/ide/ide-disk.c 2003-04-27 13:30:48.000000000 +0200 @@ -1128,18 +1128,6 @@ #endif /* CONFIG_IDEDISK_STROKE */ /* - * Tests if the drive supports Host Protected Area feature. - * Returns true if supported, false otherwise. - */ -static inline int idedisk_supports_host_protected_area(ide_drive_t *drive) -{ - int flag = (drive->id->cfs_enable_1 & 0x0400) ? 1 : 0; - if (flag) - printk("%s: host protected area => %d\n", drive->name, flag); - return flag; -} - -/* * Compute drive->capacity, the full capacity of the drive * Called with drive->id != NULL. * @@ -1165,8 +1153,6 @@ drive->capacity48 = 0; drive->select.b.lba = 0; - (void) idedisk_supports_host_protected_area(drive); - if (id->cfs_enable_2 & 0x0400) { capacity_2 = id->lba_capacity_2; drive->head = drive->bios_head = 255; - 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/ ----- End forwarded message ----- - 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/