summaryrefslogtreecommitdiff
path: root/sound/firewire
diff options
context:
space:
mode:
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>2016-05-18 16:27:43 +0300
committerTakashi Iwai <tiwai@suse.de>2016-05-18 17:32:09 +0300
commit17e1717c11a34f9b0956e33e0c4a4e4ae8c51a57 (patch)
treed56d4ec3279aa37db18de1e982be2c61e92bb491 /sound/firewire
parentc7c5856b6f6f30de622bf87947363890b286553f (diff)
downloadlinux-17e1717c11a34f9b0956e33e0c4a4e4ae8c51a57.tar.xz
ALSA: firewire-lib: change a member of event structure to suppress sparse wanings to bool type
Commit a9c4284bf5a9 ("ALSA: firewire-lib: add context information to tracepoints") adds new members to tracepoint events of this module, to represent context information. One of the members is bool type and this causes sparse warnings. 16:1: warning: expression using sizeof bool 60:1: warning: expression using sizeof bool 16:1: warning: odd constant _Bool cast (ffffffffffffffff becomes 1) 60:1: warning: odd constant _Bool cast (ffffffffffffffff becomes 1) This commit suppresses the warnings, by changing type of the member to 'unsigned int'. Additionally, this commit applies '!!' idiom to get 0/1 from 'in_interrupt()'. Fixes: a9c4284bf5a9 ("ALSA: firewire-lib: add context information to tracepoints") Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/firewire')
-rw-r--r--sound/firewire/amdtp-stream-trace.h8
1 files changed, 4 insertions, 4 deletions
diff --git a/sound/firewire/amdtp-stream-trace.h b/sound/firewire/amdtp-stream-trace.h
index 16225792b722..9c04faf206b2 100644
--- a/sound/firewire/amdtp-stream-trace.h
+++ b/sound/firewire/amdtp-stream-trace.h
@@ -26,7 +26,7 @@ TRACE_EVENT(in_packet,
__field(u32, cip_header1)
__field(unsigned int, payload_quadlets)
__field(unsigned int, packet_index)
- __field(bool, irq)
+ __field(unsigned int, irq)
__field(unsigned int, index)
),
TP_fast_assign(
@@ -39,7 +39,7 @@ TRACE_EVENT(in_packet,
__entry->cip_header1 = cip_header[1];
__entry->payload_quadlets = payload_quadlets;
__entry->packet_index = s->packet_index;
- __entry->irq = in_interrupt();
+ __entry->irq = !!in_interrupt();
__entry->index = index;
),
TP_printk(
@@ -70,7 +70,7 @@ TRACE_EVENT(out_packet,
__field(u32, cip_header1)
__field(unsigned int, payload_quadlets)
__field(unsigned int, packet_index)
- __field(bool, irq)
+ __field(unsigned int, irq)
__field(unsigned int, index)
),
TP_fast_assign(
@@ -83,7 +83,7 @@ TRACE_EVENT(out_packet,
__entry->cip_header1 = be32_to_cpu(cip_header[1]);
__entry->payload_quadlets = payload_length / 4;
__entry->packet_index = s->packet_index;
- __entry->irq = in_interrupt();
+ __entry->irq = !!in_interrupt();
__entry->index = index;
),
TP_printk(