Unverified Commit 56036d6a authored by Stuart Hayes's avatar Stuart Hayes Committed by Ilpo Järvinen
Browse files

platform/x86: dell_rbu: Remove unused struct 



Stop using an entire struct packet_data just for its embedded list_head.

Signed-off-by: default avatarStuart Hayes <stuart.w.hayes@gmail.com>
Link: https://lore.kernel.org/r/20250609184659.7210-4-stuart.w.hayes@gmail.com


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
parent b9707d46

drivers/platform/x86/dell/dell_rbu.c

+5 −5
Original line number Diff line number Diff line
@@ -77,14 +77,14 @@ struct packet_data { 
	int ordernum;

};



static struct packet_data packet_data_head;

static struct list_head packet_data_list;



static struct platform_device *rbu_device;

static int context;



static void init_packet_head(void)

{

	INIT_LIST_HEAD(&packet_data_head.list);

	INIT_LIST_HEAD(&packet_data_list);

	rbu_data.packet_read_count = 0;

	rbu_data.num_packets = 0;

	rbu_data.packetsize = 0;
@@ -183,7 +183,7 @@ static int create_packet(void *data, size_t length) __must_hold(&rbu_data.lock) 


	/* initialize the newly created packet headers */

	INIT_LIST_HEAD(&newpacket->list);

	list_add_tail(&newpacket->list, &packet_data_head.list);

	list_add_tail(&newpacket->list, &packet_data_list);



	memcpy(newpacket->data, data, length);
@@ -292,7 +292,7 @@ static int packet_read_list(char *data, size_t * pread_length) 
	remaining_bytes = *pread_length;

	bytes_read = rbu_data.packet_read_count;



	list_for_each_entry(newpacket, &packet_data_head.list, list) {

	list_for_each_entry(newpacket, &packet_data_list, list) {

		bytes_copied = do_packet_read(pdest, newpacket,

			remaining_bytes, bytes_read, &temp_count);

		remaining_bytes -= bytes_copied;
@@ -315,7 +315,7 @@ static void packet_empty_list(void) 
{

	struct packet_data *newpacket, *tmp;



	list_for_each_entry_safe(newpacket, tmp, &packet_data_head.list, list) {

	list_for_each_entry_safe(newpacket, tmp, &packet_data_list, list) {

		list_del(&newpacket->list);



		/*