From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261634AbUBZC1b (ORCPT ); Wed, 25 Feb 2004 21:27:31 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261639AbUBZC1b (ORCPT ); Wed, 25 Feb 2004 21:27:31 -0500 Received: from parcelfarce.linux.theplanet.co.uk ([195.92.249.252]:22481 "EHLO www.linux.org.uk") by vger.kernel.org with ESMTP id S262605AbUBZC11 (ORCPT ); Wed, 25 Feb 2004 21:27:27 -0500 Message-ID: <403D597C.4020708@pobox.com> Date: Wed, 25 Feb 2004 21:27:08 -0500 From: Jeff Garzik User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030703 X-Accept-Language: en-us, en MIME-Version: 1.0 To: "Mukker, Atul" CC: "'Christoph Hellwig'" , "'Arjan van de Ven'" , "'James Bottomley'" , "'matt_domsch@dell.com'" , "'Paul Wagland'" , Matthew Wilcox , "'linux-kernel@vger.kernel.org'" , "'linux-scsi@vger.kernel.org'" Subject: Re: [SUBJECT CHANGE]: megaraid unified driver version 2.20.0.0-al pha1 References: <0E3FA95632D6D047BA649F95DAB60E57033BC3E7@exa-atlanta.se.lsil.com> In-Reply-To: <0E3FA95632D6D047BA649F95DAB60E57033BC3E7@exa-atlanta.se.lsil.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Mukker, Atul wrote: >>given that they are completely different from the controller we know >>as megaraid today this is an extremly bad idea. Just put it >>into an driver >>of their own, e.g. mptraid > > Although, this simplifies the development and maintenance effort, having a > single driver to drive both controllers or two independent drivers is not > always our decision. Most often, it would be Dell's preference. If the hardware is similar, a single driver is OK. If the hardware is not similar, the preference is for separate drivers. Shared code in a third module, a "library module", is an acceptable solution. modprobe automatically loads dependent modules, so users running "modprobe driver1" or "modprobe driver2" would automatically load the shared library module. Jeff