summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHeiko Carstens <heiko.carstens@de.ibm.com>2013-09-07 11:07:22 +0200
committerHeiko Carstens <heiko.carstens@de.ibm.com>2013-09-07 11:58:18 +0200
commit7d6c3b492f7fc1b0393b03ec6582a80c3a68bc42 (patch)
tree63387f98a92bb8780009aadbaa3ea66eafca5aad
parent5eb8ae503e13c6eee3545e3e3be986db000ae8c2 (diff)
downloadop-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.c2
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;
OpenPOWER on IntegriCloud