i2400m->rx_reorder = i2400m_rx_reorder_disabled? 0 : 1;
        if (i2400m->rx_reorder) {
                unsigned itr;
-               size_t size;
                struct i2400m_roq_log *rd;
 
                result = -ENOMEM;
 
-               size = sizeof(i2400m->rx_roq[0]) * (I2400M_RO_CIN + 1);
-               i2400m->rx_roq = kzalloc(size, GFP_KERNEL);
-               if (i2400m->rx_roq == NULL) {
-                       dev_err(dev, "RX: cannot allocate %zu bytes for "
-                               "reorder queues\n", size);
+               i2400m->rx_roq = kcalloc(I2400M_RO_CIN + 1,
+                                        sizeof(i2400m->rx_roq[0]), GFP_KERNEL);
+               if (i2400m->rx_roq == NULL)
                        goto error_roq_alloc;
-               }
 
-               size = sizeof(*i2400m->rx_roq[0].log) * (I2400M_RO_CIN + 1);
-               rd = kzalloc(size, GFP_KERNEL);
+               rd = kcalloc(I2400M_RO_CIN + 1, sizeof(*i2400m->rx_roq[0].log),
+                            GFP_KERNEL);
                if (rd == NULL) {
-                       dev_err(dev, "RX: cannot allocate %zu bytes for "
-                               "reorder queues log areas\n", size);
                        result = -ENOMEM;
                        goto error_roq_log_alloc;
                }
 
        d_fnstart(4, dev, "(i2400m %p)\n", i2400mu);
        buf = kmalloc(I2400MU_MAX_NOTIFICATION_LEN, GFP_KERNEL | GFP_DMA);
        if (buf == NULL) {
-               dev_err(dev, "notification: buffer allocation failed\n");
                ret = -ENOMEM;
                goto error_buf_alloc;
        }