Re: [PATCH 1/4] drm: Add helpers for locking an array of BO reservations.

From: Eric Anholt
Date: Thu Mar 14 2019 - 12:28:16 EST


Rob Herring <robh@xxxxxxxxxx> writes:

> On Tue, Mar 12, 2019 at 12:37 PM Eric Anholt <eric@xxxxxxxxxx> wrote:
>>
>> Rob Herring <robh@xxxxxxxxxx> writes:
>>
>> > On Fri, Mar 8, 2019 at 10:17 AM Eric Anholt <eric@xxxxxxxxxx> wrote:
>> >>
>> >> Now that we have the reservation object in the GEM object, it's easy
>> >> to provide a helper for this common case. Noticed while reviewing
>> >> panfrost and lima drivers. This particular version came out of v3d,
>> >> which in turn was a copy from vc4.
>> >>
>> >> Signed-off-by: Eric Anholt <eric@xxxxxxxxxx>
>> >> ---
>> >> drivers/gpu/drm/drm_gem.c | 76 +++++++++++++++++++++++++++++++++++++++
>> >> include/drm/drm_gem.h | 4 +++
>> >> 2 files changed, 80 insertions(+)
>> >
>> > Sweet! I was about to go write this same patch. You are changing the
>> > license from GPL to MIT copying the v3d version, but I guess you have
>> > rights to do that.
>> >
>> > FWIW,
>> >
>> > Acked-by: Rob Herring <robh@xxxxxxxxxx>
>>
>> Was this just for this one patch, or the series? I don't think I can
>> merge without a consumer.
>
> Sure, it can be for patches 1-3. Patch 4 is dependent on me sending
> out the shmem helpers again.

Thanks. Pushed the first 3 now.

Attachment: signature.asc
Description: PGP signature