[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <A3397C8B8B789E45844E7EC5DEAD89D05995A36C@satlexdag05.amd.com>
Date: Fri, 14 Nov 2014 22:37:56 +0000
From: "Deucher, Alexander" <Alexander.Deucher@....com>
To: Andrey Utkin <andrey.krieger.utkin@...il.com>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"linux-kbuild@...r.kernel.org" <linux-kbuild@...r.kernel.org>,
"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
"kernel-janitors@...r.kernel.org" <kernel-janitors@...r.kernel.org>,
"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
"mgorman@...e.de" <mgorman@...e.de>,
"ddstreet@...e.org" <ddstreet@...e.org>,
"jeffrey.t.kirsher@...el.com" <jeffrey.t.kirsher@...el.com>,
"yamada.m@...panasonic.com" <yamada.m@...panasonic.com>,
"kenhelias@...email.de" <kenhelias@...email.de>,
"oleg@...hat.com" <oleg@...hat.com>,
"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"shuah.kh@...sung.com" <shuah.kh@...sung.com>,
"valentina.manea.m@...il.com" <valentina.manea.m@...il.com>,
"yann.morin.1998@...e.fr" <yann.morin.1998@...e.fr>,
"laijs@...fujitsu.com" <laijs@...fujitsu.com>,
"mathieu.desnoyers@...icios.com" <mathieu.desnoyers@...icios.com>,
"rostedt@...dmis.org" <rostedt@...dmis.org>,
"josh@...htriplett.org" <josh@...htriplett.org>,
"paulmck@...ux.vnet.ibm.com" <paulmck@...ux.vnet.ibm.com>,
"m.chehab@...sung.com" <m.chehab@...sung.com>,
"awalls@...metrocast.net" <awalls@...metrocast.net>,
"airlied@...ux.ie" <airlied@...ux.ie>,
"Koenig, Christian" <Christian.Koenig@....com>,
"trivial@...nel.org" <trivial@...nel.org>
Subject: RE: [RESUBMIT] [PATCH] Replace mentions of "list_struct" to
"list_head"
> -----Original Message-----
> From: Andrey Utkin [mailto:andrey.krieger.utkin@...il.com]
> Sent: Thursday, November 13, 2014 8:10 PM
> To: linux-usb@...r.kernel.org; linux-kbuild@...r.kernel.org; linux-
> media@...r.kernel.org; linux-kernel@...r.kernel.org; dri-
> devel@...ts.freedesktop.org; kernel-janitors@...r.kernel.org;
> gregkh@...uxfoundation.org; mgorman@...e.de; ddstreet@...e.org;
> jeffrey.t.kirsher@...el.com; yamada.m@...panasonic.com;
> kenhelias@...email.de; oleg@...hat.com; akpm@...ux-foundation.org;
> shuah.kh@...sung.com; valentina.manea.m@...il.com;
> yann.morin.1998@...e.fr; laijs@...fujitsu.com;
> mathieu.desnoyers@...icios.com; rostedt@...dmis.org;
> josh@...htriplett.org; paulmck@...ux.vnet.ibm.com;
> m.chehab@...sung.com; awalls@...metrocast.net; airlied@...ux.ie;
> Koenig, Christian; Deucher, Alexander; trivial@...nel.org
> Cc: Andrey Utkin
> Subject: [RESUBMIT] [PATCH] Replace mentions of "list_struct" to
> "list_head"
>
> There's no such thing as "list_struct".
>
> Signed-off-by: Andrey Utkin <andrey.krieger.utkin@...il.com>
Acked-by: Alex Deucher <alexander.deucher@....com>
> ---
> drivers/gpu/drm/radeon/mkregtable.c | 24 ++++++++++++------------
> drivers/media/pci/cx18/cx18-driver.h | 2 +-
> include/linux/list.h | 34 +++++++++++++++++-----------------
> include/linux/plist.h | 10 +++++-----
> include/linux/rculist.h | 8 ++++----
> scripts/kconfig/list.h | 6 +++---
> tools/usb/usbip/libsrc/list.h | 2 +-
> 7 files changed, 43 insertions(+), 43 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/mkregtable.c
> b/drivers/gpu/drm/radeon/mkregtable.c
> index 4a85bb6..b928c17 100644
> --- a/drivers/gpu/drm/radeon/mkregtable.c
> +++ b/drivers/gpu/drm/radeon/mkregtable.c
> @@ -347,7 +347,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_entry - get the struct for this entry
> * @ptr: the &struct list_head pointer.
> * @type: the type of the struct this is embedded in.
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_entry(ptr, type, member) \
> container_of(ptr, type, member)
> @@ -356,7 +356,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Note, that list is expected to be not empty.
> */
> @@ -406,7 +406,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_for_each_entry - iterate 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_for_each_entry(pos, head, member)
> \
> for (pos = list_entry((head)->next, typeof(*pos), member); \
> @@ -417,7 +417,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @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); \
> @@ -428,7 +428,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @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().
> */
> @@ -439,7 +439,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Continue to iterate over list of given type, continuing after
> * the current position.
> @@ -453,7 +453,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Start to iterate over list of given type backwards, continuing after
> * the current position.
> @@ -467,7 +467,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Iterate over list of given type, continuing from current position.
> */
> @@ -480,7 +480,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @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), \
> @@ -493,7 +493,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @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.
> @@ -509,7 +509,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @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.
> @@ -524,7 +524,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Iterate backwards over list of given type, safe against removal
> * of list entry.
> diff --git a/drivers/media/pci/cx18/cx18-driver.h
> b/drivers/media/pci/cx18/cx18-driver.h
> index 57f4688..da59ec3 100644
> --- a/drivers/media/pci/cx18/cx18-driver.h
> +++ b/drivers/media/pci/cx18/cx18-driver.h
> @@ -290,7 +290,7 @@ struct cx18_options {
> * list_entry_is_past_end - check if a previous loop cursor is off list end
> * @pos: the type * previously used as a loop cursor.
> * @head: the head for your list.
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Check if the entry's list_head is the head of the list, thus it's not a
> * real entry but was the loop cursor that walked past the end
> diff --git a/include/linux/list.h b/include/linux/list.h
> index f33f831..feb773c 100644
> --- a/include/linux/list.h
> +++ b/include/linux/list.h
> @@ -346,7 +346,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_entry - get the struct for this entry
> * @ptr: the &struct list_head pointer.
> * @type: the type of the struct this is embedded in.
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_entry(ptr, type, member) \
> container_of(ptr, type, member)
> @@ -355,7 +355,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Note, that list is expected to be not empty.
> */
> @@ -366,7 +366,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_last_entry - get the last 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Note, that list is expected to be not empty.
> */
> @@ -377,7 +377,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_first_entry_or_null - 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Note that if the list is empty, it returns NULL.
> */
> @@ -387,7 +387,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> /**
> * list_next_entry - get the next element in list
> * @pos: the type * to cursor
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_next_entry(pos, member) \
> list_entry((pos)->member.next, typeof(*(pos)), member)
> @@ -395,7 +395,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> /**
> * list_prev_entry - get the prev element in list
> * @pos: the type * to cursor
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_prev_entry(pos, member) \
> list_entry((pos)->member.prev, typeof(*(pos)), member)
> @@ -441,7 +441,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_for_each_entry - iterate 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_for_each_entry(pos, head, member)
> \
> for (pos = list_first_entry(head, typeof(*pos), member); \
> @@ -452,7 +452,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_for_each_entry_reverse(pos, head, member)
> \
> for (pos = list_last_entry(head, typeof(*pos), member);
> \
> @@ -463,7 +463,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @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().
> */
> @@ -474,7 +474,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Continue to iterate over list of given type, continuing after
> * the current position.
> @@ -488,7 +488,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Start to iterate over list of given type backwards, continuing after
> * the current position.
> @@ -502,7 +502,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Iterate over list of given type, continuing from current position.
> */
> @@ -515,7 +515,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_for_each_entry_safe(pos, n, head, member)
> \
> for (pos = list_first_entry(head, typeof(*pos), member), \
> @@ -528,7 +528,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @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.
> @@ -544,7 +544,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @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.
> @@ -559,7 +559,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * @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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Iterate backwards over list of given type, safe against removal
> * of list entry.
> @@ -574,7 +574,7 @@ static inline void list_splice_tail_init(struct list_head
> *list,
> * list_safe_reset_next - reset a stale list_for_each_entry_safe loop
> * @pos: the loop cursor used in the list_for_each_entry_safe loop
> * @n: temporary storage used in list_for_each_entry_safe
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * list_safe_reset_next is not safe to use in general if the list may be
> * modified concurrently (eg. the lock is dropped in the loop body). An
> diff --git a/include/linux/plist.h b/include/linux/plist.h
> index 8b6c970..9788360 100644
> --- a/include/linux/plist.h
> +++ b/include/linux/plist.h
> @@ -176,7 +176,7 @@ extern void plist_requeue(struct plist_node *node,
> struct plist_head *head);
> * plist_for_each_entry - iterate over list of given type
> * @pos: the type * to use as a loop counter
> * @head: the head for your list
> - * @mem: the name of the list_struct within the struct
> + * @mem: the name of the list_head within the struct
> */
> #define plist_for_each_entry(pos, head, mem) \
> list_for_each_entry(pos, &(head)->node_list, mem.node_list)
> @@ -185,7 +185,7 @@ extern void plist_requeue(struct plist_node *node,
> struct plist_head *head);
> * plist_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
> - * @m: the name of the list_struct within the struct
> + * @m: the name of the list_head within the struct
> *
> * Continue to iterate over list of given type, continuing after
> * the current position.
> @@ -198,7 +198,7 @@ extern void plist_requeue(struct plist_node *node,
> struct plist_head *head);
> * @pos: the type * to use as a loop counter
> * @n: another type * to use as temporary storage
> * @head: the head for your list
> - * @m: the name of the list_struct within the struct
> + * @m: the name of the list_head within the struct
> *
> * Iterate over list of given type, safe against removal of list entry.
> */
> @@ -229,7 +229,7 @@ static inline int plist_node_empty(const struct
> plist_node *node)
> * plist_first_entry - get the struct for the first entry
> * @head: the &struct plist_head pointer
> * @type: the type of the struct this is embedded in
> - * @member: the name of the list_struct within the struct
> + * @member: the name of the list_head within the struct
> */
> #ifdef CONFIG_DEBUG_PI_LIST
> # define plist_first_entry(head, type, member) \
> @@ -246,7 +246,7 @@ static inline int plist_node_empty(const struct
> plist_node *node)
> * plist_last_entry - get the struct for the last entry
> * @head: the &struct plist_head pointer
> * @type: the type of the struct this is embedded in
> - * @member: the name of the list_struct within the struct
> + * @member: the name of the list_head within the struct
> */
> #ifdef CONFIG_DEBUG_PI_LIST
> # define plist_last_entry(head, type, member) \
> diff --git a/include/linux/rculist.h b/include/linux/rculist.h
> index 372ad5e..664060b 100644
> --- a/include/linux/rculist.h
> +++ b/include/linux/rculist.h
> @@ -241,7 +241,7 @@ static inline void list_splice_init_rcu(struct list_head
> *list,
> * list_entry_rcu - get the struct for this entry
> * @ptr: the &struct list_head pointer.
> * @type: the type of the struct this is embedded in.
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * This primitive may safely run concurrently with the _rcu list-mutation
> * primitives such as list_add_rcu() as long as it's guarded by rcu_read_lock().
> @@ -278,7 +278,7 @@ static inline void list_splice_init_rcu(struct list_head
> *list,
> * list_first_or_null_rcu - 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Note that if the list is empty, it returns NULL.
> *
> @@ -296,7 +296,7 @@ static inline void list_splice_init_rcu(struct list_head
> *list,
> * list_for_each_entry_rcu - iterate over rcu 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * This list-traversal primitive may safely run concurrently with
> * the _rcu list-mutation primitives such as list_add_rcu()
> @@ -311,7 +311,7 @@ static inline void list_splice_init_rcu(struct list_head
> *list,
> * list_for_each_entry_continue_rcu - 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> *
> * Continue to iterate over list of given type, continuing after
> * the current position.
> diff --git a/scripts/kconfig/list.h b/scripts/kconfig/list.h
> index 685d80e..2cf23f0 100644
> --- a/scripts/kconfig/list.h
> +++ b/scripts/kconfig/list.h
> @@ -34,7 +34,7 @@ struct list_head {
> * list_entry - get the struct for this entry
> * @ptr: the &struct list_head pointer.
> * @type: the type of the struct this is embedded in.
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_entry(ptr, type, member) \
> container_of(ptr, type, member)
> @@ -43,7 +43,7 @@ struct list_head {
> * list_for_each_entry - iterate 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_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_for_each_entry(pos, head, member)
> \
> for (pos = list_entry((head)->next, typeof(*pos), member); \
> @@ -55,7 +55,7 @@ struct list_head {
> * @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_struct within the struct.
> + * @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), \
> diff --git a/tools/usb/usbip/libsrc/list.h b/tools/usb/usbip/libsrc/list.h
> index 8d0c936..5eaaa78 100644
> --- a/tools/usb/usbip/libsrc/list.h
> +++ b/tools/usb/usbip/libsrc/list.h
> @@ -98,7 +98,7 @@ static inline void list_del(struct list_head *entry)
> * list_entry - get the struct for this entry
> * @ptr: the &struct list_head pointer.
> * @type: the type of the struct this is embedded in.
> - * @member: the name of the list_struct within the struct.
> + * @member: the name of the list_head within the struct.
> */
> #define list_entry(ptr, type, member) \
> container_of(ptr, type, member)
> --
> 1.8.5.5
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists