Re: [PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-08 Thread Melissa Wen
On 08/05, Simon Ser wrote: > Hi, > > Have you seen [1] and [2]? CC'ing Sean and Hans, it seems like there's > a disagreement regarding probing early vs. setting the initial value to > UNKNOWN. > > If a driver doesn't fetch EDIDs before exposing them to user-space, > then there is an amount of

Re: [PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-08 Thread Melissa Wen
On 08/08, Hans de Goede wrote: > Hi, > > On 8/5/22 19:34, Simon Ser wrote: > > Hi, > > > > Have you seen [1] and [2]? CC'ing Sean and Hans, it seems like there's > > a disagreement regarding probing early vs. setting the initial value to > > UNKNOWN. > > At least for the discussions I've been

Re: [PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-08 Thread Hans de Goede
Hi, On 8/5/22 19:34, Simon Ser wrote: > Hi, > > Have you seen [1] and [2]? CC'ing Sean and Hans, it seems like there's > a disagreement regarding probing early vs. setting the initial value to > UNKNOWN. At least for the discussions I've been involved in so far (see the links in the originak

Re: [PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-05 Thread Simon Ser
Hi, Have you seen [1] and [2]? CC'ing Sean and Hans, it seems like there's a disagreement regarding probing early vs. setting the initial value to UNKNOWN. If a driver doesn't fetch EDIDs before exposing them to user-space, then there is an amount of time where the connectors are exposed as

Re: [PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-03 Thread Harry Wentland
On 2022-08-03 12:24, Melissa Wen wrote: > To set the panel orientation property with quirk, we need the mode size > provided by EDID. This info is available after EDID is read by > dc_link_detect() > and updated by amdgpu_dm_update_connector_after_detect(). The detection > happens at driver

Re: [PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-03 Thread Melissa Wen
On 08/03, Melissa Wen wrote: > To set the panel orientation property with quirk, we need the mode size > provided by EDID. This info is available after EDID is read by > dc_link_detect() > and updated by amdgpu_dm_update_connector_after_detect(). The detection > happens at driver load in

[PATCH] drm/amd/display: set panel orientation before drm_dev_register

2022-08-03 Thread Melissa Wen
To set the panel orientation property with quirk, we need the mode size provided by EDID. This info is available after EDID is read by dc_link_detect() and updated by amdgpu_dm_update_connector_after_detect(). The detection happens at driver load in amdgpu_dm_initialize_drm_device() and,