diff mbox series

[v2,25/25] drm/xe/bmg: Enable the display support

Message ID 20240403112253.1432390-26-balasubramani.vivekanandan@intel.com (mailing list archive)
State New, archived
Headers show
Series Enable dislay support for Battlemage | expand

Commit Message

Vivekanandan, Balasubramani April 3, 2024, 11:22 a.m. UTC
Enable the display support for Battlemage

Signed-off-by: Balasubramani Vivekanandan <balasubramani.vivekanandan@intel.com>
---
 drivers/gpu/drm/xe/xe_pci.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Chauhan, Shekhar April 8, 2024, 3:16 a.m. UTC | #1
On 4/3/2024 16:52, Balasubramani Vivekanandan wrote:
> Enable the display support for Battlemage
>
> Signed-off-by: Balasubramani Vivekanandan <balasubramani.vivekanandan@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_pci.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index b3158053baee..835c18ec8fb9 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -340,6 +340,7 @@ static const struct xe_device_desc lnl_desc = {
>   static const struct xe_device_desc bmg_desc = {
>   	DGFX_FEATURES,
>   	PLATFORM(XE_BATTLEMAGE),
> +	.has_display = true,

Can it be squashed with the other commit 'drm/xe/bmg: Add BMG platform 
definition'?

If not, then, LGTM, and with that,

Reviewed-by: Shekhar Chauhan <shekhar.chauhan@intel.com>

>   	.require_force_probe = true,
>   };
>
Vivekanandan, Balasubramani April 8, 2024, 6:23 a.m. UTC | #2
On 08.04.2024 08:46, Chauhan, Shekhar wrote:
> 
> On 4/3/2024 16:52, Balasubramani Vivekanandan wrote:
> > Enable the display support for Battlemage
> > 
> > Signed-off-by: Balasubramani Vivekanandan <balasubramani.vivekanandan@intel.com>
> > ---
> >   drivers/gpu/drm/xe/xe_pci.c | 1 +
> >   1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> > index b3158053baee..835c18ec8fb9 100644
> > --- a/drivers/gpu/drm/xe/xe_pci.c
> > +++ b/drivers/gpu/drm/xe/xe_pci.c
> > @@ -340,6 +340,7 @@ static const struct xe_device_desc lnl_desc = {
> >   static const struct xe_device_desc bmg_desc = {
> >   	DGFX_FEATURES,
> >   	PLATFORM(XE_BATTLEMAGE),
> > +	.has_display = true,
> 
> Can it be squashed with the other commit 'drm/xe/bmg: Add BMG platform
> definition'?
The other commit, I have duplicated in this series to help build this
patch series. That commit is actually part of my earlier patch series
"Add Battlemage support" which enables the basic Battlemage support.

Regards,
Bala

> 
> If not, then, LGTM, and with that,
> 
> Reviewed-by: Shekhar Chauhan <shekhar.chauhan@intel.com>
> 
> >   	.require_force_probe = true,
> >   };
> 
> -- 
> -shekhar
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index b3158053baee..835c18ec8fb9 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -340,6 +340,7 @@  static const struct xe_device_desc lnl_desc = {
 static const struct xe_device_desc bmg_desc = {
 	DGFX_FEATURES,
 	PLATFORM(XE_BATTLEMAGE),
+	.has_display = true,
 	.require_force_probe = true,
 };