mbox series

[0/4] drm/displayid: use primary use case to figure out non-desktop

Message ID cover.1676580180.git.jani.nikula@intel.com (mailing list archive)
Headers show
Series drm/displayid: use primary use case to figure out non-desktop | expand

Message

Jani Nikula Feb. 16, 2023, 8:44 p.m. UTC
Mostly this is prep work and plumbing for easier use of displayid
structure version and primary use case for parsing the displayid blocks,
but it can be nicely used for figuring out non-desktop too.

Completely untested. :)

BR,
Jani.

Cc: Iaroslav Boliukin <iam@lach.pw>
Cc: Dmitry Osipenko <dmitry.osipenko@collabora.com>

Jani Nikula (4):
  drm/displayid: add displayid_get_header() and check bounds better
  drm/displayid: return struct displayid_header from
    validate_displayid()
  drm/displayid: provide access to DisplayID version and primary use
    case
  drm/edid: update non-desktop use also from DisplayID

 drivers/gpu/drm/drm_displayid.c | 62 ++++++++++++++++++++++++++++-----
 drivers/gpu/drm/drm_edid.c      | 25 +++++++++++++
 include/drm/drm_displayid.h     | 12 ++++++-
 3 files changed, 89 insertions(+), 10 deletions(-)

Comments

Dmitry Osipenko Feb. 20, 2023, 3:44 p.m. UTC | #1
On 2/16/23 23:44, Jani Nikula wrote:
> Mostly this is prep work and plumbing for easier use of displayid
> structure version and primary use case for parsing the displayid blocks,
> but it can be nicely used for figuring out non-desktop too.
> 
> Completely untested. :)
> 
> BR,
> Jani.
> 
> Cc: Iaroslav Boliukin <iam@lach.pw>
> Cc: Dmitry Osipenko <dmitry.osipenko@collabora.com>
> 
> Jani Nikula (4):
>   drm/displayid: add displayid_get_header() and check bounds better
>   drm/displayid: return struct displayid_header from
>     validate_displayid()
>   drm/displayid: provide access to DisplayID version and primary use
>     case
>   drm/edid: update non-desktop use also from DisplayID
> 
>  drivers/gpu/drm/drm_displayid.c | 62 ++++++++++++++++++++++++++++-----
>  drivers/gpu/drm/drm_edid.c      | 25 +++++++++++++
>  include/drm/drm_displayid.h     | 12 ++++++-
>  3 files changed, 89 insertions(+), 10 deletions(-)
> 

It works now without the EDID quirk, thanks!

Tested-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
Dmitry Osipenko Feb. 21, 2023, 4:29 p.m. UTC | #2
On 2/20/23 18:44, Dmitry Osipenko wrote:
> On 2/16/23 23:44, Jani Nikula wrote:
>> Mostly this is prep work and plumbing for easier use of displayid
>> structure version and primary use case for parsing the displayid blocks,
>> but it can be nicely used for figuring out non-desktop too.
>>
>> Completely untested. :)
>>
>> BR,
>> Jani.
>>
>> Cc: Iaroslav Boliukin <iam@lach.pw>
>> Cc: Dmitry Osipenko <dmitry.osipenko@collabora.com>
>>
>> Jani Nikula (4):
>>   drm/displayid: add displayid_get_header() and check bounds better
>>   drm/displayid: return struct displayid_header from
>>     validate_displayid()
>>   drm/displayid: provide access to DisplayID version and primary use
>>     case
>>   drm/edid: update non-desktop use also from DisplayID
>>
>>  drivers/gpu/drm/drm_displayid.c | 62 ++++++++++++++++++++++++++++-----
>>  drivers/gpu/drm/drm_edid.c      | 25 +++++++++++++
>>  include/drm/drm_displayid.h     | 12 ++++++-
>>  3 files changed, 89 insertions(+), 10 deletions(-)
>>
> 
> It works now without the EDID quirk, thanks!
> 
> Tested-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
> 

I'm going to apply this to misc-next later this week if there won't be
any objections.
Dmitry Osipenko Feb. 27, 2023, 12:03 a.m. UTC | #3
On 2/21/23 19:29, Dmitry Osipenko wrote:
> On 2/20/23 18:44, Dmitry Osipenko wrote:
>> On 2/16/23 23:44, Jani Nikula wrote:
>>> Mostly this is prep work and plumbing for easier use of displayid
>>> structure version and primary use case for parsing the displayid blocks,
>>> but it can be nicely used for figuring out non-desktop too.
>>>
>>> Completely untested. :)
>>>
>>> BR,
>>> Jani.
>>>
>>> Cc: Iaroslav Boliukin <iam@lach.pw>
>>> Cc: Dmitry Osipenko <dmitry.osipenko@collabora.com>
>>>
>>> Jani Nikula (4):
>>>   drm/displayid: add displayid_get_header() and check bounds better
>>>   drm/displayid: return struct displayid_header from
>>>     validate_displayid()
>>>   drm/displayid: provide access to DisplayID version and primary use
>>>     case
>>>   drm/edid: update non-desktop use also from DisplayID
>>>
>>>  drivers/gpu/drm/drm_displayid.c | 62 ++++++++++++++++++++++++++++-----
>>>  drivers/gpu/drm/drm_edid.c      | 25 +++++++++++++
>>>  include/drm/drm_displayid.h     | 12 ++++++-
>>>  3 files changed, 89 insertions(+), 10 deletions(-)
>>>
>>
>> It works now without the EDID quirk, thanks!
>>
>> Tested-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
>>
> 
> I'm going to apply this to misc-next later this week if there won't be
> any objections.
> 

Applied all 5 patches to misc-next
Jani Nikula Feb. 27, 2023, 2:57 p.m. UTC | #4
On Mon, 27 Feb 2023, Dmitry Osipenko <dmitry.osipenko@collabora.com> wrote:
> On 2/21/23 19:29, Dmitry Osipenko wrote:
>> On 2/20/23 18:44, Dmitry Osipenko wrote:
>>> On 2/16/23 23:44, Jani Nikula wrote:
>>>> Mostly this is prep work and plumbing for easier use of displayid
>>>> structure version and primary use case for parsing the displayid blocks,
>>>> but it can be nicely used for figuring out non-desktop too.
>>>>
>>>> Completely untested. :)
>>>>
>>>> BR,
>>>> Jani.
>>>>
>>>> Cc: Iaroslav Boliukin <iam@lach.pw>
>>>> Cc: Dmitry Osipenko <dmitry.osipenko@collabora.com>
>>>>
>>>> Jani Nikula (4):
>>>>   drm/displayid: add displayid_get_header() and check bounds better
>>>>   drm/displayid: return struct displayid_header from
>>>>     validate_displayid()
>>>>   drm/displayid: provide access to DisplayID version and primary use
>>>>     case
>>>>   drm/edid: update non-desktop use also from DisplayID
>>>>
>>>>  drivers/gpu/drm/drm_displayid.c | 62 ++++++++++++++++++++++++++++-----
>>>>  drivers/gpu/drm/drm_edid.c      | 25 +++++++++++++
>>>>  include/drm/drm_displayid.h     | 12 ++++++-
>>>>  3 files changed, 89 insertions(+), 10 deletions(-)
>>>>
>>>
>>> It works now without the EDID quirk, thanks!
>>>
>>> Tested-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
>>>
>> 
>> I'm going to apply this to misc-next later this week if there won't be
>> any objections.
>> 
>
> Applied all 5 patches to misc-next

Thanks for the review and pushing!

BR,
Jani.