diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2013-09-07 11:07:22 +0200 |
---|---|---|
committer | Heiko Carstens <heiko.carstens@de.ibm.com> | 2013-09-07 11:58:18 +0200 |
commit | 7d6c3b492f7fc1b0393b03ec6582a80c3a68bc42 (patch) | |
tree | 63387f98a92bb8780009aadbaa3ea66eafca5aad | |
parent | 5eb8ae503e13c6eee3545e3e3be986db000ae8c2 (diff) | |
download | op-kernel-dev-7d6c3b492f7fc1b0393b03ec6582a80c3a68bc42.zip op-kernel-dev-7d6c3b492f7fc1b0393b03ec6582a80c3a68bc42.tar.gz |
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>
-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 d4174b8..02300dc 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, register unsigned long reg2 asm ("2") = (unsigned long) msg; register unsigned long reg3 asm ("3") = (unsigned long) length; register unsigned long reg4 asm ("4") = (unsigned int) (psmid >> 32); - register unsigned long reg5 asm ("5") = (unsigned int) psmid; + register unsigned long reg5 asm ("5") = psmid & 0xffffffff; if (special == 1) reg0 |= 0x400000UL; |