bus: mhi: core: Wait for ready after an EDL firmware download
authorBhaumik Bhatt <bbhatt@codeaurora.org>
Tue, 30 Mar 2021 01:28:19 +0000 (18:28 -0700)
committerManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Wed, 31 Mar 2021 11:17:07 +0000 (16:47 +0530)
Currently, the firmware load handler returns after the EDL image
is downloaded. Wait for an MHI READY transition instead as the
specification expects so as to proceed with further bootup such
as device entering Flash Programmer execution environment.

Signed-off-by: Bhaumik Bhatt <bbhatt@codeaurora.org>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Reviewed-by: Loic Poulain <loic.poulain@linaro.org>
Link: https://lore.kernel.org/r/1617067704-28850-3-git-send-email-bbhatt@codeaurora.org
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
drivers/bus/mhi/core/boot.c

index 84c2117..0f0ae88 100644 (file)
@@ -418,7 +418,7 @@ void mhi_fw_load_handler(struct mhi_controller *mhi_cntrl)
 
        /* If device is in pass through, do reset to ready state transition */
        if (mhi_cntrl->ee == MHI_EE_PTHRU)
-               goto fw_load_ee_pthru;
+               goto fw_load_ready_state;
 
        fw_name = (mhi_cntrl->ee == MHI_EE_EDL) ?
                mhi_cntrl->edl_image : mhi_cntrl->fw_image;
@@ -460,10 +460,10 @@ void mhi_fw_load_handler(struct mhi_controller *mhi_cntrl)
                goto error_fw_load;
        }
 
-       /* Exit if EDL image was loaded */
+       /* Wait for ready since EDL image was loaded */
        if (fw_name == mhi_cntrl->edl_image) {
                release_firmware(firmware);
-               return;
+               goto fw_load_ready_state;
        }
 
        write_lock_irq(&mhi_cntrl->pm_lock);
@@ -488,7 +488,7 @@ void mhi_fw_load_handler(struct mhi_controller *mhi_cntrl)
 
        release_firmware(firmware);
 
-fw_load_ee_pthru:
+fw_load_ready_state:
        /* Transitioning into MHI RESET->READY state */
        ret = mhi_ready_state_transition(mhi_cntrl);
        if (ret) {