From: Bhaumik Bhatt <bbhatt@codeaurora.org> To: Jeffrey Hugo <quic_jhugo@quicinc.com> Cc: manivannan.sadhasivam@linaro.org, linux-arm-msm@vger.kernel.org, hemantk@codeaurora.org, linux-kernel@vger.kernel.org, loic.poulain@linaro.org, linux-wireless@vger.kernel.org, kvalo@codeaurora.org, ath11k@lists.infradead.org Subject: Re: [PATCH v1 3/6] bus: mhi: Add MMIO region length to controller structure Date: Tue, 04 May 2021 16:04:12 -0700 [thread overview] Message-ID: <928a29dca0d2b141d4e21b7878c1c42e@codeaurora.org> (raw) In-Reply-To: <83589a09-e1a0-4611-a844-0ea52ca036b9@quicinc.com> Hi Jeff, On 2021-05-04 10:28 AM, Jeffrey Hugo wrote: > On 4/16/2021 12:47 PM, Bhaumik Bhatt wrote: >> Make controller driver specify the MMIO register region length >> for range checking of BHI or BHIe space. This can help validate >> that offsets are in acceptable memory region or not and avoid any >> boot-up issues due to BHI or BHIe memory accesses. >> >> Signed-off-by: Bhaumik Bhatt <bbhatt@codeaurora.org> >> --- >> include/linux/mhi.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/include/linux/mhi.h b/include/linux/mhi.h >> index 944aa3a..ad53b24 100644 >> --- a/include/linux/mhi.h >> +++ b/include/linux/mhi.h >> @@ -293,6 +293,7 @@ struct mhi_controller_config { >> * @mhi_dev: MHI device instance for the controller >> * @debugfs_dentry: MHI controller debugfs directory >> * @regs: Base address of MHI MMIO register space (required) >> + * @reg_len: Length of the MHI MMIO region (required) >> * @bhi: Points to base of MHI BHI register space >> * @bhie: Points to base of MHI BHIe register space >> * @wake_db: MHI WAKE doorbell register address >> @@ -375,6 +376,7 @@ struct mhi_controller { >> struct mhi_device *mhi_dev; >> struct dentry *debugfs_dentry; >> void __iomem *regs; >> + size_t reg_len; > > Didn't Mani make sure this struct was packed using pahole when things > were first upstreamed? Feels like sticking this in the middle of a > bunch of void * entries will add some padding. Am I wrong? > OK. I missed out on this detail. I can move this size_t entry in v2. >> void __iomem *bhi; >> void __iomem *bhie; >> void __iomem *wake_db; >> Thanks, Bhaumik --- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
WARNING: multiple messages have this Message-ID (diff)
From: Bhaumik Bhatt <bbhatt@codeaurora.org> To: Jeffrey Hugo <quic_jhugo@quicinc.com> Cc: manivannan.sadhasivam@linaro.org, linux-arm-msm@vger.kernel.org, hemantk@codeaurora.org, linux-kernel@vger.kernel.org, loic.poulain@linaro.org, linux-wireless@vger.kernel.org, kvalo@codeaurora.org, ath11k@lists.infradead.org Subject: Re: [PATCH v1 3/6] bus: mhi: Add MMIO region length to controller structure Date: Tue, 04 May 2021 16:04:12 -0700 [thread overview] Message-ID: <928a29dca0d2b141d4e21b7878c1c42e@codeaurora.org> (raw) In-Reply-To: <83589a09-e1a0-4611-a844-0ea52ca036b9@quicinc.com> Hi Jeff, On 2021-05-04 10:28 AM, Jeffrey Hugo wrote: > On 4/16/2021 12:47 PM, Bhaumik Bhatt wrote: >> Make controller driver specify the MMIO register region length >> for range checking of BHI or BHIe space. This can help validate >> that offsets are in acceptable memory region or not and avoid any >> boot-up issues due to BHI or BHIe memory accesses. >> >> Signed-off-by: Bhaumik Bhatt <bbhatt@codeaurora.org> >> --- >> include/linux/mhi.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/include/linux/mhi.h b/include/linux/mhi.h >> index 944aa3a..ad53b24 100644 >> --- a/include/linux/mhi.h >> +++ b/include/linux/mhi.h >> @@ -293,6 +293,7 @@ struct mhi_controller_config { >> * @mhi_dev: MHI device instance for the controller >> * @debugfs_dentry: MHI controller debugfs directory >> * @regs: Base address of MHI MMIO register space (required) >> + * @reg_len: Length of the MHI MMIO region (required) >> * @bhi: Points to base of MHI BHI register space >> * @bhie: Points to base of MHI BHIe register space >> * @wake_db: MHI WAKE doorbell register address >> @@ -375,6 +376,7 @@ struct mhi_controller { >> struct mhi_device *mhi_dev; >> struct dentry *debugfs_dentry; >> void __iomem *regs; >> + size_t reg_len; > > Didn't Mani make sure this struct was packed using pahole when things > were first upstreamed? Feels like sticking this in the middle of a > bunch of void * entries will add some padding. Am I wrong? > OK. I missed out on this detail. I can move this size_t entry in v2. >> void __iomem *bhi; >> void __iomem *bhie; >> void __iomem *wake_db; >> Thanks, Bhaumik --- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k
next prev parent reply other threads:[~2021-05-04 23:04 UTC|newest] Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-16 18:46 [PATCH v1 0/6] BHI/BHIe improvements for MHI power purposes Bhaumik Bhatt 2021-04-16 18:46 ` Bhaumik Bhatt 2021-04-16 18:47 ` [PATCH v1 1/6] bus: mhi: core: Set BHI/BHIe offsets on power up preparation Bhaumik Bhatt 2021-04-16 18:47 ` Bhaumik Bhatt 2021-05-04 17:22 ` Jeffrey Hugo 2021-05-04 17:22 ` Jeffrey Hugo 2021-04-16 18:47 ` [PATCH v1 2/6] bus: mhi: core: Set BHI and BHIe pointers to NULL in clean-up Bhaumik Bhatt 2021-04-16 18:47 ` Bhaumik Bhatt 2021-05-04 17:26 ` Jeffrey Hugo 2021-05-04 17:26 ` Jeffrey Hugo 2021-04-16 18:47 ` [PATCH v1 3/6] bus: mhi: Add MMIO region length to controller structure Bhaumik Bhatt 2021-04-16 18:47 ` Bhaumik Bhatt 2021-05-04 17:28 ` Jeffrey Hugo 2021-05-04 17:28 ` Jeffrey Hugo 2021-05-04 23:04 ` Bhaumik Bhatt [this message] 2021-05-04 23:04 ` Bhaumik Bhatt 2021-04-16 18:47 ` [PATCH v1 4/6] ath11k: set register access length for MHI driver Bhaumik Bhatt 2021-04-16 18:47 ` Bhaumik Bhatt 2021-04-16 18:47 ` [PATCH v1 5/6] bus: mhi: pci_generic: Set " Bhaumik Bhatt 2021-04-16 18:47 ` Bhaumik Bhatt 2021-04-16 18:47 ` [PATCH v1 6/6] bus: mhi: core: Add range checks for BHI and BHIe Bhaumik Bhatt 2021-04-16 18:47 ` Bhaumik Bhatt 2021-05-04 17:30 ` Jeffrey Hugo 2021-05-04 17:30 ` Jeffrey Hugo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=928a29dca0d2b141d4e21b7878c1c42e@codeaurora.org \ --to=bbhatt@codeaurora.org \ --cc=ath11k@lists.infradead.org \ --cc=hemantk@codeaurora.org \ --cc=kvalo@codeaurora.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-wireless@vger.kernel.org \ --cc=loic.poulain@linaro.org \ --cc=manivannan.sadhasivam@linaro.org \ --cc=quic_jhugo@quicinc.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.