RE: [EXTERNAL] [PATCH v2 1/1] drm/hyperv: Replace "hyperv_" with "hv_drm_" as symbol name prefix

From: Michael Kelley

Date: Thu May 28 2026 - 21:47:57 EST


From: Dexuan Cui <DECUI@xxxxxxxxxxxxx> Sent: Thursday, May 28, 2026 9:53 AM
>
> > From: Michael Kelley <mhklkml@xxxxxxxxxxxx>
> > 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 <decui@xxxxxxxxxxxxx>

Thanks for reviewing.

Michael