Load late binding firmware Signed-off-by: Badal Nilawar <badal.nila...@intel.com> --- drivers/gpu/drm/xe/xe_device.c | 2 + drivers/gpu/drm/xe/xe_late_bind_fw.c | 91 +++++++++++++++++++++++++++- drivers/gpu/drm/xe/xe_late_bind_fw.h | 1 + 3 files changed, 92 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c index d83864e7189c..30a416323b37 100644 --- a/drivers/gpu/drm/xe/xe_device.c +++ b/drivers/gpu/drm/xe/xe_device.c @@ -894,6 +894,8 @@ int xe_device_probe(struct xe_device *xe) xe_late_bind_fw_init(&xe->late_bind); + xe_late_bind_fw_load(&xe->late_bind); + err = xe_oa_init(xe); if (err) return err; diff --git a/drivers/gpu/drm/xe/xe_late_bind_fw.c b/drivers/gpu/drm/xe/xe_late_bind_fw.c index 297238fd3d16..7d2bc959027d 100644 --- a/drivers/gpu/drm/xe/xe_late_bind_fw.c +++ b/drivers/gpu/drm/xe/xe_late_bind_fw.c @@ -16,6 +16,16 @@ #include "xe_late_bind_fw.h" #include "xe_pcode.h" #include "xe_pcode_api.h" +#include "xe_pm.h" + +/* + * The component should load quite quickly in most cases, but it could take + * a bit. Using a very big timeout just to cover the worst case scenario + */ +#define LB_INIT_TIMEOUT_MS 20000 + +#define LB_FW_LOAD_RETRY_MAXCOUNT 40 +#define LB_FW_LOAD_RETRY_PAUSE_MS 50 static const char * const fw_id_to_name[] = { [FAN_CONTROL_ID] = "fan_control", @@ -45,6 +55,78 @@ static int late_bind_fw_num_fans(struct xe_late_bind *late_bind) return 0; } +static void late_bind_work(struct work_struct *work) +{ + struct xe_late_bind_fw *lbfw = container_of(work, struct xe_late_bind_fw, work); + struct xe_late_bind *late_bind = container_of(lbfw, struct xe_late_bind, + late_bind_fw[lbfw->id]); + struct xe_device *xe = late_bind_to_xe(late_bind); + int retry = LB_FW_LOAD_RETRY_MAXCOUNT; + int ret; + int slept; + + if (!late_bind->component_added) + return; + + if (!lbfw->valid) + return; + + /* we can queue this before the component is bound */ + for (slept = 0; slept < LB_INIT_TIMEOUT_MS; slept += 100) { + if (late_bind->component) + break; + msleep(100); + } + + xe_pm_runtime_get(xe); + mutex_lock(&late_bind->mutex); + drm_dbg(&xe->drm, "Load %s firmware\n", fw_id_to_name[lbfw->id]); + + do { + ret = late_bind->component->ops->push_config(late_bind->component->mei_dev, + lbfw->type, lbfw->flags, + lbfw->payload, lbfw->payload_size); + if (!ret) + break; + msleep(LB_FW_LOAD_RETRY_PAUSE_MS); + } while (--retry && ret == -EAGAIN); + + if (ret) + drm_err(&xe->drm, "Load %s firmware failed with err %d\n", + fw_id_to_name[lbfw->id], ret); + else + drm_dbg(&xe->drm, "Load %s firmware successful\n", + fw_id_to_name[lbfw->id]); + + mutex_unlock(&late_bind->mutex); + xe_pm_runtime_put(xe); +} + +int xe_late_bind_fw_load(struct xe_late_bind *late_bind) +{ + struct xe_device *xe = late_bind_to_xe(late_bind); + struct xe_late_bind_fw *lbfw; + int id; + + if (!late_bind->component_added) + return -EINVAL; + + for (id = 0; id < MAX_ID; id++) { + lbfw = &late_bind->late_bind_fw[id]; + if (lbfw->valid) { + drm_dbg(&xe->drm, "Queue work: to load %s firmware\n", + fw_id_to_name[lbfw->id]); + queue_work(late_bind->wq, &lbfw->work); + } + } + return 0; +} + +/** + * late_bind_fw_init() - initialize late bind firmware + * + * Return: 0 if the initialization was successful, a negative errno otherwise. + */ static int late_bind_fw_init(struct xe_late_bind *late_bind, u32 id) { struct xe_device *xe = late_bind_to_xe(late_bind); @@ -93,6 +175,7 @@ static int late_bind_fw_init(struct xe_late_bind *late_bind, u32 id) memcpy(lb_fw->payload, fw->data, lb_fw->payload_size); release_firmware(fw); + INIT_WORK(&lb_fw->work, late_bind_work); lb_fw->valid = true; return 0; @@ -108,12 +191,17 @@ int xe_late_bind_fw_init(struct xe_late_bind *late_bind) int id; int ret; + late_bind->wq = create_singlethread_workqueue("late-bind-ordered-wq"); + if (!late_bind->wq) + return -ENOMEM; + for (id = 0; id < MAX_ID; id++) { ret = late_bind_fw_init(late_bind, id); if (ret) return ret; } - return ret; + + return 0; } static int xe_late_bind_component_bind(struct device *xe_kdev, @@ -179,7 +267,6 @@ int xe_late_bind_init(struct xe_late_bind *late_bind) } late_bind->component_added = true; - /* the component must be removed before unload, so can't use drmm for cleanup */ return 0; } diff --git a/drivers/gpu/drm/xe/xe_late_bind_fw.h b/drivers/gpu/drm/xe/xe_late_bind_fw.h index e88c637b15a6..edd0e4c0650e 100644 --- a/drivers/gpu/drm/xe/xe_late_bind_fw.h +++ b/drivers/gpu/drm/xe/xe_late_bind_fw.h @@ -13,5 +13,6 @@ struct xe_late_bind; int xe_late_bind_init(struct xe_late_bind *late_bind); void xe_late_bind_remove(struct xe_late_bind *late_bind); int xe_late_bind_fw_init(struct xe_late_bind *late_bind); +int xe_late_bind_fw_load(struct xe_late_bind *late_bind); #endif -- 2.34.1