On Thu, May 06, 2021 at 12:51:39PM -0700, Bhaumik Bhatt wrote: > This patch series improves the power up behavior by allowing MHI host driver to > set BHI and/or BHIe offsets early on in the preparation phase and fail pre-power > up if offsets are not found or not within a limited MMIO region. This also > allows MHI host to clean up the offsets in the unprepare after power down phase. > > Going forward, controllers will be required to specify a reg_len field which > will be used to check whether the BHI/BHIe offsets are in range or not. > > This series has been tested on X86_64 architecture with the PCI generic driver > as controller and an SDX55 device. > Since there is a chance to send this series for 5.14, I've applied it now to mhi-next! Thanks, Mani > v4: > -Added reviewed-by tags > -Updated range check patch to include BHI/e offsets in the error message > > v3: > -Added reviewed-by tags > -Updated order of reg_len in mhi_controller structure documentation > > v2: > -Added reviewed-by tags > -Moved reg_len entry in mhi_controller structure to allow for a packed struct > > Bhaumik Bhatt (6): > bus: mhi: core: Set BHI/BHIe offsets on power up preparation > bus: mhi: core: Set BHI and BHIe pointers to NULL in clean-up > bus: mhi: Add MMIO region length to controller structure > ath11k: set register access length for MHI driver > bus: mhi: pci_generic: Set register access length for MHI driver > bus: mhi: core: Add range checks for BHI and BHIe > > drivers/bus/mhi/core/init.c | 61 ++++++++++++++++++++++++----------- > drivers/bus/mhi/core/pm.c | 28 +++------------- > drivers/bus/mhi/pci_generic.c | 1 + > drivers/net/wireless/ath/ath11k/mhi.c | 1 + > include/linux/mhi.h | 2 ++ > 5 files changed, 50 insertions(+), 43 deletions(-) > > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project >