staging: vchiq_arm: Rename a struct vchiq_bulk member
authorUmang Jain <umang.jain@ideasonboard.com>
Wed, 23 Oct 2024 11:04:01 +0000 (16:34 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 3 Nov 2024 23:55:06 +0000 (00:55 +0100)
Rename the struct vchiq_bulk's 'data' member to 'dma_addr' for
better readability. No functional changes intended in this patch.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20241023110406.885199-2-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h

index c2e7c2bd507136ab7e6c626698c7e189e3a88c80..f23d98a1b96081784071a05d1399d729fbd841df 100644 (file)
@@ -587,7 +587,7 @@ vchiq_blocking_bulk_transfer(struct vchiq_instance *instance, unsigned int handl
                if (bulk) {
                        /* This thread has an outstanding bulk transfer. */
                        /* FIXME: why compare a dma address to a pointer? */
-                       if ((bulk->data != (dma_addr_t)(uintptr_t)bulk_params->data) ||
+                       if ((bulk->dma_addr != (dma_addr_t)(uintptr_t)bulk_params->dma_addr) ||
                            (bulk->size != bulk_params->size)) {
                                /*
                                 * This is not a retry of the previous one.
index da18c77f3cabcb04d6df0ebd63cd1695a7a9095d..1e3b2d76fcb61826b60b91d8a2154f3a512fc3f8 100644 (file)
@@ -1369,7 +1369,7 @@ notify_bulks(struct vchiq_service *service, struct vchiq_bulk_queue *queue,
                 * Only generate callbacks for non-dummy bulk
                 * requests, and non-terminated services
                 */
-               if (bulk->data && service->instance) {
+               if (bulk->dma_addr && service->instance) {
                        status = service_notify_bulk(service, bulk);
                        if (status == -EAGAIN)
                                break;
@@ -1751,7 +1751,7 @@ vchiq_prepare_bulk_data(struct vchiq_instance *instance, struct vchiq_bulk *bulk
        if (!pagelistinfo)
                return -ENOMEM;
 
-       bulk->data = pagelistinfo->dma_addr;
+       bulk->dma_addr = pagelistinfo->dma_addr;
 
        /*
         * Store the pagelistinfo address in remote_data,
@@ -1807,7 +1807,7 @@ abort_outstanding_bulks(struct vchiq_service *service,
                                service->remoteport, bulk->size, bulk->remote_size);
                } else {
                        /* fabricate a matching dummy bulk */
-                       bulk->data = 0;
+                       bulk->dma_addr = 0;
                        bulk->size = 0;
                        bulk->actual = VCHIQ_BULK_ACTUAL_ABORTED;
                        bulk->dir = is_tx ? VCHIQ_BULK_TRANSMIT :
@@ -2112,7 +2112,7 @@ parse_message(struct vchiq_state *state, struct vchiq_header *header)
 
                        dev_dbg(state->dev, "core: %d: prs %s@%pK (%d->%d) %x@%pad\n",
                                state->id, msg_type_str(type), header, remoteport,
-                               localport, bulk->actual, &bulk->data);
+                               localport, bulk->actual, &bulk->dma_addr);
 
                        dev_dbg(state->dev, "core: %d: prs:%d %cx li=%x ri=%x p=%x\n",
                                state->id, localport,
@@ -3081,7 +3081,7 @@ vchiq_bulk_xfer_queue_msg_killable(struct vchiq_service *service,
 
        dev_dbg(state->dev, "core: %d: bt (%d->%d) %cx %x@%pad %pK\n",
                state->id, service->localport, service->remoteport,
-               dir_char, bulk->size, &bulk->data, bulk->userdata);
+               dir_char, bulk->size, &bulk->dma_addr, bulk->userdata);
 
        /*
         * The slot mutex must be held when the service is being closed, so
@@ -3095,7 +3095,7 @@ vchiq_bulk_xfer_queue_msg_killable(struct vchiq_service *service,
        if (service->srvstate != VCHIQ_SRVSTATE_OPEN)
                goto unlock_both_error_exit;
 
-       payload[0] = lower_32_bits(bulk->data);
+       payload[0] = lower_32_bits(bulk->dma_addr);
        payload[1] = bulk->size;
        status = queue_message(state,
                               NULL,
index 186b1395d3a222f46b6d5ccb5a4880a4b628b834..9ba4a2295dc38605020d5d23c096e08e4f41f86f 100644 (file)
@@ -115,7 +115,7 @@ struct vchiq_bulk {
        short mode;
        short dir;
        void *userdata;
-       dma_addr_t data;
+       dma_addr_t dma_addr;
        int size;
        void *remote_data;
        int remote_size;