diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2013-09-07 05:07:22 -0400 |
---|---|---|
committer | Heiko Carstens <heiko.carstens@de.ibm.com> | 2013-09-07 05:58:18 -0400 |
commit | 7d6c3b492f7fc1b0393b03ec6582a80c3a68bc42 (patch) | |
tree | 63387f98a92bb8780009aadbaa3ea66eafca5aad /drivers/s390/crypto/ap_bus.c | |
parent | 5eb8ae503e13c6eee3545e3e3be986db000ae8c2 (diff) |
s390/ap_bus: use and-mask instead of a cast
Let's get rid of another sparse false positive:
drivers/s390/crypto/ap_bus.c:416:64: warning:
cast truncates bits from constant value (102030405060708 becomes 5060708)
So instead of using a cast let's use an and-mask.
That way sparse remains silent and one doesn't always have to check
if this is a valid warning/bug or just a false positive.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Diffstat (limited to 'drivers/s390/crypto/ap_bus.c')
-rw-r--r-- | drivers/s390/crypto/ap_bus.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/s390/crypto/ap_bus.c b/drivers/s390/crypto/ap_bus.c index d4174b82a1a9..02300dcfac91 100644 --- a/drivers/s390/crypto/ap_bus.c +++ b/drivers/s390/crypto/ap_bus.c | |||
@@ -413,7 +413,7 @@ __ap_send(ap_qid_t qid, unsigned long long psmid, void *msg, size_t length, | |||
413 | register unsigned long reg2 asm ("2") = (unsigned long) msg; | 413 | register unsigned long reg2 asm ("2") = (unsigned long) msg; |
414 | register unsigned long reg3 asm ("3") = (unsigned long) length; | 414 | register unsigned long reg3 asm ("3") = (unsigned long) length; |
415 | register unsigned long reg4 asm ("4") = (unsigned int) (psmid >> 32); | 415 | register unsigned long reg4 asm ("4") = (unsigned int) (psmid >> 32); |
416 | register unsigned long reg5 asm ("5") = (unsigned int) psmid; | 416 | register unsigned long reg5 asm ("5") = psmid & 0xffffffff; |
417 | 417 | ||
418 | if (special == 1) | 418 | if (special == 1) |
419 | reg0 |= 0x400000UL; | 419 | reg0 |= 0x400000UL; |