RE: [PATCH v4] video: hyperv: hyperv_fb: Obtain screen resolution from Hyper-V host

[Dexuan Cui]

> From: Michael Kelley <mikelley@xxxxxxxxxxxxx>
> Sent: Thursday, September 5, 2019 7:06 AM
> 
> From: Wei Hu <weh@xxxxxxxxxxxxx> Sent: Thursday, September 5, 2019 2:12
> AM
> >
> > Beginning from Windows 10 RS5+, VM screen resolution is obtained from
> host.
> > The "video=hyperv_fb" boot time option is not needed, but still can be
> > used to overwrite what the host specifies. The VM resolution on the host
> > could be set by executing the powershell "set-vmvideo" command.
> >
> > Signed-off-by: Iouri Tarassov <iourit@xxxxxxxxxxxxx>
> > Signed-off-by: Wei Hu <weh@xxxxxxxxxxxxx>
> > ---
> >     v2:
> >     - Implemented fallback when version negotiation failed.
> >     - Defined full size for supported_resolution array.
> >
> >     v3:
> >     - Corrected the synthvid major and minor version comparison problem.
> >
> >     v4:
> >     - Changed function name to synthvid_ver_ge().
> >
> >  drivers/video/fbdev/hyperv_fb.c | 159
> +++++++++++++++++++++++++++++---
> >  1 file changed, 147 insertions(+), 12 deletions(-)
> >
> 
> Reviewed-by: Michael Kelley <mikelley@xxxxxxxxxxxxx>

Looks good to me.

Reviewed-by: Dexuan Cui <decui@xxxxxxxxxxxxx>