* [PATCH] drm/radeon/mkregtable: Delete unused list functions and macros
@ 2018-02-28 22:17 ` Matthias Kaehlcke
0 siblings, 0 replies; 5+ messages in thread
From: Matthias Kaehlcke @ 2018-02-28 22:17 UTC (permalink / raw)
To: Alex Deucher, Christian König, David Airlie
Cc: amd-gfx, dri-devel, linux-kernel, Guenter Roeck, Matthias Kaehlcke
The util mkregtable includes a copy of the kernel API for linked lists,
only a small subset of it is used. Delete the unused functions and macros.
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
---
drivers/gpu/drm/radeon/mkregtable.c | 433 ----------------------------
1 file changed, 433 deletions(-)
diff --git a/drivers/gpu/drm/radeon/mkregtable.c b/drivers/gpu/drm/radeon/mkregtable.c
index c21d8fa591ef..ba704633b072 100644
--- a/drivers/gpu/drm/radeon/mkregtable.c
+++ b/drivers/gpu/drm/radeon/mkregtable.c
@@ -43,10 +43,6 @@ struct list_head {
struct list_head *next, *prev;
};
-#define LIST_HEAD_INIT(name) { &(name), &(name) }
-
-#define LIST_HEAD(name) \
- struct list_head name = LIST_HEAD_INIT(name)
static inline void INIT_LIST_HEAD(struct list_head *list)
{
@@ -74,19 +70,6 @@ extern void __list_add(struct list_head *new,
struct list_head *prev, struct list_head *next);
#endif
-/**
- * list_add - add a new entry
- * @new: new entry to be added
- * @head: list head to add it after
- *
- * Insert a new entry after the specified head.
- * This is good for implementing stacks.
- */
-static inline void list_add(struct list_head *new, struct list_head *head)
-{
- __list_add(new, head, head->next);
-}
-
/**
* list_add_tail - add a new entry
* @new: new entry to be added
@@ -100,250 +83,6 @@ static inline void list_add_tail(struct list_head *new, struct list_head *head)
__list_add(new, head->prev, head);
}
-/*
- * Delete a list entry by making the prev/next entries
- * point to each other.
- *
- * This is only for internal list manipulation where we know
- * the prev/next entries already!
- */
-static inline void __list_del(struct list_head *prev, struct list_head *next)
-{
- next->prev = prev;
- prev->next = next;
-}
-
-/**
- * list_del - deletes entry from list.
- * @entry: the element to delete from the list.
- * Note: list_empty() on entry does not return true after this, the entry is
- * in an undefined state.
- */
-#ifndef CONFIG_DEBUG_LIST
-static inline void list_del(struct list_head *entry)
-{
- __list_del(entry->prev, entry->next);
- entry->next = (void *)0xDEADBEEF;
- entry->prev = (void *)0xBEEFDEAD;
-}
-#else
-extern void list_del(struct list_head *entry);
-#endif
-
-/**
- * list_replace - replace old entry by new one
- * @old : the element to be replaced
- * @new : the new element to insert
- *
- * If @old was empty, it will be overwritten.
- */
-static inline void list_replace(struct list_head *old, struct list_head *new)
-{
- new->next = old->next;
- new->next->prev = new;
- new->prev = old->prev;
- new->prev->next = new;
-}
-
-static inline void list_replace_init(struct list_head *old,
- struct list_head *new)
-{
- list_replace(old, new);
- INIT_LIST_HEAD(old);
-}
-
-/**
- * list_del_init - deletes entry from list and reinitialize it.
- * @entry: the element to delete from the list.
- */
-static inline void list_del_init(struct list_head *entry)
-{
- __list_del(entry->prev, entry->next);
- INIT_LIST_HEAD(entry);
-}
-
-/**
- * list_move - delete from one list and add as another's head
- * @list: the entry to move
- * @head: the head that will precede our entry
- */
-static inline void list_move(struct list_head *list, struct list_head *head)
-{
- __list_del(list->prev, list->next);
- list_add(list, head);
-}
-
-/**
- * list_move_tail - delete from one list and add as another's tail
- * @list: the entry to move
- * @head: the head that will follow our entry
- */
-static inline void list_move_tail(struct list_head *list,
- struct list_head *head)
-{
- __list_del(list->prev, list->next);
- list_add_tail(list, head);
-}
-
-/**
- * list_is_last - tests whether @list is the last entry in list @head
- * @list: the entry to test
- * @head: the head of the list
- */
-static inline int list_is_last(const struct list_head *list,
- const struct list_head *head)
-{
- return list->next == head;
-}
-
-/**
- * list_empty - tests whether a list is empty
- * @head: the list to test.
- */
-static inline int list_empty(const struct list_head *head)
-{
- return head->next == head;
-}
-
-/**
- * list_empty_careful - tests whether a list is empty and not being modified
- * @head: the list to test
- *
- * Description:
- * tests whether a list is empty _and_ checks that no other CPU might be
- * in the process of modifying either member (next or prev)
- *
- * NOTE: using list_empty_careful() without synchronization
- * can only be safe if the only activity that can happen
- * to the list entry is list_del_init(). Eg. it cannot be used
- * if another CPU could re-list_add() it.
- */
-static inline int list_empty_careful(const struct list_head *head)
-{
- struct list_head *next = head->next;
- return (next == head) && (next == head->prev);
-}
-
-/**
- * list_is_singular - tests whether a list has just one entry.
- * @head: the list to test.
- */
-static inline int list_is_singular(const struct list_head *head)
-{
- return !list_empty(head) && (head->next == head->prev);
-}
-
-static inline void __list_cut_position(struct list_head *list,
- struct list_head *head,
- struct list_head *entry)
-{
- struct list_head *new_first = entry->next;
- list->next = head->next;
- list->next->prev = list;
- list->prev = entry;
- entry->next = list;
- head->next = new_first;
- new_first->prev = head;
-}
-
-/**
- * list_cut_position - cut a list into two
- * @list: a new list to add all removed entries
- * @head: a list with entries
- * @entry: an entry within head, could be the head itself
- * and if so we won't cut the list
- *
- * This helper moves the initial part of @head, up to and
- * including @entry, from @head to @list. You should
- * pass on @entry an element you know is on @head. @list
- * should be an empty list or a list you do not care about
- * losing its data.
- *
- */
-static inline void list_cut_position(struct list_head *list,
- struct list_head *head,
- struct list_head *entry)
-{
- if (list_empty(head))
- return;
- if (list_is_singular(head) && (head->next != entry && head != entry))
- return;
- if (entry == head)
- INIT_LIST_HEAD(list);
- else
- __list_cut_position(list, head, entry);
-}
-
-static inline void __list_splice(const struct list_head *list,
- struct list_head *prev, struct list_head *next)
-{
- struct list_head *first = list->next;
- struct list_head *last = list->prev;
-
- first->prev = prev;
- prev->next = first;
-
- last->next = next;
- next->prev = last;
-}
-
-/**
- * list_splice - join two lists, this is designed for stacks
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- */
-static inline void list_splice(const struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list))
- __list_splice(list, head, head->next);
-}
-
-/**
- * list_splice_tail - join two lists, each list being a queue
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- */
-static inline void list_splice_tail(struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list))
- __list_splice(list, head->prev, head);
-}
-
-/**
- * list_splice_init - join two lists and reinitialise the emptied list.
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- *
- * The list at @list is reinitialised
- */
-static inline void list_splice_init(struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list)) {
- __list_splice(list, head, head->next);
- INIT_LIST_HEAD(list);
- }
-}
-
-/**
- * list_splice_tail_init - join two lists and reinitialise the emptied list
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- *
- * Each of the lists is a queue.
- * The list at @list is reinitialised
- */
-static inline void list_splice_tail_init(struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list)) {
- __list_splice(list, head->prev, head);
- INIT_LIST_HEAD(list);
- }
-}
-
/**
* list_entry - get the struct for this entry
* @ptr: the &struct list_head pointer.
@@ -353,56 +92,6 @@ static inline void list_splice_tail_init(struct list_head *list,
#define list_entry(ptr, type, member) \
container_of(ptr, type, member)
-/**
- * list_first_entry - get the first element from a list
- * @ptr: the list head to take the element from.
- * @type: the type of the struct this is embedded in.
- * @member: the name of the list_head within the struct.
- *
- * Note, that list is expected to be not empty.
- */
-#define list_first_entry(ptr, type, member) \
- list_entry((ptr)->next, type, member)
-
-/**
- * list_for_each - iterate over a list
- * @pos: the &struct list_head to use as a loop cursor.
- * @head: the head for your list.
- */
-#define list_for_each(pos, head) \
- for (pos = (head)->next; prefetch(pos->next), pos != (head); \
- pos = pos->next)
-
-/**
- * list_for_each_prev - iterate over a list backwards
- * @pos: the &struct list_head to use as a loop cursor.
- * @head: the head for your list.
- */
-#define list_for_each_prev(pos, head) \
- for (pos = (head)->prev; prefetch(pos->prev), pos != (head); \
- pos = pos->prev)
-
-/**
- * list_for_each_safe - iterate over a list safe against removal of list entry
- * @pos: the &struct list_head to use as a loop cursor.
- * @n: another &struct list_head to use as temporary storage
- * @head: the head for your list.
- */
-#define list_for_each_safe(pos, n, head) \
- for (pos = (head)->next, n = pos->next; pos != (head); \
- pos = n, n = pos->next)
-
-/**
- * list_for_each_prev_safe - iterate over a list backwards safe against removal of list entry
- * @pos: the &struct list_head to use as a loop cursor.
- * @n: another &struct list_head to use as temporary storage
- * @head: the head for your list.
- */
-#define list_for_each_prev_safe(pos, n, head) \
- for (pos = (head)->prev, n = pos->prev; \
- prefetch(pos->prev), pos != (head); \
- pos = n, n = pos->prev)
-
/**
* list_for_each_entry - iterate over list of given type
* @pos: the type * to use as a loop cursor.
@@ -414,128 +103,6 @@ static inline void list_splice_tail_init(struct list_head *list,
&pos->member != (head); \
pos = list_entry(pos->member.next, typeof(*pos), member))
-/**
- * list_for_each_entry_reverse - iterate backwards over list of given type.
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- */
-#define list_for_each_entry_reverse(pos, head, member) \
- for (pos = list_entry((head)->prev, typeof(*pos), member); \
- prefetch(pos->member.prev), &pos->member != (head); \
- pos = list_entry(pos->member.prev, typeof(*pos), member))
-
-/**
- * list_prepare_entry - prepare a pos entry for use in list_for_each_entry_continue()
- * @pos: the type * to use as a start point
- * @head: the head of the list
- * @member: the name of the list_head within the struct.
- *
- * Prepares a pos entry for use as a start point in list_for_each_entry_continue().
- */
-#define list_prepare_entry(pos, head, member) \
- ((pos) ? : list_entry(head, typeof(*pos), member))
-
-/**
- * list_for_each_entry_continue - continue iteration over list of given type
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Continue to iterate over list of given type, continuing after
- * the current position.
- */
-#define list_for_each_entry_continue(pos, head, member) \
- for (pos = list_entry(pos->member.next, typeof(*pos), member); \
- prefetch(pos->member.next), &pos->member != (head); \
- pos = list_entry(pos->member.next, typeof(*pos), member))
-
-/**
- * list_for_each_entry_continue_reverse - iterate backwards from the given point
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Start to iterate over list of given type backwards, continuing after
- * the current position.
- */
-#define list_for_each_entry_continue_reverse(pos, head, member) \
- for (pos = list_entry(pos->member.prev, typeof(*pos), member); \
- prefetch(pos->member.prev), &pos->member != (head); \
- pos = list_entry(pos->member.prev, typeof(*pos), member))
-
-/**
- * list_for_each_entry_from - iterate over list of given type from the current point
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate over list of given type, continuing from current position.
- */
-#define list_for_each_entry_from(pos, head, member) \
- for (; prefetch(pos->member.next), &pos->member != (head); \
- pos = list_entry(pos->member.next, typeof(*pos), member))
-
-/**
- * list_for_each_entry_safe - iterate over list of given type safe against removal of list entry
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- */
-#define list_for_each_entry_safe(pos, n, head, member) \
- for (pos = list_entry((head)->next, typeof(*pos), member), \
- n = list_entry(pos->member.next, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.next, typeof(*n), member))
-
-/**
- * list_for_each_entry_safe_continue
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate over list of given type, continuing after current point,
- * safe against removal of list entry.
- */
-#define list_for_each_entry_safe_continue(pos, n, head, member) \
- for (pos = list_entry(pos->member.next, typeof(*pos), member), \
- n = list_entry(pos->member.next, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.next, typeof(*n), member))
-
-/**
- * list_for_each_entry_safe_from
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate over list of given type from current point, safe against
- * removal of list entry.
- */
-#define list_for_each_entry_safe_from(pos, n, head, member) \
- for (n = list_entry(pos->member.next, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.next, typeof(*n), member))
-
-/**
- * list_for_each_entry_safe_reverse
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate backwards over list of given type, safe against removal
- * of list entry.
- */
-#define list_for_each_entry_safe_reverse(pos, n, head, member) \
- for (pos = list_entry((head)->prev, typeof(*pos), member), \
- n = list_entry(pos->member.prev, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.prev, typeof(*n), member))
-
struct offset {
struct list_head list;
unsigned offset;
--
2.16.2.395.g2e18187dfd-goog
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] drm/radeon/mkregtable: Delete unused list functions and macros
@ 2018-02-28 22:17 ` Matthias Kaehlcke
0 siblings, 0 replies; 5+ messages in thread
From: Matthias Kaehlcke @ 2018-02-28 22:17 UTC (permalink / raw)
To: Alex Deucher, Christian König, David Airlie
Cc: Guenter Roeck, Matthias Kaehlcke, dri-devel, amd-gfx, linux-kernel
The util mkregtable includes a copy of the kernel API for linked lists,
only a small subset of it is used. Delete the unused functions and macros.
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
---
drivers/gpu/drm/radeon/mkregtable.c | 433 ----------------------------
1 file changed, 433 deletions(-)
diff --git a/drivers/gpu/drm/radeon/mkregtable.c b/drivers/gpu/drm/radeon/mkregtable.c
index c21d8fa591ef..ba704633b072 100644
--- a/drivers/gpu/drm/radeon/mkregtable.c
+++ b/drivers/gpu/drm/radeon/mkregtable.c
@@ -43,10 +43,6 @@ struct list_head {
struct list_head *next, *prev;
};
-#define LIST_HEAD_INIT(name) { &(name), &(name) }
-
-#define LIST_HEAD(name) \
- struct list_head name = LIST_HEAD_INIT(name)
static inline void INIT_LIST_HEAD(struct list_head *list)
{
@@ -74,19 +70,6 @@ extern void __list_add(struct list_head *new,
struct list_head *prev, struct list_head *next);
#endif
-/**
- * list_add - add a new entry
- * @new: new entry to be added
- * @head: list head to add it after
- *
- * Insert a new entry after the specified head.
- * This is good for implementing stacks.
- */
-static inline void list_add(struct list_head *new, struct list_head *head)
-{
- __list_add(new, head, head->next);
-}
-
/**
* list_add_tail - add a new entry
* @new: new entry to be added
@@ -100,250 +83,6 @@ static inline void list_add_tail(struct list_head *new, struct list_head *head)
__list_add(new, head->prev, head);
}
-/*
- * Delete a list entry by making the prev/next entries
- * point to each other.
- *
- * This is only for internal list manipulation where we know
- * the prev/next entries already!
- */
-static inline void __list_del(struct list_head *prev, struct list_head *next)
-{
- next->prev = prev;
- prev->next = next;
-}
-
-/**
- * list_del - deletes entry from list.
- * @entry: the element to delete from the list.
- * Note: list_empty() on entry does not return true after this, the entry is
- * in an undefined state.
- */
-#ifndef CONFIG_DEBUG_LIST
-static inline void list_del(struct list_head *entry)
-{
- __list_del(entry->prev, entry->next);
- entry->next = (void *)0xDEADBEEF;
- entry->prev = (void *)0xBEEFDEAD;
-}
-#else
-extern void list_del(struct list_head *entry);
-#endif
-
-/**
- * list_replace - replace old entry by new one
- * @old : the element to be replaced
- * @new : the new element to insert
- *
- * If @old was empty, it will be overwritten.
- */
-static inline void list_replace(struct list_head *old, struct list_head *new)
-{
- new->next = old->next;
- new->next->prev = new;
- new->prev = old->prev;
- new->prev->next = new;
-}
-
-static inline void list_replace_init(struct list_head *old,
- struct list_head *new)
-{
- list_replace(old, new);
- INIT_LIST_HEAD(old);
-}
-
-/**
- * list_del_init - deletes entry from list and reinitialize it.
- * @entry: the element to delete from the list.
- */
-static inline void list_del_init(struct list_head *entry)
-{
- __list_del(entry->prev, entry->next);
- INIT_LIST_HEAD(entry);
-}
-
-/**
- * list_move - delete from one list and add as another's head
- * @list: the entry to move
- * @head: the head that will precede our entry
- */
-static inline void list_move(struct list_head *list, struct list_head *head)
-{
- __list_del(list->prev, list->next);
- list_add(list, head);
-}
-
-/**
- * list_move_tail - delete from one list and add as another's tail
- * @list: the entry to move
- * @head: the head that will follow our entry
- */
-static inline void list_move_tail(struct list_head *list,
- struct list_head *head)
-{
- __list_del(list->prev, list->next);
- list_add_tail(list, head);
-}
-
-/**
- * list_is_last - tests whether @list is the last entry in list @head
- * @list: the entry to test
- * @head: the head of the list
- */
-static inline int list_is_last(const struct list_head *list,
- const struct list_head *head)
-{
- return list->next == head;
-}
-
-/**
- * list_empty - tests whether a list is empty
- * @head: the list to test.
- */
-static inline int list_empty(const struct list_head *head)
-{
- return head->next == head;
-}
-
-/**
- * list_empty_careful - tests whether a list is empty and not being modified
- * @head: the list to test
- *
- * Description:
- * tests whether a list is empty _and_ checks that no other CPU might be
- * in the process of modifying either member (next or prev)
- *
- * NOTE: using list_empty_careful() without synchronization
- * can only be safe if the only activity that can happen
- * to the list entry is list_del_init(). Eg. it cannot be used
- * if another CPU could re-list_add() it.
- */
-static inline int list_empty_careful(const struct list_head *head)
-{
- struct list_head *next = head->next;
- return (next == head) && (next == head->prev);
-}
-
-/**
- * list_is_singular - tests whether a list has just one entry.
- * @head: the list to test.
- */
-static inline int list_is_singular(const struct list_head *head)
-{
- return !list_empty(head) && (head->next == head->prev);
-}
-
-static inline void __list_cut_position(struct list_head *list,
- struct list_head *head,
- struct list_head *entry)
-{
- struct list_head *new_first = entry->next;
- list->next = head->next;
- list->next->prev = list;
- list->prev = entry;
- entry->next = list;
- head->next = new_first;
- new_first->prev = head;
-}
-
-/**
- * list_cut_position - cut a list into two
- * @list: a new list to add all removed entries
- * @head: a list with entries
- * @entry: an entry within head, could be the head itself
- * and if so we won't cut the list
- *
- * This helper moves the initial part of @head, up to and
- * including @entry, from @head to @list. You should
- * pass on @entry an element you know is on @head. @list
- * should be an empty list or a list you do not care about
- * losing its data.
- *
- */
-static inline void list_cut_position(struct list_head *list,
- struct list_head *head,
- struct list_head *entry)
-{
- if (list_empty(head))
- return;
- if (list_is_singular(head) && (head->next != entry && head != entry))
- return;
- if (entry == head)
- INIT_LIST_HEAD(list);
- else
- __list_cut_position(list, head, entry);
-}
-
-static inline void __list_splice(const struct list_head *list,
- struct list_head *prev, struct list_head *next)
-{
- struct list_head *first = list->next;
- struct list_head *last = list->prev;
-
- first->prev = prev;
- prev->next = first;
-
- last->next = next;
- next->prev = last;
-}
-
-/**
- * list_splice - join two lists, this is designed for stacks
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- */
-static inline void list_splice(const struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list))
- __list_splice(list, head, head->next);
-}
-
-/**
- * list_splice_tail - join two lists, each list being a queue
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- */
-static inline void list_splice_tail(struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list))
- __list_splice(list, head->prev, head);
-}
-
-/**
- * list_splice_init - join two lists and reinitialise the emptied list.
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- *
- * The list at @list is reinitialised
- */
-static inline void list_splice_init(struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list)) {
- __list_splice(list, head, head->next);
- INIT_LIST_HEAD(list);
- }
-}
-
-/**
- * list_splice_tail_init - join two lists and reinitialise the emptied list
- * @list: the new list to add.
- * @head: the place to add it in the first list.
- *
- * Each of the lists is a queue.
- * The list at @list is reinitialised
- */
-static inline void list_splice_tail_init(struct list_head *list,
- struct list_head *head)
-{
- if (!list_empty(list)) {
- __list_splice(list, head->prev, head);
- INIT_LIST_HEAD(list);
- }
-}
-
/**
* list_entry - get the struct for this entry
* @ptr: the &struct list_head pointer.
@@ -353,56 +92,6 @@ static inline void list_splice_tail_init(struct list_head *list,
#define list_entry(ptr, type, member) \
container_of(ptr, type, member)
-/**
- * list_first_entry - get the first element from a list
- * @ptr: the list head to take the element from.
- * @type: the type of the struct this is embedded in.
- * @member: the name of the list_head within the struct.
- *
- * Note, that list is expected to be not empty.
- */
-#define list_first_entry(ptr, type, member) \
- list_entry((ptr)->next, type, member)
-
-/**
- * list_for_each - iterate over a list
- * @pos: the &struct list_head to use as a loop cursor.
- * @head: the head for your list.
- */
-#define list_for_each(pos, head) \
- for (pos = (head)->next; prefetch(pos->next), pos != (head); \
- pos = pos->next)
-
-/**
- * list_for_each_prev - iterate over a list backwards
- * @pos: the &struct list_head to use as a loop cursor.
- * @head: the head for your list.
- */
-#define list_for_each_prev(pos, head) \
- for (pos = (head)->prev; prefetch(pos->prev), pos != (head); \
- pos = pos->prev)
-
-/**
- * list_for_each_safe - iterate over a list safe against removal of list entry
- * @pos: the &struct list_head to use as a loop cursor.
- * @n: another &struct list_head to use as temporary storage
- * @head: the head for your list.
- */
-#define list_for_each_safe(pos, n, head) \
- for (pos = (head)->next, n = pos->next; pos != (head); \
- pos = n, n = pos->next)
-
-/**
- * list_for_each_prev_safe - iterate over a list backwards safe against removal of list entry
- * @pos: the &struct list_head to use as a loop cursor.
- * @n: another &struct list_head to use as temporary storage
- * @head: the head for your list.
- */
-#define list_for_each_prev_safe(pos, n, head) \
- for (pos = (head)->prev, n = pos->prev; \
- prefetch(pos->prev), pos != (head); \
- pos = n, n = pos->prev)
-
/**
* list_for_each_entry - iterate over list of given type
* @pos: the type * to use as a loop cursor.
@@ -414,128 +103,6 @@ static inline void list_splice_tail_init(struct list_head *list,
&pos->member != (head); \
pos = list_entry(pos->member.next, typeof(*pos), member))
-/**
- * list_for_each_entry_reverse - iterate backwards over list of given type.
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- */
-#define list_for_each_entry_reverse(pos, head, member) \
- for (pos = list_entry((head)->prev, typeof(*pos), member); \
- prefetch(pos->member.prev), &pos->member != (head); \
- pos = list_entry(pos->member.prev, typeof(*pos), member))
-
-/**
- * list_prepare_entry - prepare a pos entry for use in list_for_each_entry_continue()
- * @pos: the type * to use as a start point
- * @head: the head of the list
- * @member: the name of the list_head within the struct.
- *
- * Prepares a pos entry for use as a start point in list_for_each_entry_continue().
- */
-#define list_prepare_entry(pos, head, member) \
- ((pos) ? : list_entry(head, typeof(*pos), member))
-
-/**
- * list_for_each_entry_continue - continue iteration over list of given type
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Continue to iterate over list of given type, continuing after
- * the current position.
- */
-#define list_for_each_entry_continue(pos, head, member) \
- for (pos = list_entry(pos->member.next, typeof(*pos), member); \
- prefetch(pos->member.next), &pos->member != (head); \
- pos = list_entry(pos->member.next, typeof(*pos), member))
-
-/**
- * list_for_each_entry_continue_reverse - iterate backwards from the given point
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Start to iterate over list of given type backwards, continuing after
- * the current position.
- */
-#define list_for_each_entry_continue_reverse(pos, head, member) \
- for (pos = list_entry(pos->member.prev, typeof(*pos), member); \
- prefetch(pos->member.prev), &pos->member != (head); \
- pos = list_entry(pos->member.prev, typeof(*pos), member))
-
-/**
- * list_for_each_entry_from - iterate over list of given type from the current point
- * @pos: the type * to use as a loop cursor.
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate over list of given type, continuing from current position.
- */
-#define list_for_each_entry_from(pos, head, member) \
- for (; prefetch(pos->member.next), &pos->member != (head); \
- pos = list_entry(pos->member.next, typeof(*pos), member))
-
-/**
- * list_for_each_entry_safe - iterate over list of given type safe against removal of list entry
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- */
-#define list_for_each_entry_safe(pos, n, head, member) \
- for (pos = list_entry((head)->next, typeof(*pos), member), \
- n = list_entry(pos->member.next, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.next, typeof(*n), member))
-
-/**
- * list_for_each_entry_safe_continue
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate over list of given type, continuing after current point,
- * safe against removal of list entry.
- */
-#define list_for_each_entry_safe_continue(pos, n, head, member) \
- for (pos = list_entry(pos->member.next, typeof(*pos), member), \
- n = list_entry(pos->member.next, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.next, typeof(*n), member))
-
-/**
- * list_for_each_entry_safe_from
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate over list of given type from current point, safe against
- * removal of list entry.
- */
-#define list_for_each_entry_safe_from(pos, n, head, member) \
- for (n = list_entry(pos->member.next, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.next, typeof(*n), member))
-
-/**
- * list_for_each_entry_safe_reverse
- * @pos: the type * to use as a loop cursor.
- * @n: another type * to use as temporary storage
- * @head: the head for your list.
- * @member: the name of the list_head within the struct.
- *
- * Iterate backwards over list of given type, safe against removal
- * of list entry.
- */
-#define list_for_each_entry_safe_reverse(pos, n, head, member) \
- for (pos = list_entry((head)->prev, typeof(*pos), member), \
- n = list_entry(pos->member.prev, typeof(*pos), member); \
- &pos->member != (head); \
- pos = n, n = list_entry(n->member.prev, typeof(*n), member))
-
struct offset {
struct list_head list;
unsigned offset;
--
2.16.2.395.g2e18187dfd-goog
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/radeon/mkregtable: Delete unused list functions and macros
2018-02-28 22:17 ` Matthias Kaehlcke
(?)
@ 2018-03-01 15:41 ` Guenter Roeck
2018-03-01 16:27 ` Alex Deucher
-1 siblings, 1 reply; 5+ messages in thread
From: Guenter Roeck @ 2018-03-01 15:41 UTC (permalink / raw)
To: Matthias Kaehlcke
Cc: Alex Deucher, Christian König, David Airlie, amd-gfx list,
Maling list - DRI developers, linux-kernel, Guenter Roeck
On Wed, Feb 28, 2018 at 2:17 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
> The util mkregtable includes a copy of the kernel API for linked lists,
> only a small subset of it is used. Delete the unused functions and macros.
>
> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Guenter Roeck <groeck@chromium.org>
> ---
> drivers/gpu/drm/radeon/mkregtable.c | 433 ----------------------------
> 1 file changed, 433 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/mkregtable.c b/drivers/gpu/drm/radeon/mkregtable.c
> index c21d8fa591ef..ba704633b072 100644
> --- a/drivers/gpu/drm/radeon/mkregtable.c
> +++ b/drivers/gpu/drm/radeon/mkregtable.c
> @@ -43,10 +43,6 @@ struct list_head {
> struct list_head *next, *prev;
> };
>
> -#define LIST_HEAD_INIT(name) { &(name), &(name) }
> -
> -#define LIST_HEAD(name) \
> - struct list_head name = LIST_HEAD_INIT(name)
>
> static inline void INIT_LIST_HEAD(struct list_head *list)
> {
> @@ -74,19 +70,6 @@ extern void __list_add(struct list_head *new,
> struct list_head *prev, struct list_head *next);
> #endif
>
> -/**
> - * list_add - add a new entry
> - * @new: new entry to be added
> - * @head: list head to add it after
> - *
> - * Insert a new entry after the specified head.
> - * This is good for implementing stacks.
> - */
> -static inline void list_add(struct list_head *new, struct list_head *head)
> -{
> - __list_add(new, head, head->next);
> -}
> -
> /**
> * list_add_tail - add a new entry
> * @new: new entry to be added
> @@ -100,250 +83,6 @@ static inline void list_add_tail(struct list_head *new, struct list_head *head)
> __list_add(new, head->prev, head);
> }
>
> -/*
> - * Delete a list entry by making the prev/next entries
> - * point to each other.
> - *
> - * This is only for internal list manipulation where we know
> - * the prev/next entries already!
> - */
> -static inline void __list_del(struct list_head *prev, struct list_head *next)
> -{
> - next->prev = prev;
> - prev->next = next;
> -}
> -
> -/**
> - * list_del - deletes entry from list.
> - * @entry: the element to delete from the list.
> - * Note: list_empty() on entry does not return true after this, the entry is
> - * in an undefined state.
> - */
> -#ifndef CONFIG_DEBUG_LIST
> -static inline void list_del(struct list_head *entry)
> -{
> - __list_del(entry->prev, entry->next);
> - entry->next = (void *)0xDEADBEEF;
> - entry->prev = (void *)0xBEEFDEAD;
> -}
> -#else
> -extern void list_del(struct list_head *entry);
> -#endif
> -
> -/**
> - * list_replace - replace old entry by new one
> - * @old : the element to be replaced
> - * @new : the new element to insert
> - *
> - * If @old was empty, it will be overwritten.
> - */
> -static inline void list_replace(struct list_head *old, struct list_head *new)
> -{
> - new->next = old->next;
> - new->next->prev = new;
> - new->prev = old->prev;
> - new->prev->next = new;
> -}
> -
> -static inline void list_replace_init(struct list_head *old,
> - struct list_head *new)
> -{
> - list_replace(old, new);
> - INIT_LIST_HEAD(old);
> -}
> -
> -/**
> - * list_del_init - deletes entry from list and reinitialize it.
> - * @entry: the element to delete from the list.
> - */
> -static inline void list_del_init(struct list_head *entry)
> -{
> - __list_del(entry->prev, entry->next);
> - INIT_LIST_HEAD(entry);
> -}
> -
> -/**
> - * list_move - delete from one list and add as another's head
> - * @list: the entry to move
> - * @head: the head that will precede our entry
> - */
> -static inline void list_move(struct list_head *list, struct list_head *head)
> -{
> - __list_del(list->prev, list->next);
> - list_add(list, head);
> -}
> -
> -/**
> - * list_move_tail - delete from one list and add as another's tail
> - * @list: the entry to move
> - * @head: the head that will follow our entry
> - */
> -static inline void list_move_tail(struct list_head *list,
> - struct list_head *head)
> -{
> - __list_del(list->prev, list->next);
> - list_add_tail(list, head);
> -}
> -
> -/**
> - * list_is_last - tests whether @list is the last entry in list @head
> - * @list: the entry to test
> - * @head: the head of the list
> - */
> -static inline int list_is_last(const struct list_head *list,
> - const struct list_head *head)
> -{
> - return list->next == head;
> -}
> -
> -/**
> - * list_empty - tests whether a list is empty
> - * @head: the list to test.
> - */
> -static inline int list_empty(const struct list_head *head)
> -{
> - return head->next == head;
> -}
> -
> -/**
> - * list_empty_careful - tests whether a list is empty and not being modified
> - * @head: the list to test
> - *
> - * Description:
> - * tests whether a list is empty _and_ checks that no other CPU might be
> - * in the process of modifying either member (next or prev)
> - *
> - * NOTE: using list_empty_careful() without synchronization
> - * can only be safe if the only activity that can happen
> - * to the list entry is list_del_init(). Eg. it cannot be used
> - * if another CPU could re-list_add() it.
> - */
> -static inline int list_empty_careful(const struct list_head *head)
> -{
> - struct list_head *next = head->next;
> - return (next == head) && (next == head->prev);
> -}
> -
> -/**
> - * list_is_singular - tests whether a list has just one entry.
> - * @head: the list to test.
> - */
> -static inline int list_is_singular(const struct list_head *head)
> -{
> - return !list_empty(head) && (head->next == head->prev);
> -}
> -
> -static inline void __list_cut_position(struct list_head *list,
> - struct list_head *head,
> - struct list_head *entry)
> -{
> - struct list_head *new_first = entry->next;
> - list->next = head->next;
> - list->next->prev = list;
> - list->prev = entry;
> - entry->next = list;
> - head->next = new_first;
> - new_first->prev = head;
> -}
> -
> -/**
> - * list_cut_position - cut a list into two
> - * @list: a new list to add all removed entries
> - * @head: a list with entries
> - * @entry: an entry within head, could be the head itself
> - * and if so we won't cut the list
> - *
> - * This helper moves the initial part of @head, up to and
> - * including @entry, from @head to @list. You should
> - * pass on @entry an element you know is on @head. @list
> - * should be an empty list or a list you do not care about
> - * losing its data.
> - *
> - */
> -static inline void list_cut_position(struct list_head *list,
> - struct list_head *head,
> - struct list_head *entry)
> -{
> - if (list_empty(head))
> - return;
> - if (list_is_singular(head) && (head->next != entry && head != entry))
> - return;
> - if (entry == head)
> - INIT_LIST_HEAD(list);
> - else
> - __list_cut_position(list, head, entry);
> -}
> -
> -static inline void __list_splice(const struct list_head *list,
> - struct list_head *prev, struct list_head *next)
> -{
> - struct list_head *first = list->next;
> - struct list_head *last = list->prev;
> -
> - first->prev = prev;
> - prev->next = first;
> -
> - last->next = next;
> - next->prev = last;
> -}
> -
> -/**
> - * list_splice - join two lists, this is designed for stacks
> - * @list: the new list to add.
> - * @head: the place to add it in the first list.
> - */
> -static inline void list_splice(const struct list_head *list,
> - struct list_head *head)
> -{
> - if (!list_empty(list))
> - __list_splice(list, head, head->next);
> -}
> -
> -/**
> - * list_splice_tail - join two lists, each list being a queue
> - * @list: the new list to add.
> - * @head: the place to add it in the first list.
> - */
> -static inline void list_splice_tail(struct list_head *list,
> - struct list_head *head)
> -{
> - if (!list_empty(list))
> - __list_splice(list, head->prev, head);
> -}
> -
> -/**
> - * list_splice_init - join two lists and reinitialise the emptied list.
> - * @list: the new list to add.
> - * @head: the place to add it in the first list.
> - *
> - * The list at @list is reinitialised
> - */
> -static inline void list_splice_init(struct list_head *list,
> - struct list_head *head)
> -{
> - if (!list_empty(list)) {
> - __list_splice(list, head, head->next);
> - INIT_LIST_HEAD(list);
> - }
> -}
> -
> -/**
> - * list_splice_tail_init - join two lists and reinitialise the emptied list
> - * @list: the new list to add.
> - * @head: the place to add it in the first list.
> - *
> - * Each of the lists is a queue.
> - * The list at @list is reinitialised
> - */
> -static inline void list_splice_tail_init(struct list_head *list,
> - struct list_head *head)
> -{
> - if (!list_empty(list)) {
> - __list_splice(list, head->prev, head);
> - INIT_LIST_HEAD(list);
> - }
> -}
> -
> /**
> * list_entry - get the struct for this entry
> * @ptr: the &struct list_head pointer.
> @@ -353,56 +92,6 @@ static inline void list_splice_tail_init(struct list_head *list,
> #define list_entry(ptr, type, member) \
> container_of(ptr, type, member)
>
> -/**
> - * list_first_entry - get the first element from a list
> - * @ptr: the list head to take the element from.
> - * @type: the type of the struct this is embedded in.
> - * @member: the name of the list_head within the struct.
> - *
> - * Note, that list is expected to be not empty.
> - */
> -#define list_first_entry(ptr, type, member) \
> - list_entry((ptr)->next, type, member)
> -
> -/**
> - * list_for_each - iterate over a list
> - * @pos: the &struct list_head to use as a loop cursor.
> - * @head: the head for your list.
> - */
> -#define list_for_each(pos, head) \
> - for (pos = (head)->next; prefetch(pos->next), pos != (head); \
> - pos = pos->next)
> -
> -/**
> - * list_for_each_prev - iterate over a list backwards
> - * @pos: the &struct list_head to use as a loop cursor.
> - * @head: the head for your list.
> - */
> -#define list_for_each_prev(pos, head) \
> - for (pos = (head)->prev; prefetch(pos->prev), pos != (head); \
> - pos = pos->prev)
> -
> -/**
> - * list_for_each_safe - iterate over a list safe against removal of list entry
> - * @pos: the &struct list_head to use as a loop cursor.
> - * @n: another &struct list_head to use as temporary storage
> - * @head: the head for your list.
> - */
> -#define list_for_each_safe(pos, n, head) \
> - for (pos = (head)->next, n = pos->next; pos != (head); \
> - pos = n, n = pos->next)
> -
> -/**
> - * list_for_each_prev_safe - iterate over a list backwards safe against removal of list entry
> - * @pos: the &struct list_head to use as a loop cursor.
> - * @n: another &struct list_head to use as temporary storage
> - * @head: the head for your list.
> - */
> -#define list_for_each_prev_safe(pos, n, head) \
> - for (pos = (head)->prev, n = pos->prev; \
> - prefetch(pos->prev), pos != (head); \
> - pos = n, n = pos->prev)
> -
> /**
> * list_for_each_entry - iterate over list of given type
> * @pos: the type * to use as a loop cursor.
> @@ -414,128 +103,6 @@ static inline void list_splice_tail_init(struct list_head *list,
> &pos->member != (head); \
> pos = list_entry(pos->member.next, typeof(*pos), member))
>
> -/**
> - * list_for_each_entry_reverse - iterate backwards over list of given type.
> - * @pos: the type * to use as a loop cursor.
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - */
> -#define list_for_each_entry_reverse(pos, head, member) \
> - for (pos = list_entry((head)->prev, typeof(*pos), member); \
> - prefetch(pos->member.prev), &pos->member != (head); \
> - pos = list_entry(pos->member.prev, typeof(*pos), member))
> -
> -/**
> - * list_prepare_entry - prepare a pos entry for use in list_for_each_entry_continue()
> - * @pos: the type * to use as a start point
> - * @head: the head of the list
> - * @member: the name of the list_head within the struct.
> - *
> - * Prepares a pos entry for use as a start point in list_for_each_entry_continue().
> - */
> -#define list_prepare_entry(pos, head, member) \
> - ((pos) ? : list_entry(head, typeof(*pos), member))
> -
> -/**
> - * list_for_each_entry_continue - continue iteration over list of given type
> - * @pos: the type * to use as a loop cursor.
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - *
> - * Continue to iterate over list of given type, continuing after
> - * the current position.
> - */
> -#define list_for_each_entry_continue(pos, head, member) \
> - for (pos = list_entry(pos->member.next, typeof(*pos), member); \
> - prefetch(pos->member.next), &pos->member != (head); \
> - pos = list_entry(pos->member.next, typeof(*pos), member))
> -
> -/**
> - * list_for_each_entry_continue_reverse - iterate backwards from the given point
> - * @pos: the type * to use as a loop cursor.
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - *
> - * Start to iterate over list of given type backwards, continuing after
> - * the current position.
> - */
> -#define list_for_each_entry_continue_reverse(pos, head, member) \
> - for (pos = list_entry(pos->member.prev, typeof(*pos), member); \
> - prefetch(pos->member.prev), &pos->member != (head); \
> - pos = list_entry(pos->member.prev, typeof(*pos), member))
> -
> -/**
> - * list_for_each_entry_from - iterate over list of given type from the current point
> - * @pos: the type * to use as a loop cursor.
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - *
> - * Iterate over list of given type, continuing from current position.
> - */
> -#define list_for_each_entry_from(pos, head, member) \
> - for (; prefetch(pos->member.next), &pos->member != (head); \
> - pos = list_entry(pos->member.next, typeof(*pos), member))
> -
> -/**
> - * list_for_each_entry_safe - iterate over list of given type safe against removal of list entry
> - * @pos: the type * to use as a loop cursor.
> - * @n: another type * to use as temporary storage
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - */
> -#define list_for_each_entry_safe(pos, n, head, member) \
> - for (pos = list_entry((head)->next, typeof(*pos), member), \
> - n = list_entry(pos->member.next, typeof(*pos), member); \
> - &pos->member != (head); \
> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
> -
> -/**
> - * list_for_each_entry_safe_continue
> - * @pos: the type * to use as a loop cursor.
> - * @n: another type * to use as temporary storage
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - *
> - * Iterate over list of given type, continuing after current point,
> - * safe against removal of list entry.
> - */
> -#define list_for_each_entry_safe_continue(pos, n, head, member) \
> - for (pos = list_entry(pos->member.next, typeof(*pos), member), \
> - n = list_entry(pos->member.next, typeof(*pos), member); \
> - &pos->member != (head); \
> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
> -
> -/**
> - * list_for_each_entry_safe_from
> - * @pos: the type * to use as a loop cursor.
> - * @n: another type * to use as temporary storage
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - *
> - * Iterate over list of given type from current point, safe against
> - * removal of list entry.
> - */
> -#define list_for_each_entry_safe_from(pos, n, head, member) \
> - for (n = list_entry(pos->member.next, typeof(*pos), member); \
> - &pos->member != (head); \
> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
> -
> -/**
> - * list_for_each_entry_safe_reverse
> - * @pos: the type * to use as a loop cursor.
> - * @n: another type * to use as temporary storage
> - * @head: the head for your list.
> - * @member: the name of the list_head within the struct.
> - *
> - * Iterate backwards over list of given type, safe against removal
> - * of list entry.
> - */
> -#define list_for_each_entry_safe_reverse(pos, n, head, member) \
> - for (pos = list_entry((head)->prev, typeof(*pos), member), \
> - n = list_entry(pos->member.prev, typeof(*pos), member); \
> - &pos->member != (head); \
> - pos = n, n = list_entry(n->member.prev, typeof(*n), member))
> -
> struct offset {
> struct list_head list;
> unsigned offset;
> --
> 2.16.2.395.g2e18187dfd-goog
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/radeon/mkregtable: Delete unused list functions and macros
@ 2018-03-01 16:27 ` Alex Deucher
0 siblings, 0 replies; 5+ messages in thread
From: Alex Deucher @ 2018-03-01 16:27 UTC (permalink / raw)
To: Guenter Roeck
Cc: Matthias Kaehlcke, David Airlie, linux-kernel,
Maling list - DRI developers, Guenter Roeck, amd-gfx list,
Alex Deucher, Christian König
On Thu, Mar 1, 2018 at 10:41 AM, Guenter Roeck <groeck@google.com> wrote:
> On Wed, Feb 28, 2018 at 2:17 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
>> The util mkregtable includes a copy of the kernel API for linked lists,
>> only a small subset of it is used. Delete the unused functions and macros.
>>
>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>
> Reviewed-by: Guenter Roeck <groeck@chromium.org>
Applied. thanks!
Alex
>
>> ---
>> drivers/gpu/drm/radeon/mkregtable.c | 433 ----------------------------
>> 1 file changed, 433 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/radeon/mkregtable.c b/drivers/gpu/drm/radeon/mkregtable.c
>> index c21d8fa591ef..ba704633b072 100644
>> --- a/drivers/gpu/drm/radeon/mkregtable.c
>> +++ b/drivers/gpu/drm/radeon/mkregtable.c
>> @@ -43,10 +43,6 @@ struct list_head {
>> struct list_head *next, *prev;
>> };
>>
>> -#define LIST_HEAD_INIT(name) { &(name), &(name) }
>> -
>> -#define LIST_HEAD(name) \
>> - struct list_head name = LIST_HEAD_INIT(name)
>>
>> static inline void INIT_LIST_HEAD(struct list_head *list)
>> {
>> @@ -74,19 +70,6 @@ extern void __list_add(struct list_head *new,
>> struct list_head *prev, struct list_head *next);
>> #endif
>>
>> -/**
>> - * list_add - add a new entry
>> - * @new: new entry to be added
>> - * @head: list head to add it after
>> - *
>> - * Insert a new entry after the specified head.
>> - * This is good for implementing stacks.
>> - */
>> -static inline void list_add(struct list_head *new, struct list_head *head)
>> -{
>> - __list_add(new, head, head->next);
>> -}
>> -
>> /**
>> * list_add_tail - add a new entry
>> * @new: new entry to be added
>> @@ -100,250 +83,6 @@ static inline void list_add_tail(struct list_head *new, struct list_head *head)
>> __list_add(new, head->prev, head);
>> }
>>
>> -/*
>> - * Delete a list entry by making the prev/next entries
>> - * point to each other.
>> - *
>> - * This is only for internal list manipulation where we know
>> - * the prev/next entries already!
>> - */
>> -static inline void __list_del(struct list_head *prev, struct list_head *next)
>> -{
>> - next->prev = prev;
>> - prev->next = next;
>> -}
>> -
>> -/**
>> - * list_del - deletes entry from list.
>> - * @entry: the element to delete from the list.
>> - * Note: list_empty() on entry does not return true after this, the entry is
>> - * in an undefined state.
>> - */
>> -#ifndef CONFIG_DEBUG_LIST
>> -static inline void list_del(struct list_head *entry)
>> -{
>> - __list_del(entry->prev, entry->next);
>> - entry->next = (void *)0xDEADBEEF;
>> - entry->prev = (void *)0xBEEFDEAD;
>> -}
>> -#else
>> -extern void list_del(struct list_head *entry);
>> -#endif
>> -
>> -/**
>> - * list_replace - replace old entry by new one
>> - * @old : the element to be replaced
>> - * @new : the new element to insert
>> - *
>> - * If @old was empty, it will be overwritten.
>> - */
>> -static inline void list_replace(struct list_head *old, struct list_head *new)
>> -{
>> - new->next = old->next;
>> - new->next->prev = new;
>> - new->prev = old->prev;
>> - new->prev->next = new;
>> -}
>> -
>> -static inline void list_replace_init(struct list_head *old,
>> - struct list_head *new)
>> -{
>> - list_replace(old, new);
>> - INIT_LIST_HEAD(old);
>> -}
>> -
>> -/**
>> - * list_del_init - deletes entry from list and reinitialize it.
>> - * @entry: the element to delete from the list.
>> - */
>> -static inline void list_del_init(struct list_head *entry)
>> -{
>> - __list_del(entry->prev, entry->next);
>> - INIT_LIST_HEAD(entry);
>> -}
>> -
>> -/**
>> - * list_move - delete from one list and add as another's head
>> - * @list: the entry to move
>> - * @head: the head that will precede our entry
>> - */
>> -static inline void list_move(struct list_head *list, struct list_head *head)
>> -{
>> - __list_del(list->prev, list->next);
>> - list_add(list, head);
>> -}
>> -
>> -/**
>> - * list_move_tail - delete from one list and add as another's tail
>> - * @list: the entry to move
>> - * @head: the head that will follow our entry
>> - */
>> -static inline void list_move_tail(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - __list_del(list->prev, list->next);
>> - list_add_tail(list, head);
>> -}
>> -
>> -/**
>> - * list_is_last - tests whether @list is the last entry in list @head
>> - * @list: the entry to test
>> - * @head: the head of the list
>> - */
>> -static inline int list_is_last(const struct list_head *list,
>> - const struct list_head *head)
>> -{
>> - return list->next == head;
>> -}
>> -
>> -/**
>> - * list_empty - tests whether a list is empty
>> - * @head: the list to test.
>> - */
>> -static inline int list_empty(const struct list_head *head)
>> -{
>> - return head->next == head;
>> -}
>> -
>> -/**
>> - * list_empty_careful - tests whether a list is empty and not being modified
>> - * @head: the list to test
>> - *
>> - * Description:
>> - * tests whether a list is empty _and_ checks that no other CPU might be
>> - * in the process of modifying either member (next or prev)
>> - *
>> - * NOTE: using list_empty_careful() without synchronization
>> - * can only be safe if the only activity that can happen
>> - * to the list entry is list_del_init(). Eg. it cannot be used
>> - * if another CPU could re-list_add() it.
>> - */
>> -static inline int list_empty_careful(const struct list_head *head)
>> -{
>> - struct list_head *next = head->next;
>> - return (next == head) && (next == head->prev);
>> -}
>> -
>> -/**
>> - * list_is_singular - tests whether a list has just one entry.
>> - * @head: the list to test.
>> - */
>> -static inline int list_is_singular(const struct list_head *head)
>> -{
>> - return !list_empty(head) && (head->next == head->prev);
>> -}
>> -
>> -static inline void __list_cut_position(struct list_head *list,
>> - struct list_head *head,
>> - struct list_head *entry)
>> -{
>> - struct list_head *new_first = entry->next;
>> - list->next = head->next;
>> - list->next->prev = list;
>> - list->prev = entry;
>> - entry->next = list;
>> - head->next = new_first;
>> - new_first->prev = head;
>> -}
>> -
>> -/**
>> - * list_cut_position - cut a list into two
>> - * @list: a new list to add all removed entries
>> - * @head: a list with entries
>> - * @entry: an entry within head, could be the head itself
>> - * and if so we won't cut the list
>> - *
>> - * This helper moves the initial part of @head, up to and
>> - * including @entry, from @head to @list. You should
>> - * pass on @entry an element you know is on @head. @list
>> - * should be an empty list or a list you do not care about
>> - * losing its data.
>> - *
>> - */
>> -static inline void list_cut_position(struct list_head *list,
>> - struct list_head *head,
>> - struct list_head *entry)
>> -{
>> - if (list_empty(head))
>> - return;
>> - if (list_is_singular(head) && (head->next != entry && head != entry))
>> - return;
>> - if (entry == head)
>> - INIT_LIST_HEAD(list);
>> - else
>> - __list_cut_position(list, head, entry);
>> -}
>> -
>> -static inline void __list_splice(const struct list_head *list,
>> - struct list_head *prev, struct list_head *next)
>> -{
>> - struct list_head *first = list->next;
>> - struct list_head *last = list->prev;
>> -
>> - first->prev = prev;
>> - prev->next = first;
>> -
>> - last->next = next;
>> - next->prev = last;
>> -}
>> -
>> -/**
>> - * list_splice - join two lists, this is designed for stacks
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - */
>> -static inline void list_splice(const struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list))
>> - __list_splice(list, head, head->next);
>> -}
>> -
>> -/**
>> - * list_splice_tail - join two lists, each list being a queue
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - */
>> -static inline void list_splice_tail(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list))
>> - __list_splice(list, head->prev, head);
>> -}
>> -
>> -/**
>> - * list_splice_init - join two lists and reinitialise the emptied list.
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - *
>> - * The list at @list is reinitialised
>> - */
>> -static inline void list_splice_init(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list)) {
>> - __list_splice(list, head, head->next);
>> - INIT_LIST_HEAD(list);
>> - }
>> -}
>> -
>> -/**
>> - * list_splice_tail_init - join two lists and reinitialise the emptied list
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - *
>> - * Each of the lists is a queue.
>> - * The list at @list is reinitialised
>> - */
>> -static inline void list_splice_tail_init(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list)) {
>> - __list_splice(list, head->prev, head);
>> - INIT_LIST_HEAD(list);
>> - }
>> -}
>> -
>> /**
>> * list_entry - get the struct for this entry
>> * @ptr: the &struct list_head pointer.
>> @@ -353,56 +92,6 @@ static inline void list_splice_tail_init(struct list_head *list,
>> #define list_entry(ptr, type, member) \
>> container_of(ptr, type, member)
>>
>> -/**
>> - * list_first_entry - get the first element from a list
>> - * @ptr: the list head to take the element from.
>> - * @type: the type of the struct this is embedded in.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Note, that list is expected to be not empty.
>> - */
>> -#define list_first_entry(ptr, type, member) \
>> - list_entry((ptr)->next, type, member)
>> -
>> -/**
>> - * list_for_each - iterate over a list
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each(pos, head) \
>> - for (pos = (head)->next; prefetch(pos->next), pos != (head); \
>> - pos = pos->next)
>> -
>> -/**
>> - * list_for_each_prev - iterate over a list backwards
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each_prev(pos, head) \
>> - for (pos = (head)->prev; prefetch(pos->prev), pos != (head); \
>> - pos = pos->prev)
>> -
>> -/**
>> - * list_for_each_safe - iterate over a list safe against removal of list entry
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @n: another &struct list_head to use as temporary storage
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each_safe(pos, n, head) \
>> - for (pos = (head)->next, n = pos->next; pos != (head); \
>> - pos = n, n = pos->next)
>> -
>> -/**
>> - * list_for_each_prev_safe - iterate over a list backwards safe against removal of list entry
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @n: another &struct list_head to use as temporary storage
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each_prev_safe(pos, n, head) \
>> - for (pos = (head)->prev, n = pos->prev; \
>> - prefetch(pos->prev), pos != (head); \
>> - pos = n, n = pos->prev)
>> -
>> /**
>> * list_for_each_entry - iterate over list of given type
>> * @pos: the type * to use as a loop cursor.
>> @@ -414,128 +103,6 @@ static inline void list_splice_tail_init(struct list_head *list,
>> &pos->member != (head); \
>> pos = list_entry(pos->member.next, typeof(*pos), member))
>>
>> -/**
>> - * list_for_each_entry_reverse - iterate backwards over list of given type.
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - */
>> -#define list_for_each_entry_reverse(pos, head, member) \
>> - for (pos = list_entry((head)->prev, typeof(*pos), member); \
>> - prefetch(pos->member.prev), &pos->member != (head); \
>> - pos = list_entry(pos->member.prev, typeof(*pos), member))
>> -
>> -/**
>> - * list_prepare_entry - prepare a pos entry for use in list_for_each_entry_continue()
>> - * @pos: the type * to use as a start point
>> - * @head: the head of the list
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Prepares a pos entry for use as a start point in list_for_each_entry_continue().
>> - */
>> -#define list_prepare_entry(pos, head, member) \
>> - ((pos) ? : list_entry(head, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_continue - continue iteration over list of given type
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Continue to iterate over list of given type, continuing after
>> - * the current position.
>> - */
>> -#define list_for_each_entry_continue(pos, head, member) \
>> - for (pos = list_entry(pos->member.next, typeof(*pos), member); \
>> - prefetch(pos->member.next), &pos->member != (head); \
>> - pos = list_entry(pos->member.next, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_continue_reverse - iterate backwards from the given point
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Start to iterate over list of given type backwards, continuing after
>> - * the current position.
>> - */
>> -#define list_for_each_entry_continue_reverse(pos, head, member) \
>> - for (pos = list_entry(pos->member.prev, typeof(*pos), member); \
>> - prefetch(pos->member.prev), &pos->member != (head); \
>> - pos = list_entry(pos->member.prev, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_from - iterate over list of given type from the current point
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate over list of given type, continuing from current position.
>> - */
>> -#define list_for_each_entry_from(pos, head, member) \
>> - for (; prefetch(pos->member.next), &pos->member != (head); \
>> - pos = list_entry(pos->member.next, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_safe - iterate over list of given type safe against removal of list entry
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - */
>> -#define list_for_each_entry_safe(pos, n, head, member) \
>> - for (pos = list_entry((head)->next, typeof(*pos), member), \
>> - n = list_entry(pos->member.next, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
>> -
>> -/**
>> - * list_for_each_entry_safe_continue
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate over list of given type, continuing after current point,
>> - * safe against removal of list entry.
>> - */
>> -#define list_for_each_entry_safe_continue(pos, n, head, member) \
>> - for (pos = list_entry(pos->member.next, typeof(*pos), member), \
>> - n = list_entry(pos->member.next, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
>> -
>> -/**
>> - * list_for_each_entry_safe_from
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate over list of given type from current point, safe against
>> - * removal of list entry.
>> - */
>> -#define list_for_each_entry_safe_from(pos, n, head, member) \
>> - for (n = list_entry(pos->member.next, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
>> -
>> -/**
>> - * list_for_each_entry_safe_reverse
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate backwards over list of given type, safe against removal
>> - * of list entry.
>> - */
>> -#define list_for_each_entry_safe_reverse(pos, n, head, member) \
>> - for (pos = list_entry((head)->prev, typeof(*pos), member), \
>> - n = list_entry(pos->member.prev, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.prev, typeof(*n), member))
>> -
>> struct offset {
>> struct list_head list;
>> unsigned offset;
>> --
>> 2.16.2.395.g2e18187dfd-goog
>>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/radeon/mkregtable: Delete unused list functions and macros
@ 2018-03-01 16:27 ` Alex Deucher
0 siblings, 0 replies; 5+ messages in thread
From: Alex Deucher @ 2018-03-01 16:27 UTC (permalink / raw)
To: Guenter Roeck
Cc: David Airlie, linux-kernel, amd-gfx list, Alex Deucher,
Matthias Kaehlcke, Maling list - DRI developers, Guenter Roeck,
Christian König
On Thu, Mar 1, 2018 at 10:41 AM, Guenter Roeck <groeck@google.com> wrote:
> On Wed, Feb 28, 2018 at 2:17 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
>> The util mkregtable includes a copy of the kernel API for linked lists,
>> only a small subset of it is used. Delete the unused functions and macros.
>>
>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>
> Reviewed-by: Guenter Roeck <groeck@chromium.org>
Applied. thanks!
Alex
>
>> ---
>> drivers/gpu/drm/radeon/mkregtable.c | 433 ----------------------------
>> 1 file changed, 433 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/radeon/mkregtable.c b/drivers/gpu/drm/radeon/mkregtable.c
>> index c21d8fa591ef..ba704633b072 100644
>> --- a/drivers/gpu/drm/radeon/mkregtable.c
>> +++ b/drivers/gpu/drm/radeon/mkregtable.c
>> @@ -43,10 +43,6 @@ struct list_head {
>> struct list_head *next, *prev;
>> };
>>
>> -#define LIST_HEAD_INIT(name) { &(name), &(name) }
>> -
>> -#define LIST_HEAD(name) \
>> - struct list_head name = LIST_HEAD_INIT(name)
>>
>> static inline void INIT_LIST_HEAD(struct list_head *list)
>> {
>> @@ -74,19 +70,6 @@ extern void __list_add(struct list_head *new,
>> struct list_head *prev, struct list_head *next);
>> #endif
>>
>> -/**
>> - * list_add - add a new entry
>> - * @new: new entry to be added
>> - * @head: list head to add it after
>> - *
>> - * Insert a new entry after the specified head.
>> - * This is good for implementing stacks.
>> - */
>> -static inline void list_add(struct list_head *new, struct list_head *head)
>> -{
>> - __list_add(new, head, head->next);
>> -}
>> -
>> /**
>> * list_add_tail - add a new entry
>> * @new: new entry to be added
>> @@ -100,250 +83,6 @@ static inline void list_add_tail(struct list_head *new, struct list_head *head)
>> __list_add(new, head->prev, head);
>> }
>>
>> -/*
>> - * Delete a list entry by making the prev/next entries
>> - * point to each other.
>> - *
>> - * This is only for internal list manipulation where we know
>> - * the prev/next entries already!
>> - */
>> -static inline void __list_del(struct list_head *prev, struct list_head *next)
>> -{
>> - next->prev = prev;
>> - prev->next = next;
>> -}
>> -
>> -/**
>> - * list_del - deletes entry from list.
>> - * @entry: the element to delete from the list.
>> - * Note: list_empty() on entry does not return true after this, the entry is
>> - * in an undefined state.
>> - */
>> -#ifndef CONFIG_DEBUG_LIST
>> -static inline void list_del(struct list_head *entry)
>> -{
>> - __list_del(entry->prev, entry->next);
>> - entry->next = (void *)0xDEADBEEF;
>> - entry->prev = (void *)0xBEEFDEAD;
>> -}
>> -#else
>> -extern void list_del(struct list_head *entry);
>> -#endif
>> -
>> -/**
>> - * list_replace - replace old entry by new one
>> - * @old : the element to be replaced
>> - * @new : the new element to insert
>> - *
>> - * If @old was empty, it will be overwritten.
>> - */
>> -static inline void list_replace(struct list_head *old, struct list_head *new)
>> -{
>> - new->next = old->next;
>> - new->next->prev = new;
>> - new->prev = old->prev;
>> - new->prev->next = new;
>> -}
>> -
>> -static inline void list_replace_init(struct list_head *old,
>> - struct list_head *new)
>> -{
>> - list_replace(old, new);
>> - INIT_LIST_HEAD(old);
>> -}
>> -
>> -/**
>> - * list_del_init - deletes entry from list and reinitialize it.
>> - * @entry: the element to delete from the list.
>> - */
>> -static inline void list_del_init(struct list_head *entry)
>> -{
>> - __list_del(entry->prev, entry->next);
>> - INIT_LIST_HEAD(entry);
>> -}
>> -
>> -/**
>> - * list_move - delete from one list and add as another's head
>> - * @list: the entry to move
>> - * @head: the head that will precede our entry
>> - */
>> -static inline void list_move(struct list_head *list, struct list_head *head)
>> -{
>> - __list_del(list->prev, list->next);
>> - list_add(list, head);
>> -}
>> -
>> -/**
>> - * list_move_tail - delete from one list and add as another's tail
>> - * @list: the entry to move
>> - * @head: the head that will follow our entry
>> - */
>> -static inline void list_move_tail(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - __list_del(list->prev, list->next);
>> - list_add_tail(list, head);
>> -}
>> -
>> -/**
>> - * list_is_last - tests whether @list is the last entry in list @head
>> - * @list: the entry to test
>> - * @head: the head of the list
>> - */
>> -static inline int list_is_last(const struct list_head *list,
>> - const struct list_head *head)
>> -{
>> - return list->next == head;
>> -}
>> -
>> -/**
>> - * list_empty - tests whether a list is empty
>> - * @head: the list to test.
>> - */
>> -static inline int list_empty(const struct list_head *head)
>> -{
>> - return head->next == head;
>> -}
>> -
>> -/**
>> - * list_empty_careful - tests whether a list is empty and not being modified
>> - * @head: the list to test
>> - *
>> - * Description:
>> - * tests whether a list is empty _and_ checks that no other CPU might be
>> - * in the process of modifying either member (next or prev)
>> - *
>> - * NOTE: using list_empty_careful() without synchronization
>> - * can only be safe if the only activity that can happen
>> - * to the list entry is list_del_init(). Eg. it cannot be used
>> - * if another CPU could re-list_add() it.
>> - */
>> -static inline int list_empty_careful(const struct list_head *head)
>> -{
>> - struct list_head *next = head->next;
>> - return (next == head) && (next == head->prev);
>> -}
>> -
>> -/**
>> - * list_is_singular - tests whether a list has just one entry.
>> - * @head: the list to test.
>> - */
>> -static inline int list_is_singular(const struct list_head *head)
>> -{
>> - return !list_empty(head) && (head->next == head->prev);
>> -}
>> -
>> -static inline void __list_cut_position(struct list_head *list,
>> - struct list_head *head,
>> - struct list_head *entry)
>> -{
>> - struct list_head *new_first = entry->next;
>> - list->next = head->next;
>> - list->next->prev = list;
>> - list->prev = entry;
>> - entry->next = list;
>> - head->next = new_first;
>> - new_first->prev = head;
>> -}
>> -
>> -/**
>> - * list_cut_position - cut a list into two
>> - * @list: a new list to add all removed entries
>> - * @head: a list with entries
>> - * @entry: an entry within head, could be the head itself
>> - * and if so we won't cut the list
>> - *
>> - * This helper moves the initial part of @head, up to and
>> - * including @entry, from @head to @list. You should
>> - * pass on @entry an element you know is on @head. @list
>> - * should be an empty list or a list you do not care about
>> - * losing its data.
>> - *
>> - */
>> -static inline void list_cut_position(struct list_head *list,
>> - struct list_head *head,
>> - struct list_head *entry)
>> -{
>> - if (list_empty(head))
>> - return;
>> - if (list_is_singular(head) && (head->next != entry && head != entry))
>> - return;
>> - if (entry == head)
>> - INIT_LIST_HEAD(list);
>> - else
>> - __list_cut_position(list, head, entry);
>> -}
>> -
>> -static inline void __list_splice(const struct list_head *list,
>> - struct list_head *prev, struct list_head *next)
>> -{
>> - struct list_head *first = list->next;
>> - struct list_head *last = list->prev;
>> -
>> - first->prev = prev;
>> - prev->next = first;
>> -
>> - last->next = next;
>> - next->prev = last;
>> -}
>> -
>> -/**
>> - * list_splice - join two lists, this is designed for stacks
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - */
>> -static inline void list_splice(const struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list))
>> - __list_splice(list, head, head->next);
>> -}
>> -
>> -/**
>> - * list_splice_tail - join two lists, each list being a queue
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - */
>> -static inline void list_splice_tail(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list))
>> - __list_splice(list, head->prev, head);
>> -}
>> -
>> -/**
>> - * list_splice_init - join two lists and reinitialise the emptied list.
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - *
>> - * The list at @list is reinitialised
>> - */
>> -static inline void list_splice_init(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list)) {
>> - __list_splice(list, head, head->next);
>> - INIT_LIST_HEAD(list);
>> - }
>> -}
>> -
>> -/**
>> - * list_splice_tail_init - join two lists and reinitialise the emptied list
>> - * @list: the new list to add.
>> - * @head: the place to add it in the first list.
>> - *
>> - * Each of the lists is a queue.
>> - * The list at @list is reinitialised
>> - */
>> -static inline void list_splice_tail_init(struct list_head *list,
>> - struct list_head *head)
>> -{
>> - if (!list_empty(list)) {
>> - __list_splice(list, head->prev, head);
>> - INIT_LIST_HEAD(list);
>> - }
>> -}
>> -
>> /**
>> * list_entry - get the struct for this entry
>> * @ptr: the &struct list_head pointer.
>> @@ -353,56 +92,6 @@ static inline void list_splice_tail_init(struct list_head *list,
>> #define list_entry(ptr, type, member) \
>> container_of(ptr, type, member)
>>
>> -/**
>> - * list_first_entry - get the first element from a list
>> - * @ptr: the list head to take the element from.
>> - * @type: the type of the struct this is embedded in.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Note, that list is expected to be not empty.
>> - */
>> -#define list_first_entry(ptr, type, member) \
>> - list_entry((ptr)->next, type, member)
>> -
>> -/**
>> - * list_for_each - iterate over a list
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each(pos, head) \
>> - for (pos = (head)->next; prefetch(pos->next), pos != (head); \
>> - pos = pos->next)
>> -
>> -/**
>> - * list_for_each_prev - iterate over a list backwards
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each_prev(pos, head) \
>> - for (pos = (head)->prev; prefetch(pos->prev), pos != (head); \
>> - pos = pos->prev)
>> -
>> -/**
>> - * list_for_each_safe - iterate over a list safe against removal of list entry
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @n: another &struct list_head to use as temporary storage
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each_safe(pos, n, head) \
>> - for (pos = (head)->next, n = pos->next; pos != (head); \
>> - pos = n, n = pos->next)
>> -
>> -/**
>> - * list_for_each_prev_safe - iterate over a list backwards safe against removal of list entry
>> - * @pos: the &struct list_head to use as a loop cursor.
>> - * @n: another &struct list_head to use as temporary storage
>> - * @head: the head for your list.
>> - */
>> -#define list_for_each_prev_safe(pos, n, head) \
>> - for (pos = (head)->prev, n = pos->prev; \
>> - prefetch(pos->prev), pos != (head); \
>> - pos = n, n = pos->prev)
>> -
>> /**
>> * list_for_each_entry - iterate over list of given type
>> * @pos: the type * to use as a loop cursor.
>> @@ -414,128 +103,6 @@ static inline void list_splice_tail_init(struct list_head *list,
>> &pos->member != (head); \
>> pos = list_entry(pos->member.next, typeof(*pos), member))
>>
>> -/**
>> - * list_for_each_entry_reverse - iterate backwards over list of given type.
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - */
>> -#define list_for_each_entry_reverse(pos, head, member) \
>> - for (pos = list_entry((head)->prev, typeof(*pos), member); \
>> - prefetch(pos->member.prev), &pos->member != (head); \
>> - pos = list_entry(pos->member.prev, typeof(*pos), member))
>> -
>> -/**
>> - * list_prepare_entry - prepare a pos entry for use in list_for_each_entry_continue()
>> - * @pos: the type * to use as a start point
>> - * @head: the head of the list
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Prepares a pos entry for use as a start point in list_for_each_entry_continue().
>> - */
>> -#define list_prepare_entry(pos, head, member) \
>> - ((pos) ? : list_entry(head, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_continue - continue iteration over list of given type
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Continue to iterate over list of given type, continuing after
>> - * the current position.
>> - */
>> -#define list_for_each_entry_continue(pos, head, member) \
>> - for (pos = list_entry(pos->member.next, typeof(*pos), member); \
>> - prefetch(pos->member.next), &pos->member != (head); \
>> - pos = list_entry(pos->member.next, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_continue_reverse - iterate backwards from the given point
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Start to iterate over list of given type backwards, continuing after
>> - * the current position.
>> - */
>> -#define list_for_each_entry_continue_reverse(pos, head, member) \
>> - for (pos = list_entry(pos->member.prev, typeof(*pos), member); \
>> - prefetch(pos->member.prev), &pos->member != (head); \
>> - pos = list_entry(pos->member.prev, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_from - iterate over list of given type from the current point
>> - * @pos: the type * to use as a loop cursor.
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate over list of given type, continuing from current position.
>> - */
>> -#define list_for_each_entry_from(pos, head, member) \
>> - for (; prefetch(pos->member.next), &pos->member != (head); \
>> - pos = list_entry(pos->member.next, typeof(*pos), member))
>> -
>> -/**
>> - * list_for_each_entry_safe - iterate over list of given type safe against removal of list entry
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - */
>> -#define list_for_each_entry_safe(pos, n, head, member) \
>> - for (pos = list_entry((head)->next, typeof(*pos), member), \
>> - n = list_entry(pos->member.next, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
>> -
>> -/**
>> - * list_for_each_entry_safe_continue
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate over list of given type, continuing after current point,
>> - * safe against removal of list entry.
>> - */
>> -#define list_for_each_entry_safe_continue(pos, n, head, member) \
>> - for (pos = list_entry(pos->member.next, typeof(*pos), member), \
>> - n = list_entry(pos->member.next, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
>> -
>> -/**
>> - * list_for_each_entry_safe_from
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate over list of given type from current point, safe against
>> - * removal of list entry.
>> - */
>> -#define list_for_each_entry_safe_from(pos, n, head, member) \
>> - for (n = list_entry(pos->member.next, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.next, typeof(*n), member))
>> -
>> -/**
>> - * list_for_each_entry_safe_reverse
>> - * @pos: the type * to use as a loop cursor.
>> - * @n: another type * to use as temporary storage
>> - * @head: the head for your list.
>> - * @member: the name of the list_head within the struct.
>> - *
>> - * Iterate backwards over list of given type, safe against removal
>> - * of list entry.
>> - */
>> -#define list_for_each_entry_safe_reverse(pos, n, head, member) \
>> - for (pos = list_entry((head)->prev, typeof(*pos), member), \
>> - n = list_entry(pos->member.prev, typeof(*pos), member); \
>> - &pos->member != (head); \
>> - pos = n, n = list_entry(n->member.prev, typeof(*n), member))
>> -
>> struct offset {
>> struct list_head list;
>> unsigned offset;
>> --
>> 2.16.2.395.g2e18187dfd-goog
>>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-03-01 16:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-28 22:17 [PATCH] drm/radeon/mkregtable: Delete unused list functions and macros Matthias Kaehlcke
2018-02-28 22:17 ` Matthias Kaehlcke
2018-03-01 15:41 ` Guenter Roeck
2018-03-01 16:27 ` Alex Deucher
2018-03-01 16:27 ` Alex Deucher
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.