linux-kernel - Re: [PATCH 22/56] message/fusion: Remove void casts

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 08 Apr 2009 14:55:17 +0100
From:	Jack Stone <jwjstone@...tmail.fm>
To:	Bert Wesarg <bert.wesarg@...glemail.com>
CC:	linux-kernel@...r.kernel.org, jeff@...zik.org,
	kernel-janitors@...r.kernel.org
Subject: Re: [PATCH 22/56] message/fusion: Remove void casts

Fixed patch

Thanks,

Jack

--

message/fusion: Remove void casts

From: Jack Stone <jwjstone@...tmail.fm>

Remove uneeded void casts

Signed-Off-By: Jack Stone <jwjstone@...tmail.fm>
---

 drivers/message/fusion/mptbase.c |    6 +++---
 drivers/message/fusion/mptctl.c  |    6 +++---
 drivers/message/fusion/mptfc.c   |    7 +++----
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/drivers/message/fusion/mptbase.c
b/drivers/message/fusion/mptbase.c
index d0d126c..656f15f 100644
--- a/drivers/message/fusion/mptbase.c
+++ b/drivers/message/fusion/mptbase.c
@@ -4666,7 +4666,7 @@ GetLanConfigPages(MPT_ADAPTER *ioc)
 
     if (hdr.PageLength > 0) {
         data_sz = hdr.PageLength * 4;
-        ppage0_alloc = (LANPage0_t *) pci_alloc_consistent(ioc->pcidev,
data_sz, &page0_dma);
+        ppage0_alloc = pci_alloc_consistent(ioc->pcidev, data_sz,
&page0_dma);
         rc = -ENOMEM;
         if (ppage0_alloc) {
             memset((u8 *)ppage0_alloc, 0, data_sz);
@@ -4712,7 +4712,7 @@ GetLanConfigPages(MPT_ADAPTER *ioc)
 
     data_sz = hdr.PageLength * 4;
     rc = -ENOMEM;
-    ppage1_alloc = (LANPage1_t *) pci_alloc_consistent(ioc->pcidev,
data_sz, &page1_dma);
+    ppage1_alloc = pci_alloc_consistent(ioc->pcidev, data_sz, &page1_dma);
     if (ppage1_alloc) {
         memset((u8 *)ppage1_alloc, 0, data_sz);
         cfg.physAddr = page1_dma;
@@ -4986,7 +4986,7 @@ GetIoUnitPage2(MPT_ADAPTER *ioc)
     /* Read the config page */
     data_sz = hdr.PageLength * 4;
     rc = -ENOMEM;
-    ppage_alloc = (IOUnitPage2_t *) pci_alloc_consistent(ioc->pcidev,
data_sz, &page_dma);
+    ppage_alloc = pci_alloc_consistent(ioc->pcidev, data_sz, &page_dma);
     if (ppage_alloc) {
         memset((u8 *)ppage_alloc, 0, data_sz);
         cfg.physAddr = page_dma;
diff --git a/drivers/message/fusion/mptctl.c
b/drivers/message/fusion/mptctl.c
index c638171..7893a2c 100644
--- a/drivers/message/fusion/mptctl.c
+++ b/drivers/message/fusion/mptctl.c
@@ -2603,7 +2603,7 @@ mptctl_hp_targetinfo(unsigned long arg)
        /* Get the data transfer speeds
         */
     data_sz = ioc->spi_data.sdp0length * 4;
-    pg0_alloc = (SCSIDevicePage0_t *) pci_alloc_consistent(ioc->pcidev,
data_sz, &page_dma);
+    pg0_alloc = pci_alloc_consistent(ioc->pcidev, data_sz, &page_dma);
     if (pg0_alloc) {
         hdr.PageVersion = ioc->spi_data.sdp0version;
         hdr.PageLength = data_sz;
@@ -2667,8 +2667,8 @@ mptctl_hp_targetinfo(unsigned long arg)
         /* Issue the second config page request */
         cfg.action = MPI_CONFIG_ACTION_PAGE_READ_CURRENT;
         data_sz = (int) cfg.cfghdr.hdr->PageLength * 4;
-        pg3_alloc = (SCSIDevicePage3_t *) pci_alloc_consistent(
-                            ioc->pcidev, data_sz, &page_dma);
+        pg3_alloc = pci_alloc_consistent(ioc->pcidev, data_sz,
+                &page_dma);
         if (pg3_alloc) {
             cfg.physAddr = page_dma;
             cfg.pageAddr = (karg.hdr.channel << 8) | karg.hdr.id;
diff --git a/drivers/message/fusion/mptfc.c b/drivers/message/fusion/mptfc.c
index c3c24fd..baee13b 100644
--- a/drivers/message/fusion/mptfc.c
+++ b/drivers/message/fusion/mptfc.c
@@ -766,7 +766,7 @@ mptfc_GetFcPortPage0(MPT_ADAPTER *ioc, int portnum)
 
     data_sz = hdr.PageLength * 4;
     rc = -ENOMEM;
-    ppage0_alloc = (FCPortPage0_t *) pci_alloc_consistent(ioc->pcidev,
data_sz, &page0_dma);
+    ppage0_alloc = pci_alloc_consistent(ioc->pcidev, data_sz, &page0_dma);
     if (ppage0_alloc) {
 
  try_again:
@@ -907,9 +907,8 @@ start_over:
         if (data_sz < sizeof(FCPortPage1_t))
             data_sz = sizeof(FCPortPage1_t);
 
-        page1_alloc = (FCPortPage1_t *) pci_alloc_consistent(ioc->pcidev,
-                        data_sz,
-                        &page1_dma);
+        page1_alloc = pci_alloc_consistent(ioc->pcidev, data_sz,
+                &page1_dma);
         if (!page1_alloc)
             return -ENOMEM;
     }
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/