diff mbox series

[v5,11/11] drm/xe/ptl: Enable PTL display

Message ID 20241025204743.211510-12-clinton.a.taylor@intel.com (mailing list archive)
State New, archived
Headers show
Series drm/i915/xe3lpd: ptl display patches | expand

Commit Message

Clint Taylor Oct. 25, 2024, 8:47 p.m. UTC
From: Haridhar Kalvala <haridhar.kalvala@intel.com>

Signed-off-by: Haridhar Kalvala <haridhar.kalvala@intel.com>
Signed-off-by: Clint Taylor <Clinton.A.Taylor@intel.com>
---
 drivers/gpu/drm/xe/xe_pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Saarinen, Jani Oct. 28, 2024, 6:45 p.m. UTC | #1
Hi, 

> -----Original Message-----
> From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Clint
> Taylor
> Sent: Friday, 25 October 2024 23.48
> To: intel-gfx@lists.freedesktop.org; intel-xe@lists.freedesktop.org
> Subject: [PATCH v5 11/11] drm/xe/ptl: Enable PTL display
> 
> From: Haridhar Kalvala <haridhar.kalvala@intel.com>
> 
> Signed-off-by: Haridhar Kalvala <haridhar.kalvala@intel.com>
> Signed-off-by: Clint Taylor <Clinton.A.Taylor@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c index
> 4085bb3b6550..6f73a243c24c 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -352,7 +352,7 @@ static const struct xe_device_desc bmg_desc = {

Acked-by: Jani Saarinen <jani.saarinen@intel.com>
Tested-by: Jani Saarinen<jani.saarinen@intel.com>

> 
>  static const struct xe_device_desc ptl_desc = {
>  	PLATFORM(PANTHERLAKE),
> -	.has_display = false,
> +	.has_display = true,
>  	.require_force_probe = true,
>  };
> 
> --
> 2.25.1
Br,
Jani
Rodrigo Vivi Oct. 28, 2024, 7:12 p.m. UTC | #2
On Fri, Oct 25, 2024 at 01:47:43PM -0700, Clint Taylor wrote:
> From: Haridhar Kalvala <haridhar.kalvala@intel.com>

Every patch needs a commit message. No exception.

> 
> Signed-off-by: Haridhar Kalvala <haridhar.kalvala@intel.com>
> Signed-off-by: Clint Taylor <Clinton.A.Taylor@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 4085bb3b6550..6f73a243c24c 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -352,7 +352,7 @@ static const struct xe_device_desc bmg_desc = {
>  
>  static const struct xe_device_desc ptl_desc = {
>  	PLATFORM(PANTHERLAKE),
> -	.has_display = false,
> +	.has_display = true,
>  	.require_force_probe = true,
>  };
>  
> -- 
> 2.25.1
>
Matt Roper Oct. 28, 2024, 7:17 p.m. UTC | #3
With an added commit message (something simple like "At this point we
should have enough support landed to turn on and start testing basic
display functionality" would be fine),

Reviewed-by: Matt Roper <matthew.d.roper@intel.com>


Matt

On Fri, Oct 25, 2024 at 01:47:43PM -0700, Clint Taylor wrote:
> From: Haridhar Kalvala <haridhar.kalvala@intel.com>
> 
> Signed-off-by: Haridhar Kalvala <haridhar.kalvala@intel.com>
> Signed-off-by: Clint Taylor <Clinton.A.Taylor@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 4085bb3b6550..6f73a243c24c 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -352,7 +352,7 @@ static const struct xe_device_desc bmg_desc = {
>  
>  static const struct xe_device_desc ptl_desc = {
>  	PLATFORM(PANTHERLAKE),
> -	.has_display = false,
> +	.has_display = true,
>  	.require_force_probe = true,
>  };
>  
> -- 
> 2.25.1
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 4085bb3b6550..6f73a243c24c 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -352,7 +352,7 @@  static const struct xe_device_desc bmg_desc = {
 
 static const struct xe_device_desc ptl_desc = {
 	PLATFORM(PANTHERLAKE),
-	.has_display = false,
+	.has_display = true,
 	.require_force_probe = true,
 };