* [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
@ 2020-01-01 3:37 Brian Masney
2020-01-01 11:15 ` Stephan Gerhold
2020-01-02 7:36 ` Bjorn Andersson
0 siblings, 2 replies; 7+ messages in thread
From: Brian Masney @ 2020-01-01 3:37 UTC (permalink / raw)
To: bjorn.andersson; +Cc: agross, linux-arm-msm, linux-kernel
Add 32 bit implmentations of the functions
__qcom_scm_iommu_secure_ptbl_size() and
__qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
driver.
Signed-off-by: Brian Masney <masneyb@onstation.org>
---
drivers/firmware/qcom_scm-32.c | 32 ++++++++++++++++++++++++++++++--
1 file changed, 30 insertions(+), 2 deletions(-)
diff --git a/drivers/firmware/qcom_scm-32.c b/drivers/firmware/qcom_scm-32.c
index 48e2ef794ea3..f149a85d36b0 100644
--- a/drivers/firmware/qcom_scm-32.c
+++ b/drivers/firmware/qcom_scm-32.c
@@ -638,13 +638,41 @@ int __qcom_scm_restore_sec_cfg(struct device *dev, u32 device_id,
int __qcom_scm_iommu_secure_ptbl_size(struct device *dev, u32 spare,
size_t *size)
{
- return -ENODEV;
+ int psize[2] = { 0, 0 };
+ int ret;
+
+ ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
+ QCOM_SCM_IOMMU_SECURE_PTBL_SIZE,
+ &spare, sizeof(spare), &psize, sizeof(psize));
+ if (ret || psize[1])
+ return ret ? ret : -EINVAL;
+
+ *size = psize[0];
+
+ return 0;
}
int __qcom_scm_iommu_secure_ptbl_init(struct device *dev, u64 addr, u32 size,
u32 spare)
{
- return -ENODEV;
+ struct msm_scm_ptbl_init {
+ __le32 paddr;
+ __le32 size;
+ __le32 spare;
+ } req;
+ int ret, scm_ret = 0;
+
+ req.paddr = addr;
+ req.size = size;
+ req.spare = spare;
+
+ ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
+ QCOM_SCM_IOMMU_SECURE_PTBL_INIT,
+ &req, sizeof(req), &scm_ret, sizeof(scm_ret));
+ if (ret || scm_ret)
+ return ret ? ret : -EINVAL;
+
+ return 0;
}
int __qcom_scm_io_readl(struct device *dev, phys_addr_t addr,
--
2.21.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
2020-01-01 3:37 [PATCH] firmware: qcom: scm: add 32 bit iommu page table support Brian Masney
@ 2020-01-01 11:15 ` Stephan Gerhold
2020-01-01 20:14 ` Rob Clark
2020-01-02 7:36 ` Bjorn Andersson
1 sibling, 1 reply; 7+ messages in thread
From: Stephan Gerhold @ 2020-01-01 11:15 UTC (permalink / raw)
To: Brian Masney; +Cc: bjorn.andersson, agross, linux-arm-msm, linux-kernel
On Tue, Dec 31, 2019 at 10:37:04PM -0500, Brian Masney wrote:
> Add 32 bit implmentations of the functions
> __qcom_scm_iommu_secure_ptbl_size() and
> __qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
> driver.
>
> Signed-off-by: Brian Masney <masneyb@onstation.org>
> ---
> drivers/firmware/qcom_scm-32.c | 32 ++++++++++++++++++++++++++++++--
> 1 file changed, 30 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/firmware/qcom_scm-32.c b/drivers/firmware/qcom_scm-32.c
> index 48e2ef794ea3..f149a85d36b0 100644
> --- a/drivers/firmware/qcom_scm-32.c
> +++ b/drivers/firmware/qcom_scm-32.c
> @@ -638,13 +638,41 @@ int __qcom_scm_restore_sec_cfg(struct device *dev, u32 device_id,
> int __qcom_scm_iommu_secure_ptbl_size(struct device *dev, u32 spare,
> size_t *size)
> {
> - return -ENODEV;
> + int psize[2] = { 0, 0 };
I would use an explicit size (i.e. __le32) here.
> + int ret;
> +
> + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> + QCOM_SCM_IOMMU_SECURE_PTBL_SIZE,
> + &spare, sizeof(spare), &psize, sizeof(psize));
> + if (ret || psize[1])
> + return ret ? ret : -EINVAL;
> +
> + *size = psize[0];
> +
> + return 0;
> }
>
> int __qcom_scm_iommu_secure_ptbl_init(struct device *dev, u64 addr, u32 size,
> u32 spare)
> {
> - return -ENODEV;
> + struct msm_scm_ptbl_init {
> + __le32 paddr;
> + __le32 size;
> + __le32 spare;
> + } req;
> + int ret, scm_ret = 0;
> +
> + req.paddr = addr;
> + req.size = size;
> + req.spare = spare;
I'm not sure if there is actually anyone using qcom in BE mode (does
that even work?), but all the other methods in this file explicitly
convert using cpu_to_le32(), so this method should do the same :)
> +
> + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> + QCOM_SCM_IOMMU_SECURE_PTBL_INIT,
> + &req, sizeof(req), &scm_ret, sizeof(scm_ret));
> + if (ret || scm_ret)
> + return ret ? ret : -EINVAL;
> +
> + return 0;
> }
>
> int __qcom_scm_io_readl(struct device *dev, phys_addr_t addr,
> --
> 2.21.0
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
2020-01-01 11:15 ` Stephan Gerhold
@ 2020-01-01 20:14 ` Rob Clark
2020-01-05 6:56 ` Stephen Boyd
0 siblings, 1 reply; 7+ messages in thread
From: Rob Clark @ 2020-01-01 20:14 UTC (permalink / raw)
To: Stephan Gerhold
Cc: Brian Masney, Bjorn Andersson, Andy Gross, linux-arm-msm,
Linux Kernel Mailing List, Stephen Boyd
On Wed, Jan 1, 2020 at 3:16 AM Stephan Gerhold <stephan@gerhold.net> wrote:
>
> On Tue, Dec 31, 2019 at 10:37:04PM -0500, Brian Masney wrote:
> > Add 32 bit implmentations of the functions
> > __qcom_scm_iommu_secure_ptbl_size() and
> > __qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
> > driver.
> >
> > Signed-off-by: Brian Masney <masneyb@onstation.org>
> > ---
> > drivers/firmware/qcom_scm-32.c | 32 ++++++++++++++++++++++++++++++--
> > 1 file changed, 30 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/firmware/qcom_scm-32.c b/drivers/firmware/qcom_scm-32.c
> > index 48e2ef794ea3..f149a85d36b0 100644
> > --- a/drivers/firmware/qcom_scm-32.c
> > +++ b/drivers/firmware/qcom_scm-32.c
> > @@ -638,13 +638,41 @@ int __qcom_scm_restore_sec_cfg(struct device *dev, u32 device_id,
> > int __qcom_scm_iommu_secure_ptbl_size(struct device *dev, u32 spare,
> > size_t *size)
> > {
> > - return -ENODEV;
> > + int psize[2] = { 0, 0 };
>
> I would use an explicit size (i.e. __le32) here.
>
> > + int ret;
> > +
> > + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> > + QCOM_SCM_IOMMU_SECURE_PTBL_SIZE,
> > + &spare, sizeof(spare), &psize, sizeof(psize));
> > + if (ret || psize[1])
> > + return ret ? ret : -EINVAL;
> > +
> > + *size = psize[0];
> > +
> > + return 0;
> > }
> >
> > int __qcom_scm_iommu_secure_ptbl_init(struct device *dev, u64 addr, u32 size,
> > u32 spare)
> > {
> > - return -ENODEV;
> > + struct msm_scm_ptbl_init {
> > + __le32 paddr;
> > + __le32 size;
> > + __le32 spare;
> > + } req;
> > + int ret, scm_ret = 0;
> > +
> > + req.paddr = addr;
> > + req.size = size;
> > + req.spare = spare;
>
> I'm not sure if there is actually anyone using qcom in BE mode (does
> that even work?), but all the other methods in this file explicitly
> convert using cpu_to_le32(), so this method should do the same :)
sboyd used to occasionally fix things related to qcom in BE back in
the day.. not sure if modern snapdragons still support BE.
(I'm willing to just pretend that they don't.. that lessens the chance
that someday someone gets far enough to try the GPU in BE mode, and
realizes they've wasted their time getting that far ;-))
BR,
-R
> > +
> > + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> > + QCOM_SCM_IOMMU_SECURE_PTBL_INIT,
> > + &req, sizeof(req), &scm_ret, sizeof(scm_ret));
> > + if (ret || scm_ret)
> > + return ret ? ret : -EINVAL;
> > +
> > + return 0;
> > }
> >
> > int __qcom_scm_io_readl(struct device *dev, phys_addr_t addr,
> > --
> > 2.21.0
> >
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
2020-01-01 20:14 ` Rob Clark
@ 2020-01-05 6:56 ` Stephen Boyd
0 siblings, 0 replies; 7+ messages in thread
From: Stephen Boyd @ 2020-01-05 6:56 UTC (permalink / raw)
To: Rob Clark, Stephan Gerhold
Cc: Brian Masney, Bjorn Andersson, Andy Gross, linux-arm-msm,
Linux Kernel Mailing List
Quoting Rob Clark (2020-01-01 12:14:35)
> On Wed, Jan 1, 2020 at 3:16 AM Stephan Gerhold <stephan@gerhold.net> wrote:
> >
> > On Tue, Dec 31, 2019 at 10:37:04PM -0500, Brian Masney wrote:
> > > Add 32 bit implmentations of the functions
> > > __qcom_scm_iommu_secure_ptbl_size() and
> > > __qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
> > > driver.
> > >
> > > Signed-off-by: Brian Masney <masneyb@onstation.org>
> > > ---
> > > drivers/firmware/qcom_scm-32.c | 32 ++++++++++++++++++++++++++++++--
> > > 1 file changed, 30 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/firmware/qcom_scm-32.c b/drivers/firmware/qcom_scm-32.c
> > > index 48e2ef794ea3..f149a85d36b0 100644
> > > --- a/drivers/firmware/qcom_scm-32.c
> > > +++ b/drivers/firmware/qcom_scm-32.c
> > > @@ -638,13 +638,41 @@ int __qcom_scm_restore_sec_cfg(struct device *dev, u32 device_id,
> > > int __qcom_scm_iommu_secure_ptbl_size(struct device *dev, u32 spare,
> > > size_t *size)
> > > {
> > > - return -ENODEV;
> > > + int psize[2] = { 0, 0 };
> >
> > I would use an explicit size (i.e. __le32) here.
> >
> > > + int ret;
> > > +
> > > + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> > > + QCOM_SCM_IOMMU_SECURE_PTBL_SIZE,
> > > + &spare, sizeof(spare), &psize, sizeof(psize));
> > > + if (ret || psize[1])
> > > + return ret ? ret : -EINVAL;
> > > +
> > > + *size = psize[0];
> > > +
> > > + return 0;
> > > }
> > >
> > > int __qcom_scm_iommu_secure_ptbl_init(struct device *dev, u64 addr, u32 size,
> > > u32 spare)
> > > {
> > > - return -ENODEV;
> > > + struct msm_scm_ptbl_init {
> > > + __le32 paddr;
> > > + __le32 size;
> > > + __le32 spare;
> > > + } req;
> > > + int ret, scm_ret = 0;
> > > +
> > > + req.paddr = addr;
> > > + req.size = size;
> > > + req.spare = spare;
> >
> > I'm not sure if there is actually anyone using qcom in BE mode (does
> > that even work?), but all the other methods in this file explicitly
> > convert using cpu_to_le32(), so this method should do the same :)
>
> sboyd used to occasionally fix things related to qcom in BE back in
> the day.. not sure if modern snapdragons still support BE.
>
> (I'm willing to just pretend that they don't.. that lessens the chance
> that someday someone gets far enough to try the GPU in BE mode, and
> realizes they've wasted their time getting that far ;-))
>
Yeah it used to work many ages ago, but I don't think anyone besides me
tested it, except maybe for the folks working in QCA back then.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
2020-01-01 3:37 [PATCH] firmware: qcom: scm: add 32 bit iommu page table support Brian Masney
2020-01-01 11:15 ` Stephan Gerhold
@ 2020-01-02 7:36 ` Bjorn Andersson
2020-01-02 8:50 ` Brian Masney
2020-01-04 2:24 ` Brian Masney
1 sibling, 2 replies; 7+ messages in thread
From: Bjorn Andersson @ 2020-01-02 7:36 UTC (permalink / raw)
To: Brian Masney; +Cc: agross, linux-arm-msm, linux-kernel
On Tue 31 Dec 19:37 PST 2019, Brian Masney wrote:
> Add 32 bit implmentations of the functions
> __qcom_scm_iommu_secure_ptbl_size() and
> __qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
> driver.
>
Hi Brian,
This looks good, but I was hoping to hoping to reach a conclusion and
merge [1] - which in patch 16 squashes the argument filling boiler plate
code of the 32 and 64-bit version of scm and hence implements the same.
If you have time to take a peek at this series I would greatly
appreciate it (not a lot of people testing 32-bit these days...)
[1] https://patchwork.kernel.org/project/linux-arm-msm/list/?series=215943
Regards,
Bjorn
> Signed-off-by: Brian Masney <masneyb@onstation.org>
> ---
> drivers/firmware/qcom_scm-32.c | 32 ++++++++++++++++++++++++++++++--
> 1 file changed, 30 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/firmware/qcom_scm-32.c b/drivers/firmware/qcom_scm-32.c
> index 48e2ef794ea3..f149a85d36b0 100644
> --- a/drivers/firmware/qcom_scm-32.c
> +++ b/drivers/firmware/qcom_scm-32.c
> @@ -638,13 +638,41 @@ int __qcom_scm_restore_sec_cfg(struct device *dev, u32 device_id,
> int __qcom_scm_iommu_secure_ptbl_size(struct device *dev, u32 spare,
> size_t *size)
> {
> - return -ENODEV;
> + int psize[2] = { 0, 0 };
> + int ret;
> +
> + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> + QCOM_SCM_IOMMU_SECURE_PTBL_SIZE,
> + &spare, sizeof(spare), &psize, sizeof(psize));
> + if (ret || psize[1])
> + return ret ? ret : -EINVAL;
> +
> + *size = psize[0];
> +
> + return 0;
> }
>
> int __qcom_scm_iommu_secure_ptbl_init(struct device *dev, u64 addr, u32 size,
> u32 spare)
> {
> - return -ENODEV;
> + struct msm_scm_ptbl_init {
> + __le32 paddr;
> + __le32 size;
> + __le32 spare;
> + } req;
> + int ret, scm_ret = 0;
> +
> + req.paddr = addr;
> + req.size = size;
> + req.spare = spare;
> +
> + ret = qcom_scm_call(dev, QCOM_SCM_SVC_MP,
> + QCOM_SCM_IOMMU_SECURE_PTBL_INIT,
> + &req, sizeof(req), &scm_ret, sizeof(scm_ret));
> + if (ret || scm_ret)
> + return ret ? ret : -EINVAL;
> +
> + return 0;
> }
>
> int __qcom_scm_io_readl(struct device *dev, phys_addr_t addr,
> --
> 2.21.0
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
2020-01-02 7:36 ` Bjorn Andersson
@ 2020-01-02 8:50 ` Brian Masney
2020-01-04 2:24 ` Brian Masney
1 sibling, 0 replies; 7+ messages in thread
From: Brian Masney @ 2020-01-02 8:50 UTC (permalink / raw)
To: Bjorn Andersson; +Cc: agross, linux-arm-msm, linux-kernel
Hi Bjorn,
On Wed, Jan 01, 2020 at 11:36:07PM -0800, Bjorn Andersson wrote:
> On Tue 31 Dec 19:37 PST 2019, Brian Masney wrote:
>
> > Add 32 bit implmentations of the functions
> > __qcom_scm_iommu_secure_ptbl_size() and
> > __qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
> > driver.
> >
>
> Hi Brian,
>
> This looks good, but I was hoping to hoping to reach a conclusion and
> merge [1] - which in patch 16 squashes the argument filling boiler plate
> code of the 32 and 64-bit version of scm and hence implements the same.
>
> If you have time to take a peek at this series I would greatly
> appreciate it (not a lot of people testing 32-bit these days...)
>
> [1] https://patchwork.kernel.org/project/linux-arm-msm/list/?series=215943
I'll look at that series this weekend and test it on arm32.
Brian
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] firmware: qcom: scm: add 32 bit iommu page table support
2020-01-02 7:36 ` Bjorn Andersson
2020-01-02 8:50 ` Brian Masney
@ 2020-01-04 2:24 ` Brian Masney
1 sibling, 0 replies; 7+ messages in thread
From: Brian Masney @ 2020-01-04 2:24 UTC (permalink / raw)
To: Bjorn Andersson; +Cc: agross, linux-arm-msm, linux-kernel
On Wed, Jan 01, 2020 at 11:36:07PM -0800, Bjorn Andersson wrote:
> On Tue 31 Dec 19:37 PST 2019, Brian Masney wrote:
>
> > Add 32 bit implmentations of the functions
> > __qcom_scm_iommu_secure_ptbl_size() and
> > __qcom_scm_iommu_secure_ptbl_init() that are required by the qcom_iommu
> > driver.
> >
>
> Hi Brian,
>
> This looks good, but I was hoping to hoping to reach a conclusion and
> merge [1] - which in patch 16 squashes the argument filling boiler plate
> code of the 32 and 64-bit version of scm and hence implements the same.
>
> If you have time to take a peek at this series I would greatly
> appreciate it (not a lot of people testing 32-bit these days...)
>
> [1] https://patchwork.kernel.org/project/linux-arm-msm/list/?series=215943
I agree to drop what I proposed here and go with that series instead.
It's a big improvement over what's upstream now. I left a Tested-by on
that cover letter.
Brian
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-01-05 6:57 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-01 3:37 [PATCH] firmware: qcom: scm: add 32 bit iommu page table support Brian Masney
2020-01-01 11:15 ` Stephan Gerhold
2020-01-01 20:14 ` Rob Clark
2020-01-05 6:56 ` Stephen Boyd
2020-01-02 7:36 ` Bjorn Andersson
2020-01-02 8:50 ` Brian Masney
2020-01-04 2:24 ` Brian Masney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).