drivers/scsi/53c* MIN/MAX removal
Subsystems (2)
PATCH 1/1 drivers/scsi/53c* MIN/MAX removal
Patch (against 2.6.6-rc1) removes unnecessary min/max macros and changes calls to use kernel.h macros instead.
Feedback is always welcome
Michael
drivers/scsi/ncr53c8xx.c +3 -3
@@ -307,7 +307,7 @@** The maximum number of segments a transfer is split into. ** We support up to 127 segments for both read and write. ** The data scripts are broken into 2 sub-scripts. -** 80 (MAX_SCATTERL) segments are moved from a sub-script +** 80 (MAX_SCATTERL) segments are moved from a sub-script ** in on-chip RAM. This makes data transfers shorter than ** 80k (assuming 1k fs) as fast as possible. */
@@ -4542,7 +4542,7 @@/* ** command */ -memcpy (cp->cdb_buf, cmd->cmnd,MIN (cmd->cmd_len,sizeof (cp->cdb_buf))); +memcpy (cp->cdb_buf, cmd->cmnd,min_t (int, cmd->cmd_len,sizeof (cp->cdb_buf))); cp->phys.cmd.addr =cpu_to_scr (CCB_PHYS (cp, cdb_buf[0 ])); cp->phys.cmd.size =cpu_to_scr (cmd->cmd_len);
@@ -5172,7 +5172,7 @@** Copy back sense data to caller's buffer. */ memcpy (cmd->sense_buffer, cp->sense_buf, -MIN (sizeof (cmd->sense_buffer),sizeof (cp->sense_buf))); +min (sizeof (cmd->sense_buffer),sizeof (cp->sense_buf)));if (DEBUG_FLAGS & (DEBUG_RESULT |DEBUG_TINY )) {u_char * p = (u_char *) & cmd->sense_buffer;
drivers/scsi/sym53c8xx_comm.h +1 -4
@@ -59,12 +59,9 @@** sym53c8xx and ncr53c8xx drivers should share. ** The sharing will be achieved in a further version ** of the driver bundle. For now, only the ncr53c8xx -** driver includes this file. +** driver includes this file. */ -#define MIN (a,b) (((a) < (b)) ? (a) : (b)) -#define MAX (a,b) (((a) > (b)) ? (a) : (b)) -/*========================================================== ** ** Hmmm... What complex some PCI-HOST bridges actually
_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors
Oops, those patches were against RC-2, not RC-1 of course.
Quoted 61 lines · Michael Veeck
Patch (against 2.6.6-rc1) removes unnecessary min/max macros and changes calls to use
kernel.hmacros instead.Feedback is always welcome
Michael------------------------------------------------------------------------
diff -Naur -X dontdiff-osdl
linux-2.6.5.org/drivers/scsi/ncr53c8xx.clinux-2.6.5.new/drivers/scsi/ncr53c8xx.cdrivers/scsi/ncr53c8xx.c +3
-3@@ -307,7 +307,7 @@** The maximum number of segments a transfer is split into. ** We support up to 127 segments for both read and write. ** The data scripts are broken into 2 sub-scripts. -** 80 (MAX_SCATTERL) segments are moved from a sub-script +** 80 (MAX_SCATTERL) segments are moved from a sub-script ** in on-chip RAM. This makes data transfers shorter than ** 80k (assuming 1k fs) as fast as possible. */ @@ -4542,7 +4542,7 @@/* ** command */ -memcpy (cp->cdb_buf, cmd->cmnd,MIN (cmd->cmd_len,sizeof (cp->cdb_buf))); +memcpy (cp->cdb_buf, cmd->cmnd,min_t (int, cmd->cmd_len,sizeof (cp->cdb_buf))); cp->phys.cmd.addr =cpu_to_scr (CCB_PHYS (cp, cdb_buf[0 ])); cp->phys.cmd.size =cpu_to_scr (cmd->cmd_len);@@ -5172,7 +5172,7 @@** Copy back sense data to caller's buffer. */ memcpy (cmd->sense_buffer, cp->sense_buf, -MIN (sizeof (cmd->sense_buffer),sizeof (cp->sense_buf))); +min (sizeof (cmd->sense_buffer),sizeof (cp->sense_buf)));if (DEBUG_FLAGS & (DEBUG_RESULT |DEBUG_TINY )) {u_char * p = (u_char *) & cmd->sense_buffer;diff -Naur -X dontdiff-osdl
linux-2.6.5.org/drivers/scsi/sym53c8xx_comm.hlinux-2.6.5.new/drivers/scsi/sym53c8xx_comm.hdrivers/scsi/sym53c8xx_comm.h +1
-5@@ -59,12 +59,9 @@** sym53c8xx and ncr53c8xx drivers should share. ** The sharing will be achieved in a further version ** of the driver bundle. For now, only the ncr53c8xx -** driver includes this file. +** driver includes this file. */ -#define MIN (a,b) (((a) < (b)) ? (a) : (b)) -#define MAX (a,b) (((a) > (b)) ? (a) : (b)) -/*========================================================== ** ** Hmmm... What complex some PCI-HOST bridges actually ------------------------------------------------------------------------
Quoted 4 lines
_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors
_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors
On Fri, 23 Apr 2004 10:00:22 +0200 Michael Veeck wrote:
| Oops, those patches were against RC-2, not RC-1 of course.
Yes, I've applied them to -rc2.
--
~Randy
_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors