[PATCH v3 2/2] OMAP: IOMMU: add support to callback during fault handling
David Cohen
dacohen at gmail.com
Wed Feb 23 04:45:17 EST 2011
On Wed, Feb 23, 2011 at 3:17 AM, Guzman Lugo, Fernando
<fernando.lugo at ti.com> wrote:
> On Wed, Feb 16, 2011 at 1:35 PM, David Cohen <dacohen at gmail.com> wrote:
>> Add support to register an isr for IOMMU fault situations and adapt it
>> to allow such (*isr)() to be used as fault callback. Drivers using IOMMU
>> module might want to be informed when errors happen in order to debug it
>> or react.
>>
>> Signed-off-by: David Cohen <dacohen at gmail.com>
>> ---
>> arch/arm/mach-omap2/iommu2.c | 17 +++++++++-
>> arch/arm/plat-omap/include/plat/iommu.h | 14 ++++++++-
>> arch/arm/plat-omap/iommu.c | 52 ++++++++++++++++++++++---------
>> 3 files changed, 65 insertions(+), 18 deletions(-)
>>
> ....
>
>> @@ -917,6 +912,33 @@ void iommu_put(struct iommu *obj)
>> }
>> EXPORT_SYMBOL_GPL(iommu_put);
>>
>> +int iommu_set_isr(const char *name,
>> + int (*isr)(struct iommu *obj, u32 da, u32 iommu_errs,
>> + void *priv),
>> + void *isr_priv)
>> +{
>> + struct device *dev;
>> + struct iommu *obj;
>> +
>
> if the driver support multiple user for the same iommu why can only
> one callback be registered? should it support register multiple
> callback function (one per user)?
Can you define a scenario for that?
On OMAP3 ISP the multiple users are the multiple ISP submodule, but I
don't think it's necessary all submodule to have a specific callback.
ISP core layer should handle.
Br,
David
>
> Regards,
> Fernando.
>
More information about the linux-arm-kernel
mailing list