Re: [PATCH 1/3] debloat aic7xxx and aic79xx drivers

2007-11-01 Thread Matthew Wilcox
On Sun, Oct 14, 2007 at 04:00:10PM +0100, Denys Vlasenko wrote:
 Deinlines and moves big functions from .h to .c files.
 Adds prototypes for ahc_lookup_scb and ahd_lookup_scb to .h files.

Adds trailing whitespace.
.dotest/patch:216:   * We also set the full residual flag which the 
Adds trailing whitespace.
.dotest/patch:2383:{   
warning: 2 lines add trailing whitespaces.

-- 
Intel are signing my paycheques ... these opinions are still mine
Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step.
-
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/3] debloat aic7xxx and aic79xx drivers

2007-09-24 Thread Hannes Reinecke
Denys Vlasenko wrote:
 On Friday 31 August 2007 16:13, Denys Vlasenko wrote:
 Attached are three patches which fix that:

textdata bss dec hex filename
  261433   500181172  312623   4c52f 
 linux-2.6.23-rc1.org.t/drivers/scsi/aic7xxx/built-in.o
  199654   500181172  250844   3d3dc 
 linux-2.6.23-rc1.aic.t/drivers/scsi/aic7xxx/built-in.o
  184014   213141172  206500   326a4 
 linux-2.6.23-rc1.aic1.t/drivers/scsi/aic7xxx/built-in.o
  20237828501172  206400   32640 
 linux-2.6.23-rc1.aic2.t/drivers/scsi/aic7xxx/built-in.o

 1-debloat.patchdeinlines a lot of functions
 2-addstatic.patch  adds statics, #ifdefs out huge amount of unused code, 
 adds consts
 3-addconst.patch   adds more consts
 --
 vda
 
 
Signed-off-by: Hannes Reinecke [EMAIL PROTECTED]

-- 
Dr. Hannes Reinecke   zSeries  Storage
[EMAIL PROTECTED] +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)
-
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html