Re: [ patch 7/7] drivers/serial/jsm: new serial device driver

[Christoph Hellwig]

> +config SERIAL_JSM
> +        tristate "Digi International NEO PCI Support"
> +        select SERIAL_CORE

shouldn't this depend on CONFIG_PCI?

> diff -Nuar linux-2.6.9.orig/drivers/serial/jsm/Makefile linux-2.6.9.new/drivers/serial/jsm/Makefile
> --- linux-2.6.9.orig/drivers/serial/jsm/Makefile	1969-12-31 18:00:00.000000000 -0600
> +++ linux-2.6.9.new/drivers/serial/jsm/Makefile	2005-02-27 17:01:43.725941288 -0600
> @@ -0,0 +1,36 @@
> +##################################################################
> +# Copyright 2003 Digi International (www.digi.com)
> +# Scott H Kilau <Scott_Kilau at digi dot com>
> +# 
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; either version 2, or (at your option)
> +# any later version.
> +# 
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY, EXPRESS OR IMPLIED; without even the
> +# implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR
> +# PURPOSE.  See the GNU General Public License for more details.
> +# 
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> +# 
> +# 
> +#       NOTE TO LINUX KERNEL HACKERS:  DO NOT REFORMAT THIS CODE!
> +# 
> +#       Send any bug fixes/changes to:  Eng.Linux at digi dot com.
> +#       Thank you.
> +# 
> +# 
> +##################################################################
> +
> +#
> +# Makefile for Jasmine adapter
> +#
> +
> +obj-$(CONFIG_SERIAL_JSM) += jsm.o
> +
> +jsm-objs :=    jsm_driver.o jsm_mgmt.o jsm_neo.o\
> +               jsm_proc.o jsm_tty.o
> +

I doubt anyone can register a copyright on a trivial three-line makefile.
Also please use jsm-y instead of jsm-objs.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/