From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from silver.osuosl.org (silver.osuosl.org [140.211.166.136]) by ash.osuosl.org (Postfix) with ESMTP id 3628D1BF9B7 for ; Thu, 18 Jul 2013 18:33:50 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id 2AC77328AA for ; Thu, 18 Jul 2013 18:33:43 +0000 (UTC) Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6oP+ookLQ867 for ; Thu, 18 Jul 2013 18:33:39 +0000 (UTC) Received: from mail.linuxfoundation.org (mail.linuxfoundation.org [140.211.169.12]) by silver.osuosl.org (Postfix) with ESMTP id 48E34328BD for ; Thu, 18 Jul 2013 18:33:26 +0000 (UTC) Date: Thu, 18 Jul 2013 11:33:33 -0700 From: Greg KH Subject: Re: [PATCH] Adds Digi driver to staging. Message-ID: <20130718183333.GA19558@kroah.com> References: <20130717201801.GA3642@lidza-Satellite-L505D> <20130717214905.GA14660@kroah.com> <51E7DF0F.2060800@compro.net> <20130718174007.GA1646@kroah.com> <51E83307.5060006@compro.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <51E83307.5060006@compro.net> List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: driverdev-devel-bounces@linuxdriverproject.org To: Mark Hounschell Cc: Lidza Louina , driverdev-devel@linuxdriverproject.org On Thu, Jul 18, 2013 at 02:25:11PM -0400, Mark Hounschell wrote: > On 07/18/2013 01:40 PM, Greg KH wrote: > > On Thu, Jul 18, 2013 at 08:26:55AM -0400, Mark Hounschell wrote: > >> On 07/17/2013 05:49 PM, Greg KH wrote: > >>> On Wed, Jul 17, 2013 at 04:18:01PM -0400, Lidza Louina wrote: > >>>> This patch adds the Digi driver to staging. > >>> > >>> Which digi driver? For what hardware? What type of driver is it? > >>> Please provide some more details here. > >>> > >> > >> From the readme: > >> > >> This Digi software package includes device drivers for the PCI models > >> of the Digi Neo and ClassicBoard series of products. > >> It is currently supported on the following hardware platforms: > >> > >> o Standard i386/i486 and Pentium PC (x86 32bit) > >> o x86 64bit > > > > I was "hinting" that this needed to be explained in the changelog entry. > > The part about the architectures is not needed, as, unless something > > really strange is going on in the driver, it should work on all cpus > > that support PCI, right? > > > > That I cannot attest to. The readme also says: > > o Standard i386/i486 and Pentium PC (x86 32bit) > o x86 64bit > > I can only personally vouch for Intel/Amd CPUs running in 32 bit mode. I'll look at the driver code, but odds are, unless you are assuming a specific endian format of some data, all should be fine on other cpu types. > >>>> It builds. Needs work done > >>>> to adapt to the API changes. > >>>> > >>>> Signed-off-by: Lidza Louina > >>> > >>> If at all possible, can we attribute this to the original author(s)? > >>> Are their names in the driver somewhere? > >>> > >> > >> * Copyright 2003 Digi International (www.digi.com) > >> * Scott H Kilau > > > > Cool, that's a good start. Lidza, can you use this as the "From:" email > > address for the initial import? Then do follow-on patches that add it > > to the build with your own Makefile and Kbuild files, and then any > > follow-on patches fixing issues. That way Scott gets the proper credit > > for the initial code, and you get the credit for doing all of the work > > to include it into the tree. > > > > thanks, > > > > greg k-h > > . > > > > Just to insure there is no duplication of work going on, I see 4 Digi > drivers referenced at: > > http://linuxdriverproject.org/mediawiki/index.php/Out_Of_Tree_Drivers#Multiport_Serial_Devices > > Should I assume that info was put there prior to and expecting for Lidza > to start this project?? That's a wiki, I have no idea who wrote that, and I don't think it has anything to do with Lidza's project, other than proof that it should be done :) thanks, greg k-h _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel