[PATCH v2] drm/amdgpu: Clear the hotplug interrupt ack bit before hpd initialization
Qiang Ma
maqianga at uniontech.com
Wed Mar 27 03:31:00 UTC 2024
On Thu, 14 Mar 2024 14:40:40 +0000
"Deucher, Alexander" <Alexander.Deucher at amd.com> wrote:
> [Public]
>
> > -----Original Message-----
> > From: Qiang Ma <maqianga at uniontech.com>
> > Sent: Wednesday, March 13, 2024 2:18 AM
> > To: Deucher, Alexander <Alexander.Deucher at amd.com>; Koenig,
> > Christian <Christian.Koenig at amd.com>; Pan, Xinhui
> > <Xinhui.Pan at amd.com>; airlied at gmail.com; daniel at ffwll.ch;
> > SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM at amd.com>;
> > sunran001 at 208suo.com Cc: amd-gfx at lists.freedesktop.org;
> > dri-devel at lists.freedesktop.org; linux- kernel at vger.kernel.org
> > Subject: Re: [PATCH v2] drm/amdgpu: Clear the hotplug interrupt ack
> > bit before hpd initialization
> >
> > On Wed, 31 Jan 2024 15:57:03 +0800
> > Qiang Ma <maqianga at uniontech.com> wrote:
> >
> > Hello everyone, please help review this patch.
>
> This was applied back in January, sorry if I forget to reply.
>
> Alex
Hi, Alex, it doesn't matter, please take some time to help review this
patch.
This patch mainly solves the problem that after unplugging the HDMI
display during bios initialization, the display does not light up after
the system starts.
Qiang Ma
>
> >
> > Qiang Ma
> >
> > > Problem:
> > > The computer in the bios initialization process, unplug the HDMI
> > > display, wait until the system up, plug in the HDMI display, did
> > > not enter the hotplug interrupt function, the display is not
> > > bright.
> > >
> > > Fix:
> > > After the above problem occurs, and the hpd ack interrupt bit is
> > > 1, the interrupt should be cleared during hpd_init initialization
> > > so that when the driver is ready, it can respond to the hpd
> > > interrupt normally.
> > >
> > > Signed-off-by: Qiang Ma <maqianga at uniontech.com>
> > > ---
> > > v2:
> > > - Remove unused variable 'tmp'
> > > - Fixed function spelling errors
> > >
> > > drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 2 ++
> > > drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 2 ++
> > > drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 22
> > > ++++++++++++++++++---
> > -
> > > drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 22
> > > ++++++++++++++++++---
> > -
> > > 4 files changed, 40 insertions(+), 8 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c index
> > > bb666cb7522e..12a8ba929a72 100644 ---
> > > a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c +++
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c @@ -51,6 +51,7 @@
> > >
> > > static void dce_v10_0_set_display_funcs(struct amdgpu_device
> > > *adev); static void dce_v10_0_set_irq_funcs(struct amdgpu_device
> > > *adev); +static void dce_v10_0_hpd_int_ack(struct amdgpu_device
> > > *adev, int hpd);
> > > static const u32 crtc_offsets[] = {
> > > CRTC0_REGISTER_OFFSET,
> > > @@ -363,6 +364,7 @@ static void dce_v10_0_hpd_init(struct
> > > amdgpu_device *adev) AMDGPU_HPD_DISCONNECT_INT_DELAY_IN_MS);
> > > WREG32(mmDC_HPD_TOGGLE_FILT_CNTL +
> > > hpd_offsets[amdgpu_connector->hpd.hpd], tmp);
> > > + dce_v10_0_hpd_int_ack(adev,
> > > amdgpu_connector->hpd.hpd); dce_v10_0_hpd_set_polarity(adev,
> > > amdgpu_connector->hpd.hpd); amdgpu_irq_get(adev, &adev->hpd_irq,
> > > amdgpu_connector->hpd.hpd); diff --git
> > > a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c index
> > > 7af277f61cca..745e4fdffade 100644 ---
> > > a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c +++
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c @@ -51,6 +51,7 @@
> > >
> > > static void dce_v11_0_set_display_funcs(struct amdgpu_device
> > > *adev); static void dce_v11_0_set_irq_funcs(struct amdgpu_device
> > > *adev); +static void dce_v11_0_hpd_int_ack(struct amdgpu_device
> > > *adev, int hpd);
> > > static const u32 crtc_offsets[] =
> > > {
> > > @@ -387,6 +388,7 @@ static void dce_v11_0_hpd_init(struct
> > > amdgpu_device *adev) AMDGPU_HPD_DISCONNECT_INT_DELAY_IN_MS);
> > > WREG32(mmDC_HPD_TOGGLE_FILT_CNTL +
> > > hpd_offsets[amdgpu_connector->hpd.hpd], tmp);
> > > + dce_v11_0_hpd_int_ack(adev,
> > > amdgpu_connector->hpd.hpd); dce_v11_0_hpd_set_polarity(adev,
> > > amdgpu_connector->hpd.hpd); amdgpu_irq_get(adev, &adev->hpd_irq,
> > > amdgpu_connector->hpd.hpd); } diff --git
> > > a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c index
> > > 143efc37a17f..28c4a735716b 100644 ---
> > > a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c +++
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c @@ -272,6 +272,21 @@
> > static
> > > void dce_v6_0_hpd_set_polarity(struct amdgpu_device *adev,
> > > WREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd], tmp); }
> > >
> > > +static void dce_v6_0_hpd_int_ack(struct amdgpu_device *adev,
> > > + int hpd)
> > > +{
> > > + u32 tmp;
> > > +
> > > + if (hpd >= adev->mode_info.num_hpd) {
> > > + DRM_DEBUG("invalid hdp %d\n", hpd);
> > > + return;
> > > + }
> > > +
> > > + tmp = RREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd]);
> > > + tmp |= DC_HPD1_INT_CONTROL__DC_HPD1_INT_ACK_MASK;
> > > + WREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd], tmp); }
> > > +
> > > /**
> > > * dce_v6_0_hpd_init - hpd setup callback.
> > > *
> > > @@ -311,6 +326,7 @@ static void dce_v6_0_hpd_init(struct
> > amdgpu_device
> > > *adev) continue;
> > > }
> > >
> > > + dce_v6_0_hpd_int_ack(adev,
> > > amdgpu_connector->hpd.hpd); dce_v6_0_hpd_set_polarity(adev,
> > > amdgpu_connector->hpd.hpd); amdgpu_irq_get(adev, &adev->hpd_irq,
> > > amdgpu_connector->hpd.hpd); } @@ -3088,7 +3104,7 @@ static int
> > > dce_v6_0_hpd_irq(struct amdgpu_device *adev, struct amdgpu_irq_src
> > > *source,
> > > struct amdgpu_iv_entry *entry) {
> > > - uint32_t disp_int, mask, tmp;
> > > + uint32_t disp_int, mask;
> > > unsigned hpd;
> > >
> > > if (entry->src_data[0] >= adev->mode_info.num_hpd) { @@
> > > -3101,9 +3117,7 @@ static int dce_v6_0_hpd_irq(struct
> > > amdgpu_device *adev, mask = interrupt_status_offsets[hpd].hpd;
> > >
> > > if (disp_int & mask) {
> > > - tmp = RREG32(mmDC_HPD1_INT_CONTROL +
> > > hpd_offsets[hpd]);
> > > - tmp |=
> > DC_HPD1_INT_CONTROL__DC_HPD1_INT_ACK_MASK;
> > > - WREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd],
> > > tmp);
> > > + dce_v6_0_hpd_int_ack(adev, hpd);
> > > schedule_delayed_work(&adev->hotplug_work, 0);
> > > DRM_DEBUG("IH: HPD%d\n", hpd + 1);
> > > }
> > > diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c index
> > > adeddfb7ff12..8ff2b5adfd95 100644 ---
> > > a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c +++
> > > b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c @@ -264,6 +264,21 @@
> > static
> > > void dce_v8_0_hpd_set_polarity(struct amdgpu_device *adev,
> > > WREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd], tmp); }
> > >
> > > +static void dce_v8_0_hpd_int_ack(struct amdgpu_device *adev,
> > > + int hpd)
> > > +{
> > > + u32 tmp;
> > > +
> > > + if (hpd >= adev->mode_info.num_hpd) {
> > > + DRM_DEBUG("invalid hdp %d\n", hpd);
> > > + return;
> > > + }
> > > +
> > > + tmp = RREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd]);
> > > + tmp |= DC_HPD1_INT_CONTROL__DC_HPD1_INT_ACK_MASK;
> > > + WREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd], tmp); }
> > > +
> > > /**
> > > * dce_v8_0_hpd_init - hpd setup callback.
> > > *
> > > @@ -303,6 +318,7 @@ static void dce_v8_0_hpd_init(struct
> > amdgpu_device
> > > *adev) continue;
> > > }
> > >
> > > + dce_v8_0_hpd_int_ack(adev,
> > > amdgpu_connector->hpd.hpd); dce_v8_0_hpd_set_polarity(adev,
> > > amdgpu_connector->hpd.hpd); amdgpu_irq_get(adev, &adev->hpd_irq,
> > > amdgpu_connector->hpd.hpd); } @@ -3176,7 +3192,7 @@ static int
> > > dce_v8_0_hpd_irq(struct amdgpu_device *adev, struct amdgpu_irq_src
> > > *source,
> > > struct amdgpu_iv_entry *entry) {
> > > - uint32_t disp_int, mask, tmp;
> > > + uint32_t disp_int, mask;
> > > unsigned hpd;
> > >
> > > if (entry->src_data[0] >= adev->mode_info.num_hpd) { @@
> > > -3189,9 +3205,7 @@ static int dce_v8_0_hpd_irq(struct
> > > amdgpu_device *adev, mask = interrupt_status_offsets[hpd].hpd;
> > >
> > > if (disp_int & mask) {
> > > - tmp = RREG32(mmDC_HPD1_INT_CONTROL +
> > > hpd_offsets[hpd]);
> > > - tmp |=
> > DC_HPD1_INT_CONTROL__DC_HPD1_INT_ACK_MASK;
> > > - WREG32(mmDC_HPD1_INT_CONTROL + hpd_offsets[hpd],
> > > tmp);
> > > + dce_v8_0_hpd_int_ack(adev, hpd);
> > > schedule_delayed_work(&adev->hotplug_work, 0);
> > > DRM_DEBUG("IH: HPD%d\n", hpd + 1);
> > > }
>
>
More information about the amd-gfx
mailing list