From: John Harrison <john.c.harrison@intel.com>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Greg KH <greg@kroah.com>, Daniel Vetter <daniel.vetter@ffwll.ch>,
"Jani Nikula" <jani.nikula@linux.intel.com>,
Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
Rodrigo Vivi <rodrigo.vivi@intel.com>,
Intel Graphics <intel-gfx@lists.freedesktop.org>,
DRI <dri-devel@lists.freedesktop.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Linux Next Mailing List <linux-next@vger.kernel.org>
Subject: Re: linux-next: manual merge of the usb tree with the drm-intel-fixes tree
Date: Tue, 31 Jan 2023 10:27:29 -0800 [thread overview]
Message-ID: <9566dc52-2ff1-760d-c9cb-fdfef9278f05@intel.com> (raw)
In-Reply-To: <Y9kNRVppj5Uxa9ub@smile.fi.intel.com>
On 1/31/2023 04:44, Andy Shevchenko wrote:
> On Tue, Jan 31, 2023 at 01:03:05PM +1100, Stephen Rothwell wrote:
>> Hi all,
>>
>> Today's linux-next merge of the usb tree got a conflict in:
>>
>> drivers/gpu/drm/i915/gt/intel_engine_cs.c
>>
>> between commit:
>>
>> 5bc4b43d5c6c ("drm/i915: Fix up locking around dumping requests lists")
>>
>> from the drm-intel-fixes tree and commit:
>>
>> 4d70c74659d9 ("i915: Move list_count() to list.h as list_count_nodes() for broader use")
>>
>> from the usb tree.
>>
>> I fixed it up (the former removed the code changed by the latter)
> Hmm... Currently I see that 20230127002842.3169194-4-John.C.Harrison@Intel.com
> moves the code to the drivers/gpu/drm/i915/gt/intel_execlists_submission.c.
>
> Is there any new series beside the above mentioned that touches that file and
> actually _removes_ that code?
As long as the removal is limited to list_count/list_count_nodes, that's
fine. I only moved it from one file to another because the one and only
function that was using it was being moved to the other file. If someone
else has found a use for the same and wants to move it to a more common
place then great. I assume there was no conflict happening in the i915
specific code.
John.
>
>> and
>> can carry the fix as necessary. This is now fixed as far as linux-next
>> is concerned, but any non trivial conflicts should be mentioned to your
>> upstream maintainer when your tree is submitted for merging. You may
>> also want to consider cooperating with the maintainer of the conflicting
>> tree to minimise any particularly complex conflicts.
next prev parent reply other threads:[~2023-01-31 18:27 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-31 2:03 linux-next: manual merge of the usb tree with the drm-intel-fixes tree Stephen Rothwell
2023-01-31 6:41 ` Greg KH
2023-01-31 12:44 ` Andy Shevchenko
2023-01-31 18:27 ` John Harrison [this message]
2023-02-01 4:11 ` Stephen Rothwell
2023-02-01 15:31 ` Rodrigo Vivi
2023-02-01 18:37 ` John Harrison
2023-02-01 21:05 ` Rodrigo Vivi
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=9566dc52-2ff1-760d-c9cb-fdfef9278f05@intel.com \
--to=john.c.harrison@intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=greg@kroah.com \
--cc=gregkh@linuxfoundation.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@linux.intel.com \
--cc=joonas.lahtinen@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=rodrigo.vivi@intel.com \
--cc=sfr@canb.auug.org.au \
/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