The problem is that everyone reading [patch 1/6] thinks you're renaming hv_device to vm_device or introducing a new struct vm_device. That makes people annoyed. If you had written the patch description like this: In the original code, the structs vm_device included a struct hv_device. This patch moves the members from hv_device directly into struct vm_device. The members ->dev_type and ->dev_instance from hv_device were the same as ->class_id and ->device_id in vm_device so those were not copied over. Now that everything is included into vm_device directly, hv_device is unused and we can delete the definition. There still might be issues with the patch, but at least you would be talking about the same thing. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel