Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-06-01 Thread Daniel Vetter
On Thu, May 26, 2022 at 09:01:03AM -0700, Doug Anderson wrote: > Hi, > > On Thu, May 26, 2022 at 8:42 AM Daniel Vetter wrote: > > > > On Thu, 26 May 2022 at 03:28, Sean Paul wrote: > > > > > > On Wed, May 25, 2022 at 9:26 AM Daniel Vetter wrote: > > > > > > > > On Mon, May 23, 2022 at 05:59:02P

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-26 Thread Doug Anderson
Hi, On Thu, May 26, 2022 at 8:42 AM Daniel Vetter wrote: > > On Thu, 26 May 2022 at 03:28, Sean Paul wrote: > > > > On Wed, May 25, 2022 at 9:26 AM Daniel Vetter wrote: > > > > > > On Mon, May 23, 2022 at 05:59:02PM -0700, Doug Anderson wrote: > > > > Hi, > > > > > > > > On Fri, May 20, 2022 at

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-26 Thread Daniel Vetter
On Thu, 26 May 2022 at 03:28, Sean Paul wrote: > > On Wed, May 25, 2022 at 9:26 AM Daniel Vetter wrote: > > > > On Mon, May 23, 2022 at 05:59:02PM -0700, Doug Anderson wrote: > > > Hi, > > > > > > On Fri, May 20, 2022 at 5:01 PM Doug Anderson > > > wrote: > > > > > > > > Hi, > > > > > > > > On

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-25 Thread Sean Paul
On Wed, May 25, 2022 at 9:26 AM Daniel Vetter wrote: > > On Mon, May 23, 2022 at 05:59:02PM -0700, Doug Anderson wrote: > > Hi, > > > > On Fri, May 20, 2022 at 5:01 PM Doug Anderson wrote: > > > > > > Hi, > > > > > > On Mon, May 16, 2022 at 3:28 AM Thomas Zimmermann > > > wrote: > > > > > > > >

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-25 Thread Daniel Vetter
On Mon, May 23, 2022 at 05:59:02PM -0700, Doug Anderson wrote: > Hi, > > On Fri, May 20, 2022 at 5:01 PM Doug Anderson wrote: > > > > Hi, > > > > On Mon, May 16, 2022 at 3:28 AM Thomas Zimmermann > > wrote: > > > > > > Hi Douglas, > > > > > > I understand that you're trying to tell userspace th

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-23 Thread Doug Anderson
Hi, On Fri, May 20, 2022 at 5:01 PM Doug Anderson wrote: > > Hi, > > On Mon, May 16, 2022 at 3:28 AM Thomas Zimmermann wrote: > > > > Hi Douglas, > > > > I understand that you're trying to tell userspace that the modelist has > > been made up, but it's not something that should be done via fragi

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-20 Thread Doug Anderson
Hi, On Mon, May 16, 2022 at 3:28 AM Thomas Zimmermann wrote: > > Hi Douglas, > > I understand that you're trying to tell userspace that the modelist has > been made up, but it's not something that should be done via fragile > heuristics IMHO. > > I looked at the Chromium source code that you link

Re: [PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-16 Thread Thomas Zimmermann
Hi Douglas, I understand that you're trying to tell userspace that the modelist has been made up, but it's not something that should be done via fragile heuristics IMHO. I looked at the Chromium source code that you linked, but I cannot say whether it's doing the correct thing. It all depend

[PATCH v3] drm/probe-helper: Make 640x480 first if no EDID

2022-05-13 Thread Douglas Anderson
If we're unable to read the EDID for a display because it's corrupt / bogus / invalid then we'll add a set of standard modes for the display. Since we have no true information about the connected display, these modes are essentially guesses but better than nothing. None of the modes returned is mar