return 0;
 }
 
+static int smu_smc_table_hw_init(struct smu_context *smu)
+{
+       int ret;
+
+       ret = smu_read_pptable_from_vbios(smu);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
 static int smu_hw_init(void *handle)
 {
        int ret;
 
        mutex_lock(&smu->mutex);
 
-       /* TODO */
+       ret = smu_smc_table_hw_init(smu);
+       if (ret)
+               goto failed;
 
        mutex_unlock(&smu->mutex);
 
        pr_info("SMU is initialized successfully!\n");
 
        return 0;
+
+failed:
+       mutex_unlock(&smu->mutex);
+       return ret;
 }
 
 static int smu_hw_fini(void *handle)
 
        int (*init_microcode)(struct smu_context *smu);
        int (*load_microcode)(struct smu_context *smu);
        int (*check_fw_status)(struct smu_context *smu);
+       int (*read_pptable_from_vbios)(struct smu_context *smu);
 };
 
 #define smu_init_microcode(smu) \
        ((smu)->funcs->load_microcode ? (smu)->funcs->load_microcode((smu)) : 0)
 #define smu_check_fw_status(smu) \
        ((smu)->funcs->check_fw_status ? (smu)->funcs->check_fw_status((smu)) : 0)
+#define smu_read_pptable_from_vbios(smu) \
+       ((smu)->funcs->read_pptable_from_vbios ? (smu)->funcs->read_pptable_from_vbios((smu)) : 0)
 
 extern const struct amd_ip_funcs smu_ip_funcs;