diff mbox series

nightly.conf: Add the xe repo to drm-tip

Message ID 20231222113640.14038-1-thomas.hellstrom@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series nightly.conf: Add the xe repo to drm-tip | expand

Commit Message

Thomas Hellström Dec. 22, 2023, 11:36 a.m. UTC
Add the xe repo to drm-tip and the dim tools.
For now use the sha1 of the first drm-xe-next pull request for drm-tip,
since that branch tip is currently adapted for our CI testing.

Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Cc: Oded Gabbay <ogabbay@kernel.org>
Cc: daniel.vetter@ffwll.ch
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: dim-tools@lists.freedesktop.org
Cc: dri-devel@lists.freedesktop.org
Cc: intel-gfx@lists.freedesktop.org
Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
---
 nightly.conf | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Jani Nikula Dec. 22, 2023, 12:01 p.m. UTC | #1
On Fri, 22 Dec 2023, Thomas Hellström <thomas.hellstrom@linux.intel.com> wrote:
> Add the xe repo to drm-tip and the dim tools.
> For now use the sha1 of the first drm-xe-next pull request for drm-tip,
> since that branch tip is currently adapted for our CI testing.

I guess we'll need xe CI to switch to drm-tip based testing, and then
force push drm-xe-next to drop that core-for-CI thing. But I think it's
fine.

However, please first push v6.7-rc6 to drm-xe-fixes and current drm-next
to drm-xe-next-fixes as a starting point. AFAICT the branches don't
exist yet.

Other than that, LGTM.

BR,
Jani.

>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Cc: Oded Gabbay <ogabbay@kernel.org>
> Cc: daniel.vetter@ffwll.ch
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: dim-tools@lists.freedesktop.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: intel-gfx@lists.freedesktop.org
> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> ---
>  nightly.conf | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/nightly.conf b/nightly.conf
> index 24126b61b797..accd3ff2cc39 100644
> --- a/nightly.conf
> +++ b/nightly.conf
> @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
>  https://anongit.freedesktop.org/git/drm/drm-tip
>  https://anongit.freedesktop.org/git/drm/drm-tip.git
>  "
> +drm_tip_repos[drm-xe]="
> +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
> +https://gitlab.freedesktop.org/drm/xe/kernel.git
> +"
>  drm_tip_repos[drm-intel]="
>  ssh://git.freedesktop.org/git/drm/drm-intel
>  ssh://git.freedesktop.org/git/drm-intel
> @@ -65,14 +69,17 @@ drm_tip_config=(
>  	"drm			drm-fixes"
>  	"drm-misc		drm-misc-fixes"
>  	"drm-intel		drm-intel-fixes"
> +	"drm-xe			drm-xe-fixes"
>  
>  	"drm			drm-next"
>  	"drm-misc		drm-misc-next-fixes"
>  	"drm-intel		drm-intel-next-fixes"
> +	"drm-xe			drm-xe-next-fixes"
>  
>  	"drm-misc		drm-misc-next"
>  	"drm-intel		drm-intel-next"
>  	"drm-intel		drm-intel-gt-next"
> +	"drm-xe			drm-xe-next b6e1b7081768"
>  
>  	"drm-intel		topic/core-for-CI"
>  	"drm-misc		topic/i915-ttm"
Thomas Hellström Dec. 22, 2023, 12:25 p.m. UTC | #2
Hi,

On 12/22/23 13:01, Jani Nikula wrote:
> On Fri, 22 Dec 2023, Thomas Hellström <thomas.hellstrom@linux.intel.com> wrote:
>> Add the xe repo to drm-tip and the dim tools.
>> For now use the sha1 of the first drm-xe-next pull request for drm-tip,
>> since that branch tip is currently adapted for our CI testing.
> I guess we'll need xe CI to switch to drm-tip based testing, and then
> force push drm-xe-next to drop that core-for-CI thing. But I think it's
> fine.

Yes, that's in the plan.

>
> However, please first push v6.7-rc6 to drm-xe-fixes and current drm-next
> to drm-xe-next-fixes as a starting point. AFAICT the branches don't
> exist yet.

Yes, I have that queued-up. Still waiting for the right people to wake 
up, though, since I'm ATM lacking the right to create those branches in 
the xe repo. I did a dry-run building drm-tip with these patches present 
in a clone of the xe repo for testing purposes...

>
> Other than that, LGTM.

Thanks,

Thomas


>
> BR,
> Jani.
>
>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>> Cc: Oded Gabbay <ogabbay@kernel.org>
>> Cc: daniel.vetter@ffwll.ch
>> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>> Cc: dim-tools@lists.freedesktop.org
>> Cc: dri-devel@lists.freedesktop.org
>> Cc: intel-gfx@lists.freedesktop.org
>> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>> ---
>>   nightly.conf | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/nightly.conf b/nightly.conf
>> index 24126b61b797..accd3ff2cc39 100644
>> --- a/nightly.conf
>> +++ b/nightly.conf
>> @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
>>   https://anongit.freedesktop.org/git/drm/drm-tip
>>   https://anongit.freedesktop.org/git/drm/drm-tip.git
>>   "
>> +drm_tip_repos[drm-xe]="
>> +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
>> +https://gitlab.freedesktop.org/drm/xe/kernel.git
>> +"
>>   drm_tip_repos[drm-intel]="
>>   ssh://git.freedesktop.org/git/drm/drm-intel
>>   ssh://git.freedesktop.org/git/drm-intel
>> @@ -65,14 +69,17 @@ drm_tip_config=(
>>   	"drm			drm-fixes"
>>   	"drm-misc		drm-misc-fixes"
>>   	"drm-intel		drm-intel-fixes"
>> +	"drm-xe			drm-xe-fixes"
>>   
>>   	"drm			drm-next"
>>   	"drm-misc		drm-misc-next-fixes"
>>   	"drm-intel		drm-intel-next-fixes"
>> +	"drm-xe			drm-xe-next-fixes"
>>   
>>   	"drm-misc		drm-misc-next"
>>   	"drm-intel		drm-intel-next"
>>   	"drm-intel		drm-intel-gt-next"
>> +	"drm-xe			drm-xe-next b6e1b7081768"
>>   
>>   	"drm-intel		topic/core-for-CI"
>>   	"drm-misc		topic/i915-ttm"
Thomas Hellström Dec. 22, 2023, 12:29 p.m. UTC | #3
On 12/22/23 13:25, Thomas Hellström wrote:
> Hi,
>
> On 12/22/23 13:01, Jani Nikula wrote:
>> On Fri, 22 Dec 2023, Thomas Hellström 
>> <thomas.hellstrom@linux.intel.com> wrote:
>>> Add the xe repo to drm-tip and the dim tools.
>>> For now use the sha1 of the first drm-xe-next pull request for drm-tip,
>>> since that branch tip is currently adapted for our CI testing.
>> I guess we'll need xe CI to switch to drm-tip based testing, and then
>> force push drm-xe-next to drop that core-for-CI thing. But I think it's
>> fine.
>
> Yes, that's in the plan.
>
>>
>> However, please first push v6.7-rc6 to drm-xe-fixes and current drm-next
>> to drm-xe-next-fixes as a starting point. AFAICT the branches don't
>> exist yet.
>
> Yes, I have that queued-up. Still waiting for the right people to wake 
> up, though, since I'm ATM lacking the right to create those branches 
> in the xe repo. I did a dry-run building drm-tip with these patches
s/patches/branches/ ...
> present in a clone of the xe repo for testing purposes...
>
>>
>> Other than that, LGTM.
>
> Thanks,
>
> Thomas
>
>
>>
>> BR,
>> Jani.
>>
>>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>>> Cc: Oded Gabbay <ogabbay@kernel.org>
>>> Cc: daniel.vetter@ffwll.ch
>>> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>>> Cc: dim-tools@lists.freedesktop.org
>>> Cc: dri-devel@lists.freedesktop.org
>>> Cc: intel-gfx@lists.freedesktop.org
>>> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>>> ---
>>>   nightly.conf | 7 +++++++
>>>   1 file changed, 7 insertions(+)
>>>
>>> diff --git a/nightly.conf b/nightly.conf
>>> index 24126b61b797..accd3ff2cc39 100644
>>> --- a/nightly.conf
>>> +++ b/nightly.conf
>>> @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
>>>   https://anongit.freedesktop.org/git/drm/drm-tip
>>>   https://anongit.freedesktop.org/git/drm/drm-tip.git
>>>   "
>>> +drm_tip_repos[drm-xe]="
>>> +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
>>> +https://gitlab.freedesktop.org/drm/xe/kernel.git
>>> +"
>>>   drm_tip_repos[drm-intel]="
>>>   ssh://git.freedesktop.org/git/drm/drm-intel
>>>   ssh://git.freedesktop.org/git/drm-intel
>>> @@ -65,14 +69,17 @@ drm_tip_config=(
>>>       "drm            drm-fixes"
>>>       "drm-misc        drm-misc-fixes"
>>>       "drm-intel        drm-intel-fixes"
>>> +    "drm-xe            drm-xe-fixes"
>>>         "drm            drm-next"
>>>       "drm-misc        drm-misc-next-fixes"
>>>       "drm-intel        drm-intel-next-fixes"
>>> +    "drm-xe            drm-xe-next-fixes"
>>>         "drm-misc        drm-misc-next"
>>>       "drm-intel        drm-intel-next"
>>>       "drm-intel        drm-intel-gt-next"
>>> +    "drm-xe            drm-xe-next b6e1b7081768"
>>>         "drm-intel        topic/core-for-CI"
>>>       "drm-misc        topic/i915-ttm"
Lucas De Marchi Dec. 22, 2023, 4:28 p.m. UTC | #4
On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
>Add the xe repo to drm-tip and the dim tools.
>For now use the sha1 of the first drm-xe-next pull request for drm-tip,
>since that branch tip is currently adapted for our CI testing.
>
>Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>Cc: Oded Gabbay <ogabbay@kernel.org>
>Cc: daniel.vetter@ffwll.ch
>Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>Cc: dim-tools@lists.freedesktop.org
>Cc: dri-devel@lists.freedesktop.org
>Cc: intel-gfx@lists.freedesktop.org
>Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>


Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>

Lucas De Marchi

>---
> nightly.conf | 7 +++++++
> 1 file changed, 7 insertions(+)
>
>diff --git a/nightly.conf b/nightly.conf
>index 24126b61b797..accd3ff2cc39 100644
>--- a/nightly.conf
>+++ b/nightly.conf
>@@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
> https://anongit.freedesktop.org/git/drm/drm-tip
> https://anongit.freedesktop.org/git/drm/drm-tip.git
> "
>+drm_tip_repos[drm-xe]="
>+ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
>+https://gitlab.freedesktop.org/drm/xe/kernel.git
>+"
> drm_tip_repos[drm-intel]="
> ssh://git.freedesktop.org/git/drm/drm-intel
> ssh://git.freedesktop.org/git/drm-intel
>@@ -65,14 +69,17 @@ drm_tip_config=(
> 	"drm			drm-fixes"
> 	"drm-misc		drm-misc-fixes"
> 	"drm-intel		drm-intel-fixes"
>+	"drm-xe			drm-xe-fixes"
>
> 	"drm			drm-next"
> 	"drm-misc		drm-misc-next-fixes"
> 	"drm-intel		drm-intel-next-fixes"
>+	"drm-xe			drm-xe-next-fixes"
>
> 	"drm-misc		drm-misc-next"
> 	"drm-intel		drm-intel-next"
> 	"drm-intel		drm-intel-gt-next"
>+	"drm-xe			drm-xe-next b6e1b7081768"
>
> 	"drm-intel		topic/core-for-CI"
> 	"drm-misc		topic/i915-ttm"
>-- 
>2.42.0
>
Rodrigo Vivi Dec. 26, 2023, 6:30 p.m. UTC | #5
On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
> Add the xe repo to drm-tip and the dim tools.
> For now use the sha1 of the first drm-xe-next pull request for drm-tip,
> since that branch tip is currently adapted for our CI testing.
> 
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Cc: Oded Gabbay <ogabbay@kernel.org>
> Cc: daniel.vetter@ffwll.ch
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: dim-tools@lists.freedesktop.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: intel-gfx@lists.freedesktop.org
> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> ---
>  nightly.conf | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/nightly.conf b/nightly.conf
> index 24126b61b797..accd3ff2cc39 100644
> --- a/nightly.conf
> +++ b/nightly.conf
> @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
>  https://anongit.freedesktop.org/git/drm/drm-tip
>  https://anongit.freedesktop.org/git/drm/drm-tip.git
>  "
> +drm_tip_repos[drm-xe]="
> +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
> +https://gitlab.freedesktop.org/drm/xe/kernel.git
> +"
>  drm_tip_repos[drm-intel]="
>  ssh://git.freedesktop.org/git/drm/drm-intel
>  ssh://git.freedesktop.org/git/drm-intel
> @@ -65,14 +69,17 @@ drm_tip_config=(
>  	"drm			drm-fixes"
>  	"drm-misc		drm-misc-fixes"
>  	"drm-intel		drm-intel-fixes"
> +	"drm-xe			drm-xe-fixes"
>  
>  	"drm			drm-next"
>  	"drm-misc		drm-misc-next-fixes"
>  	"drm-intel		drm-intel-next-fixes"
> +	"drm-xe			drm-xe-next-fixes"
>  
>  	"drm-misc		drm-misc-next"
>  	"drm-intel		drm-intel-next"
>  	"drm-intel		drm-intel-gt-next"
> +	"drm-xe			drm-xe-next b6e1b7081768"

yeap, up to this commit nothing else should change, but
then we will need an extra rebase of the rest on top of drm/drm-next.

But then we need to decide where these following patches will live:
880277f31cc69 drm/xe/guc: define LNL FW
2cfc5ae1b8267 drm/xe/guc: define PVC FW
52383b58eb8cf mei/hdcp: Also enable for XE
bea27d7369855 mei: gsc: add support for auxiliary device created by Xe driver
fcb3410197f05 fault-inject: Include linux/types.h by default.
8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs


Will it be the topic/core-for-CI?
or topic/xe-extras?
or what?

Anyway, for the inclusion like this, after our CI is ready:

Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

>  
>  	"drm-intel		topic/core-for-CI"
>  	"drm-misc		topic/i915-ttm"
> -- 
> 2.42.0
>
Thomas Hellström Jan. 3, 2024, 1:50 p.m. UTC | #6
On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote:
> On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
> > Add the xe repo to drm-tip and the dim tools.
> > For now use the sha1 of the first drm-xe-next pull request for drm-
> > tip,
> > since that branch tip is currently adapted for our CI testing.
> > 
> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> > Cc: Oded Gabbay <ogabbay@kernel.org>
> > Cc: daniel.vetter@ffwll.ch
> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > Cc: dim-tools@lists.freedesktop.org
> > Cc: dri-devel@lists.freedesktop.org
> > Cc: intel-gfx@lists.freedesktop.org
> > Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > ---
> >  nightly.conf | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/nightly.conf b/nightly.conf
> > index 24126b61b797..accd3ff2cc39 100644
> > --- a/nightly.conf
> > +++ b/nightly.conf
> > @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
> >  https://anongit.freedesktop.org/git/drm/drm-tip
> >  https://anongit.freedesktop.org/git/drm/drm-tip.git
> >  "
> > +drm_tip_repos[drm-xe]="
> > +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
> > +https://gitlab.freedesktop.org/drm/xe/kernel.git
> > +"
> >  drm_tip_repos[drm-intel]="
> >  ssh://git.freedesktop.org/git/drm/drm-intel
> >  ssh://git.freedesktop.org/git/drm-intel
> > @@ -65,14 +69,17 @@ drm_tip_config=(
> >  	"drm			drm-fixes"
> >  	"drm-misc		drm-misc-fixes"
> >  	"drm-intel		drm-intel-fixes"
> > +	"drm-xe			drm-xe-fixes"
> >  
> >  	"drm			drm-next"
> >  	"drm-misc		drm-misc-next-fixes"
> >  	"drm-intel		drm-intel-next-fixes"
> > +	"drm-xe			drm-xe-next-fixes"
> >  
> >  	"drm-misc		drm-misc-next"
> >  	"drm-intel		drm-intel-next"
> >  	"drm-intel		drm-intel-gt-next"
> > +	"drm-xe			drm-xe-next b6e1b7081768"
> 
> yeap, up to this commit nothing else should change, but
> then we will need an extra rebase of the rest on top of drm/drm-next.
> 
> But then we need to decide where these following patches will live:
> 880277f31cc69 drm/xe/guc: define LNL FW
> 2cfc5ae1b8267 drm/xe/guc: define PVC FW
> 52383b58eb8cf mei/hdcp: Also enable for XE
> bea27d7369855 mei: gsc: add support for auxiliary device created by
> Xe driver
> fcb3410197f05 fault-inject: Include linux/types.h by default.
> 8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs
> 
> 
> Will it be the topic/core-for-CI?
> or topic/xe-extras?
> or what?

This sounds to me like topic/core-for-CI? Or is there any drawback with
that?

> 
> Anyway, for the inclusion like this, after our CI is ready:

Could we merge this patch already at this point, considering it will,
at least for now, only update drm-tip with our fixes?

Thanks,

/Thomas


> 
> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> 
> >  
> >  	"drm-intel		topic/core-for-CI"
> >  	"drm-misc		topic/i915-ttm"
> > -- 
> > 2.42.0
> >
Lucas De Marchi Jan. 4, 2024, 5:59 a.m. UTC | #7
On Wed, Jan 03, 2024 at 02:50:57PM +0100, Thomas Hellström wrote:
>On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote:
>> On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
>> > Add the xe repo to drm-tip and the dim tools.
>> > For now use the sha1 of the first drm-xe-next pull request for drm-
>> > tip,
>> > since that branch tip is currently adapted for our CI testing.
>> >
>> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>> > Cc: Oded Gabbay <ogabbay@kernel.org>
>> > Cc: daniel.vetter@ffwll.ch
>> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>> > Cc: dim-tools@lists.freedesktop.org
>> > Cc: dri-devel@lists.freedesktop.org
>> > Cc: intel-gfx@lists.freedesktop.org
>> > Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>> > ---
>> >  nightly.conf | 7 +++++++
>> >  1 file changed, 7 insertions(+)
>> >
>> > diff --git a/nightly.conf b/nightly.conf
>> > index 24126b61b797..accd3ff2cc39 100644
>> > --- a/nightly.conf
>> > +++ b/nightly.conf
>> > @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
>> >  https://anongit.freedesktop.org/git/drm/drm-tip
>> >  https://anongit.freedesktop.org/git/drm/drm-tip.git
>> >  "
>> > +drm_tip_repos[drm-xe]="
>> > +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
>> > +https://gitlab.freedesktop.org/drm/xe/kernel.git
>> > +"
>> >  drm_tip_repos[drm-intel]="
>> >  ssh://git.freedesktop.org/git/drm/drm-intel
>> >  ssh://git.freedesktop.org/git/drm-intel
>> > @@ -65,14 +69,17 @@ drm_tip_config=(
>> >  	"drm			drm-fixes"
>> >  	"drm-misc		drm-misc-fixes"
>> >  	"drm-intel		drm-intel-fixes"
>> > +	"drm-xe			drm-xe-fixes"
>> >  
>> >  	"drm			drm-next"
>> >  	"drm-misc		drm-misc-next-fixes"
>> >  	"drm-intel		drm-intel-next-fixes"
>> > +	"drm-xe			drm-xe-next-fixes"
>> >  
>> >  	"drm-misc		drm-misc-next"
>> >  	"drm-intel		drm-intel-next"
>> >  	"drm-intel		drm-intel-gt-next"
>> > +	"drm-xe			drm-xe-next b6e1b7081768"
>>
>> yeap, up to this commit nothing else should change, but
>> then we will need an extra rebase of the rest on top of drm/drm-next.
>>
>> But then we need to decide where these following patches will live:
>> 880277f31cc69 drm/xe/guc: define LNL FW
>> 2cfc5ae1b8267 drm/xe/guc: define PVC FW
>> 52383b58eb8cf mei/hdcp: Also enable for XE
>> bea27d7369855 mei: gsc: add support for auxiliary device created by
>> Xe driver
>> fcb3410197f05 fault-inject: Include linux/types.h by default.
>> 8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs
>>
>>
>> Will it be the topic/core-for-CI?
>> or topic/xe-extras?
>> or what?
>
>This sounds to me like topic/core-for-CI? Or is there any drawback with
>that?

I think some of them are not really a "for CI". It's more like the
workflow we are adopting e.g. with guc/huc, not sending it to linux-firmware
until we are confident on what version we will start officially
supporting.

This one can't go to topic/core-for-CI neither:
	fcb3410197f05 fault-inject: Include linux/types.h by default.

what it would do would be that we would not see the build error anymore,
but everyone else would (and it's not a CI-only configuration).
Unless it's merged to another branch, we shouldn't merge it.

"52383b58eb8cf mei/hdcp: Also enable for XE" could be material for
topic/core-for-CI and  "8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs"
could either be on that branch or another xe-specific one.

>
>>
>> Anyway, for the inclusion like this, after our CI is ready:
>
>Could we merge this patch already at this point, considering it will,
>at least for now, only update drm-tip with our fixes?

ack

Lucas De Marchi

>
>Thanks,
>
>/Thomas
>
>
>>
>> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>
>> >  
>> >  	"drm-intel		topic/core-for-CI"
>> >  	"drm-misc		topic/i915-ttm"
>> > --
>> > 2.42.0
>> >
>
Rodrigo Vivi Jan. 8, 2024, 10:13 p.m. UTC | #8
On Wed, Jan 03, 2024 at 11:59:16PM -0600, Lucas De Marchi wrote:
> On Wed, Jan 03, 2024 at 02:50:57PM +0100, Thomas Hellström wrote:
> > On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote:
> > > On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
> > > > Add the xe repo to drm-tip and the dim tools.
> > > > For now use the sha1 of the first drm-xe-next pull request for drm-
> > > > tip,
> > > > since that branch tip is currently adapted for our CI testing.
> > > >
> > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> > > > Cc: Oded Gabbay <ogabbay@kernel.org>
> > > > Cc: daniel.vetter@ffwll.ch
> > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > > Cc: dim-tools@lists.freedesktop.org
> > > > Cc: dri-devel@lists.freedesktop.org
> > > > Cc: intel-gfx@lists.freedesktop.org
> > > > Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > > > ---
> > > >  nightly.conf | 7 +++++++
> > > >  1 file changed, 7 insertions(+)
> > > >
> > > > diff --git a/nightly.conf b/nightly.conf
> > > > index 24126b61b797..accd3ff2cc39 100644
> > > > --- a/nightly.conf
> > > > +++ b/nightly.conf
> > > > @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
> > > >  https://anongit.freedesktop.org/git/drm/drm-tip
> > > >  https://anongit.freedesktop.org/git/drm/drm-tip.git
> > > >  "
> > > > +drm_tip_repos[drm-xe]="
> > > > +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
> > > > +https://gitlab.freedesktop.org/drm/xe/kernel.git
> > > > +"
> > > >  drm_tip_repos[drm-intel]="
> > > >  ssh://git.freedesktop.org/git/drm/drm-intel
> > > >  ssh://git.freedesktop.org/git/drm-intel
> > > > @@ -65,14 +69,17 @@ drm_tip_config=(
> > > >  	"drm			drm-fixes"
> > > >  	"drm-misc		drm-misc-fixes"
> > > >  	"drm-intel		drm-intel-fixes"
> > > > +	"drm-xe			drm-xe-fixes"
> > > >  
> > > >  	"drm			drm-next"
> > > >  	"drm-misc		drm-misc-next-fixes"
> > > >  	"drm-intel		drm-intel-next-fixes"
> > > > +	"drm-xe			drm-xe-next-fixes"
> > > >  
> > > >  	"drm-misc		drm-misc-next"
> > > >  	"drm-intel		drm-intel-next"
> > > >  	"drm-intel		drm-intel-gt-next"
> > > > +	"drm-xe			drm-xe-next b6e1b7081768"
> > > 
> > > yeap, up to this commit nothing else should change, but
> > > then we will need an extra rebase of the rest on top of drm/drm-next.
> > > 
> > > But then we need to decide where these following patches will live:
> > > 880277f31cc69 drm/xe/guc: define LNL FW
> > > 2cfc5ae1b8267 drm/xe/guc: define PVC FW
> > > 52383b58eb8cf mei/hdcp: Also enable for XE
> > > bea27d7369855 mei: gsc: add support for auxiliary device created by
> > > Xe driver
> > > fcb3410197f05 fault-inject: Include linux/types.h by default.
> > > 8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs
> > > 
> > > 
> > > Will it be the topic/core-for-CI?
> > > or topic/xe-extras?
> > > or what?
> > 
> > This sounds to me like topic/core-for-CI? Or is there any drawback with
> > that?
> 
> I think some of them are not really a "for CI". It's more like the
> workflow we are adopting e.g. with guc/huc, not sending it to linux-firmware
> until we are confident on what version we will start officially
> supporting.

yeap, I kind of agree here, but at the same time it is our way to run
our CI with the firmware blobs that we need while not final, and also
this was already used for i915's MTL firmware.

> 
> This one can't go to topic/core-for-CI neither:
> 	fcb3410197f05 fault-inject: Include linux/types.h by default.
> 
> what it would do would be that we would not see the build error anymore,
> but everyone else would (and it's not a CI-only configuration).
> Unless it's merged to another branch, we shouldn't merge it.

yeap. it is sad that we were ignored there. let's just drop this then.
our driver is workarounding this bug anyway already.


> 
> "52383b58eb8cf mei/hdcp: Also enable for XE" could be material for
> topic/core-for-CI and  "8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs"
> could either be on that branch or another xe-specific one.

yeap. For the MEI we probably need to ping Greg on the original
submission and ask his ack so we can put that in the final drm-xe-next
for good and not even include in a topic branch.

for the PVC IDs, the topic branch could be okay as well. But if we
end up with an exclusive branch for xe, then it is better there.

> 
> > 
> > > 
> > > Anyway, for the inclusion like this, after our CI is ready:
> > 
> > Could we merge this patch already at this point, considering it will,
> > at least for now, only update drm-tip with our fixes?
> 
> ack
> 
> Lucas De Marchi
> 
> > 
> > Thanks,
> > 
> > /Thomas
> > 
> > 
> > > 
> > > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > 
> > > >  
> > > >  	"drm-intel		topic/core-for-CI"
> > > >  	"drm-misc		topic/i915-ttm"
> > > > --
> > > > 2.42.0
> > > >
> >
Rodrigo Vivi Jan. 8, 2024, 10:15 p.m. UTC | #9
On Wed, Jan 03, 2024 at 02:50:57PM +0100, Thomas Hellström wrote:
> On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote:
> > On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
> > > Add the xe repo to drm-tip and the dim tools.
> > > For now use the sha1 of the first drm-xe-next pull request for drm-
> > > tip,
> > > since that branch tip is currently adapted for our CI testing.
> > > 
> > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> > > Cc: Oded Gabbay <ogabbay@kernel.org>
> > > Cc: daniel.vetter@ffwll.ch
> > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > Cc: dim-tools@lists.freedesktop.org
> > > Cc: dri-devel@lists.freedesktop.org
> > > Cc: intel-gfx@lists.freedesktop.org
> > > Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > > ---
> > >  nightly.conf | 7 +++++++
> > >  1 file changed, 7 insertions(+)
> > > 
> > > diff --git a/nightly.conf b/nightly.conf
> > > index 24126b61b797..accd3ff2cc39 100644
> > > --- a/nightly.conf
> > > +++ b/nightly.conf
> > > @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
> > >  https://anongit.freedesktop.org/git/drm/drm-tip
> > >  https://anongit.freedesktop.org/git/drm/drm-tip.git
> > >  "
> > > +drm_tip_repos[drm-xe]="
> > > +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
> > > +https://gitlab.freedesktop.org/drm/xe/kernel.git
> > > +"
> > >  drm_tip_repos[drm-intel]="
> > >  ssh://git.freedesktop.org/git/drm/drm-intel
> > >  ssh://git.freedesktop.org/git/drm-intel
> > > @@ -65,14 +69,17 @@ drm_tip_config=(
> > >  	"drm			drm-fixes"
> > >  	"drm-misc		drm-misc-fixes"
> > >  	"drm-intel		drm-intel-fixes"
> > > +	"drm-xe			drm-xe-fixes"
> > >  
> > >  	"drm			drm-next"
> > >  	"drm-misc		drm-misc-next-fixes"
> > >  	"drm-intel		drm-intel-next-fixes"
> > > +	"drm-xe			drm-xe-next-fixes"
> > >  
> > >  	"drm-misc		drm-misc-next"
> > >  	"drm-intel		drm-intel-next"
> > >  	"drm-intel		drm-intel-gt-next"
> > > +	"drm-xe			drm-xe-next b6e1b7081768"
> > 
> > yeap, up to this commit nothing else should change, but
> > then we will need an extra rebase of the rest on top of drm/drm-next.
> > 
> > But then we need to decide where these following patches will live:
> > 880277f31cc69 drm/xe/guc: define LNL FW
> > 2cfc5ae1b8267 drm/xe/guc: define PVC FW
> > 52383b58eb8cf mei/hdcp: Also enable for XE
> > bea27d7369855 mei: gsc: add support for auxiliary device created by
> > Xe driver
> > fcb3410197f05 fault-inject: Include linux/types.h by default.
> > 8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs
> > 
> > 
> > Will it be the topic/core-for-CI?
> > or topic/xe-extras?
> > or what?
> 
> This sounds to me like topic/core-for-CI? Or is there any drawback with
> that?
> 
> > 
> > Anyway, for the inclusion like this, after our CI is ready:
> 
> Could we merge this patch already at this point, considering it will,
> at least for now, only update drm-tip with our fixes?

yeap, likely a good idea.
Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

But we just move the final drm-xe-next after figuring out the topic
branches and the final rebases fixing the tags and the commiter's s-o-b
and the fixes tags.

> 
> Thanks,
> 
> /Thomas
> 
> 
> > 
> > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > 
> > >  
> > >  	"drm-intel		topic/core-for-CI"
> > >  	"drm-misc		topic/i915-ttm"
> > > -- 
> > > 2.42.0
> > > 
>
Lucas De Marchi Jan. 8, 2024, 10:22 p.m. UTC | #10
On Mon, Jan 08, 2024 at 05:13:51PM -0500, Rodrigo Vivi wrote:
>On Wed, Jan 03, 2024 at 11:59:16PM -0600, Lucas De Marchi wrote:
>> On Wed, Jan 03, 2024 at 02:50:57PM +0100, Thomas Hellström wrote:
>> > On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote:
>> > > On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote:
>> > > > Add the xe repo to drm-tip and the dim tools.
>> > > > For now use the sha1 of the first drm-xe-next pull request for drm-
>> > > > tip,
>> > > > since that branch tip is currently adapted for our CI testing.
>> > > >
>> > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> > > > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>> > > > Cc: Oded Gabbay <ogabbay@kernel.org>
>> > > > Cc: daniel.vetter@ffwll.ch
>> > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>> > > > Cc: dim-tools@lists.freedesktop.org
>> > > > Cc: dri-devel@lists.freedesktop.org
>> > > > Cc: intel-gfx@lists.freedesktop.org
>> > > > Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>> > > > ---
>> > > >  nightly.conf | 7 +++++++
>> > > >  1 file changed, 7 insertions(+)
>> > > >
>> > > > diff --git a/nightly.conf b/nightly.conf
>> > > > index 24126b61b797..accd3ff2cc39 100644
>> > > > --- a/nightly.conf
>> > > > +++ b/nightly.conf
>> > > > @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
>> > > >  https://anongit.freedesktop.org/git/drm/drm-tip
>> > > >  https://anongit.freedesktop.org/git/drm/drm-tip.git
>> > > >  "
>> > > > +drm_tip_repos[drm-xe]="
>> > > > +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
>> > > > +https://gitlab.freedesktop.org/drm/xe/kernel.git
>> > > > +"
>> > > >  drm_tip_repos[drm-intel]="
>> > > >  ssh://git.freedesktop.org/git/drm/drm-intel
>> > > >  ssh://git.freedesktop.org/git/drm-intel
>> > > > @@ -65,14 +69,17 @@ drm_tip_config=(
>> > > >  	"drm			drm-fixes"
>> > > >  	"drm-misc		drm-misc-fixes"
>> > > >  	"drm-intel		drm-intel-fixes"
>> > > > +	"drm-xe			drm-xe-fixes"
>> > > >  
>> > > >  	"drm			drm-next"
>> > > >  	"drm-misc		drm-misc-next-fixes"
>> > > >  	"drm-intel		drm-intel-next-fixes"
>> > > > +	"drm-xe			drm-xe-next-fixes"
>> > > >  
>> > > >  	"drm-misc		drm-misc-next"
>> > > >  	"drm-intel		drm-intel-next"
>> > > >  	"drm-intel		drm-intel-gt-next"
>> > > > +	"drm-xe			drm-xe-next b6e1b7081768"
>> > >
>> > > yeap, up to this commit nothing else should change, but
>> > > then we will need an extra rebase of the rest on top of drm/drm-next.
>> > >
>> > > But then we need to decide where these following patches will live:
>> > > 880277f31cc69 drm/xe/guc: define LNL FW
>> > > 2cfc5ae1b8267 drm/xe/guc: define PVC FW
>> > > 52383b58eb8cf mei/hdcp: Also enable for XE
>> > > bea27d7369855 mei: gsc: add support for auxiliary device created by
>> > > Xe driver
>> > > fcb3410197f05 fault-inject: Include linux/types.h by default.
>> > > 8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs
>> > >
>> > >
>> > > Will it be the topic/core-for-CI?
>> > > or topic/xe-extras?
>> > > or what?
>> >
>> > This sounds to me like topic/core-for-CI? Or is there any drawback with
>> > that?
>>
>> I think some of them are not really a "for CI". It's more like the
>> workflow we are adopting e.g. with guc/huc, not sending it to linux-firmware
>> until we are confident on what version we will start officially
>> supporting.
>
>yeap, I kind of agree here, but at the same time it is our way to run
>our CI with the firmware blobs that we need while not final, and also
>this was already used for i915's MTL firmware.

ok

>
>>
>> This one can't go to topic/core-for-CI neither:
>> 	fcb3410197f05 fault-inject: Include linux/types.h by default.
>>
>> what it would do would be that we would not see the build error anymore,
>> but everyone else would (and it's not a CI-only configuration).
>> Unless it's merged to another branch, we shouldn't merge it.
>
>yeap. it is sad that we were ignored there. let's just drop this then.
>our driver is workarounding this bug anyway already.

agreed, let's drop it.

>
>
>>
>> "52383b58eb8cf mei/hdcp: Also enable for XE" could be material for
>> topic/core-for-CI and  "8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs"
>> could either be on that branch or another xe-specific one.
>
>yeap. For the MEI we probably need to ping Greg on the original
>submission and ask his ack so we can put that in the final drm-xe-next
>for good and not even include in a topic branch.
>
>for the PVC IDs, the topic branch could be okay as well. But if we
>end up with an exclusive branch for xe, then it is better there.

fair enough... I'm ok starting with topic/core-for-CI and if it ever
becomes a problem, to create a xe-specific topic branch.

thanks
Lucas De Marchi

>
>>
>> >
>> > >
>> > > Anyway, for the inclusion like this, after our CI is ready:
>> >
>> > Could we merge this patch already at this point, considering it will,
>> > at least for now, only update drm-tip with our fixes?
>>
>> ack
>>
>> Lucas De Marchi
>>
>> >
>> > Thanks,
>> >
>> > /Thomas
>> >
>> >
>> > >
>> > > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> > >
>> > > >  
>> > > >  	"drm-intel		topic/core-for-CI"
>> > > >  	"drm-misc		topic/i915-ttm"
>> > > > --
>> > > > 2.42.0
>> > > >
>> >
Thomas Hellström Jan. 9, 2024, 1:46 p.m. UTC | #11
On Mon, 2024-01-08 at 16:22 -0600, Lucas De Marchi wrote:
> On Mon, Jan 08, 2024 at 05:13:51PM -0500, Rodrigo Vivi wrote:
> > On Wed, Jan 03, 2024 at 11:59:16PM -0600, Lucas De Marchi wrote:
> > > On Wed, Jan 03, 2024 at 02:50:57PM +0100, Thomas Hellström wrote:
> > > > On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote:
> > > > > On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström
> > > > > wrote:
> > > > > > Add the xe repo to drm-tip and the dim tools.
> > > > > > For now use the sha1 of the first drm-xe-next pull request
> > > > > > for drm-
> > > > > > tip,
> > > > > > since that branch tip is currently adapted for our CI
> > > > > > testing.
> > > > > > 
> > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> > > > > > Cc: Oded Gabbay <ogabbay@kernel.org>
> > > > > > Cc: daniel.vetter@ffwll.ch
> > > > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > > > > Cc: dim-tools@lists.freedesktop.org
> > > > > > Cc: dri-devel@lists.freedesktop.org
> > > > > > Cc: intel-gfx@lists.freedesktop.org
> > > > > > Signed-off-by: Thomas Hellström
> > > > > > <thomas.hellstrom@linux.intel.com>
> > > > > > ---
> > > > > >  nightly.conf | 7 +++++++
> > > > > >  1 file changed, 7 insertions(+)
> > > > > > 
> > > > > > diff --git a/nightly.conf b/nightly.conf
> > > > > > index 24126b61b797..accd3ff2cc39 100644
> > > > > > --- a/nightly.conf
> > > > > > +++ b/nightly.conf
> > > > > > @@ -24,6 +24,10 @@ git://anongit.freedesktop.org/drm-tip
> > > > > >  https://anongit.freedesktop.org/git/drm/drm-tip
> > > > > >  https://anongit.freedesktop.org/git/drm/drm-tip.git
> > > > > >  "
> > > > > > +drm_tip_repos[drm-xe]="
> > > > > > +ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
> > > > > > +https://gitlab.freedesktop.org/drm/xe/kernel.git
> > > > > > +"
> > > > > >  drm_tip_repos[drm-intel]="
> > > > > >  ssh://git.freedesktop.org/git/drm/drm-intel
> > > > > >  ssh://git.freedesktop.org/git/drm-intel
> > > > > > @@ -65,14 +69,17 @@ drm_tip_config=(
> > > > > >  	"drm			drm-fixes"
> > > > > >  	"drm-misc		drm-misc-fixes"
> > > > > >  	"drm-intel		drm-intel-fixes"
> > > > > > +	"drm-xe			drm-xe-fixes"
> > > > > >  
> > > > > >  	"drm			drm-next"
> > > > > >  	"drm-misc		drm-misc-next-fixes"
> > > > > >  	"drm-intel		drm-intel-next-fixes"
> > > > > > +	"drm-xe			drm-xe-next-fixes"
> > > > > >  
> > > > > >  	"drm-misc		drm-misc-next"
> > > > > >  	"drm-intel		drm-intel-next"
> > > > > >  	"drm-intel		drm-intel-gt-next"
> > > > > > +	"drm-xe			drm-xe-next
> > > > > > b6e1b7081768"
> > > > > 
> > > > > yeap, up to this commit nothing else should change, but
> > > > > then we will need an extra rebase of the rest on top of
> > > > > drm/drm-next.
> > > > > 
> > > > > But then we need to decide where these following patches will
> > > > > live:
> > > > > 880277f31cc69 drm/xe/guc: define LNL FW
> > > > > 2cfc5ae1b8267 drm/xe/guc: define PVC FW
> > > > > 52383b58eb8cf mei/hdcp: Also enable for XE
> > > > > bea27d7369855 mei: gsc: add support for auxiliary device
> > > > > created by
> > > > > Xe driver
> > > > > fcb3410197f05 fault-inject: Include linux/types.h by default.
> > > > > 8ebd9cd71f8ac drm/xe: Add PVC's PCI device IDs
> > > > > 
> > > > > 
> > > > > Will it be the topic/core-for-CI?
> > > > > or topic/xe-extras?
> > > > > or what?
> > > > 
> > > > This sounds to me like topic/core-for-CI? Or is there any
> > > > drawback with
> > > > that?
> > > 
> > > I think some of them are not really a "for CI". It's more like
> > > the
> > > workflow we are adopting e.g. with guc/huc, not sending it to
> > > linux-firmware
> > > until we are confident on what version we will start officially
> > > supporting.
> > 
> > yeap, I kind of agree here, but at the same time it is our way to
> > run
> > our CI with the firmware blobs that we need while not final, and
> > also
> > this was already used for i915's MTL firmware.
> 
> ok
> 
> > 
> > > 
> > > This one can't go to topic/core-for-CI neither:
> > > 	fcb3410197f05 fault-inject: Include linux/types.h by
> > > default.
> > > 
> > > what it would do would be that we would not see the build error
> > > anymore,
> > > but everyone else would (and it's not a CI-only configuration).
> > > Unless it's merged to another branch, we shouldn't merge it.
> > 
> > yeap. it is sad that we were ignored there. let's just drop this
> > then.
> > our driver is workarounding this bug anyway already.
> 
> agreed, let's drop it.
> 
> > 
> > 
> > > 
> > > "52383b58eb8cf mei/hdcp: Also enable for XE" could be material
> > > for
> > > topic/core-for-CI and  "8ebd9cd71f8ac drm/xe: Add PVC's PCI
> > > device IDs"
> > > could either be on that branch or another xe-specific one.
> > 
> > yeap. For the MEI we probably need to ping Greg on the original
> > submission and ask his ack so we can put that in the final drm-xe-
> > next
> > for good and not even include in a topic branch.
> > 
> > for the PVC IDs, the topic branch could be okay as well. But if we
> > end up with an exclusive branch for xe, then it is better there.
> 
> fair enough... I'm ok starting with topic/core-for-CI and if it ever
> becomes a problem, to create a xe-specific topic branch.

Actually looking at the documented rules for core-for-CI, I think we
should create a topic branch drm-xe/xe-for-CI or something similar that
hold the patches that touch the xe driver. core-for-CI is meant for
changes outside of drm and when rebased should be rooted in a linux
upstream release or -rc. I figure we'd want xe-for-CI to be rooted in
drm-xe-next assuming we may want to change the code these patches touch
and quickly rebase xe-for-CI accordingly?

/Thomas

> 
> thanks
> Lucas De Marchi
> 
> > 
> > > 
> > > > 
> > > > > 
> > > > > Anyway, for the inclusion like this, after our CI is ready:
> > > > 
> > > > Could we merge this patch already at this point, considering it
> > > > will,
> > > > at least for now, only update drm-tip with our fixes?
> > > 
> > > ack
> > > 
> > > Lucas De Marchi
> > > 
> > > > 
> > > > Thanks,
> > > > 
> > > > /Thomas
> > > > 
> > > > 
> > > > > 
> > > > > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > 
> > > > > >  
> > > > > >  	"drm-intel		topic/core-for-CI"
> > > > > >  	"drm-misc		topic/i915-ttm"
> > > > > > --
> > > > > > 2.42.0
> > > > > > 
> > > >
diff mbox series

Patch

diff --git a/nightly.conf b/nightly.conf
index 24126b61b797..accd3ff2cc39 100644
--- a/nightly.conf
+++ b/nightly.conf
@@ -24,6 +24,10 @@  git://anongit.freedesktop.org/drm-tip
 https://anongit.freedesktop.org/git/drm/drm-tip
 https://anongit.freedesktop.org/git/drm/drm-tip.git
 "
+drm_tip_repos[drm-xe]="
+ssh://git@gitlab.freedesktop.org/drm/xe/kernel.git
+https://gitlab.freedesktop.org/drm/xe/kernel.git
+"
 drm_tip_repos[drm-intel]="
 ssh://git.freedesktop.org/git/drm/drm-intel
 ssh://git.freedesktop.org/git/drm-intel
@@ -65,14 +69,17 @@  drm_tip_config=(
 	"drm			drm-fixes"
 	"drm-misc		drm-misc-fixes"
 	"drm-intel		drm-intel-fixes"
+	"drm-xe			drm-xe-fixes"
 
 	"drm			drm-next"
 	"drm-misc		drm-misc-next-fixes"
 	"drm-intel		drm-intel-next-fixes"
+	"drm-xe			drm-xe-next-fixes"
 
 	"drm-misc		drm-misc-next"
 	"drm-intel		drm-intel-next"
 	"drm-intel		drm-intel-gt-next"
+	"drm-xe			drm-xe-next b6e1b7081768"
 
 	"drm-intel		topic/core-for-CI"
 	"drm-misc		topic/i915-ttm"