<p dir="ltr">Reviewed-by: Zhi Wang <<a href="mailto:zhiwang@kernel.org">zhiwang@kernel.org</a>></p>
<br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Garg, Nemesa <<a href="mailto:nemesa.garg@intel.com">nemesa.garg@intel.com</a>> 于 2024年12月18日周三 下午3:37写道：<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
> -----Original Message-----<br>
> From: Intel-gfx <<a href="mailto:intel-gfx-bounces@lists.freedesktop.org" target="_blank" rel="noreferrer">intel-gfx-bounces@lists.freedesktop.org</a>> On Behalf Of Jani<br>
> Nikula<br>
> Sent: Wednesday, December 18, 2024 7:50 PM<br>
> To: <a href="mailto:intel-gfx@lists.freedesktop.org" target="_blank" rel="noreferrer">intel-gfx@lists.freedesktop.org</a><br>
> Cc: <a href="mailto:intel-gvt-dev@lists.freedesktop.org" target="_blank" rel="noreferrer">intel-gvt-dev@lists.freedesktop.org</a>; Nikula, Jani <<a href="mailto:jani.nikula@intel.com" target="_blank" rel="noreferrer">jani.nikula@intel.com</a>><br>
> Subject: [PATCH] drm/i915/gvt: store virtual_dp_monitor_edid in rodata<br>
> <br>
> The virtual DP EDID isn't modified. Add const modifier to store it in rodata.<br>
> <br>
> Signed-off-by: Jani Nikula <<a href="mailto:jani.nikula@intel.com" target="_blank" rel="noreferrer">jani.nikula@intel.com</a>><br>
<br>
LGTM,<br>
Reviewed-by: Nemesa Garg <<a href="mailto:nemesa.garg@intel.com" target="_blank" rel="noreferrer">nemesa.garg@intel.com</a>><br>
> ---<br>
> drivers/gpu/drm/i915/gvt/display.c | 2 +-<br>
> 1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/drivers/gpu/drm/i915/gvt/display.c<br>
> b/drivers/gpu/drm/i915/gvt/display.c<br>
> index 95570cabdf27..c98dfcc3d0de 100644<br>
> --- a/drivers/gpu/drm/i915/gvt/display.c<br>
> +++ b/drivers/gpu/drm/i915/gvt/display.c<br>
> @@ -97,7 +97,7 @@ int pipe_is_enabled(struct intel_vgpu *vgpu, int pipe)<br>
> return 0;<br>
> }<br>
> <br>
> -static unsigned char virtual_dp_monitor_edid[GVT_EDID_NUM][EDID_SIZE]<br>
> = {<br>
> +static const unsigned char<br>
> +virtual_dp_monitor_edid[GVT_EDID_NUM][EDID_SIZE] = {<br>
> {<br>
> /* EDID with 1024x768 as its resolution */<br>
> /*Header*/<br>
> --<br>
> 2.39.5<br>
<br>
</blockquote></div>