From: Dexuan Cui <[email protected]> Sent: Thursday, May 28, 2026 9:53 AM
> 
> > From: Michael Kelley <[email protected]>
> > Sent: Thursday, May 28, 2026 6:51 AM
> >  ...
> > -#define to_hv(_dev) container_of(_dev, struct hyperv_drm_device, dev)
> > +#define to_hv(_dev) container_of(_dev, struct hv_drm_device, dev)
> 
> A minor nit:  change "to_hv" to "to_hv_drm"? Otherwise, LGTM.

Yes, that makes sense. It's not a symbol that would appear in, and cause
confusion in, a global symbol list. But the "hv" in "to_hv" is effectively
the prefix, so for completeness change it as well.

I'll send a v3 shortly.

> 
> Reviewed-by: Dexuan Cui <[email protected]>

Thanks for reviewing.

Michael


Reply via email to