dmaengine: pl330: remove debugfs file on teardown

init_pl330_debugfs() creates a debugfs file with struct pl330_dmac as
private data. pl330_remove() then unregisters the DMA device and frees
the PL330 channel/thread state without removing that file.

Keep the debugfs dentry and remove it before tearing down the DMAC state
used by the debugfs show callback.

Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
Link: https://patch.msgid.link/20260615091645.28878-1-pengpeng@iscas.ac.cn
Signed-off-by: Vinod Koul <vkoul@kernel.org>
This commit is contained in:
Pengpeng Hou 2026-06-15 17:16:45 +08:00 committed by Vinod Koul
parent 89aba9c39b
commit 0d3e3376b2

View File

@ -502,6 +502,7 @@ struct pl330_dmac {
struct dma_pl330_chan *peripherals; /* keep at end */
int quirks;
struct dentry *dbgfs;
struct reset_control *rstc;
struct reset_control *rstc_ocp;
};
@ -2952,14 +2953,24 @@ DEFINE_SHOW_ATTRIBUTE(pl330_debugfs);
static inline void init_pl330_debugfs(struct pl330_dmac *pl330)
{
debugfs_create_file(dev_name(pl330->ddma.dev),
S_IFREG | 0444, NULL, pl330,
&pl330_debugfs_fops);
pl330->dbgfs = debugfs_create_file(dev_name(pl330->ddma.dev),
S_IFREG | 0444, NULL, pl330,
&pl330_debugfs_fops);
}
static inline void deinit_pl330_debugfs(struct pl330_dmac *pl330)
{
debugfs_remove(pl330->dbgfs);
pl330->dbgfs = NULL;
}
#else
static inline void init_pl330_debugfs(struct pl330_dmac *pl330)
{
}
static inline void deinit_pl330_debugfs(struct pl330_dmac *pl330)
{
}
#endif
/*
@ -3204,6 +3215,8 @@ static void pl330_remove(struct amba_device *adev)
struct dma_pl330_chan *pch, *_p;
int i, irq;
deinit_pl330_debugfs(pl330);
pm_runtime_get_noresume(pl330->ddma.dev);
if (adev->dev.of_node)