Re: [PATCH 1/2] remoteproc: use a flag to detect the presence of IOMMU
On Mon, Aug 4, 2014 at 6:48 PM, Suman Anna s-a...@ti.com wrote: On 08/04/2014 06:50 AM, Ohad Ben-Cohen wrote: On Tue, Jul 29, 2014 at 7:10 PM, Suman Anna s-a...@ti.com wrote: We are trying to add a remoteproc driver for a small Cortex M3 called the WkupM3 used for suspend/resume management on TI AM335/AM437x SoCs. This processor does not have an MMU. Same is the case with another processor subsystem PRU-ICSS on AM335/AM437x. All these are platform devices, and the current iommu_present check will not scale for the same kernel image to support OMAP4/OMAP5 and AM335/AM437x. That's perfect, thanks. Can you please add this use case description to the commit log? I kept the current description generic, but sure, I can add this specific usecase examples to the commit log. I will post the revised patches once rc1 is out. Thanks! -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] remoteproc: use a flag to detect the presence of IOMMU
On Tue, Jul 29, 2014 at 7:10 PM, Suman Anna s-a...@ti.com wrote: We are trying to add a remoteproc driver for a small Cortex M3 called the WkupM3 used for suspend/resume management on TI AM335/AM437x SoCs. This processor does not have an MMU. Same is the case with another processor subsystem PRU-ICSS on AM335/AM437x. All these are platform devices, and the current iommu_present check will not scale for the same kernel image to support OMAP4/OMAP5 and AM335/AM437x. That's perfect, thanks. Can you please add this use case description to the commit log? This way we'll be able to recover it easily one day if needed. Thanks, Ohad. -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] remoteproc: use a flag to detect the presence of IOMMU
On 08/04/2014 06:50 AM, Ohad Ben-Cohen wrote: On Tue, Jul 29, 2014 at 7:10 PM, Suman Anna s-a...@ti.com wrote: We are trying to add a remoteproc driver for a small Cortex M3 called the WkupM3 used for suspend/resume management on TI AM335/AM437x SoCs. This processor does not have an MMU. Same is the case with another processor subsystem PRU-ICSS on AM335/AM437x. All these are platform devices, and the current iommu_present check will not scale for the same kernel image to support OMAP4/OMAP5 and AM335/AM437x. That's perfect, thanks. Can you please add this use case description to the commit log? I kept the current description generic, but sure, I can add this specific usecase examples to the commit log. I will post the revised patches once rc1 is out. regards Suman -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] remoteproc: use a flag to detect the presence of IOMMU
Hi Suman, On Tue, Jul 8, 2014 at 7:22 PM, Suman Anna s-a...@ti.com wrote: The remoteproc driver core currently relies on iommu_present() on the bus the device is on, to perform MMU management. However, this logic doesn't scale for multi-arch, especially for processors that do not have an IOMMU. Is there a specific hw/scenario where you need this? Can you please provide more details about it? Ideally we should add them to the commit log as well. The individual platform implementations are required to set this flag appropriately. The default setting is to not have an MMU. Let's explicitly set the default please so this would be clear for users reading the code. Cc: Sjur Brændeland sjur.brandel...@stericsson.com Sjur is no longer with STE, so no point in cc'ing his old email address. + /* +* All existing OMAP IPU and DSP processors do have an MMU, and +* are expected to use MMU, so this statement suffices. +* XXX: Replace this logic if and when a need arises. The last XXX comment is always true for any kernel code, so I'd drop it. Thanks, Ohad. -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] remoteproc: use a flag to detect the presence of IOMMU
Hi Ohad, On 07/29/2014 05:57 AM, Ohad Ben-Cohen wrote: Hi Suman, On Tue, Jul 8, 2014 at 7:22 PM, Suman Anna s-a...@ti.com wrote: The remoteproc driver core currently relies on iommu_present() on the bus the device is on, to perform MMU management. However, this logic doesn't scale for multi-arch, especially for processors that do not have an IOMMU. Is there a specific hw/scenario where you need this? Can you please provide more details about it? We are trying to add a remoteproc driver for a small Cortex M3 called the WkupM3 used for suspend/resume management on TI AM335/AM437x SoCs. This processor does not have an MMU. Same is the case with another processor subsystem PRU-ICSS on AM335/AM437x. All these are platform devices, and the current iommu_present check will not scale for the same kernel image to support OMAP4/OMAP5 and AM335/AM437x. This patch mainly addresses the existing comments in the code, -* This works for simple cases, but will easily fail with -* platforms that do have an IOMMU, but not for this specific -* rproc. -* -* This will be easily solved by introducing hw capabilities -* that will be set by the remoteproc driver. Ideally we should add them to the commit log as well. The individual platform implementations are required to set this flag appropriately. The default setting is to not have an MMU. Let's explicitly set the default please so this would be clear for users reading the code. OK, I can update the existing drivers to explicitly set this field. Cc: Sjur Brændeland sjur.brandel...@stericsson.com Sjur is no longer with STE, so no point in cc'ing his old email address. Yeah, I wasn't aware until I got a bounced email. + /* +* All existing OMAP IPU and DSP processors do have an MMU, and +* are expected to use MMU, so this statement suffices. +* XXX: Replace this logic if and when a need arises. The last XXX comment is always true for any kernel code, so I'd drop it. Sure. regards Suman -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 1/2] remoteproc: use a flag to detect the presence of IOMMU
The remoteproc driver core currently relies on iommu_present() on
the bus the device is on, to perform MMU management. However, this
logic doesn't scale for multi-arch, especially for processors that
do not have an IOMMU. Replace this logic instead by using a h/w
capability flag for the presence of IOMMU in the rproc structure.
The individual platform implementations are required to set this
flag appropriately. The default setting is to not have an MMU.
The OMAP remoteproc driver is updated while at this, to maintain
the functionality with the IOMMU detection logic change in this
patch.
Cc: Sjur Brændeland sjur.brandel...@stericsson.com
Cc: Robert Tivy rt...@ti.com
Signed-off-by: Suman Anna s-a...@ti.com
---
drivers/remoteproc/omap_remoteproc.c | 6 ++
drivers/remoteproc/remoteproc_core.c | 15 ++-
include/linux/remoteproc.h | 2 ++
3 files changed, 10 insertions(+), 13 deletions(-)
diff --git a/drivers/remoteproc/omap_remoteproc.c
b/drivers/remoteproc/omap_remoteproc.c
index 5168972..858abf0 100644
--- a/drivers/remoteproc/omap_remoteproc.c
+++ b/drivers/remoteproc/omap_remoteproc.c
@@ -199,6 +199,12 @@ static int omap_rproc_probe(struct platform_device *pdev)
oproc = rproc-priv;
oproc-rproc = rproc;
+ /*
+* All existing OMAP IPU and DSP processors do have an MMU, and
+* are expected to use MMU, so this statement suffices.
+* XXX: Replace this logic if and when a need arises.
+*/
+ rproc-has_iommu = true;
platform_set_drvdata(pdev, rproc);
diff --git a/drivers/remoteproc/remoteproc_core.c
b/drivers/remoteproc/remoteproc_core.c
index 3cd85a6..11cdb11 100644
--- a/drivers/remoteproc/remoteproc_core.c
+++ b/drivers/remoteproc/remoteproc_core.c
@@ -94,19 +94,8 @@ static int rproc_enable_iommu(struct rproc *rproc)
struct device *dev = rproc-dev.parent;
int ret;
- /*
-* We currently use iommu_present() to decide if an IOMMU
-* setup is needed.
-*
-* This works for simple cases, but will easily fail with
-* platforms that do have an IOMMU, but not for this specific
-* rproc.
-*
-* This will be easily solved by introducing hw capabilities
-* that will be set by the remoteproc driver.
-*/
- if (!iommu_present(dev-bus)) {
- dev_dbg(dev, iommu not found\n);
+ if (!rproc-has_iommu) {
+ dev_dbg(dev, iommu not present\n);
return 0;
}
diff --git a/include/linux/remoteproc.h b/include/linux/remoteproc.h
index 9e7e745..78b8a9b 100644
--- a/include/linux/remoteproc.h
+++ b/include/linux/remoteproc.h
@@ -404,6 +404,7 @@ enum rproc_crash_type {
* @table_ptr: pointer to the resource table in effect
* @cached_table: copy of the resource table
* @table_csum: checksum of the resource table
+ * @has_iommu: flag to indicate if remote processor is behind an MMU
*/
struct rproc {
struct klist_node node;
@@ -435,6 +436,7 @@ struct rproc {
struct resource_table *table_ptr;
struct resource_table *cached_table;
u32 table_csum;
+ bool has_iommu;
};
/* we currently support only two vrings per rvdev */
--
2.0.0
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
