unserialize_header(dev->buffer, h);
- debug_dump_header(dev->ctx, h);
-
if (h->direction == AM7XXX_DIRECTION_IN) {
ret = 0;
} else {
error(dev->ctx,
- "Received a packet with direction AM7XXX_DIRECTION_OUT, weird!\n");
+ "Expected an AM7XXX_DIRECTION_IN packet, got one with direction = %d. Weird!\n",
+ h->direction);
ret = -EINVAL;
}
+ debug_dump_header(dev->ctx, h);
+
out:
return ret;
}
debug_dump_header(dev->ctx, h);
+ /* For symmetry with read_header() we should check here for
+ * h->direction == AM7XXX_DIRECTION_OUT but we just ensure that in all
+ * the callers and save some cycles here.
+ */
+
serialize_header(h, dev->buffer);
+
ret = send_data(dev, dev->buffer, AM7XXX_HEADER_WIRE_SIZE);
if (ret < 0)
error(dev->ctx, "failed to send data\n");
return NULL;
}
- devices_list = &(ctx->devices_list);
-
new_device = malloc(sizeof(*new_device));
if (new_device == NULL) {
fatal("cannot allocate a new device (%s)\n", strerror(errno));
new_device->ctx = ctx;
+ devices_list = &(ctx->devices_list);
+
if (*devices_list == NULL) {
*devices_list = new_device;
} else {