Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261327AbVCIQfv (ORCPT ); Wed, 9 Mar 2005 11:35:51 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261440AbVCIQfv (ORCPT ); Wed, 9 Mar 2005 11:35:51 -0500 Received: from mail.kroah.org ([69.55.234.183]:22411 "EHLO perch.kroah.org") by vger.kernel.org with ESMTP id S261327AbVCIQfs (ORCPT ); Wed, 9 Mar 2005 11:35:48 -0500 Date: Wed, 9 Mar 2005 08:35:18 -0800 From: Greg KH To: Wen Xiong Cc: linux-kernel@vger.kernel.org Subject: Re: [ patch 4/7] drivers/serial/jsm: new serial device driver Message-ID: <20050309163518.GC25079@kroah.com> References: <42225A47.3060206@us.ltcfwd.linux.ibm.com> <20050228063954.GB23595@kroah.com> <4228CE41.2000102@us.ltcfwd.linux.ibm.com> <20050304220116.GA1201@kroah.com> <422CD9DB.10103@us.ltcfwd.linux.ibm.com> <20050308064424.GF17022@kroah.com> <422DF525.8030606@us.ltcfwd.linux.ibm.com> <20050308235807.GA11807@kroah.com> <422F1A8A.4000106@us.ltcfwd.linux.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <422F1A8A.4000106@us.ltcfwd.linux.ibm.com> User-Agent: Mutt/1.5.8i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 649 Lines: 19 On Wed, Mar 09, 2005 at 10:47:22AM -0500, Wen Xiong wrote: > +static ssize_t jsm_driver_debug_show(struct device_driver *ddp, char *buf) > +{ > + return snprintf(buf, PAGE_SIZE, "0x%x\n", jsm_debug); > +} > +static DRIVER_ATTR(debug, S_IRUSR, jsm_driver_debug_show, NULL); Should just be a module paramater, right? So you can drop this too... This file is getting quite small now :) thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/