• Bean Huo's avatar
    scsi: ufs: Split ufshcd_probe_hba() based on its called flow · 1b9e2141
    Bean Huo authored
    This patch has two major non-functionality changes:
    
    1. Take scanning host if-statement out from ufshcd_probe_hba(), and move
    into a new added function ufshcd_add_lus().  In this new function
    ufshcd_add_lus(), the main functionalitis include: ICC initialization, add
    well-known LUs, devfreq initialization, UFS bsg probe and scsi host
    scan. The reason for this change is that these functionalities only being
    called during booting stage flow ufshcd_init()->ufshcd_async_scan(). In the
    processes of error handling and power management ufshcd_suspend(),
    ufshcd_resume(), ufshcd_probe_hba() being called, but these functionalitis
    above metioned are not hit.
    
    2. Move context of initialization of parameters associated with the UFS
    device to a new added function ufshcd_device_params_init().  The reason of
    this change is that all of these parameters are used by driver, but only
    need to be initialized once when booting. Combine them into an integral
    function, make them easier maintain.
    
    Link: https://lore.kernel.org/r/20200120130820.1737-4-huobean@gmail.comReviewed-by: default avatarAsutosh Das <asutoshd@codeaurora.org>
    Signed-off-by: default avatarBean Huo <beanhuo@micron.com>
    Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
    1b9e2141
ufshcd.c 227 KB