diff options
author | Martin Bligh <mbligh@google.com> | 2007-05-23 19:11:46 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2007-05-26 12:46:38 -0400 |
commit | de5952e91caf41bd838a787c7b9b6ca2e1292484 (patch) | |
tree | b1551cb27f64df74b16e18c63cfe4f39f7f8db33 /drivers/scsi/megaraid.c | |
parent | c6a6c81cfdd0f9b95bf215d1b5626f4dbf1aba7d (diff) |
[SCSI] megaraid: fix compiler warnings
The user ioctl mailbox can only support a 32 bit address for the
commands structure. This is fine, since the area it's pointing to is
allocated with pci_alloc_consistent(), so it should be physically <
4GB. Thus kill the ptr to u32 conversion warnings on 64 bit.
Signed-off-by: Martin J. Bligh <mbligh@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Acked-by: "Patro, Sumant" <Sumant.Patro@lsi.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/megaraid.c')
-rw-r--r-- | drivers/scsi/megaraid.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/scsi/megaraid.c b/drivers/scsi/megaraid.c index 3cce75d70263..40ee07dab450 100644 --- a/drivers/scsi/megaraid.c +++ b/drivers/scsi/megaraid.c | |||
@@ -3571,7 +3571,7 @@ megadev_ioctl(struct inode *inode, struct file *filep, unsigned int cmd, | |||
3571 | /* | 3571 | /* |
3572 | * The user passthru structure | 3572 | * The user passthru structure |
3573 | */ | 3573 | */ |
3574 | upthru = (mega_passthru __user *)MBOX(uioc)->xferaddr; | 3574 | upthru = (mega_passthru __user *)(unsigned long)MBOX(uioc)->xferaddr; |
3575 | 3575 | ||
3576 | /* | 3576 | /* |
3577 | * Copy in the user passthru here. | 3577 | * Copy in the user passthru here. |
@@ -3623,7 +3623,7 @@ megadev_ioctl(struct inode *inode, struct file *filep, unsigned int cmd, | |||
3623 | /* | 3623 | /* |
3624 | * Get the user data | 3624 | * Get the user data |
3625 | */ | 3625 | */ |
3626 | if( copy_from_user(data, (char __user *)uxferaddr, | 3626 | if( copy_from_user(data, (char __user *)(unsigned long) uxferaddr, |
3627 | pthru->dataxferlen) ) { | 3627 | pthru->dataxferlen) ) { |
3628 | rval = (-EFAULT); | 3628 | rval = (-EFAULT); |
3629 | goto freemem_and_return; | 3629 | goto freemem_and_return; |
@@ -3649,7 +3649,7 @@ megadev_ioctl(struct inode *inode, struct file *filep, unsigned int cmd, | |||
3649 | * Is data going up-stream | 3649 | * Is data going up-stream |
3650 | */ | 3650 | */ |
3651 | if( pthru->dataxferlen && (uioc.flags & UIOC_RD) ) { | 3651 | if( pthru->dataxferlen && (uioc.flags & UIOC_RD) ) { |
3652 | if( copy_to_user((char __user *)uxferaddr, data, | 3652 | if( copy_to_user((char __user *)(unsigned long) uxferaddr, data, |
3653 | pthru->dataxferlen) ) { | 3653 | pthru->dataxferlen) ) { |
3654 | rval = (-EFAULT); | 3654 | rval = (-EFAULT); |
3655 | } | 3655 | } |
@@ -3702,7 +3702,7 @@ freemem_and_return: | |||
3702 | /* | 3702 | /* |
3703 | * Get the user data | 3703 | * Get the user data |
3704 | */ | 3704 | */ |
3705 | if( copy_from_user(data, (char __user *)uxferaddr, | 3705 | if( copy_from_user(data, (char __user *)(unsigned long) uxferaddr, |
3706 | uioc.xferlen) ) { | 3706 | uioc.xferlen) ) { |
3707 | 3707 | ||
3708 | pci_free_consistent(pdev, | 3708 | pci_free_consistent(pdev, |
@@ -3742,7 +3742,7 @@ freemem_and_return: | |||
3742 | * Is data going up-stream | 3742 | * Is data going up-stream |
3743 | */ | 3743 | */ |
3744 | if( uioc.xferlen && (uioc.flags & UIOC_RD) ) { | 3744 | if( uioc.xferlen && (uioc.flags & UIOC_RD) ) { |
3745 | if( copy_to_user((char __user *)uxferaddr, data, | 3745 | if( copy_to_user((char __user *)(unsigned long) uxferaddr, data, |
3746 | uioc.xferlen) ) { | 3746 | uioc.xferlen) ) { |
3747 | 3747 | ||
3748 | rval = (-EFAULT); | 3748 | rval = (-EFAULT); |