Hi, In "clean_up:" section of v4l2_device_register_subdev_nodes() we have: if (!sd->devnode) break; Maybe I miss something, but shouldn't it be rather "continue" instead of "break"? Regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html