s390/page: Cleanup page_set_storage_key() inline assemblies

Add extra lines, indentations, and symbolic names for operands in
order to make the two page_set_storage_key() inline assemblies a bit
more readable.

Reviewed-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Alexander Gordeev <agordeev@linux.ibm.com>

diff --git a/arch/s390/include/asm/page.h b/arch/s390/include/asm/page.h
index 4e5dbabdf202560dacd364772e1c6d6628252312..79db884b8e3e5e2c50779cb39b94cb7f52efe403 100644 (file)
--- a/arch/s390/include/asm/page.h
+++ b/arch/s390/include/asm/page.h
@@ -130,11 +130,17 @@ typedef pte_t *pgtable_t;
 static inline void page_set_storage_key(unsigned long addr,
                                        unsigned char skey, int mapped)
 {
-       if (!mapped)
-               asm volatile(".insn rrf,0xb22b0000,%0,%1,8,0"
-                            : : "d" (skey), "a" (addr));
-       else
-               asm volatile("sske %0,%1" : : "d" (skey), "a" (addr));
+       if (!mapped) {
+               asm volatile(
+                       "       .insn   rrf,0xb22b0000,%[skey],%[addr],8,0"
+                       :
+                       : [skey] "d" (skey), [addr] "a" (addr));
+       } else {
+               asm volatile(
+                       "       sske     %[skey],%[addr]"
+                       :
+                       : [skey] "d" (skey), [addr] "a" (addr));
+       }
 }
 
 static inline unsigned char page_get_storage_key(unsigned long addr)