Skip to content
Snippets Groups Projects
Unverified Commit dbca8048 authored by Peter Colberg's avatar Peter Colberg Committed by Xu Yilun
Browse files

fpga: dfl: remove unused member pdata from struct dfl_{afu,fme}


The member pdata in struct dfl_{afu,fme} is set in function
{afu,fme}_dev_init(), respectively, but never used.

Fixes: 857a2622 ("fpga: dfl: afu: add afu sub feature support")
Fixes: 29de7624 ("fpga: dfl: fme: add partial reconfiguration sub feature support")
Signed-off-by: default avatarPeter Colberg <peter.colberg@intel.com>
Reviewed-by: default avatarMatthew Gerlach <matthew.gerlach@linux.intel.com>
Acked-by: default avatarXu Yilun <yilun.xu@intel.com>
Link: https://lore.kernel.org/r/20240415235937.3121-1-peter.colberg@intel.com


Signed-off-by: default avatarXu Yilun <yilun.xu@linux.intel.com>
parent 7c2dafa6
No related branches found
No related tags found
No related merge requests found
...@@ -858,8 +858,6 @@ static int afu_dev_init(struct platform_device *pdev) ...@@ -858,8 +858,6 @@ static int afu_dev_init(struct platform_device *pdev)
if (!afu) if (!afu)
return -ENOMEM; return -ENOMEM;
afu->pdata = pdata;
mutex_lock(&pdata->lock); mutex_lock(&pdata->lock);
dfl_fpga_pdata_set_private(pdata, afu); dfl_fpga_pdata_set_private(pdata, afu);
afu_mmio_region_init(pdata); afu_mmio_region_init(pdata);
......
...@@ -67,7 +67,6 @@ struct dfl_afu_dma_region { ...@@ -67,7 +67,6 @@ struct dfl_afu_dma_region {
* @regions: the mmio region linked list of this afu feature device. * @regions: the mmio region linked list of this afu feature device.
* @dma_regions: root of dma regions rb tree. * @dma_regions: root of dma regions rb tree.
* @num_umsgs: num of umsgs. * @num_umsgs: num of umsgs.
* @pdata: afu platform device's pdata.
*/ */
struct dfl_afu { struct dfl_afu {
u64 region_cur_offset; u64 region_cur_offset;
...@@ -75,8 +74,6 @@ struct dfl_afu { ...@@ -75,8 +74,6 @@ struct dfl_afu {
u8 num_umsgs; u8 num_umsgs;
struct list_head regions; struct list_head regions;
struct rb_root dma_regions; struct rb_root dma_regions;
struct dfl_feature_platform_data *pdata;
}; };
/* hold pdata->lock when call __afu_port_enable/disable */ /* hold pdata->lock when call __afu_port_enable/disable */
......
...@@ -679,8 +679,6 @@ static int fme_dev_init(struct platform_device *pdev) ...@@ -679,8 +679,6 @@ static int fme_dev_init(struct platform_device *pdev)
if (!fme) if (!fme)
return -ENOMEM; return -ENOMEM;
fme->pdata = pdata;
mutex_lock(&pdata->lock); mutex_lock(&pdata->lock);
dfl_fpga_pdata_set_private(pdata, fme); dfl_fpga_pdata_set_private(pdata, fme);
mutex_unlock(&pdata->lock); mutex_unlock(&pdata->lock);
......
...@@ -24,13 +24,11 @@ ...@@ -24,13 +24,11 @@
* @mgr: FME's FPGA manager platform device. * @mgr: FME's FPGA manager platform device.
* @region_list: linked list of FME's FPGA regions. * @region_list: linked list of FME's FPGA regions.
* @bridge_list: linked list of FME's FPGA bridges. * @bridge_list: linked list of FME's FPGA bridges.
* @pdata: fme platform device's pdata.
*/ */
struct dfl_fme { struct dfl_fme {
struct platform_device *mgr; struct platform_device *mgr;
struct list_head region_list; struct list_head region_list;
struct list_head bridge_list; struct list_head bridge_list;
struct dfl_feature_platform_data *pdata;
}; };
extern const struct dfl_feature_ops fme_pr_mgmt_ops; extern const struct dfl_feature_ops fme_pr_mgmt_ops;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment