Forgot this:

Reviewed-by: Alexandre Courbot <[email protected]>

On Fri, Apr 3, 2015 at 8:13 AM, David Ung <[email protected]> wrote:
>> -----Original Message-----
>> From: Alexandre Courbot [mailto:[email protected]]
>> Sent: Monday, March 30, 2015 11:30 PM
>> To: David Ung; [email protected]
>> Cc: [email protected]; [email protected];
>> [email protected]; Arto Merilainen; Terje Bergstrom
>> Subject: Re: [PATCH V2 1/3] gpu: host1x: Reduce host1x job allocation size
>>
>> On Tue, Mar 24, 2015 at 4:39 AM, David Ung <[email protected]> wrote:
>> >
>> >> -----Original Message-----
>> >> From: Alexandre Courbot [mailto:[email protected]]
>> >> Sent: Monday, March 23, 2015 12:09 AM
>> >> To: David Ung
>> >> Cc: [email protected]; [email protected];
>> >> [email protected]; [email protected]; Arto Merilainen;
>> >> Terje Bergstrom
>> >> Subject: Re: [PATCH V2 1/3] gpu: host1x: Reduce host1x job allocation
>> >> size
>> >>
>> >> The series looks ok to me, the only problem I see is that we are
>> >> lacking means to thoroughly test host1x patches across all the
>> >> supported Tegra chips.
>> >>
>> >> Could you detail what kind of testing this patch series has
>> >> undergone, and which chips/boards have been tested?
>> >
>> > Tested the host1x channel tests for T132 on A44.
>> > Also ran Paul's EIMT set up which only tests booting kernel to console
>> > for T20/T30/T114/T124 and QEMU
>>
>> Thierry, what do you think?
>>
>> I know we are lacking proper testing infrastructure for host1x, but these
>> patches really looks harmless to me.
>
>
> Thank you Alex for the review.
> It would be good to have closure to this patchset.
>
> Thierry ping?
>
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to