deepin-ci-robot
commented
2 months ago deepin pr auto review
关键摘要:
- 在
phytium_dwmac_probe函数中，新增的ACPI设备ID匹配逻辑可能会影响现有功能，需要确认是否与现有代码库兼容。 - 在
phytium_dwmac_probe函数中，对pdev->dev.dma_ops的操作可能会导致DMA操作的不可用，需要确保这是预期的行为。 - 在
phytium_dwmac_probe函数中，新增了对plat->host_dma_width的赋值操作，需要确保这一改动不会影响其他模块或功能。 - 在
phytium_dwmac_probe函数中，新增了对plat->axi->axi_lpi_en、plat->axi->axi_xit_frm和plat->axi->axi_wr_osr_lmt的赋值操作，需要确保这些改动不会影响其他模块或功能。
是否建议立即修改:
- 是，需要确保新增的代码逻辑不会引入新的问题，并且与现有的代码库兼容。同时，应该添加相应的注释来解释这些改动的目的和影响。
Compat with some quirk platform FT2000/4 as id "FTGM0001", use rgmii-rxid instead of rgmii (wrong ACPI DSDT), and set dma_ops and host_dma_width (fix code style).
origin code in k4.19 is following: acpi_dma_configure(priv->device,DEV_DMA_COHERENT); static u64 my_mask=0xffffffff; priv->device->dma_mask = &my_mask; priv->device->coherent_dma_mask = (u32)~0; port to k5.10 add: pdev->dev.dma_ops = NULL;