Re: [PATCH v2 15/27] gpu: host1x: Add support for Tegra114

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

 



On Fri, Oct 11, 2013 at 04:13:07PM -0600, Stephen Warren wrote:
> On 10/07/2013 02:34 AM, Thierry Reding wrote:
> > Tegra114 uses a slightly updated version of host1x with an additional
> > syncpoint.
> 
> >  drivers/gpu/host1x/hw/host1x02.c            |  42 +++++
> >  drivers/gpu/host1x/hw/host1x02.h            |  26 +++
> >  drivers/gpu/host1x/hw/hw_host1x02_channel.h | 121 ++++++++++++++
> >  drivers/gpu/host1x/hw/hw_host1x02_sync.h    | 243 ++++++++++++++++++++++++++++
> >  drivers/gpu/host1x/hw/hw_host1x02_uclass.h  | 175 ++++++++++++++++++++
> 
> That seems like an awful lot of extra lines to support just one extra
> syncpoint. Are there other changes? If not, can the code be
> shared/parameterized somehow?

Yeah, I don't like very much how this is currently done. I mean about
half of this is actually duplicate code because of the static inline
functions used for register defines. As discussed elsewhere this was
originally meant to be used for coverage testing, but nobody's done
anything like that as far as I know. I'm also not convinced that these
would be very useful in coverage testing, but adding Terje on Cc and
unless he or anyone else has any (strong) objections I'll go and remove
the duplicate definitions and while at it see if I can come up with a
way to share more code/definitions between versions of host1x.

Do you see this as a blocker for 3.13 or can I do the cleanup after
that? As far as I know Tegra124 has a more heavily modified version of
host1x so implementing Tegra124 support might be a good opportunity to
clean this up anyway.

Thierry

Attachment: pgp2VlPWcd3yQ.pgp
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux