From: Imre Deak <imre.deak@intel.com>
To: "Kandpal, Suraj" <suraj.kandpal@intel.com>
Cc: "intel-xe@lists.freedesktop.org" <intel-xe@lists.freedesktop.org>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
"De Marchi, Lucas" <lucas.demarchi@intel.com>
Subject: Re: [PATCH] drm/xe/dp: Fix non-display builds with DP tunnelling incorrectly enabled
Date: Tue, 21 Jan 2025 16:57:44 +0200 [thread overview]
Message-ID: <Z4-16NeQg7m4YSd2@ideak-desk.fi.intel.com> (raw)
In-Reply-To: <SN7PR11MB6750D685E73BDAFC75FE733EE3E62@SN7PR11MB6750.namprd11.prod.outlook.com>
On Tue, Jan 21, 2025 at 06:37:54AM +0200, Kandpal, Suraj wrote:
>
>
> > -----Original Message-----
> > From: Deak, Imre <imre.deak@intel.com>
> > Sent: Friday, January 17, 2025 9:09 PM
> > To: intel-xe@lists.freedesktop.org; intel-gfx@lists.freedesktop.org
> > Cc: Kandpal, Suraj <suraj.kandpal@intel.com>; De Marchi, Lucas
> > <lucas.demarchi@intel.com>
> > Subject: [PATCH] drm/xe/dp: Fix non-display builds with DP tunnelling
> > incorrectly enabled
> >
> > Code for the DP tunnelling functionality in the xe driver can be built only if
> > the display code is also built, adjust the kconfig dependency accordingly.
> >
> > Cc: Suraj Kandpal <suraj.kandpal@intel.com>
> > Fixes: 73900dce57e4 ("drm/xe/dp: Enable DP tunneling")
> > Reported-by: Lucas De Marchi <lucas.demarchi@intel.com>
> > Signed-off-by: Imre Deak <imre.deak@intel.com>
>
> LGTM,
> Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Thanks, patch is pushed to drm-intel-next.
> > ---
> > drivers/gpu/drm/xe/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/xe/Kconfig b/drivers/gpu/drm/xe/Kconfig index
> > 50cf80df51900..99219c16e8aac 100644
> > --- a/drivers/gpu/drm/xe/Kconfig
> > +++ b/drivers/gpu/drm/xe/Kconfig
> > @@ -61,7 +61,7 @@ config DRM_XE_DISPLAY
> >
> > config DRM_XE_DP_TUNNEL
> > bool "Enable DP tunnel support"
> > - depends on DRM_XE
> > + depends on DRM_XE_DISPLAY
> > depends on USB4
> > select DRM_DISPLAY_DP_TUNNEL
> > default y
> > --
> > 2.44.2
>
prev parent reply other threads:[~2025-01-21 14:56 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-17 15:38 [PATCH] drm/xe/dp: Fix non-display builds with DP tunnelling incorrectly enabled Imre Deak
2025-01-17 16:23 ` ✓ CI.Patch_applied: success for " Patchwork
2025-01-17 16:23 ` ✗ CI.checkpatch: warning " Patchwork
2025-01-17 16:24 ` ✓ CI.KUnit: success " Patchwork
2025-01-17 16:43 ` ✓ CI.Build: " Patchwork
2025-01-17 16:45 ` ✓ CI.Hooks: " Patchwork
2025-01-17 16:46 ` ✓ CI.checksparse: " Patchwork
2025-01-17 17:14 ` ✓ Xe.CI.BAT: " Patchwork
2025-01-18 2:01 ` ✗ Xe.CI.Full: failure " Patchwork
2025-01-21 4:37 ` [PATCH] " Kandpal, Suraj
2025-01-21 14:57 ` Imre Deak [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Z4-16NeQg7m4YSd2@ideak-desk.fi.intel.com \
--to=imre.deak@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
--cc=lucas.demarchi@intel.com \
--cc=suraj.kandpal@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox