Commit fc69aa86 authored by Tomer Tayar's avatar Tomer Tayar Committed by Oded Gabbay

habanalabs: fix PCIe access to SRAM via debugfs

hl_access_sram_dram_region() uses a region base which is set within the
hl_set_dram_bar() function. However, for SRAM access this function is
not called, and we end up with a wrong value of region base and with a
bad calculated address.
Fix it by initializing the region base value independently of whether
hl_set_dram_bar() is called or not.
Signed-off-by: default avatarTomer Tayar <ttayar@habana.ai>
Reviewed-by: default avatarOded Gabbay <ogabbay@kernel.org>
Signed-off-by: default avatarOded Gabbay <ogabbay@kernel.org>
parent 679e9689
...@@ -69,17 +69,17 @@ int hl_access_sram_dram_region(struct hl_device *hdev, u64 addr, u64 *val, ...@@ -69,17 +69,17 @@ int hl_access_sram_dram_region(struct hl_device *hdev, u64 addr, u64 *val,
enum debugfs_access_type acc_type, enum pci_region region_type, bool set_dram_bar) enum debugfs_access_type acc_type, enum pci_region region_type, bool set_dram_bar)
{ {
struct pci_mem_region *region = &hdev->pci_mem_region[region_type]; struct pci_mem_region *region = &hdev->pci_mem_region[region_type];
u64 old_base = 0, rc, new_bar_region_base = 0; u64 old_base = 0, rc, bar_region_base = region->region_base;
void __iomem *acc_addr; void __iomem *acc_addr;
if (set_dram_bar) { if (set_dram_bar) {
old_base = hl_set_dram_bar(hdev, addr, region, &new_bar_region_base); old_base = hl_set_dram_bar(hdev, addr, region, &bar_region_base);
if (old_base == U64_MAX) if (old_base == U64_MAX)
return -EIO; return -EIO;
} }
acc_addr = hdev->pcie_bar[region->bar_id] + region->offset_in_bar + acc_addr = hdev->pcie_bar[region->bar_id] + region->offset_in_bar +
(addr - new_bar_region_base); (addr - bar_region_base);
switch (acc_type) { switch (acc_type) {
case DEBUGFS_READ8: case DEBUGFS_READ8:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment