mbox series

[RESEND,0/6] drm, nouveau/radeon/amdpgu: edid_blob_ptr cleanups

Message ID cover.1715353572.git.jani.nikula@intel.com (mailing list archive)
Headers show
Series drm, nouveau/radeon/amdpgu: edid_blob_ptr cleanups | expand

Message

Jani Nikula May 10, 2024, 3:08 p.m. UTC
I've sent this some moths ago, let's try again...

BR,
Jani.

Jani Nikula (6):
  drm/nouveau: convert to using is_hdmi and has_audio from display info
  drm/radeon: convert to using is_hdmi and has_audio from display info
  drm/radeon: remove radeon_connector_edid() and stop using
    edid_blob_ptr
  drm/amdgpu: remove amdgpu_connector_edid() and stop using
    edid_blob_ptr
  drm/edid: add a helper for EDID sysfs property show
  drm/connector: update edid_blob_ptr documentation

 .../gpu/drm/amd/amdgpu/amdgpu_connectors.c    | 16 ---------
 .../gpu/drm/amd/amdgpu/amdgpu_connectors.h    |  1 -
 drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |  4 +--
 drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |  4 +--
 drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |  4 +--
 drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |  4 +--
 drivers/gpu/drm/drm_crtc_internal.h           |  2 ++
 drivers/gpu/drm/drm_edid.c                    | 33 +++++++++++++++++++
 drivers/gpu/drm/drm_sysfs.c                   | 24 ++------------
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  8 ++---
 drivers/gpu/drm/nouveau/dispnv50/head.c       |  8 +----
 drivers/gpu/drm/nouveau/nouveau_connector.c   |  2 +-
 drivers/gpu/drm/radeon/atombios_encoders.c    | 10 +++---
 drivers/gpu/drm/radeon/evergreen_hdmi.c       |  5 ++-
 drivers/gpu/drm/radeon/radeon_audio.c         | 13 ++++----
 drivers/gpu/drm/radeon/radeon_connectors.c    | 27 ++++-----------
 drivers/gpu/drm/radeon/radeon_display.c       |  2 +-
 drivers/gpu/drm/radeon/radeon_encoders.c      |  4 +--
 drivers/gpu/drm/radeon/radeon_mode.h          |  2 --
 include/drm/drm_connector.h                   |  6 +++-
 20 files changed, 79 insertions(+), 100 deletions(-)

Comments

Alex Deucher May 10, 2024, 3:45 p.m. UTC | #1
On Fri, May 10, 2024 at 11:17 AM Jani Nikula <jani.nikula@intel.com> wrote:
>
> I've sent this some moths ago, let's try again...
>
> BR,
> Jani.
>
> Jani Nikula (6):
>   drm/nouveau: convert to using is_hdmi and has_audio from display info
>   drm/radeon: convert to using is_hdmi and has_audio from display info
>   drm/radeon: remove radeon_connector_edid() and stop using
>     edid_blob_ptr
>   drm/amdgpu: remove amdgpu_connector_edid() and stop using
>     edid_blob_ptr
>   drm/edid: add a helper for EDID sysfs property show
>   drm/connector: update edid_blob_ptr documentation

Series is:
Acked-by: Alex Deucher <alexander.deucher@amd.com>

>
>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.c    | 16 ---------
>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.h    |  1 -
>  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |  4 +--
>  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |  4 +--
>  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |  4 +--
>  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |  4 +--
>  drivers/gpu/drm/drm_crtc_internal.h           |  2 ++
>  drivers/gpu/drm/drm_edid.c                    | 33 +++++++++++++++++++
>  drivers/gpu/drm/drm_sysfs.c                   | 24 ++------------
>  drivers/gpu/drm/nouveau/dispnv50/disp.c       |  8 ++---
>  drivers/gpu/drm/nouveau/dispnv50/head.c       |  8 +----
>  drivers/gpu/drm/nouveau/nouveau_connector.c   |  2 +-
>  drivers/gpu/drm/radeon/atombios_encoders.c    | 10 +++---
>  drivers/gpu/drm/radeon/evergreen_hdmi.c       |  5 ++-
>  drivers/gpu/drm/radeon/radeon_audio.c         | 13 ++++----
>  drivers/gpu/drm/radeon/radeon_connectors.c    | 27 ++++-----------
>  drivers/gpu/drm/radeon/radeon_display.c       |  2 +-
>  drivers/gpu/drm/radeon/radeon_encoders.c      |  4 +--
>  drivers/gpu/drm/radeon/radeon_mode.h          |  2 --
>  include/drm/drm_connector.h                   |  6 +++-
>  20 files changed, 79 insertions(+), 100 deletions(-)
>
> --
> 2.39.2
>
Jani Nikula May 13, 2024, 12:19 p.m. UTC | #2
On Fri, 10 May 2024, Alex Deucher <alexdeucher@gmail.com> wrote:
> On Fri, May 10, 2024 at 11:17 AM Jani Nikula <jani.nikula@intel.com> wrote:
>>
>> I've sent this some moths ago, let's try again...
>>
>> BR,
>> Jani.
>>
>> Jani Nikula (6):
>>   drm/nouveau: convert to using is_hdmi and has_audio from display info
>>   drm/radeon: convert to using is_hdmi and has_audio from display info
>>   drm/radeon: remove radeon_connector_edid() and stop using
>>     edid_blob_ptr
>>   drm/amdgpu: remove amdgpu_connector_edid() and stop using
>>     edid_blob_ptr
>>   drm/edid: add a helper for EDID sysfs property show
>>   drm/connector: update edid_blob_ptr documentation
>
> Series is:
> Acked-by: Alex Deucher <alexander.deucher@amd.com>

Thanks, do you want to pick these up via your tree? And do you expect a
proper R-b before merging?

BR,
Jani.


>
>>
>>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.c    | 16 ---------
>>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.h    |  1 -
>>  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |  4 +--
>>  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |  4 +--
>>  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |  4 +--
>>  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |  4 +--
>>  drivers/gpu/drm/drm_crtc_internal.h           |  2 ++
>>  drivers/gpu/drm/drm_edid.c                    | 33 +++++++++++++++++++
>>  drivers/gpu/drm/drm_sysfs.c                   | 24 ++------------
>>  drivers/gpu/drm/nouveau/dispnv50/disp.c       |  8 ++---
>>  drivers/gpu/drm/nouveau/dispnv50/head.c       |  8 +----
>>  drivers/gpu/drm/nouveau/nouveau_connector.c   |  2 +-
>>  drivers/gpu/drm/radeon/atombios_encoders.c    | 10 +++---
>>  drivers/gpu/drm/radeon/evergreen_hdmi.c       |  5 ++-
>>  drivers/gpu/drm/radeon/radeon_audio.c         | 13 ++++----
>>  drivers/gpu/drm/radeon/radeon_connectors.c    | 27 ++++-----------
>>  drivers/gpu/drm/radeon/radeon_display.c       |  2 +-
>>  drivers/gpu/drm/radeon/radeon_encoders.c      |  4 +--
>>  drivers/gpu/drm/radeon/radeon_mode.h          |  2 --
>>  include/drm/drm_connector.h                   |  6 +++-
>>  20 files changed, 79 insertions(+), 100 deletions(-)
>>
>> --
>> 2.39.2
>>
Alex Deucher May 13, 2024, 1:53 p.m. UTC | #3
On Mon, May 13, 2024 at 8:20 AM Jani Nikula <jani.nikula@intel.com> wrote:
>
> On Fri, 10 May 2024, Alex Deucher <alexdeucher@gmail.com> wrote:
> > On Fri, May 10, 2024 at 11:17 AM Jani Nikula <jani.nikula@intel.com> wrote:
> >>
> >> I've sent this some moths ago, let's try again...
> >>
> >> BR,
> >> Jani.
> >>
> >> Jani Nikula (6):
> >>   drm/nouveau: convert to using is_hdmi and has_audio from display info
> >>   drm/radeon: convert to using is_hdmi and has_audio from display info
> >>   drm/radeon: remove radeon_connector_edid() and stop using
> >>     edid_blob_ptr
> >>   drm/amdgpu: remove amdgpu_connector_edid() and stop using
> >>     edid_blob_ptr
> >>   drm/edid: add a helper for EDID sysfs property show
> >>   drm/connector: update edid_blob_ptr documentation
> >
> > Series is:
> > Acked-by: Alex Deucher <alexander.deucher@amd.com>
>
> Thanks, do you want to pick these up via your tree? And do you expect a
> proper R-b before merging?

Feel free to take them via drm-misc if you'd prefer to land the whole
set together, otherwise, I can pick up the radeon/amdgpu patches.

Alex


>
> BR,
> Jani.
>
>
> >
> >>
> >>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.c    | 16 ---------
> >>  .../gpu/drm/amd/amdgpu/amdgpu_connectors.h    |  1 -
> >>  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |  4 +--
> >>  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |  4 +--
> >>  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |  4 +--
> >>  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |  4 +--
> >>  drivers/gpu/drm/drm_crtc_internal.h           |  2 ++
> >>  drivers/gpu/drm/drm_edid.c                    | 33 +++++++++++++++++++
> >>  drivers/gpu/drm/drm_sysfs.c                   | 24 ++------------
> >>  drivers/gpu/drm/nouveau/dispnv50/disp.c       |  8 ++---
> >>  drivers/gpu/drm/nouveau/dispnv50/head.c       |  8 +----
> >>  drivers/gpu/drm/nouveau/nouveau_connector.c   |  2 +-
> >>  drivers/gpu/drm/radeon/atombios_encoders.c    | 10 +++---
> >>  drivers/gpu/drm/radeon/evergreen_hdmi.c       |  5 ++-
> >>  drivers/gpu/drm/radeon/radeon_audio.c         | 13 ++++----
> >>  drivers/gpu/drm/radeon/radeon_connectors.c    | 27 ++++-----------
> >>  drivers/gpu/drm/radeon/radeon_display.c       |  2 +-
> >>  drivers/gpu/drm/radeon/radeon_encoders.c      |  4 +--
> >>  drivers/gpu/drm/radeon/radeon_mode.h          |  2 --
> >>  include/drm/drm_connector.h                   |  6 +++-
> >>  20 files changed, 79 insertions(+), 100 deletions(-)
> >>
> >> --
> >> 2.39.2
> >>
>
> --
> Jani Nikula, Intel
Jani Nikula May 23, 2024, 1 p.m. UTC | #4
On Mon, 13 May 2024, Alex Deucher <alexdeucher@gmail.com> wrote:
> On Mon, May 13, 2024 at 8:20 AM Jani Nikula <jani.nikula@intel.com> wrote:
>>
>> On Fri, 10 May 2024, Alex Deucher <alexdeucher@gmail.com> wrote:
>> > On Fri, May 10, 2024 at 11:17 AM Jani Nikula <jani.nikula@intel.com> wrote:
>> > Series is:
>> > Acked-by: Alex Deucher <alexander.deucher@amd.com>
>>
>> Thanks, do you want to pick these up via your tree? And do you expect a
>> proper R-b before merging?
>
> Feel free to take them via drm-misc if you'd prefer to land the whole
> set together, otherwise, I can pick up the radeon/amdgpu patches.

Thanks, merged everything to drm-misc-next.

BR,
Jani.