On Tue, 2020-06-23 at 09:59 +0200, Thomas Zimmermann wrote:

Hi Deepak

I did not receive you pat series, so I can only comment on Sam's
reply.
See below for some points.

Hi Thomas, Thanks for the review. I wanted to add you in cc list but
messed it up with final git send-email. Sorry about that. I am not sure
why you didn't received it via dri-devel. The patch series do show up
in dri-devel archive. I wonder if other people also have similar
issues.

quoted

quoted

+	struct hv_device *hdev;
+};
+
+#define to_hv(_dev) container_of(_dev, struct hyperv_device,
dev)

Could this be a function?

Is there a reason to use a function here?

quoted

quoted

+
+/* -----------------------------------------------------------
----------- */
+/* Hyper-V Synthetic Video
Protocol                                       */

The comments look awkward. Unless this style has been used within
DRM,
maybe just use

 /*
  * ...
  */

This style is copy-paste from cirrus, and bochs also have same style.
Perhaps historical. Anyway I agree to I should get rid of this.