From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754264Ab1KGKh5 (ORCPT ); Mon, 7 Nov 2011 05:37:57 -0500 Received: from exprod5og102.obsmtp.com ([64.18.0.143]:34395 "EHLO exprod5og102.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754138Ab1KGKh4 (ORCPT ); Mon, 7 Nov 2011 05:37:56 -0500 Message-ID: <4EB7B4FF.7060806@ge.com> Date: Mon, 07 Nov 2011 10:37:51 +0000 From: Martyn Welch Organization: GE Intelligent Platforms User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.23) Gecko/20110921 Thunderbird/3.1.15 MIME-Version: 1.0 To: Greg KH CC: Paul Bolle , Grant Likely , devel@driverdev.osuosl.org, manohar.vanga@cern.ch, cota@braap.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Driver for GE PIO2 VME Card References: <20111104173126.30940.28135.stgit@ES-BJ21R4J> <1320436514.14409.309.camel@x61.thuisdomein> <20111104200057.GB400@suse.de> In-Reply-To: <20111104200057.GB400@suse.de> X-Enigmail-Version: 1.1.2 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 07 Nov 2011 10:37:54.0124 (UTC) FILETIME=[4E6B18C0:01CC9D39] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/11/11 20:00, Greg KH wrote: > On Fri, Nov 04, 2011 at 08:55:14PM +0100, Paul Bolle wrote: >> A few random remarks follow. Ie, things that came up while quickly >> scanning this. >> >> On Fri, 2011-11-04 at 17:33 +0000, Martyn Welch wrote: >>> diff --git a/drivers/staging/vme/devices/Kconfig b/drivers/staging/vme/devices/Kconfig >>> index ca5ba89..99f5414 100644 >>> --- a/drivers/staging/vme/devices/Kconfig >>> +++ b/drivers/staging/vme/devices/Kconfig >>> @@ -6,3 +6,13 @@ config VME_USER >>> If you say Y here you want to be able to access a limited number of >>> VME windows in a manner at least semi-compatible with the interface >>> provided with the original driver at http://vmelinux.org/. >> >> Blank line here. >> >>> +config VME_PIO2 >>> + tristate "GE PIO2 VME" >>> + help >>> + If you say Y here you have a GE PIO2. The PIO2 is a 6U VME Card, >> >> Maybe something like: "Say Y here if you have a GE PIO2."? >> >>> + implementing 32 solid-state relay switched IO lines, in 4 groups of 8. >>> + The IO lines are provided as input, output or both as a build time >>> + option. >> >> What option would that be? >> >>> + Otherwise it is safe to say N here. >>> + >> >> A lot of modules have the "if you say M here the module will be called" >> boilerplate here (which actually doesn't match the line I just made up). >> >>> +/* These are required for each board */ >>> +MODULE_PARM_DESC(bus, "Enumeration of VMEbus to which the board is connected"); >>> +module_param_array(bus, int, &bus_num, 0); >>> + >>> +MODULE_PARM_DESC(base, "Base VME address for PIO2 Registers"); >>> +module_param_array(base, long, &base_num, 0); >>> + >>> +MODULE_PARM_DESC(vector, "VME IRQ Vector (Lower 4 bits masked)"); >>> +module_param_array(vector, int, &vector_num, 0); >>> + >>> +MODULE_PARM_DESC(level, "VME IRQ Level"); >>> +module_param_array(level, int, &level_num, 0); >>> + >>> +MODULE_PARM_DESC(variant, "Last 4 characters of PIO2 board variant"); >>> +module_param_array(variant, charp, &variant_num, 0); >>> + >>> +/* This is for debugging */ >>> +MODULE_PARM_DESC(loopback, "Enable loopback mode on all cards"); >>> +module_param(loopback, bool, 0); >> >> All module parameters have a sysfs visibility (or permission) of zero. >> Why is that? (This might very well be a naive question. But I often >> wonder why a certain parameter's permission isn't at least 400, just to >> allow a quick check of that parameter.) Are arrays tricky in sysfs? > > Yes, arrays are tricky, and in the end, all of these need to be "real" > sysfs files, not module parameters, but for now, it's ok to leave it as > is until the code is cleaned up. > Hi Greg, Am I right in thinking we wouldn't then be able to pass the parameters in the boot args? Martyn > thanks, > > greg k-h -- Martyn Welch (Principal Software Engineer) | Registered in England and GE Intelligent Platforms | Wales (3828642) at 100 T +44(0)1327322748 | Barbirolli Square, Manchester, E martyn.welch@ge.com | M2 3AB VAT:GB 927559189