diff mbox series

[V3] x86/gpu: add JasperLake to gen11 early quirks

Message ID 20210608053411.394166-1-tejaskumarx.surendrakumar.upadhyay@intel.com (mailing list archive)
State New, archived
Headers show
Series [V3] x86/gpu: add JasperLake to gen11 early quirks | expand

Commit Message

Surendrakumar Upadhyay, TejaskumarX June 8, 2021, 5:34 a.m. UTC
Let's reserve JSL stolen memory for graphics.

JasperLake is a gen11 platform which is compatible with
ICL/EHL changes.

Required due to below reference patch:

commit 24ea098b7c0d80b56d62a200608e0b029056baf6
drm/i915/jsl: Split EHL/JSL platform info and PCI ids

V2:
    - Added maintainer list in cc
    - Added patch ref in commit message
V1:
    - Added Cc: x86@kernel.org

Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: x86@kernel.org
Cc: José Roberto de Souza <jose.souza@intel.com>
Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com>
---
 arch/x86/kernel/early-quirks.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Maarten Lankhorst June 8, 2021, 10:31 a.m. UTC | #1
Op 08-06-2021 om 07:34 schreef Tejas Upadhyay:
> Let's reserve JSL stolen memory for graphics.
>
> JasperLake is a gen11 platform which is compatible with
> ICL/EHL changes.
>
> Required due to below reference patch:
>
> commit 24ea098b7c0d80b56d62a200608e0b029056baf6
> drm/i915/jsl: Split EHL/JSL platform info and PCI ids
>
> V2:
>     - Added maintainer list in cc
>     - Added patch ref in commit message
> V1:
>     - Added Cc: x86@kernel.org
>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: Borislav Petkov <bp@alien8.de>
> Cc: "H. Peter Anvin" <hpa@zytor.com>
> Cc: x86@kernel.org
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com>
> ---
>  arch/x86/kernel/early-quirks.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/x86/kernel/early-quirks.c b/arch/x86/kernel/early-quirks.c
> index b553ffe9b985..38837dad46e6 100644
> --- a/arch/x86/kernel/early-quirks.c
> +++ b/arch/x86/kernel/early-quirks.c
> @@ -549,6 +549,7 @@ static const struct pci_device_id intel_early_ids[] __initconst = {
>  	INTEL_CNL_IDS(&gen9_early_ops),
>  	INTEL_ICL_11_IDS(&gen11_early_ops),
>  	INTEL_EHL_IDS(&gen11_early_ops),
> +	INTEL_JSL_IDS(&gen11_early_ops),
>  	INTEL_TGL_12_IDS(&gen11_early_ops),
>  	INTEL_RKL_IDS(&gen11_early_ops),
>  	INTEL_ADLS_IDS(&gen11_early_ops),

I cleaned up the commit message slightly, added cc stable and pushed. :)

Thanks for patch.

~Maarten
Borislav Petkov June 8, 2021, 11:05 a.m. UTC | #2
On Tue, Jun 08, 2021 at 11:04:11AM +0530, Tejas Upadhyay wrote:
> Let's reserve JSL stolen memory for graphics.
> 
> JasperLake is a gen11 platform which is compatible with
> ICL/EHL changes.
> 
> Required due to below reference patch:
> 
> commit 24ea098b7c0d80b56d62a200608e0b029056baf6
> drm/i915/jsl: Split EHL/JSL platform info and PCI ids
> 
> V2:
>     - Added maintainer list in cc
>     - Added patch ref in commit message
> V1:
>     - Added Cc: x86@kernel.org
> 
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: Borislav Petkov <bp@alien8.de>
> Cc: "H. Peter Anvin" <hpa@zytor.com>
> Cc: x86@kernel.org
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com>
> ---
>  arch/x86/kernel/early-quirks.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/x86/kernel/early-quirks.c b/arch/x86/kernel/early-quirks.c
> index b553ffe9b985..38837dad46e6 100644
> --- a/arch/x86/kernel/early-quirks.c
> +++ b/arch/x86/kernel/early-quirks.c
> @@ -549,6 +549,7 @@ static const struct pci_device_id intel_early_ids[] __initconst = {
>  	INTEL_CNL_IDS(&gen9_early_ops),
>  	INTEL_ICL_11_IDS(&gen11_early_ops),
>  	INTEL_EHL_IDS(&gen11_early_ops),
> +	INTEL_JSL_IDS(&gen11_early_ops),
>  	INTEL_TGL_12_IDS(&gen11_early_ops),
>  	INTEL_RKL_IDS(&gen11_early_ops),
>  	INTEL_ADLS_IDS(&gen11_early_ops),
> -- 

I see it has been applied already. Regardless:

Acked-by: Borislav Petkov <bp@suse.de>
Surendrakumar Upadhyay, TejaskumarX June 8, 2021, 11:08 a.m. UTC | #3
> -----Original Message-----
> From: Borislav Petkov <bp@alien8.de>
> Sent: 08 June 2021 16:35
> To: Surendrakumar Upadhyay, TejaskumarX
> <tejaskumarx.surendrakumar.upadhyay@intel.com>
> Cc: intel-gfx@lists.freedesktop.org; tglx@linutronix.de; mingo@redhat.com;
> hpa@zytor.com; x86@kernel.org; Souza, Jose <jose.souza@intel.com>
> Subject: Re: [PATCH V3] x86/gpu: add JasperLake to gen11 early quirks
> 
> On Tue, Jun 08, 2021 at 11:04:11AM +0530, Tejas Upadhyay wrote:
> > Let's reserve JSL stolen memory for graphics.
> >
> > JasperLake is a gen11 platform which is compatible with ICL/EHL
> > changes.
> >
> > Required due to below reference patch:
> >
> > commit 24ea098b7c0d80b56d62a200608e0b029056baf6
> > drm/i915/jsl: Split EHL/JSL platform info and PCI ids
> >
> > V2:
> >     - Added maintainer list in cc
> >     - Added patch ref in commit message
> > V1:
> >     - Added Cc: x86@kernel.org
> >
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Ingo Molnar <mingo@redhat.com>
> > Cc: Borislav Petkov <bp@alien8.de>
> > Cc: "H. Peter Anvin" <hpa@zytor.com>
> > Cc: x86@kernel.org
> > Cc: José Roberto de Souza <jose.souza@intel.com>
> > Signed-off-by: Tejas Upadhyay
> > <tejaskumarx.surendrakumar.upadhyay@intel.com>
> > ---
> >  arch/x86/kernel/early-quirks.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/x86/kernel/early-quirks.c
> > b/arch/x86/kernel/early-quirks.c index b553ffe9b985..38837dad46e6
> > 100644
> > --- a/arch/x86/kernel/early-quirks.c
> > +++ b/arch/x86/kernel/early-quirks.c
> > @@ -549,6 +549,7 @@ static const struct pci_device_id intel_early_ids[]
> __initconst = {
> >  	INTEL_CNL_IDS(&gen9_early_ops),
> >  	INTEL_ICL_11_IDS(&gen11_early_ops),
> >  	INTEL_EHL_IDS(&gen11_early_ops),
> > +	INTEL_JSL_IDS(&gen11_early_ops),
> >  	INTEL_TGL_12_IDS(&gen11_early_ops),
> >  	INTEL_RKL_IDS(&gen11_early_ops),
> >  	INTEL_ADLS_IDS(&gen11_early_ops),
> > --
> 
> I see it has been applied already. Regardless:
> 
> Acked-by: Borislav Petkov <bp@suse.de>

Yes, thanks for Ack though.

Thanks,
Tejas

> 
> --
> Regards/Gruss,
>     Boris.
> 
> https://people.kernel.org/tglx/notes-about-netiquette
diff mbox series

Patch

diff --git a/arch/x86/kernel/early-quirks.c b/arch/x86/kernel/early-quirks.c
index b553ffe9b985..38837dad46e6 100644
--- a/arch/x86/kernel/early-quirks.c
+++ b/arch/x86/kernel/early-quirks.c
@@ -549,6 +549,7 @@  static const struct pci_device_id intel_early_ids[] __initconst = {
 	INTEL_CNL_IDS(&gen9_early_ops),
 	INTEL_ICL_11_IDS(&gen11_early_ops),
 	INTEL_EHL_IDS(&gen11_early_ops),
+	INTEL_JSL_IDS(&gen11_early_ops),
 	INTEL_TGL_12_IDS(&gen11_early_ops),
 	INTEL_RKL_IDS(&gen11_early_ops),
 	INTEL_ADLS_IDS(&gen11_early_ops),