* [PATCH] firmware: Drop unused pages field from struct firmware
@ 2020-04-15 16:45 Takashi Iwai
2020-04-16 5:16 ` Luis Chamberlain
0 siblings, 1 reply; 2+ messages in thread
From: Takashi Iwai @ 2020-04-15 16:45 UTC (permalink / raw)
To: Luis Chamberlain; +Cc: Greg Kroah-Hartman, Rafael J . Wysocki, linux-kernel
The struct firmware contains a page table pointer that was used only
internally in the past. Since the actual page tables are referred
from struct fw_priv and should be never from struct firmware, we can
drop this unused field gracefully.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
drivers/base/firmware_loader/main.c | 3 ---
include/linux/firmware.h | 1 -
2 files changed, 4 deletions(-)
diff --git a/drivers/base/firmware_loader/main.c b/drivers/base/firmware_loader/main.c
index 76f79913916d..5296aaca35cf 100644
--- a/drivers/base/firmware_loader/main.c
+++ b/drivers/base/firmware_loader/main.c
@@ -548,9 +548,6 @@ static void firmware_free_data(const struct firmware *fw)
static void fw_set_page_data(struct fw_priv *fw_priv, struct firmware *fw)
{
fw->priv = fw_priv;
-#ifdef CONFIG_FW_LOADER_USER_HELPER
- fw->pages = fw_priv->pages;
-#endif
fw->size = fw_priv->size;
fw->data = fw_priv->data;
diff --git a/include/linux/firmware.h b/include/linux/firmware.h
index 4bbd0afd91b7..cb3e2c06ed8a 100644
--- a/include/linux/firmware.h
+++ b/include/linux/firmware.h
@@ -12,7 +12,6 @@
struct firmware {
size_t size;
const u8 *data;
- struct page **pages;
/* firmware loader private fields */
void *priv;
--
2.25.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] firmware: Drop unused pages field from struct firmware
2020-04-15 16:45 [PATCH] firmware: Drop unused pages field from struct firmware Takashi Iwai
@ 2020-04-16 5:16 ` Luis Chamberlain
0 siblings, 0 replies; 2+ messages in thread
From: Luis Chamberlain @ 2020-04-16 5:16 UTC (permalink / raw)
To: Takashi Iwai; +Cc: Greg Kroah-Hartman, Rafael J . Wysocki, linux-kernel
On Wed, Apr 15, 2020 at 06:45:00PM +0200, Takashi Iwai wrote:
> The struct firmware contains a page table pointer that was used only
> internally in the past. Since the actual page tables are referred
> from struct fw_priv and should be never from struct firmware, we can
> drop this unused field gracefully.
>
> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Acked-by: Luis Chamberlain <mcgrof@kernel.org>
Luis
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-16 5:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-15 16:45 [PATCH] firmware: Drop unused pages field from struct firmware Takashi Iwai
2020-04-16 5:16 ` Luis Chamberlain
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).