.vendor_id = 0x21e7,
.product_id = 0x000e,
},
+ {
+ .name = "Philips/Sagemcom PicoPix 2055",
+ .vendor_id = 0x21e7,
+ .product_id = 0x0016,
+ },
};
/* The header size on the wire is known to be always 24 bytes, regardless of
debug(ctx, "BEGIN\n");
debug(ctx, "packet_type: 0x%08x (%u)\n", h->packet_type, h->packet_type);
- debug(ctx, "direction: 0x%02hhx (%hhu)\n", h->direction, h->direction);
+ debug(ctx, "direction: 0x%02hhx (%hhu) (%s)\n", h->direction, h->direction,
+ h->direction == AM7XXX_DIRECTION_IN ? "IN" :
+ h->direction == AM7XXX_DIRECTION_OUT ? "OUT" :
+ "UNKNOWN");
debug(ctx, "header_data_len: 0x%02hhx (%hhu)\n", h->header_data_len, h->header_data_len);
debug(ctx, "unknown2: 0x%02hhx (%hhu)\n", h->unknown2, h->unknown2);
debug(ctx, "unknown3: 0x%02hhx (%hhu)\n", h->unknown3, h->unknown3);
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 {
return send_data(dev, image, image_size);
}
-AM7XXX_PUBLIC int am7xxx_set_power_mode(am7xxx_device *dev, am7xxx_power_mode mode)
+AM7XXX_PUBLIC int am7xxx_set_power_mode(am7xxx_device *dev, am7xxx_power_mode power)
{
int ret;
struct am7xxx_header h = {
.unknown3 = 0x10,
};
- switch(mode) {
+ switch(power) {
case AM7XXX_POWER_OFF:
h.header_data.power.bit2 = 0;
h.header_data.power.bit1 = 0;
break;
default:
- error(dev->ctx, "Power mode not supported!\n");
+ error(dev->ctx, "Unsupported power mode.\n");
return -EINVAL;
};