Hi Vinod On Fri, Nov 04, 2022 at 07:12:22PM +0530, Vinod Koul wrote: > On 21-09-22, 14:48, Cai Huoqing wrote: > > From: caihuoqing <caihuoqing@xxxxxxxxx> > > > > Rename dw_edma_core_ops structure to dw_edma_plat_ops, > > because the 'ops' is related to platform device > > Is that really a case, the device can be non platform too right? The patchlog is misleading in the suggested naming convention. The dw_edma_pcie_plat_ops name was supposed to refer to the platform which the DW eDMA engine is embedded to, like PCIe end-point (accessible via the PCIe bus) or a PCIe root port (directly accessible by CPU). Needless to say that for them the IRQ-vector and PCI-addresses are differently determined. The suggested name has a connection with the kernel platform device only as a private case of the eDMA/hDMA embedded into the DW PCI Root ports, though basically it was supposed to refer to any platform in which the DMA hardware lives. Anyway the renaming was necessary to distinguish two types of the implementation callbacks: 1. DW eDMA/hDMA IP-core specific operations: device-specific CSR setups in one or another aspect of the DMA-engine initialization. 2. DW eDMA/hDMA platform specific operations: the DMA device environment configs like IRQs, address translation, etc. dw_edma_pcie_core_ops is supposed to be used for the case 1, and dw_edma_pcie_plat_ops - for the case 2. -Serge(y) > > -- > ~Vinod
