drm/xe: Privatize xe_ggtt_node

Nothing requires it any more, make the member private.

Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
Link: https://patch.msgid.link/20260108101014.579906-16-dev@lankhorst.se

diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
index 1d748d1aeb3c08faed9e1b6dd7c147e9b728068a..cb23d97845a8651b7ca4bd112b3a31a35a44ff80 100644 (file)
--- a/drivers/gpu/drm/xe/xe_ggtt.c
+++ b/drivers/gpu/drm/xe/xe_ggtt.c
@@ -67,6 +67,24 @@
  * give us the correct placement for free.
  */
 
+/**
+ * struct xe_ggtt_node - A node in GGTT.
+ *
+ * This struct needs to be initialized (only-once) with xe_ggtt_node_init() before any node
+ * insertion, reservation, or 'ballooning'.
+ * It will, then, be finalized by either xe_ggtt_node_remove() or xe_ggtt_node_deballoon().
+ */
+struct xe_ggtt_node {
+       /** @ggtt: Back pointer to xe_ggtt where this region will be inserted at */
+       struct xe_ggtt *ggtt;
+       /** @base: A drm_mm_node */
+       struct drm_mm_node base;
+       /** @delayed_removal_work: The work struct for the delayed removal */
+       struct work_struct delayed_removal_work;
+       /** @invalidate_on_remove: If it needs invalidation upon removal */
+       bool invalidate_on_remove;
+};
+
 static u64 xelp_ggtt_pte_flags(struct xe_bo *bo, u16 pat_index)
 {
        u64 pte = XE_PAGE_PRESENT;

diff --git a/drivers/gpu/drm/xe/xe_ggtt_types.h b/drivers/gpu/drm/xe/xe_ggtt_types.h
index 5fd723ce810878f445b72cc05781b089239b4120..d82b71a198bc2bf6d7e697a441af9bed2c589011 100644 (file)
--- a/drivers/gpu/drm/xe/xe_ggtt_types.h
+++ b/drivers/gpu/drm/xe/xe_ggtt_types.h
@@ -11,6 +11,7 @@
 #include "xe_pt_types.h"
 
 struct xe_bo;
+struct xe_ggtt_node;
 struct xe_gt;
 
 /**
@@ -53,24 +54,6 @@ struct xe_ggtt {
        struct workqueue_struct *wq;
 };
 
-/**
- * struct xe_ggtt_node - A node in GGTT.
- *
- * This struct needs to be initialized (only-once) with xe_ggtt_node_init() before any node
- * insertion, reservation, or 'ballooning'.
- * It will, then, be finalized by either xe_ggtt_node_remove() or xe_ggtt_node_deballoon().
- */
-struct xe_ggtt_node {
-       /** @ggtt: Back pointer to xe_ggtt where this region will be inserted at */
-       struct xe_ggtt *ggtt;
-       /** @base: A drm_mm_node */
-       struct drm_mm_node base;
-       /** @delayed_removal_work: The work struct for the delayed removal */
-       struct work_struct delayed_removal_work;
-       /** @invalidate_on_remove: If it needs invalidation upon removal */
-       bool invalidate_on_remove;
-};
-
 typedef void (*xe_ggtt_set_pte_fn)(struct xe_ggtt *ggtt, u64 addr, u64 pte);
 typedef void (*xe_ggtt_transform_cb)(struct xe_ggtt *ggtt,
                                     struct xe_ggtt_node *node,