From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [PATCH 17/21] ide: drop 'initializing' argument from ide_register_hw() Date: Thu, 13 Dec 2007 14:54:48 +0100 Message-ID: <200712131454.48336.bzolnier@gmail.com> References: <200711182331.23988.bzolnier@gmail.com> <200712122236.46043.bzolnier@gmail.com> <47612D60.4050408@ru.mvista.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from ug-out-1314.google.com ([66.249.92.170]:9182 "EHLO ug-out-1314.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753994AbXLMNps (ORCPT ); Thu, 13 Dec 2007 08:45:48 -0500 Received: by ug-out-1314.google.com with SMTP id z38so871393ugc.16 for ; Thu, 13 Dec 2007 05:45:39 -0800 (PST) In-Reply-To: <47612D60.4050408@ru.mvista.com> Content-Disposition: inline Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Sergei Shtylyov Cc: linux-ide@vger.kernel.org, Geert Uytterhoeven , Roman Zippel Hi, On Thursday 13 December 2007, Sergei Shtylyov wrote: > Bartlomiej Zolnierkiewicz wrote: > > >>>* Rename init_hwif_data() to ide_init_port_data() and export it. > > >>>* For all users of ide_register_hw() with 'initializing' argument set > >>> hwif->present and hwif->hold are always zero so convert these host > >>> drivers to use ide_find_port()+ide_init_port_data()+ide_init_port_hw() > >>> instead (also no need for init_hwif_default() call since the setup > >>> done by it gets over-ridden by ide_init_port_data() call). > > >> But init_hwif_default() gets called *after* ide_init_port_data() call, at > >>least in ide_register_hw()... > > > This is a mistake in the patch description: > > > s/ide_init_port_data/ide_init_port_hw/ > > > [ I fixed it locally ] > > >>>* Drop 'initializing' argument from ide_register_hw(). > > >>>Signed-off-by: Bartlomiej Zolnierkiewicz > > >>>Index: b/drivers/ide/ide.c > >>>=================================================================== > >>>--- a/drivers/ide/ide.c > >>>+++ b/drivers/ide/ide.c > >>>@@ -116,7 +116,7 @@ EXPORT_SYMBOL(ide_hwifs); > >>> /* > >>> * Do not even *think* about calling this! > >>> */ > > >> Isn't this comment being obsoleted by the patch? > > > It is still valid because ide_init_port_data() destroys > > "hdx=" and "idex=" kernel parameters... > > But the drivers are now calling it nevertheless... ...and they also called it before this patch (indirectly through ide_register_hw() call). Needs fixing but not necessarily in this particular patch (since IDE uses static hwif objects it may be that hwif clearing was explicitely needed by some host driver, caution is needed). Incremental patches are welcomed. Bart