Thread (20 messages) 20 messages, 3 authors, 2012-10-25
STALE4990d
Revisions (5)
  1. v3 [diff vs current]
  2. v3 [diff vs current]
  3. v3 current
  4. v3 [diff vs current]
  5. v4 [diff vs current]

[PATCH 5/6] ARM: OMAP2+: Make some definitions local

From: tony@atomide.com (Tony Lindgren)
Date: 2012-10-19 16:17:43
Also in: linux-omap

* Laurent Pinchart [off-list ref] [121019 02:45]:
On Thursday 18 October 2012 13:28:48 Tony Lindgren wrote:
quoted
@@ -117,13 +112,6 @@ static inline struct omap_iommu
*dev_to_omap_iommu(struct device *dev) }
 #endif

-/* IOMMU errors */
-#define OMAP_IOMMU_ERR_TLB_MISS		(1 << 0)
-#define OMAP_IOMMU_ERR_TRANS_FAULT	(1 << 1)
-#define OMAP_IOMMU_ERR_EMU_MISS		(1 << 2)
-#define OMAP_IOMMU_ERR_TBLWALK_FAULT	(1 << 3)
-#define OMAP_IOMMU_ERR_MULTIHIT_FAULT	(1 << 4)
-
I'll use those in the tidspbridge driver, in patches that I plan to push soon.

I will apply this patch set on top of mine, see what breaks. Would you like me 
to propose a modified version of this set, or add additional patches in my set 
?
Sure, let's try to expose only the minimal amount of omap
iommu things with this patch set so we don't break things.
Then the iommu development can continue on it's own independent
of the core omap code except for the platform data.

But again, if there are nasty layering violations using this
code, I would just remove those features. Those things tend to
just get worse unless they are fixed properly to start with.

Regards,

Tony
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help