Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-03-04 Thread Jani Nikula
On Mon, 04 Mar 2024, Hsin-Yi Wang wrote: > On Mon, Mar 4, 2024 at 8:17 AM Doug Anderson wrote: >> >> Hi, >> >> On Sun, Mar 3, 2024 at 1:30 PM Dmitry Baryshkov >> wrote: >> > >> > > The problem is that Dmitry didn't like the idea of using a hash and in >> > > v2 Hsin-Yi has moved to using the

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-03-04 Thread Hsin-Yi Wang
On Mon, Mar 4, 2024 at 8:17 AM Doug Anderson wrote: > > Hi, > > On Sun, Mar 3, 2024 at 1:30 PM Dmitry Baryshkov > wrote: > > > > > The problem is that Dmitry didn't like the idea of using a hash and in > > > v2 Hsin-Yi has moved to using the name of the display. ...except of > > > course that

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-03-04 Thread Doug Anderson
Hi, On Sun, Mar 3, 2024 at 1:30 PM Dmitry Baryshkov wrote: > > > The problem is that Dmitry didn't like the idea of using a hash and in > > v2 Hsin-Yi has moved to using the name of the display. ...except of > > course that eDP panels don't always properly specify > > "EDID_DETAIL_MONITOR_NAME".

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-03-03 Thread Dmitry Baryshkov
On Thu, 29 Feb 2024 at 19:11, Doug Anderson wrote: > > Hi, > > On Thu, Feb 29, 2024 at 8:43 AM Jani Nikula > wrote: > > > > On Wed, 28 Feb 2024, Doug Anderson wrote: > > > Hi, > > > > > > On Tue, Feb 27, 2024 at 5:27 PM Hsin-Yi Wang wrote: > > >> > > >> On Tue, Feb 27, 2024 at 1:09 AM Jani

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-29 Thread Doug Anderson
Hi, On Thu, Feb 29, 2024 at 8:43 AM Jani Nikula wrote: > > On Wed, 28 Feb 2024, Doug Anderson wrote: > > Hi, > > > > On Tue, Feb 27, 2024 at 5:27 PM Hsin-Yi Wang wrote: > >> > >> On Tue, Feb 27, 2024 at 1:09 AM Jani Nikula > >> wrote: > >> > > >> > On Fri, 23 Feb 2024, Hsin-Yi Wang wrote: >

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-29 Thread Jani Nikula
On Wed, 28 Feb 2024, Doug Anderson wrote: > Hi, > > On Tue, Feb 27, 2024 at 5:27 PM Hsin-Yi Wang wrote: >> >> On Tue, Feb 27, 2024 at 1:09 AM Jani Nikula >> wrote: >> > >> > On Fri, 23 Feb 2024, Hsin-Yi Wang wrote: >> > > It's found that some panels have variants that they share the same

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-28 Thread Doug Anderson
Hi, On Tue, Feb 27, 2024 at 5:27 PM Hsin-Yi Wang wrote: > > On Tue, Feb 27, 2024 at 1:09 AM Jani Nikula > wrote: > > > > On Fri, 23 Feb 2024, Hsin-Yi Wang wrote: > > > It's found that some panels have variants that they share the same panel > > > id > > > although their EDID and names are

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-27 Thread Hsin-Yi Wang
On Tue, Feb 27, 2024 at 1:09 AM Jani Nikula wrote: > > On Fri, 23 Feb 2024, Hsin-Yi Wang wrote: > > It's found that some panels have variants that they share the same panel id > > although their EDID and names are different. Besides panel id, now we need > > the hash of entire EDID base block to

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-27 Thread Doug Anderson
Hi, On Tue, Feb 27, 2024 at 1:09 AM Jani Nikula wrote: > > On Fri, 23 Feb 2024, Hsin-Yi Wang wrote: > > It's found that some panels have variants that they share the same panel id > > although their EDID and names are different. Besides panel id, now we need > > the hash of entire EDID base

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-27 Thread Jani Nikula
On Fri, 23 Feb 2024, Hsin-Yi Wang wrote: > It's found that some panels have variants that they share the same panel id > although their EDID and names are different. Besides panel id, now we need > the hash of entire EDID base block to distinguish these panel variants. > > Add

Re: [PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-26 Thread Doug Anderson
Hi, On Fri, Feb 23, 2024 at 2:40 PM Hsin-Yi Wang wrote: > > @@ -2770,58 +2770,63 @@ static u32 edid_extract_panel_id(const struct edid > *edid) > } > > /** > - * drm_edid_get_panel_id - Get a panel's ID through DDC > - * @adapter: I2C adapter to use for DDC > + * drm_edid_get_panel_id - Get a

[PATCH 1/2] drm_edid: Add a function to get EDID base block

2024-02-23 Thread Hsin-Yi Wang
It's found that some panels have variants that they share the same panel id although their EDID and names are different. Besides panel id, now we need the hash of entire EDID base block to distinguish these panel variants. Add drm_edid_get_base_block to returns the EDID base block, so caller can