Forgot this: Reviewed-by: Alexandre Courbot <acourbot@xxxxxxxxxx> On Fri, Apr 3, 2015 at 8:13 AM, David Ung <davidu@xxxxxxxxxx> wrote: >> -----Original Message----- >> From: Alexandre Courbot [mailto:gnurou@xxxxxxxxx] >> Sent: Monday, March 30, 2015 11:30 PM >> To: David Ung; thierry.reding@xxxxxxxxx >> Cc: airlied@xxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx; >> swarren@xxxxxxxxxxxxx; 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 <davidu@xxxxxxxxxx> wrote: >> > >> >> -----Original Message----- >> >> From: Alexandre Courbot [mailto:gnurou@xxxxxxxxx] >> >> Sent: Monday, March 23, 2015 12:09 AM >> >> To: David Ung >> >> Cc: airlied@xxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx; >> >> thierry.reding@xxxxxxxxx; swarren@xxxxxxxxxxxxx; 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
