From: Bryan Thompson <bryan.thomp...@unisys.com>

Call POSTCODE_LINUX directly instead of passing through POSTCODE_LINUX_2.

Signed-off-by: Bryan Thompson <bryan.thomp...@unisys.com>
Signed-off-by: David Kershner <david.kersh...@unisys.com>
---
 drivers/staging/unisys/visorbus/visorbus_main.c   |  4 +-
 drivers/staging/unisys/visorbus/visorchipset.c    | 65 ++++++++++++-----------
 drivers/staging/unisys/visorbus/vmcallinterface.h |  3 --
 3 files changed, 35 insertions(+), 37 deletions(-)

diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c 
b/drivers/staging/unisys/visorbus/visorbus_main.c
index b7001ed..4536c633d 100644
--- a/drivers/staging/unisys/visorbus/visorbus_main.c
+++ b/drivers/staging/unisys/visorbus/visorbus_main.c
@@ -966,7 +966,7 @@ int visorbus_register_visor_driver(struct visor_driver *drv)
        int err;
        struct spar_vbus_headerinfo *hdr_info;
 
-       POSTCODE_LINUX_2(BUS_CREATE_ENTRY_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(BUS_CREATE_ENTRY_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 
        hdr_info = kzalloc(sizeof(*hdr_info), GFP_KERNEL);
        if (!hdr_info)
@@ -1307,7 +1307,7 @@ int visorbus_register_visor_driver(struct visor_driver 
*drv)
 
        err = create_bus_type();
        if (err < 0) {
-               POSTCODE_LINUX_2(BUS_CREATE_ENTRY_PC, DIAG_SEVERITY_ERR);
+               POSTCODE_LINUX(BUS_CREATE_ENTRY_PC, 0, 0, DIAG_SEVERITY_ERR);
                goto error;
        }
 
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c 
b/drivers/staging/unisys/visorbus/visorchipset.c
index 95a2cf1..25da22c 100644
--- a/drivers/staging/unisys/visorbus/visorchipset.c
+++ b/drivers/staging/unisys/visorbus/visorchipset.c
@@ -519,14 +519,14 @@ struct visor_device *visorbus_get_device_by_id(u32 
bus_no, u32 dev_no,
        int rc = CONTROLVM_RESP_SUCCESS;
        int res = 0;
 
-       POSTCODE_LINUX_2(CHIPSET_INIT_ENTRY_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(CHIPSET_INIT_ENTRY_PC, 0, 0, POSTCODE_SEVERITY_INFO);
        if (chipset_inited) {
                rc = -CONTROLVM_RESP_ERROR_ALREADY_DONE;
                res = -EIO;
                goto out_respond;
        }
        chipset_inited = 1;
-       POSTCODE_LINUX_2(CHIPSET_INIT_EXIT_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(CHIPSET_INIT_EXIT_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 
        /*
         * Set features to indicate we support parahotplug (if Command
@@ -591,8 +591,8 @@ enum crash_obj_type {
                                         saved_crash_message_count),
                                &local_crash_msg_count, sizeof(u16));
        if (err) {
-               POSTCODE_LINUX_2(CRASH_DEV_CTRL_RD_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_CTRL_RD_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return err;
        }
 
@@ -608,8 +608,8 @@ enum crash_obj_type {
                                         saved_crash_message_offset),
                                &local_crash_msg_offset, sizeof(u32));
        if (err) {
-               POSTCODE_LINUX_2(CRASH_DEV_CTRL_RD_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_CTRL_RD_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return err;
        }
 
@@ -619,8 +619,8 @@ enum crash_obj_type {
                                         msg,
                                        sizeof(struct controlvm_message));
                if (err) {
-                       POSTCODE_LINUX_2(SAVE_MSG_BUS_FAILURE_PC,
-                                        POSTCODE_SEVERITY_ERR);
+                       POSTCODE_LINUX(SAVE_MSG_BUS_FAILURE_PC, 0, 0,
+                                      POSTCODE_SEVERITY_ERR);
                        return err;
                }
        } else {
@@ -630,8 +630,8 @@ enum crash_obj_type {
                                         msg,
                                         sizeof(struct controlvm_message));
                if (err) {
-                       POSTCODE_LINUX_2(SAVE_MSG_DEV_FAILURE_PC,
-                                        POSTCODE_SEVERITY_ERR);
+                       POSTCODE_LINUX(SAVE_MSG_DEV_FAILURE_PC, 0, 0,
+                                      POSTCODE_SEVERITY_ERR);
                        return err;
                }
        }
@@ -1135,16 +1135,16 @@ enum crash_obj_type {
                              offsetof(struct spar_controlvm_channel_protocol,
                                       request_payload_offset),
                              &payload_offset, sizeof(payload_offset)) < 0) {
-               POSTCODE_LINUX_2(CONTROLVM_INIT_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CONTROLVM_INIT_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
        if (visorchannel_read(controlvm_channel,
                              offsetof(struct spar_controlvm_channel_protocol,
                                       request_payload_bytes),
                              &payload_bytes, sizeof(payload_bytes)) < 0) {
-               POSTCODE_LINUX_2(CONTROLVM_INIT_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CONTROLVM_INIT_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
        initialize_controlvm_payload_info(phys_addr,
@@ -1564,7 +1564,7 @@ static u64 controlvm_get_channel_address(void)
        u32 local_crash_msg_offset;
        u16 local_crash_msg_count;
 
-       POSTCODE_LINUX_2(CRASH_DEV_ENTRY_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(CRASH_DEV_ENTRY_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 
        /* send init chipset msg */
        msg.hdr.id = CONTROLVM_CHIPSET_INIT;
@@ -1578,8 +1578,8 @@ static u64 controlvm_get_channel_address(void)
                              offsetof(struct spar_controlvm_channel_protocol,
                                       saved_crash_message_count),
                              &local_crash_msg_count, sizeof(u16)) < 0) {
-               POSTCODE_LINUX_2(CRASH_DEV_CTRL_RD_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_CTRL_RD_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
 
@@ -1595,8 +1595,8 @@ static u64 controlvm_get_channel_address(void)
                              offsetof(struct spar_controlvm_channel_protocol,
                                       saved_crash_message_offset),
                              &local_crash_msg_offset, sizeof(u32)) < 0) {
-               POSTCODE_LINUX_2(CRASH_DEV_CTRL_RD_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_CTRL_RD_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
 
@@ -1605,8 +1605,8 @@ static u64 controlvm_get_channel_address(void)
                              local_crash_msg_offset,
                              &local_crash_bus_msg,
                              sizeof(struct controlvm_message)) < 0) {
-               POSTCODE_LINUX_2(CRASH_DEV_RD_BUS_FAIULRE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_RD_BUS_FAIULRE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
 
@@ -1616,8 +1616,8 @@ static u64 controlvm_get_channel_address(void)
                              sizeof(struct controlvm_message),
                              &local_crash_dev_msg,
                              sizeof(struct controlvm_message)) < 0) {
-               POSTCODE_LINUX_2(CRASH_DEV_RD_DEV_FAIULRE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_RD_DEV_FAIULRE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
 
@@ -1625,8 +1625,8 @@ static u64 controlvm_get_channel_address(void)
        if (local_crash_bus_msg.cmd.create_bus.channel_addr) {
                bus_create(&local_crash_bus_msg);
        } else {
-               POSTCODE_LINUX_2(CRASH_DEV_BUS_NULL_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_BUS_NULL_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
 
@@ -1634,11 +1634,11 @@ static u64 controlvm_get_channel_address(void)
        if (local_crash_dev_msg.cmd.create_device.channel_addr) {
                my_device_create(&local_crash_dev_msg);
        } else {
-               POSTCODE_LINUX_2(CRASH_DEV_DEV_NULL_FAILURE_PC,
-                                POSTCODE_SEVERITY_ERR);
+               POSTCODE_LINUX(CRASH_DEV_DEV_NULL_FAILURE_PC, 0, 0,
+                              POSTCODE_SEVERITY_ERR);
                return;
        }
-       POSTCODE_LINUX_2(CRASH_DEV_EXIT_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(CRASH_DEV_EXIT_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 }
 
 void
@@ -2174,11 +2174,12 @@ static long visorchipset_ioctl(struct file *file, 
unsigned int cmd,
 
        visorchipset_platform_device.dev.devt = major_dev;
        if (platform_device_register(&visorchipset_platform_device) < 0) {
-               POSTCODE_LINUX_2(DEVICE_REGISTER_FAILURE_PC, DIAG_SEVERITY_ERR);
+               POSTCODE_LINUX(DEVICE_REGISTER_FAILURE_PC, 0, 0,
+                              DIAG_SEVERITY_ERR);
                err = -ENODEV;
                goto error_cancel_work;
        }
-       POSTCODE_LINUX_2(CHIPSET_INIT_SUCCESS_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(CHIPSET_INIT_SUCCESS_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 
        err = visorbus_init();
        if (err < 0)
@@ -2207,7 +2208,7 @@ static long visorchipset_ioctl(struct file *file, 
unsigned int cmd,
 static int
 visorchipset_exit(struct acpi_device *acpi_device)
 {
-       POSTCODE_LINUX_2(DRIVER_EXIT_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(DRIVER_EXIT_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 
        visorbus_exit();
 
@@ -2218,7 +2219,7 @@ static long visorchipset_ioctl(struct file *file, 
unsigned int cmd,
 
        visorchipset_file_cleanup(visorchipset_platform_device.dev.devt);
        platform_device_unregister(&visorchipset_platform_device);
-       POSTCODE_LINUX_2(DRIVER_EXIT_PC, POSTCODE_SEVERITY_INFO);
+       POSTCODE_LINUX(DRIVER_EXIT_PC, 0, 0, POSTCODE_SEVERITY_INFO);
 
        return 0;
 }
diff --git a/drivers/staging/unisys/visorbus/vmcallinterface.h 
b/drivers/staging/unisys/visorbus/vmcallinterface.h
index b6181f9..89c2d67 100644
--- a/drivers/staging/unisys/visorbus/vmcallinterface.h
+++ b/drivers/staging/unisys/visorbus/vmcallinterface.h
@@ -254,9 +254,6 @@ enum event_pc {                     /* POSTCODE event 
identifier tuples */
 } while (0)
 
 /* MOST COMMON */
-#define POSTCODE_LINUX_2(EVENT_PC, severity)                           \
-       POSTCODE_LINUX(EVENT_PC, 0, 0, severity)
-
 #define POSTCODE_LINUX_3(EVENT_PC, pc32bit, severity)                  \
        POSTCODE_LINUX(EVENT_PC, (pc32bit >> 16), (pc32bit & 0xFFFF),   \
                       severity)
-- 
1.9.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to