Re: [PATCH 3/6] gpu: drm: tegra: Remove redundant host1x

[Thierry Reding <thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1110 bytes --]

On Thu, Nov 22, 2012 at 11:48:01AM +0200, Terje Bergstrom wrote:
> From: Arto Merilainen <amerilainen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> This patch removes the redundant host1x driver from tegradrm and
> makes necessary bindings to the separate host driver.

I'd rather keep the host1x driver where it is for now. There aren't any
users external to DRM yet, so I don't see a reason for moving it out.
Instead we can use DRM as a playground, see what we really need and once
external users start to appear we can think about moving the host1x code
somewhere else.

> This modification introduces a regression: Because there is no
> general framework for attaching separate devices into the
> same address space, this patch removes the ability to use IOMMU
> in tegradrm.

So the IOMMU support was up for discussion anyway and I wasn't going to
submit it for inclusion anytime soon either. No harm done.

Aside from ripping out the existing host1x code, the remainder of this
patch is mostly renaming from host1x_client to tegra_drm_client, so I
won't comment on it.

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]