staging: vc04_services: parse_rx_slots() - Fix compiler warning
authorMichael Zoran <mzoran@crowfest.net>
Tue, 1 Nov 2016 15:21:14 +0000 (08:21 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 7 Nov 2016 10:04:10 +0000 (11:04 +0100)
vc04_services contains a debug logging mechanism.  The log is
maintained in a shared memory area between the kernel and the
firmware.  Changing the sizes of the data in this area would
require a firmware change which is distributed independently
from the kernel binary.

One of the items logged is the address of received messages.
This address is a pointer, but the debugging slot used to store
the information is a 32 bit integer.

Luckily, this value is never interpreted by anything other
then debug tools and it is expected that a human debugging
the kernel interpret it.

This change adds a cast to long before the original cast
to int to silence the warning.

Signed-off-by: Michael Zoran <mzoran@crowfest.net>
Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c

index a771062..5978017 100644 (file)
@@ -1668,7 +1668,7 @@ parse_rx_slots(VCHIQ_STATE_T *state)
 
                header = (VCHIQ_HEADER_T *)(state->rx_data +
                        (state->rx_pos & VCHIQ_SLOT_MASK));
-               DEBUG_VALUE(PARSE_HEADER, (int)header);
+               DEBUG_VALUE(PARSE_HEADER, (int)(long)header);
                msgid = header->msgid;
                DEBUG_VALUE(PARSE_MSGID, msgid);
                size = header->size;