firewire: core: code cleanup after vm_map_pages_zero introduction
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Tue, 5 Nov 2019 13:49:39 +0000 (14:49 +0100)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Wed, 13 Nov 2019 19:59:57 +0000 (20:59 +0100)
Commit 22660db89262 turned fw_iso_buffer_map_vma into a one-liner.
There is no need to keep this in the core-iso.c collection of buffer
management functions; put it inline into the sole user, the character
device file driver.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
drivers/firewire/core-cdev.c
drivers/firewire/core-iso.c
drivers/firewire/core.h

index 1da7ba1..7197918 100644 (file)
@@ -1694,7 +1694,8 @@ static int fw_device_op_mmap(struct file *file, struct vm_area_struct *vma)
        if (ret < 0)
                goto fail;
 
-       ret = fw_iso_buffer_map_vma(&client->buffer, vma);
+       ret = vm_map_pages_zero(vma, client->buffer.pages,
+                               client->buffer.page_count);
        if (ret < 0)
                goto fail;
 
index df8a56a..185b0b7 100644 (file)
@@ -91,13 +91,6 @@ int fw_iso_buffer_init(struct fw_iso_buffer *buffer, struct fw_card *card,
 }
 EXPORT_SYMBOL(fw_iso_buffer_init);
 
-int fw_iso_buffer_map_vma(struct fw_iso_buffer *buffer,
-                         struct vm_area_struct *vma)
-{
-       return vm_map_pages_zero(vma, buffer->pages,
-                                       buffer->page_count);
-}
-
 void fw_iso_buffer_destroy(struct fw_iso_buffer *buffer,
                           struct fw_card *card)
 {
index 0f0bed3..4b0e4ee 100644 (file)
@@ -158,8 +158,6 @@ void fw_node_event(struct fw_card *card, struct fw_node *node, int event);
 int fw_iso_buffer_alloc(struct fw_iso_buffer *buffer, int page_count);
 int fw_iso_buffer_map_dma(struct fw_iso_buffer *buffer, struct fw_card *card,
                          enum dma_data_direction direction);
-int fw_iso_buffer_map_vma(struct fw_iso_buffer *buffer,
-                         struct vm_area_struct *vma);
 
 
 /* -topology */