linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [0/3] add MODULE_VERSION to drivers/scsi
@ 2004-07-21 22:01 Matt Domsch
  2004-07-21 22:02 ` [1/3] " Matt Domsch
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Matt Domsch @ 2004-07-21 22:01 UTC (permalink / raw)
  To: linux-scsi, jgarzik

James, please do a

	bk pull http://mdomsch.bkbits.net/linux-2.6-modversions

This will update the following files:

 drivers/scsi/aacraid/linit.c           |    1 
 drivers/scsi/aic7xxx/aic79xx_osm.c     |    5 ---
 drivers/scsi/aic7xxx/aic7xxx_osm.c     |    5 ---
 drivers/scsi/aic7xxx_old.c             |    1 
 drivers/scsi/ata_piix.c                |    1 
 drivers/scsi/libata-core.c             |    1 
 drivers/scsi/megaraid.c                |    3 +
 drivers/scsi/qla1280.c                 |    1 
 drivers/scsi/qla2xxx/ql2100.c          |    1 
 drivers/scsi/qla2xxx/ql2200.c          |    1 
 drivers/scsi/qla2xxx/ql2300.c          |    1 
 drivers/scsi/qla2xxx/ql2322.c          |    1 
 drivers/scsi/qla2xxx/ql6312.c          |    1 
 drivers/scsi/qla2xxx/ql6322.c          |    1 
 drivers/scsi/qla2xxx/qla_os.c          |    1 
 drivers/scsi/sata_nv.c                 |    1 
 drivers/scsi/sata_promise.c            |    1 
 drivers/scsi/sata_sil.c                |    1 
 drivers/scsi/sata_sis.c                |    1 
 drivers/scsi/sata_svw.c                |    1 
 drivers/scsi/sata_sx4.c                |    1 
 drivers/scsi/sata_via.c                |    1 
 drivers/scsi/sata_vsc.c                |    1 
 drivers/scsi/sym53c8xx_2/sym_glue.c    |    2 +
 drivers/scsi/sym53c8xx_2/sym_hipd.c    |    5 ---
 drivers/scsi/sym53c8xx_2/sym_version.h |   53 +++++++++++++++++++++++++++++++++
 26 files changed, 81 insertions, 12 deletions

through these ChangeSets:

<Matt_Domsch@dell.com> (04/07/19 1.1842.1.3)
   Set MODULE_VERSION
   
   Move SYM_VERSION and SYM_DRIVER_NAME to new sym_version.h file

<Matt_Domsch@dell.com> (04/07/19 1.1842.1.2)
   Set MODULE_VERSION for drivers/scsi
   
   for Adaptec SCSI, megaraid qla1280, qla2x00

<Matt_Domsch@dell.com> (04/07/19 1.1842.1.1)
   Set MODULE_VERSION for drivers/scsi/ SATA, libata, and ata_piix.



Patches will follow as separate mails.

Thanks,
Matt

-- 
Matt Domsch
Sr. Software Engineer, Lead Engineer
Dell Linux Solutions linux.dell.com & www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [1/3] add MODULE_VERSION to drivers/scsi
  2004-07-21 22:01 [0/3] add MODULE_VERSION to drivers/scsi Matt Domsch
@ 2004-07-21 22:02 ` Matt Domsch
  2004-07-21 22:02 ` [2/3] " Matt Domsch
  2004-07-21 22:02 ` [3/3] " Matt Domsch
  2 siblings, 0 replies; 8+ messages in thread
From: Matt Domsch @ 2004-07-21 22:02 UTC (permalink / raw)
  To: linux-scsi, jgarzik

You can import this changeset into BK by piping this whole message to
'| bk receive [path to repository]' or apply the patch as usual.

===================================================================


ChangeSet@1.1842.1.1, 2004-07-19 17:57:27-04:00, Matt_Domsch@dell.com
  Set MODULE_VERSION for drivers/scsi/ SATA, libata, and ata_piix.


 ata_piix.c     |    1 +
 libata-core.c  |    1 +
 sata_nv.c      |    1 +
 sata_promise.c |    1 +
 sata_sil.c     |    1 +
 sata_sis.c     |    1 +
 sata_svw.c     |    1 +
 sata_sx4.c     |    1 +
 sata_via.c     |    1 +
 sata_vsc.c     |    1 +
 10 files changed, 10 insertions


diff -Nru a/drivers/scsi/ata_piix.c b/drivers/scsi/ata_piix.c
--- a/drivers/scsi/ata_piix.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/ata_piix.c	2004-07-21 16:57:05 -05:00
@@ -226,6 +226,7 @@
 MODULE_DESCRIPTION("SCSI low-level driver for Intel PIIX/ICH ATA controllers");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, piix_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 /**
  *	piix_pata_cbl_detect - Probe host controller cable detect info
diff -Nru a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c
--- a/drivers/scsi/libata-core.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/libata-core.c	2004-07-21 16:57:05 -05:00
@@ -62,6 +62,7 @@
 MODULE_AUTHOR("Jeff Garzik");
 MODULE_DESCRIPTION("Library module for ATA devices");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(DRV_VERSION);
 
 /**
  *	ata_tf_load_pio - send taskfile registers to host controller
diff -Nru a/drivers/scsi/sata_nv.c b/drivers/scsi/sata_nv.c
--- a/drivers/scsi/sata_nv.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_nv.c	2004-07-21 16:57:05 -05:00
@@ -154,6 +154,7 @@
 MODULE_DESCRIPTION("low-level driver for NVIDIA nForce SATA controller");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, nv_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 irqreturn_t nv_interrupt (int irq, void *dev_instance, struct pt_regs *regs)
 {
diff -Nru a/drivers/scsi/sata_promise.c b/drivers/scsi/sata_promise.c
--- a/drivers/scsi/sata_promise.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_promise.c	2004-07-21 16:57:05 -05:00
@@ -670,6 +670,7 @@
 MODULE_DESCRIPTION("Promise SATA TX2/TX4 low-level driver");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, pdc_sata_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 module_init(pdc_sata_init);
 module_exit(pdc_sata_exit);
diff -Nru a/drivers/scsi/sata_sil.c b/drivers/scsi/sata_sil.c
--- a/drivers/scsi/sata_sil.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_sil.c	2004-07-21 16:57:05 -05:00
@@ -185,6 +185,7 @@
 MODULE_DESCRIPTION("low-level driver for Silicon Image SATA controller");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, sil_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 static void sil_post_set_mode (struct ata_port *ap)
 {
diff -Nru a/drivers/scsi/sata_sis.c b/drivers/scsi/sata_sis.c
--- a/drivers/scsi/sata_sis.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_sis.c	2004-07-21 16:57:05 -05:00
@@ -116,6 +116,7 @@
 MODULE_DESCRIPTION("low-level driver for Silicon Integratad Systems SATA controller");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, sis_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 static unsigned int get_scr_cfg_addr(unsigned int port_no, unsigned int sc_reg)
 {
diff -Nru a/drivers/scsi/sata_svw.c b/drivers/scsi/sata_svw.c
--- a/drivers/scsi/sata_svw.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_svw.c	2004-07-21 16:57:05 -05:00
@@ -398,6 +398,7 @@
 MODULE_DESCRIPTION("low-level driver for K2 SATA controller");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, k2_sata_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 module_init(k2_sata_init);
 module_exit(k2_sata_exit);
diff -Nru a/drivers/scsi/sata_sx4.c b/drivers/scsi/sata_sx4.c
--- a/drivers/scsi/sata_sx4.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_sx4.c	2004-07-21 16:57:05 -05:00
@@ -1457,6 +1457,7 @@
 MODULE_DESCRIPTION("Promise SATA low-level driver");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, pdc_sata_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 module_init(pdc_sata_init);
 module_exit(pdc_sata_exit);
diff -Nru a/drivers/scsi/sata_via.c b/drivers/scsi/sata_via.c
--- a/drivers/scsi/sata_via.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_via.c	2004-07-21 16:57:05 -05:00
@@ -127,6 +127,7 @@
 MODULE_DESCRIPTION("SCSI low-level driver for VIA SATA controllers");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, svia_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 static u32 svia_scr_read (struct ata_port *ap, unsigned int sc_reg)
 {
diff -Nru a/drivers/scsi/sata_vsc.c b/drivers/scsi/sata_vsc.c
--- a/drivers/scsi/sata_vsc.c	2004-07-21 16:57:05 -05:00
+++ b/drivers/scsi/sata_vsc.c	2004-07-21 16:57:05 -05:00
@@ -382,6 +382,7 @@
 MODULE_DESCRIPTION("low-level driver for Vitesse VSC7174 SATA controller");
 MODULE_LICENSE("GPL");
 MODULE_DEVICE_TABLE(pci, vsc_sata_pci_tbl);
+MODULE_VERSION(DRV_VERSION);
 
 module_init(vsc_sata_init);
 module_exit(vsc_sata_exit);

===================================================================


This BitKeeper patch contains the following changesets:
1.1842.1.1
## Wrapped with gzip_uu ##


M'XL( +'F_D   \V9;6_:2!" /\>_PE*_W.F*V=GWI<J)7!/UJFLO4=+T2U5%
MR]H$7P!'7H>FE7_\K4T TY"P6+D3O BTLH>9>79>=G@57MHD[QU\U$5Q=9Q-
MK!D%K\(_,UOT#N)D/(Y,-G$+YUGF%KJC;))T)W%]67=PTYUD\2S);7><3N_N
M.SCBG8>5-)O:P-UWI@LS"JN5W@%$9+E2?+]->@?G)^\N/QR=!\'A8?AVI*?7
MR452A(>'09'E,SV.;5\7HW$VC8I<3^TD*72E3KF\M,0(8?=D( ABO 2.J"@-
MQ ":0A(C3"6GP22]&6>V;W\D>1*GT>CNRT+^UW59% D0""BEJ@0L$ J.0XA 
M4ARYCQ#1+A)=4"&('A,]+#J(]A *&[[K+WP6_@8H[*#@C_!E;7D;F+!RTL?3
MX\L/)U>?3\XOWI_^'0ZS/(SSM(9AC4V[X<71IZ/7X3@=Z$*_#O4T#MV7J]LT
MO8^"OT(@1/#@;.7UH+/C(PB01L'OX3_7.O^1WO2=:T>ZJ&U:4\16/VMGWR)3
M64@ 8< $ W$6TLI"1;0F3%).-6)J&&_TYG,B*V@* Q- 2BP)$&^ETO$&I0@E
MN$PXT\.AY@#) &#@K]1"9%,I9QSU\=2<5<=D>;)!+R0$+1,#*I9&*TT30(1Z
MZ/5(:E,U<-L-5ZJYR)A\[]OKM!+PY4'3KQLLG%FS$$.0 A<O%)<(@4(EYER 
M01 ;B1D@Y>NTALB&9IQ1\"=IETIA<+<2YS',"<>EXH"'0AH@2(N!V8&DW;"]
M! 'AJ]0LU8\Q8H8H+ZED;EL9%X*<,A@(;T\M1#:5HL!PTU.WV2"[?T+ =+:X
MG[M$1UP TM))H:@<<AD+.<1#C(D6ROBJM)+8V/!8,NS-[IYNV.U8*%9R/$RT
M451KA8S!VIO=0N2Z4I0Q7Z5N\VR2VDUA6"N6""(21F(Q$#&19G,%V"9V;;>#
MY#XI8IG"-^0'H&XO*#;0P@R',# * ?90;%WDFE*$$U+7YB=BMJK4_TGF"'SV
M<C-MN#=VFQE<3:$ \ZK-?Z[70)ZOU_]?N:X*<)W?3L-._JU^N8)Z]I2?6]3F
M]T32$(+UG_WE^/SSXONO;YX@6P5T#?9%T\F.XE9 &1(8UT#%7O.<Y[RM/"L3
MV^ $QG?&N8KL)L^72BW;B#[**ZL8!89(C12SO68Z3X#/,5T9V08JQG)GJ&OM
MG _7%EUE<)'DUVD6O4O&NLCZVA9Y%MF[R"9/Y/>-/2;%"("P$DDBYQ$LT7[C
MKIOAYW"OV=F&.&^9E)>-@P_Q-AV,5W[^N7VI\SXFLD3*>;Q&3.5^(Z[[K*U9
M>FEH*\8"MX-<'QZ] >]P>O6"VSRZKJHO=])KL.31^&.OP,[/V%O!UD:VJK]2
MM(5J=X+J?9#UA&H?0W4MMA3S'GF_H<Z/VQY0;4NHL'O]7<V>_*'Z#[_\H#8F
M7RNH'.1#4[7G4.L1W7:HE9%MH%)G9SNHU=3 'ZK_V,(/:F-FX=[4\:P./P2S
M&BK;;ZCSX<IVJ)61K2*5,M6.:CTR\Z:ZP\S.;T;1&-@M0Y4HHN8S"KS?#?%\
FLKA]1E$9V8HJW@YU\8>1&27FQMY-#H4R.%; @W\!N[(4(KH:    

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2/3] add MODULE_VERSION to drivers/scsi
  2004-07-21 22:01 [0/3] add MODULE_VERSION to drivers/scsi Matt Domsch
  2004-07-21 22:02 ` [1/3] " Matt Domsch
@ 2004-07-21 22:02 ` Matt Domsch
  2004-07-21 22:02 ` [3/3] " Matt Domsch
  2 siblings, 0 replies; 8+ messages in thread
From: Matt Domsch @ 2004-07-21 22:02 UTC (permalink / raw)
  To: linux-scsi, jgarzik

You can import this changeset into BK by piping this whole message to
'| bk receive [path to repository]' or apply the patch as usual.

===================================================================


ChangeSet@1.1842.1.2, 2004-07-19 18:00:35-04:00, Matt_Domsch@dell.com
  Set MODULE_VERSION for drivers/scsi
  
  for Adaptec SCSI, megaraid qla1280, qla2x00


 aacraid/linit.c       |    1 +
 aic7xxx/aic79xx_osm.c |    5 +----
 aic7xxx/aic7xxx_osm.c |    5 +----
 aic7xxx_old.c         |    1 +
 megaraid.c            |    3 +++
 qla1280.c             |    1 +
 qla2xxx/ql2100.c      |    1 +
 qla2xxx/ql2200.c      |    1 +
 qla2xxx/ql2300.c      |    1 +
 qla2xxx/ql2322.c      |    1 +
 qla2xxx/ql6312.c      |    1 +
 qla2xxx/ql6322.c      |    1 +
 qla2xxx/qla_os.c      |    1 +
 13 files changed, 15 insertions, 8 deletions


diff -Nru a/drivers/scsi/aacraid/linit.c b/drivers/scsi/aacraid/linit.c
--- a/drivers/scsi/aacraid/linit.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/aacraid/linit.c	2004-07-21 16:57:31 -05:00
@@ -62,6 +62,7 @@
 		   "Adaptec Advanced Raid Products, "
 		   "and HP NetRAID-4M SCSI driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(AAC_DRIVER_VERSION);
 
 
 int nondasd = -1;
diff -Nru a/drivers/scsi/aic7xxx/aic79xx_osm.c b/drivers/scsi/aic7xxx/aic79xx_osm.c
--- a/drivers/scsi/aic7xxx/aic79xx_osm.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/aic7xxx/aic79xx_osm.c	2004-07-21 16:57:31 -05:00
@@ -415,7 +415,6 @@
 /*
  * Module information and settable options.
  */
-#ifdef MODULE
 static char *aic79xx = NULL;
 /*
  * Just in case someone uses commas to separate items on the insmod
@@ -426,9 +425,8 @@
 
 MODULE_AUTHOR("Maintainer: Justin T. Gibbs <gibbs@scsiguy.com>");
 MODULE_DESCRIPTION("Adaptec Aic790X U320 SCSI Host Bus Adapter driver");
-#ifdef MODULE_LICENSE
 MODULE_LICENSE("Dual BSD/GPL");
-#endif
+MODULE_VERSION(AIC79XX_DRIVER_VERSION);
 MODULE_PARM(aic79xx, "s");
 MODULE_PARM_DESC(aic79xx,
 "period delimited, options string.\n"
@@ -463,7 +461,6 @@
 "		Change Read Streaming for Controller's 2 and 3\n"
 "\n"
 "	options aic79xx 'aic79xx=rd_strm:{..0xFFF0.0xC0F0}'");
-#endif
 
 static void ahd_linux_handle_scsi_status(struct ahd_softc *,
 					 struct ahd_linux_device *,
diff -Nru a/drivers/scsi/aic7xxx/aic7xxx_osm.c b/drivers/scsi/aic7xxx/aic7xxx_osm.c
--- a/drivers/scsi/aic7xxx/aic7xxx_osm.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/aic7xxx/aic7xxx_osm.c	2004-07-21 16:57:31 -05:00
@@ -436,7 +436,6 @@
 /*
  * Module information and settable options.
  */
-#ifdef MODULE
 static char *aic7xxx = NULL;
 /*
  * Just in case someone uses commas to separate items on the insmod
@@ -447,9 +446,8 @@
 
 MODULE_AUTHOR("Maintainer: Justin T. Gibbs <gibbs@scsiguy.com>");
 MODULE_DESCRIPTION("Adaptec Aic77XX/78XX SCSI Host Bus Adapter driver");
-#ifdef MODULE_LICENSE
 MODULE_LICENSE("Dual BSD/GPL");
-#endif
+MODULE_VERSION(AIC7XXX_DRIVER_VERSION);
 MODULE_PARM(aic7xxx, "s");
 MODULE_PARM_DESC(aic7xxx,
 "period delimited, options string.\n"
@@ -479,7 +477,6 @@
 "\n"
 "	options aic7xxx 'aic7xxx=probe_eisa_vl.tag_info:{{}.{.10}}.seltime:1'\n"
 );
-#endif
 
 static void ahc_linux_handle_scsi_status(struct ahc_softc *,
 					 struct ahc_linux_device *,
diff -Nru a/drivers/scsi/aic7xxx_old.c b/drivers/scsi/aic7xxx_old.c
--- a/drivers/scsi/aic7xxx_old.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/aic7xxx_old.c	2004-07-21 16:57:31 -05:00
@@ -11139,6 +11139,7 @@
 #include "aic7xxx_old/aic7xxx_proc.c"
 
 MODULE_LICENSE("Dual BSD/GPL");
+MODULE_VERSION(AIC7XXX_H_VERSION);
 
 
 static Scsi_Host_Template driver_template = {
diff -Nru a/drivers/scsi/megaraid.c b/drivers/scsi/megaraid.c
--- a/drivers/scsi/megaraid.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/megaraid.c	2004-07-21 16:57:31 -05:00
@@ -53,9 +53,12 @@
 
 #include "megaraid.h"
 
+#define MEGARAID_MODULE_VERSION "2.00.3"
+
 MODULE_AUTHOR ("LSI Logic Corporation");
 MODULE_DESCRIPTION ("LSI Logic MegaRAID driver");
 MODULE_LICENSE ("GPL");
+MODULE_VERSION(MEGARAID_MODULE_VERSION);
 
 static unsigned int max_cmd_per_lun = DEF_CMD_PER_LUN;
 MODULE_PARM(max_cmd_per_lun, "i");
diff -Nru a/drivers/scsi/qla1280.c b/drivers/scsi/qla1280.c
--- a/drivers/scsi/qla1280.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla1280.c	2004-07-21 16:57:31 -05:00
@@ -4936,6 +4936,7 @@
 MODULE_AUTHOR("Qlogic & Jes Sorensen");
 MODULE_DESCRIPTION("Qlogic ISP SCSI (qla1x80/qla1x160) driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA1280_VERSION);
 
 /*
  * Overrides for Emacs so that we almost follow Linus's tabbing style.
diff -Nru a/drivers/scsi/qla2xxx/ql2100.c b/drivers/scsi/qla2xxx/ql2100.c
--- a/drivers/scsi/qla2xxx/ql2100.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/ql2100.c	2004-07-21 16:57:31 -05:00
@@ -89,3 +89,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic ISP21xx FC-SCSI Host Bus Adapter driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);
diff -Nru a/drivers/scsi/qla2xxx/ql2200.c b/drivers/scsi/qla2xxx/ql2200.c
--- a/drivers/scsi/qla2xxx/ql2200.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/ql2200.c	2004-07-21 16:57:31 -05:00
@@ -89,3 +89,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic ISP22xx FC-SCSI Host Bus Adapter driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);
diff -Nru a/drivers/scsi/qla2xxx/ql2300.c b/drivers/scsi/qla2xxx/ql2300.c
--- a/drivers/scsi/qla2xxx/ql2300.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/ql2300.c	2004-07-21 16:57:31 -05:00
@@ -100,3 +100,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic ISP2300 FC-SCSI Host Bus Adapter driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);
diff -Nru a/drivers/scsi/qla2xxx/ql2322.c b/drivers/scsi/qla2xxx/ql2322.c
--- a/drivers/scsi/qla2xxx/ql2322.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/ql2322.c	2004-07-21 16:57:31 -05:00
@@ -105,3 +105,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic ISP2322 FC-SCSI Host Bus Adapter driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);
diff -Nru a/drivers/scsi/qla2xxx/ql6312.c b/drivers/scsi/qla2xxx/ql6312.c
--- a/drivers/scsi/qla2xxx/ql6312.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/ql6312.c	2004-07-21 16:57:31 -05:00
@@ -87,3 +87,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic ISP6312 FC-SCSI Host Bus Adapter driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);
diff -Nru a/drivers/scsi/qla2xxx/ql6322.c b/drivers/scsi/qla2xxx/ql6322.c
--- a/drivers/scsi/qla2xxx/ql6322.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/ql6322.c	2004-07-21 16:57:31 -05:00
@@ -105,3 +105,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic ISP6322 FC-SCSI Host Bus Adapter driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);
diff -Nru a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c
--- a/drivers/scsi/qla2xxx/qla_os.c	2004-07-21 16:57:31 -05:00
+++ b/drivers/scsi/qla2xxx/qla_os.c	2004-07-21 16:57:31 -05:00
@@ -4517,3 +4517,4 @@
 MODULE_AUTHOR("QLogic Corporation");
 MODULE_DESCRIPTION("QLogic Fibre Channel HBA Driver");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(QLA2XXX_VERSION);

===================================================================


This BitKeeper patch contains the following changesets:
1.1842.1.2
## Wrapped with gzip_uu ##


M'XL( ,OF_D   ]5:;4_;2!#^C'^%U7YI=<79V5<O)TZA!+7<M=<>54](O0JM
M[0U)26(:FY>>_.-O;),2$B?8IN4 $$XL>SP[SS,SS^[ZJ?LQL=.MC;<F38]Z
M\3@)!\Y3]W6<I%L;D1V-O# >XXF#.,83G4$\MIUQ5%S6"4XZXS@ZM].D,QI.
MSBXWJ2<WK\X,XTGBX'WO31H.W/S,U@9X[/N9]-NIW=HXV'OU\<W.@>-L;[N[
M S,YMA]LZFYO.VD\/3>C*.F:=#"*)UXZ-9-D;%.3NY-]OS2CA%#\%: 8$3(#
M2;C*0H@ # <;$<I]R9VYL75G8[IIA!,%FH)05&7 F)).SP4/?$X]\*A+>(>H
M#F@7_"U"MIC8)!P_N%6&W5^ N9O$>>G^V$'L.J&;1^?MN]['-WM'?^\=?-A_
M]Z?;CZ=N-!T6*"1A,L2K\"\_NQ.9T]3B3;L?]E^X8WMLIF88N5]'!JA/7N0?
MZ"4ASA^N$H0[[Z\1<#8;_C@.,<3Y[981S[O9N7+#"^>&SPG1&9.:^ID,=1\$
M"[BT(B"F7QGJ-19+0'/3-"-22H'>F4DTM1?>N4F^GME_NU]'\?$PK+1#+R\O
M\2@9I3-SZ"(" X@/EQ)8%DJ(K TTZ,A8H+J>@Q6&K_T$(8 WBZ(9A@I-+D91
M90(Q)1D/:!1"WT8\XJ1OHAI.7ED\BD?1HH> D=0:T,,O]DO0'9^-CJ?FW'K/
M)O'$/O^4G_R\:L3F*$YF]@ $2 H$[0DJ_8R&@>B#0I[+?I\&LE$HYPW/0ZZT
M:N4H^D46'249<) BLWTM@!K%)/=5Y#?R<][NM9^,^VW]I)5^^LJG&8\(4Q"&
M5$!H222:.4JK'=6T90[!4@XAY!A#DH7:$BL8^)1;#CQHF$-0D4.,42';!915
M!90RK6@6"3_L(_C"$N)+J9H%E%4%5"A"6+-DGY7PY9I)@"F>^43W31 R&O8U
M4Z1.'MTT>2.00$O 1V;2'5W&T^.S$^_LY.S,PX_X8:%JF# WD^N 87KE'X:/
M4;0I()-$,4R?/'%"@AGN^P+Y6:<:+=N=KT>@)>3I<SP,@J0;XWW%W5Z0AIXI
MV]_*"G==Z9+Q+**:<NSYF-T9!9]IK,@A09QT7_@R(E$=@JZT/N^W4#[(-ME$
MESH2)2PC2DB:*6;ZQ*<R4"HPD>0-2;JB(RE^E_CJY?@J(!PR3%-DK/0!@ZLE
MY21 YVW#^.I5\<704%6HRG5LRH7F3Z2W<SZ<QMU3,PWMJ)__]PJ5[*4#>XI/
MM6@HQB=]FE6 S[6HCXV(*2%0)6H%A4IE?%&?4EBO3^]/GCY[CO*RS-)W[N;T
MHOA#O?A^+3 M].<^\AV<A8?O[.P>]0[V\>OLU/-?EUFQF 0%+7YF8CI5S:F.
M89FG#U5Y8C+@O$!_&7SZL, O2L@Z\!?'V@9](&H9_K_>[-##P\-ZV)=RHA7V
M321.(^SG#5]CSW(N/0KL2S%6#_MRK&VP]_6=H6^=]DUFB VAKTY[XC^:M(>Z
M:2_O/>TK141.@/L2.HXY.1UWXR0:Y7JC:F*R1N<(M,MQ4B%0I0).(G(^B!8B
M@-\G(4I)ME8$5(VX!2MZJ.&1%3U.=7%@@(?]\K H$/9WE4:J+(F$'J8P7KZ6
M.)<_A#@-9R#.[V9L$^]EG*;Q>&2_=9/46CS:_"FW\6AA/B*II"B8J<[0N"S%
MI) /G$?EU*DNCR[OQ"-6$DB0\D +'A6'"AX=5O/(IU4\6EP7RBGT4]>JFFG.
M.;MSNH-RJ1Y%\RE7U6IJSF*L;9J/KB@HC71'N=+6$OH&RW_-L*<KL-?BD6"?
M+U36Q)[^?]BS.V#?8*6RX5RS"GO!A2I%QG)O>%C8EVNJ=>>:;;$'4M$!&H%?
M[EFT [_)1DHC\.<-S\!G&>'LJN@S_X&C7VSYU ._'&L;\+F =M/-&_MJMV]Q
MWV67SS'3+V9RWIW::&#26X3A_$:?+#@%@B#L^*VU(+Q?V/,MR1IZL!QHJX0'
ME%2K1=_K=;!?;[ TQ+SI9D^YVGQQ<7&UREPN8Z]:7%[>]\G7+OQ\2BF8**N]
M@H8)?X^O/92MOMBB6@?]]3#;X"Z$2YVGD>T/)]9]N_=JYV!GOW>T\/[%$^IA
M(V%/G'_P!G^9)2ON6]$@RA<8&E*EX:L4]<K#PML4L]( >4?@5Z5!/_2.D+_W
M<4M'* ?9JA5H5H$WMH+<Y#S.LW>>PH$-3Y*S\7;@VY!@ZCK_ :K<,,]])0  
 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [3/3] add MODULE_VERSION to drivers/scsi
  2004-07-21 22:01 [0/3] add MODULE_VERSION to drivers/scsi Matt Domsch
  2004-07-21 22:02 ` [1/3] " Matt Domsch
  2004-07-21 22:02 ` [2/3] " Matt Domsch
@ 2004-07-21 22:02 ` Matt Domsch
  2004-07-22 12:28   ` Matthew Wilcox
  2 siblings, 1 reply; 8+ messages in thread
From: Matt Domsch @ 2004-07-21 22:02 UTC (permalink / raw)
  To: linux-scsi, jgarzik

You can import this changeset into BK by piping this whole message to
'| bk receive [path to repository]' or apply the patch as usual.

===================================================================


ChangeSet@1.1842.1.3, 2004-07-19 18:06:39-04:00, Matt_Domsch@dell.com
  Set MODULE_VERSION
  
  Move SYM_VERSION and SYM_DRIVER_NAME to new sym_version.h file


 sym_glue.c    |    2 ++
 sym_hipd.c    |    5 +----
 sym_version.h |   53 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 56 insertions, 4 deletions


diff -Nru a/drivers/scsi/sym53c8xx_2/sym_glue.c b/drivers/scsi/sym53c8xx_2/sym_glue.c
--- a/drivers/scsi/sym53c8xx_2/sym_glue.c	2004-07-21 16:57:58 -05:00
+++ b/drivers/scsi/sym53c8xx_2/sym_glue.c	2004-07-21 16:57:58 -05:00
@@ -64,6 +64,7 @@
 
 #include "sym_glue.h"
 #include "sym_nvram.h"
+#include "sym_version.h"
 
 #define NAME53C		"sym53c"
 #define NAME53C8XX	"sym53c8xx"
@@ -2283,6 +2284,7 @@
 }
 
 MODULE_LICENSE("Dual BSD/GPL");
+MODULE_VERSION(SYM_VERSION);
 
 /*
  * Driver host template.
diff -Nru a/drivers/scsi/sym53c8xx_2/sym_hipd.c b/drivers/scsi/sym53c8xx_2/sym_hipd.c
--- a/drivers/scsi/sym53c8xx_2/sym_hipd.c	2004-07-21 16:57:58 -05:00
+++ b/drivers/scsi/sym53c8xx_2/sym_hipd.c	2004-07-21 16:57:58 -05:00
@@ -49,12 +49,9 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
-
-#define SYM_VERSION "2.1.18j"
-#define SYM_DRIVER_NAME	"sym-" SYM_VERSION
-
 #include "sym_glue.h"
 #include "sym_nvram.h"
+#include "sym_version.h"
 
 #if 0
 #define SYM_DEBUG_GENERIC_SUPPORT
diff -Nru a/drivers/scsi/sym53c8xx_2/sym_version.h b/drivers/scsi/sym53c8xx_2/sym_version.h
--- /dev/null	Wed Dec 31 16:00:00 196900
+++ b/drivers/scsi/sym53c8xx_2/sym_version.h	2004-07-21 16:57:58 -05:00
@@ -0,0 +1,53 @@
+/*
+ * Device driver for the SYMBIOS/LSILOGIC 53C8XX and 53C1010 family 
+ * of PCI-SCSI IO processors.
+ *
+ * Copyright (C) 1999-2001  Gerard Roudier <groudier@free.fr>
+ *
+ * This driver is derived from the Linux sym53c8xx driver.
+ * Copyright (C) 1998-2000  Gerard Roudier
+ *
+ * The sym53c8xx driver is derived from the ncr53c8xx driver that had been 
+ * a port of the FreeBSD ncr driver to Linux-1.2.13.
+ *
+ * The original ncr driver has been written for 386bsd and FreeBSD by
+ *         Wolfgang Stanglmeier        <wolf@cologne.de>
+ *         Stefan Esser                <se@mi.Uni-Koeln.de>
+ * Copyright (C) 1994  Wolfgang Stanglmeier
+ *
+ * Other major contributions:
+ *
+ * NVRAM detection and reading.
+ * Copyright (C) 1997 Richard Waltham <dormouse@farsrobt.demon.co.uk>
+ *
+ *-----------------------------------------------------------------------------
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. The name of the author may not be used to endorse or promote products
+ *    derived from this software without specific prior written permission.
+ *
+ * Where this Software is combined with software released under the terms of 
+ * the GNU Public License ("GPL") and the terms of the GPL would require the 
+ * combined work to also be released under the terms of the GPL, the terms
+ * and conditions of this License will apply in addition to those of the
+ * GPL with the exception of any terms or conditions of this License that
+ * conflict with, or are expressly prohibited by, the GPL.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHORS AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE FOR
+ * ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+#define SYM_VERSION "2.1.18j"
+#define SYM_DRIVER_NAME	"sym-" SYM_VERSION

===================================================================


This BitKeeper patch contains the following changesets:
1.1842.1.3
## Wrapped with gzip_uu ##


M'XL( .;F_D   [U8:W.;R!+];'Y%E_-AXUT]> D]-DX9(6Q3D806D!]UZY87
MP<@B0: +R(\J_?C;/4*RG+@<VWG83CP>>DZ?/MW38N8=C'.6=?8&?E%<]=)Y
M'LR$=W":YD5G+V1Q7 O2.4XX:8H3]5DZ9_5YR,WJDR_U>1K>L"ROQU&RO*O*
M-:U:SD1ID@NX;N07P0QHIK,GU93M3'&_8)T]QSP9]W5'$ X/P9CYR35S60&'
MAT*19C=^'.9'?C&+TZ169'Z2SUGA$YW5UG0EBZ*,WPVIJ8@-;25IHMI<!5(H
M2;XJL5"4U9:F"CNQ'6UB>@RBBDVI+>- ::S$9D-4A1Y(-:FERC5D#:):%YMU
MJ0U2JR-J':5=%=6.*,)3P/"7 E51Z,+/C<$0 B!Q!G9OW#>OSDS'M>PA3N+/
M(+UAX%X.-K/@)R'_N^=8.'4UU <FTH&$W4)^/[\J$U2;P32*F? )9%G6%&'T
MD .A^LHO01!]4?@("\KNTQ&'6<1K)0_RJ(XT&DK0NKN[DFE\-8L682W8:B&+
MBB*I*U%NR=HJ5-FDK3*FBN%$4X(G57\I^B;1&N(VFFKK1SA?QTOV%&=955:^
MI+4FS8G?;FB!KVIOX/R OLM9:6F:C)Q?C[=-^@.D*"N-]DI6-2P[I-]H*\U0
M\T4MF$X:/\N#*HHK46GCCJ)-_K+UU &&6*M4G9T7KA&FP$ 5SO#?T_O]9XK#
MNX.XTQ:TCJQT&L^W!?%7M(4 NE'QB;$%R[A<L.G.\DY[IF:\;=&-^@L%Q;9 
MP6-;>.$"![[1Z0+$NY8LO:&=_*8D2J],8N,7-?=CD%!POD]LJ&:W_ =U>+'X
MKU?8$J&A"/4_!?@3>NPF"ECI"Z9I!L6,?Z1T+=NM]UVK;Y]8!MH;K8L+_O&"
M0TF41)CZ\RB^!P))IS RK*IKN!98-BRR-&!YGF9Y#9^2@9$N[K/H>E; >^, 
MI':[745])( 3EOE9B"\9RS!"_Q^NL_7H:)HQ5IMF'TL ;Q;E&Y(T8C0,89IA
M:HAPGVH<MB*5IK4G?;?(M_BU[ZTC]@W,DQZ3('ML5,S\ F9^"!/&$BZ+#XLT
M*T@=6G",$77='BW<+DG7Q*M2#5\WE-H.AQ0I1XD?[YK/_'P-?IM%18&_*5WX
MN3#)0YZ9C8?)/:%LOL[3>'J-50EN@?_'<T9"EU\?;O'A49#&Z77":B'[N+O0
M+=C43\#,\X<%VX4Y.YI'M7$253^E+$XV:[\16WV:0!FIC<)D,/<_8R!!FA19
M-%D6U+0ZI<'PS-$'*'[! IKG86;,#Z/D^NGD-L&)@AGE]=R/,25S^!"FV3Q=
M(N&IG^59.BF0[!RW3I#6EE_* GOM)GI^AZVY.RR,\FU(G#JR@"B!/%UFN.EH
M9H))SNXID_.\ K=1,</4\]_ILB 4[.31- I\PJB GS' CC^G_(>TT6ZB$ >\
M]JC(IFD<I[>H#LD91EQ+7HJX#IL4R8J][RMJ.95HR2E(0[1<Y@7*7/C(E5#]
M";UG!ANMRR))T@);1P4M<'_$B$<P#VYY>(\YH=,@]J-YN3/E&B_UQ)^SS2;Q
MEQ@X5<0]P6.UDV0A;1268!YSVA@4]CPM&/T.ET&1EWR^VJ)(*D^GQ2V%7NH)
M^8(%I"8NC4CF<A]Q17/>3\O<G6-=LC6(NP'!,;9X3!CZX(G:PF<L9C[Q7"8A
M6W?0 B&YKH1&$R?#,8R6DQB=]U&V!$-YOW\RZN\?;(7:KN'VHS[<ILN8ZOU_
MRXBS81SM@42:?2%I_#A/2:KG:)20E8=97A;H>2=CW [#W!"\C>(8_,4"VSQ6
M@A^N[<@ERIEODD9 G"UI0O#L+F +;HD&?G*_89$]YXM*>!U=,D61"@Y7H46D
M,+M;9/B1@D0PZ;-H$E'Y3^XKF[BVK?/4<L&UC[USW3$!QR/'/K-Z9@^ZE_C0
M!'WLG=J."_JP!X8]]!RK._9HXM]_=1<7_/$'/2(H?7@)YL7(,5T7; >LP:AO
M(0X"._K0LTRW M;0Z(][UO"D H@"0]N#OC6P/#3S[ KY(Z!O5X)]# /3,4[Q
M3[UK]2WODA,ZMKPAN3M&?SJ,=,>S##HFPVCLC&R7HU%</<LU^KHU,'LUY(!^
MP3PSAQZXIWJ_OQ,F\7X49-=$@GJW;Y*+39 ]RS$-CZ)Y&!FH&7+#>G%'IF'1
MP+PP,1#=N:R4L*[YSQB-\"%_D= '^@F&]OX[HF!"C+%C#H@ORN".NZYG>6//
MA!/;[KG\4\$!UW3.+,-T_X:^[7*]QJY902>>SMTC"HJ%CW'<';L6E\T:>J;C
MC$<>GH0/".C4/D=AD*R.JWM<8CPC4\RHD>U<$B[IP3-0@?-3$^<=DI2KII,6
M+JIG>(2V8XE>44]O)U@8FB=]Z\0<&B8]M0GHW'+- TR8Y9*!M?9\KE_R&,<\
M?,H5<EL/=TJWPC,*UC'HO3.+R)?&6 :N59:,?4Q([M@X+=6G75 7WH5LBOWA
MT;W /EUG2*W/^X^>[MP2[.WC6T]U?W?1]\]MZY,J'=I^PTE:H&YTCV_DD\A/
M:FEV_9_/[//DOR\_2^.WI$F***\456Z*_!2@JE]?\<C:L\< ^3=>\7R"]:'_
M->>"=<!O.11H&IY$WD5)$"_Q%6#_T2%C7[!DN=5 @\<4W^^4R\'?WR^8]77,
MCQ3,*ZZ+WE0PNQ=&#P6CB9HLEC>#KRP8":KJ+RF8'[[[6]^"O::XUN*\H;AZ
D#1E4P6HTGRNQ[15Q,&/!EWPY/U0T29NTFI+P?UKA.D:L%@  
 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [3/3] add MODULE_VERSION to drivers/scsi
  2004-07-21 22:02 ` [3/3] " Matt Domsch
@ 2004-07-22 12:28   ` Matthew Wilcox
  2004-07-22 13:38     ` Matt Domsch
  0 siblings, 1 reply; 8+ messages in thread
From: Matthew Wilcox @ 2004-07-22 12:28 UTC (permalink / raw)
  To: Matt Domsch; +Cc: linux-scsi, jgarzik

On Wed, Jul 21, 2004 at 05:02:58PM -0500, Matt Domsch wrote:
> You can import this changeset into BK by piping this whole message to
> '| bk receive [path to repository]' or apply the patch as usual.

> diff -Nru a/drivers/scsi/sym53c8xx_2/sym_version.h b/drivers/scsi/sym53c8xx_2/sym_version.h
> --- /dev/null	Wed Dec 31 16:00:00 196900
> +++ b/drivers/scsi/sym53c8xx_2/sym_version.h	2004-07-21 16:57:58 -05:00
> @@ -0,0 +1,53 @@
> +/*
> + * Device driver for the SYMBIOS/LSILOGIC 53C8XX and 53C1010 family 
> + * of PCI-SCSI IO processors.
> + *
> + * Copyright (C) 1999-2001  Gerard Roudier <groudier@free.fr>
> + *
> + * This driver is derived from the Linux sym53c8xx driver.
> + * Copyright (C) 1998-2000  Gerard Roudier
> + *
> + * The sym53c8xx driver is derived from the ncr53c8xx driver that had been 
> + * a port of the FreeBSD ncr driver to Linux-1.2.13.
> + *
> + * The original ncr driver has been written for 386bsd and FreeBSD by
> + *         Wolfgang Stanglmeier        <wolf@cologne.de>
> + *         Stefan Esser                <se@mi.Uni-Koeln.de>
> + * Copyright (C) 1994  Wolfgang Stanglmeier
> + *
> + * Other major contributions:
> + *
> + * NVRAM detection and reading.
> + * Copyright (C) 1997 Richard Waltham <dormouse@farsrobt.demon.co.uk>
> + *
> + *-----------------------------------------------------------------------------
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions
> + * are met:
> + * 1. Redistributions of source code must retain the above copyright
> + *    notice, this list of conditions and the following disclaimer.
> + * 2. The name of the author may not be used to endorse or promote products
> + *    derived from this software without specific prior written permission.
> + *
> + * Where this Software is combined with software released under the terms of 
> + * the GNU Public License ("GPL") and the terms of the GPL would require the 
> + * combined work to also be released under the terms of the GPL, the terms
> + * and conditions of this License will apply in addition to those of the
> + * GPL with the exception of any terms or conditions of this License that
> + * conflict with, or are expressly prohibited by, the GPL.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE AUTHORS AND CONTRIBUTORS ``AS IS'' AND
> + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
> + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE FOR
> + * ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
> + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
> + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
> + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
> + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
> + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
> + * SUCH DAMAGE.
> + */
> +#define SYM_VERSION "2.1.18j"
> +#define SYM_DRIVER_NAME	"sym-" SYM_VERSION

Umm.  No.  This is ridiculous.  I'm not having a 51 line copyright message
with 2 lines of #define.  It's not even clear to me that these two lines
have artisitic merit.  How about putting these defines in sym_defs.h,
at the top?  It's already included in both sym_glue.c and sym_hipd.c by
being included by sym_glue.h.

-- 
"Next the statesmen will invent cheap lies, putting the blame upon 
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince 
himself that the war is just, and will thank God for the better sleep 
he enjoys after this process of grotesque self-deception." -- Mark Twain

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [3/3] add MODULE_VERSION to drivers/scsi
  2004-07-22 12:28   ` Matthew Wilcox
@ 2004-07-22 13:38     ` Matt Domsch
  2004-07-22 14:07       ` Matt Domsch
  0 siblings, 1 reply; 8+ messages in thread
From: Matt Domsch @ 2004-07-22 13:38 UTC (permalink / raw)
  To: Matthew Wilcox; +Cc: linux-scsi, jgarzik

On Thu, Jul 22, 2004 at 01:28:23PM +0100, Matthew Wilcox wrote:
> > +#define SYM_VERSION "2.1.18j"
> > +#define SYM_DRIVER_NAME	"sym-" SYM_VERSION
> 
> Umm.  No.  This is ridiculous.  I'm not having a 51 line copyright message
> with 2 lines of #define.  It's not even clear to me that these two lines
> have artisitic merit.  How about putting these defines in sym_defs.h,
> at the top?  It's already included in both sym_glue.c and sym_hipd.c by
> being included by sym_glue.h.

Indeed, point well taken.  Done, they're now at the top of defines in
sym_defs.h, and I've updated my BK tree accordingly.

Thanks,
Matt

-- 
Matt Domsch
Sr. Software Engineer, Lead Engineer
Dell Linux Solutions linux.dell.com & www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [3/3] add MODULE_VERSION to drivers/scsi
  2004-07-22 13:38     ` Matt Domsch
@ 2004-07-22 14:07       ` Matt Domsch
  2004-07-22 19:02         ` Matthew Wilcox
  0 siblings, 1 reply; 8+ messages in thread
From: Matt Domsch @ 2004-07-22 14:07 UTC (permalink / raw)
  To: Matthew Wilcox; +Cc: linux-scsi, jgarzik


On Thu, Jul 22, 2004 at 08:38:55AM -0500, Matt Domsch wrote:
> On Thu, Jul 22, 2004 at 01:28:23PM +0100, Matthew Wilcox wrote:
> > > +#define SYM_VERSION "2.1.18j"
> > > +#define SYM_DRIVER_NAME	"sym-" SYM_VERSION
> > 
> > Umm.  No.  This is ridiculous.  I'm not having a 51 line copyright message
> > with 2 lines of #define.  It's not even clear to me that these two lines
> > have artisitic merit.  How about putting these defines in sym_defs.h,
> > at the top?  It's already included in both sym_glue.c and sym_hipd.c by
> > being included by sym_glue.h.
> 
> Indeed, point well taken.  Done, they're now at the top of defines in
> sym_defs.h, and I've updated my BK tree accordingly.

whole patch to sym53c8xx_2 below.

-- 
Matt Domsch
Sr. Software Engineer, Lead Engineer
Dell Linux Solutions linux.dell.com & www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com

diff -urNp --exclude-from=/mdomsch2/excludes --minimal ../linux-2.5/drivers/scsi/sym53c8xx_2/sym_defs.h linux-2.5/drivers/scsi/sym53c8xx_2/sym_defs.h
--- ../linux-2.5/drivers/scsi/sym53c8xx_2/sym_defs.h	2004-07-22 10:10:50.277283936 -0400
+++ linux-2.5/drivers/scsi/sym53c8xx_2/sym_defs.h	2004-07-22 10:12:02.931238856 -0400
@@ -53,6 +53,9 @@
 #ifndef SYM_DEFS_H
 #define SYM_DEFS_H
 
+#define SYM_VERSION "2.1.18j"
+#define SYM_DRIVER_NAME	"sym-" SYM_VERSION
+
 /*
  *  Vendor.
  */
diff -urNp --exclude-from=/mdomsch2/excludes --minimal ../linux-2.5/drivers/scsi/sym53c8xx_2/sym_glue.c linux-2.5/drivers/scsi/sym53c8xx_2/sym_glue.c
--- ../linux-2.5/drivers/scsi/sym53c8xx_2/sym_glue.c	2004-07-22 10:10:50.646227848 -0400
+++ linux-2.5/drivers/scsi/sym53c8xx_2/sym_glue.c	2004-07-22 10:12:02.973232472 -0400
@@ -2283,6 +2283,7 @@ static int sym_detach(struct sym_hcb *np
 }
 
 MODULE_LICENSE("Dual BSD/GPL");
+MODULE_VERSION(SYM_VERSION);
 
 /*
  * Driver host template.
diff -urNp --exclude-from=/mdomsch2/excludes --minimal ../linux-2.5/drivers/scsi/sym53c8xx_2/sym_hipd.c linux-2.5/drivers/scsi/sym53c8xx_2/sym_hipd.c
--- ../linux-2.5/drivers/scsi/sym53c8xx_2/sym_hipd.c	2004-07-22 10:10:50.736214168 -0400
+++ linux-2.5/drivers/scsi/sym53c8xx_2/sym_hipd.c	2004-07-22 10:12:02.985230648 -0400
@@ -49,10 +49,6 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
-
-#define SYM_VERSION "2.1.18j"
-#define SYM_DRIVER_NAME	"sym-" SYM_VERSION
-
 #include "sym_glue.h"
 #include "sym_nvram.h"
 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [3/3] add MODULE_VERSION to drivers/scsi
  2004-07-22 14:07       ` Matt Domsch
@ 2004-07-22 19:02         ` Matthew Wilcox
  0 siblings, 0 replies; 8+ messages in thread
From: Matthew Wilcox @ 2004-07-22 19:02 UTC (permalink / raw)
  To: Matt Domsch; +Cc: Matthew Wilcox, linux-scsi, jgarzik

On Thu, Jul 22, 2004 at 09:07:53AM -0500, Matt Domsch wrote:
> whole patch to sym53c8xx_2 below.

I'm happy with this.  James, please apply.

-- 
"Next the statesmen will invent cheap lies, putting the blame upon 
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince 
himself that the war is just, and will thank God for the better sleep 
he enjoys after this process of grotesque self-deception." -- Mark Twain

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2004-07-22 19:02 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-07-21 22:01 [0/3] add MODULE_VERSION to drivers/scsi Matt Domsch
2004-07-21 22:02 ` [1/3] " Matt Domsch
2004-07-21 22:02 ` [2/3] " Matt Domsch
2004-07-21 22:02 ` [3/3] " Matt Domsch
2004-07-22 12:28   ` Matthew Wilcox
2004-07-22 13:38     ` Matt Domsch
2004-07-22 14:07       ` Matt Domsch
2004-07-22 19:02         ` Matthew Wilcox

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).