From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from whitealder.osuosl.org (whitealder.osuosl.org [140.211.166.138]) by ash.osuosl.org (Postfix) with ESMTP id 0CC4A1BF9B5 for ; Thu, 18 Jul 2013 20:49:08 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 3BB4A8D36D for ; Thu, 18 Jul 2013 20:49:01 +0000 (UTC) Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 10hPEGMx4KBb for ; Thu, 18 Jul 2013 20:49:00 +0000 (UTC) Received: from mail-vb0-f42.google.com (mail-vb0-f42.google.com [209.85.212.42]) by whitealder.osuosl.org (Postfix) with ESMTPS id 861588D527 for ; Thu, 18 Jul 2013 20:49:00 +0000 (UTC) Received: by mail-vb0-f42.google.com with SMTP id i3so2714089vbh.1 for ; Thu, 18 Jul 2013 13:48:59 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20130718183845.8FF0379@viridian.itc.virginia.edu> References: <20130718183333.GA19558@kroah.com> <20130718183845.8FF0379@viridian.itc.virginia.edu> Date: Thu, 18 Jul 2013 16:48:59 -0400 Message-ID: Subject: Re: [PATCH] Adds Digi driver to staging. From: Lidza Louina 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: Bill Pemberton Cc: Mark Hounschell , Greg KH , driverdev-devel@linuxdriverproject.org I've got all of your feedback. I'll send my patch up again with: * CVS directory removed * Makefile modified so it builds into the kernel * Better description of the driver * Attribute the original author * .h files in the main folder If there's anything else I need to add/modify, please let me know. Thanks, Lidza On Thu, Jul 18, 2013 at 2:38 PM, Bill Pemberton wrote: > Greg KH writes: >> >> 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. >> > > If it's anything like dgrp, it will have some places that it assumes a > specific endian format. > > -- > Bill _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel