On Tue, May 07, 2024 at 03:03:39PM GMT, Duoming Zhou wrote:

 typedef struct ax25_dev {
-	struct ax25_dev		*next;
+	struct list_head	list;

Would it make sense to replace this with:

LIST_HEAD(ax25_dev_list);

And then get rid of:

+	/* Initialized the list for the first entry */
+	if (!ax25_dev_list.next)
+		INIT_LIST_HEAD(&ax25_dev_list);

-- 
Lars Kellogg-Stedman [off-list ref] | larsks @ {irc,twitter,github}
http://blog.oddbit.com/                | N1LKS