[PATCH v2] bus: mhi: host: Allocate entire MHI control config once
Jeff Hugo
jeff.hugo at oss.qualcomm.com
Mon May 12 11:31:41 PDT 2025
On 4/27/2025 7:57 PM, Baochen Qiang wrote:
>
>
> On 4/26/2025 1:04 AM, Jeff Hugo wrote:
>> From: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy at quicinc.com>
>> int mhi_init_mmio(struct mhi_controller *mhi_cntrl)
>> @@ -475,6 +497,7 @@ int mhi_init_mmio(struct mhi_controller *mhi_cntrl)
>> struct mhi_event *mhi_event;
>> void __iomem *base = mhi_cntrl->regs;
>> struct device *dev = &mhi_cntrl->mhi_dev->dev;
>> + dma_addr_t mhi_ctrl_limit = mhi_cntrl->ctrl_config_dma + mhi_cntrl->ctrl_config_size - 1;
>
> but the range [ctrl_config_dma, mhi_ctrl_limit] could still be possbile to cross the 4GB
> boundary, no?
Its possible, yes. From a practical standpoint, that seems to be
unlikely as the control area is not terribly large on supported platforms.
More information about the ath11k
mailing list