4 files changed, 11 insertions, 52 deletions

diff --git a/drivers/s390/crypto/ap_bus.c b/drivers/s390/crypto/ap_bus.c
index e99713041591..67aaff3e668d 100644
--- a/drivers/s390/crypto/ap_bus.c
+++ b/drivers/s390/crypto/ap_bus.c
@@ -1225,8 +1225,9 @@ static void ap_reset_domain(void)
 {
         int i;
 
-        for (i = 0; i < AP_DEVICES; i++)
-                ap_reset_queue(AP_MKQID(i, ap_domain_index));
+        if (ap_domain_index != -1)
+                for (i = 0; i < AP_DEVICES; i++)
+                        ap_reset_queue(AP_MKQID(i, ap_domain_index));
 }
 
 static void ap_reset_all(void)
diff --git a/drivers/s390/crypto/zcrypt_mono.c b/drivers/s390/crypto/zcrypt_mono.c
index 2a9349ad68b7..44253fdd4136 100644
--- a/drivers/s390/crypto/zcrypt_mono.c
+++ b/drivers/s390/crypto/zcrypt_mono.c
@@ -45,7 +45,7 @@
 /**
  * The module initialization code.
  */
-int __init zcrypt_init(void)
+static int __init zcrypt_init(void)
 {
         int rc;
 
@@ -86,7 +86,7 @@ out:
 /**
  * The module termination code.
  */
-void __exit zcrypt_exit(void)
+static void __exit zcrypt_exit(void)
 {
         zcrypt_cex2a_exit();
         zcrypt_pcixcc_exit();
diff --git a/drivers/s390/crypto/zcrypt_pcixcc.c b/drivers/s390/crypto/zcrypt_pcixcc.c
index 64948788d301..70b9ddc8cf9d 100644
--- a/drivers/s390/crypto/zcrypt_pcixcc.c
+++ b/drivers/s390/crypto/zcrypt_pcixcc.c
@@ -277,7 +277,7 @@ static int XCRB_msg_to_type6CPRB_msgX(struct zcrypt_device *zdev,
         };
         struct {
                 struct type6_hdr hdr;
-                struct ica_CPRBX cprbx;
+                struct CPRBX cprbx;
         } __attribute__((packed)) *msg = ap_msg->message;
 
         int rcblen = CEIL4(xcRB->request_control_blk_length);
@@ -432,14 +432,17 @@ static int convert_type86_ica(struct zcrypt_device *zdev,
                 }
                 if (service_rc == 8 && service_rs == 770) {
                         PDEBUG("Invalid key length on PCIXCC/CEX2C\n");
-                        zdev->min_mod_size = PCIXCC_MIN_MOD_SIZE_OLD;
-                        return -EAGAIN;
+                        return -EINVAL;
                 }
                 if (service_rc == 8 && service_rs == 783) {
                         PDEBUG("Extended bitlengths not enabled on PCIXCC/CEX2C\n");
                         zdev->min_mod_size = PCIXCC_MIN_MOD_SIZE_OLD;
                         return -EAGAIN;
                 }
+                if (service_rc == 12 && service_rs == 769) {
+                        PDEBUG("Invalid key on PCIXCC/CEX2C\n");
+                        return -EINVAL;
+                }
                 PRINTK("Unknown service rc/rs (PCIXCC/CEX2C): %d/%d\n",
                        service_rc, service_rs);
                 zdev->online = 0;
diff --git a/drivers/s390/crypto/zcrypt_pcixcc.h b/drivers/s390/crypto/zcrypt_pcixcc.h
index a78ff307fd19..8cb7d7a6973b 100644
--- a/drivers/s390/crypto/zcrypt_pcixcc.h
+++ b/drivers/s390/crypto/zcrypt_pcixcc.h
@@ -28,51 +28,6 @@
 #ifndef _ZCRYPT_PCIXCC_H_
 #define _ZCRYPT_PCIXCC_H_
 
-/**
- * CPRBX
- *        Note that all shorts and ints are big-endian.
- *        All pointer fields are 16 bytes long, and mean nothing.
- *
- *        A request CPRB is followed by a request_parameter_block.
- *
- *        The request (or reply) parameter block is organized thus:
- *          function code
- *          VUD block
- *          key block
- */
-struct CPRBX {
-        unsigned short cprb_len;        /* CPRB length        220        */
-        unsigned char  cprb_ver_id;     /* CPRB version id.   0x02       */
-        unsigned char  pad_000[3];      /* Alignment pad bytes           */
-        unsigned char  func_id[2];      /* function id        0x5432     */
-        unsigned char  cprb_flags[4];   /* Flags                         */
-        unsigned int   req_parml;       /* request parameter buffer len  */
-        unsigned int   req_datal;       /* request data buffer           */
-        unsigned int   rpl_msgbl;       /* reply  message block length   */
-        unsigned int   rpld_parml;      /* replied parameter block len   */
-        unsigned int   rpl_datal;       /* reply data block len          */
-        unsigned int   rpld_datal;      /* replied data block len        */
-        unsigned int   req_extbl;       /* request extension block len   */
-        unsigned char  pad_001[4];      /* reserved                      */
-        unsigned int   rpld_extbl;      /* replied extension block len   */
-        unsigned char  req_parmb[16];   /* request parm block 'address'  */
-        unsigned char  req_datab[16];   /* request data block 'address'  */
-        unsigned char  rpl_parmb[16];   /* reply parm block 'address'    */
-        unsigned char  rpl_datab[16];   /* reply data block 'address'    */
-        unsigned char  req_extb[16];    /* request extension block 'addr'*/
-        unsigned char  rpl_extb[16];    /* reply extension block 'addres'*/
-        unsigned short ccp_rtcode;      /* server return code            */
-        unsigned short ccp_rscode;      /* server reason code            */
-        unsigned int   mac_data_len;    /* Mac Data Length               */
-        unsigned char  logon_id[8];     /* Logon Identifier              */
-        unsigned char  mac_value[8];    /* Mac Value                     */
-        unsigned char  mac_content_flgs;/* Mac content flag byte         */
-        unsigned char  pad_002;         /* Alignment                     */
-        unsigned short domain;          /* Domain                        */
-        unsigned char  pad_003[12];     /* Domain masks                  */
-        unsigned char  pad_004[36];     /* reserved                      */
-} __attribute__((packed));
-
 int zcrypt_pcixcc_init(void);
 void zcrypt_pcixcc_exit(void);