habanalabs: make set_pci_regions asic function
authorOhad Sharabi <osharabi@habana.ai>
Thu, 15 Jul 2021 12:20:44 +0000 (15:20 +0300)
committerOded Gabbay <ogabbay@kernel.org>
Wed, 1 Sep 2021 15:38:24 +0000 (18:38 +0300)
In order to better support variants of the same ASIC
the set_pci_regions function is now an ASIC function which
allows each ASIC to implement it internally, thus keeping
all definitions static to the file.

Signed-off-by: Ohad Sharabi <osharabi@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
drivers/misc/habanalabs/common/habanalabs.h
drivers/misc/habanalabs/gaudi/gaudi.c
drivers/misc/habanalabs/goya/goya.c

index 43d938b..7cee08d 100644 (file)
@@ -1159,6 +1159,7 @@ struct fw_load_mgr {
  * @init_cpu_scrambler_dram: Enable CPU specific DRAM scrambling
  * @state_dump_init: initialize constants required for state dump
  * @get_sob_addr: get SOB base address offset.
+ * @set_pci_memory_regions: setting properties of PCI memory regions
  */
 struct hl_asic_funcs {
        int (*early_init)(struct hl_device *hdev);
@@ -1287,6 +1288,7 @@ struct hl_asic_funcs {
        void (*init_cpu_scrambler_dram)(struct hl_device *hdev);
        void (*state_dump_init)(struct hl_device *hdev);
        u32 (*get_sob_addr)(struct hl_device *hdev, u32 sob_id);
+       void (*set_pci_memory_regions)(struct hl_device *hdev);
 };
 
 
index d3bd584..18fec7d 100644 (file)
@@ -1857,7 +1857,7 @@ static int gaudi_sw_init(struct hl_device *hdev)
        hdev->supports_staged_submission = true;
        hdev->supports_wait_for_multi_cs = true;
 
-       gaudi_set_pci_memory_regions(hdev);
+       hdev->asic_funcs->set_pci_memory_regions(hdev);
 
        return 0;
 
@@ -9377,7 +9377,8 @@ static const struct hl_asic_funcs gaudi_funcs = {
        .init_firmware_loader = gaudi_init_firmware_loader,
        .init_cpu_scrambler_dram = gaudi_init_scrambler_hbm,
        .state_dump_init = gaudi_state_dump_init,
-       .get_sob_addr = gaudi_get_sob_addr
+       .get_sob_addr = gaudi_get_sob_addr,
+       .set_pci_memory_regions = gaudi_set_pci_memory_regions
 };
 
 /**
index 8d890c7..e690161 100644 (file)
@@ -962,7 +962,7 @@ static int goya_sw_init(struct hl_device *hdev)
        hdev->allow_external_soft_reset = true;
        hdev->supports_wait_for_multi_cs = false;
 
-       goya_set_pci_memory_regions(hdev);
+       hdev->asic_funcs->set_pci_memory_regions(hdev);
 
        return 0;
 
@@ -5669,7 +5669,8 @@ static const struct hl_asic_funcs goya_funcs = {
        .init_firmware_loader = goya_init_firmware_loader,
        .init_cpu_scrambler_dram = goya_cpu_init_scrambler_dram,
        .state_dump_init = goya_state_dump_init,
-       .get_sob_addr = &goya_get_sob_addr
+       .get_sob_addr = &goya_get_sob_addr,
+       .set_pci_memory_regions = goya_set_pci_memory_regions,
 };
 
 /*