Re: [PATCH V2 1/3] gpu: host1x: Reduce host1x job allocation size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 02, 2015 at 11:13:38PM +0000, David Ung 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. 

We'll get closure on this once the testing infrastructure appears. I
won't apply these patches before then.

Thierry

Attachment: pgpQYZrONqsiz.pgp
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux