Re: [RFC PATCH v3 0/6] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-04-08 Thread Melissa Wen
On 03/28, Jani Nikula wrote:
> On Wed, 27 Mar 2024, Melissa Wen  wrote:
> > 2. Most of the edid data handled by `dm_helpers_parse_edid_caps()` are
> >in drm_edid halpers, but there are still a few that are not managed by
> >them yet. For example:
> >```
> > edid_caps->serial_number = edid_buf->serial;
> > edid_caps->manufacture_week = edid_buf->mfg_week;
> > edid_caps->manufacture_year = edid_buf->mfg_year;
> >```
> >AFAIU I see the same pending issue in i915/pnpid_get_panel_type() -
> >that still uses drm_edid_raw().
> 
> See https://lore.kernel.org/r/cover.1711015462.git.jani.nik...@intel.com

Thanks!
I'll work on top of your series.

Melissa

> 
> BR,
> Jani.
> 
> 
> -- 
> Jani Nikula, Intel


Re: [RFC PATCH v3 0/6] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-03-28 Thread Jani Nikula
On Wed, 27 Mar 2024, Melissa Wen  wrote:
> 2. Most of the edid data handled by `dm_helpers_parse_edid_caps()` are
>in drm_edid halpers, but there are still a few that are not managed by
>them yet. For example:
>```
>   edid_caps->serial_number = edid_buf->serial;
>   edid_caps->manufacture_week = edid_buf->mfg_week;
>   edid_caps->manufacture_year = edid_buf->mfg_year;
>```
>AFAIU I see the same pending issue in i915/pnpid_get_panel_type() -
>that still uses drm_edid_raw().

See https://lore.kernel.org/r/cover.1711015462.git.jani.nik...@intel.com

BR,
Jani.


-- 
Jani Nikula, Intel